Skip to content

feat: expose resolve_bundle_target for tenant/team resolution#64

Open
BimaPangestu28 wants to merge 2 commits into
mainfrom
feat/expose-resolve-bundle-target
Open

feat: expose resolve_bundle_target for tenant/team resolution#64
BimaPangestu28 wants to merge 2 commits into
mainfrom
feat/expose-resolve-bundle-target

Conversation

@BimaPangestu28
Copy link
Copy Markdown
Member

Summary

  • Expose public resolve_bundle_target() API that resolves effective tenant/team from bundle config (bundle.yaml, bundle-manifest.json, resolved/ directory) with optional CLI overrides
  • Extract load_normalized_bundle_config() to share logic between runtime and the new public API
  • Add BundleTarget struct as the return type

This allows callers like gtc to display accurate tenant/team values instead of hardcoded fallbacks (fixes greenticai/greentic#111).

Test plan

  • 10 unit tests pass (cargo test --lib -- bundle_config)
  • cargo clippy -- -D warnings clean
  • Tests cover: manifest resolution, CLI overrides, resolved/ fallback, bundle.yaml tenant

Extract normalized bundle config loading into a shared function and expose
a public resolve_bundle_target() API that resolves effective tenant/team
from bundle configuration (bundle.yaml, bundle-manifest.json, resolved/)
with optional CLI overrides. This allows callers like gtc to display
accurate tenant/team values instead of hardcoded fallbacks.
@BimaPangestu28 BimaPangestu28 force-pushed the feat/expose-resolve-bundle-target branch from 39dd1a7 to b7cf518 Compare April 7, 2026 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant