Skip to content

fix: complete the first hosted composition delivery handoff (environment profiles from a configuration source, module-qualified values, isolated publication roots) - #1123

Merged
antoinetoussaint-byte merged 6 commits into
mainfrom
takeover/first-hosted-delivery
Oct 23, 2026

Conversation

@antoinetoussaint-byte

Copy link
Copy Markdown
Contributor

Carries #1118 by @crisfiorini-oss (its five commits, unchanged) merged with main at 0.1.203, so a composition's first hosted render on a new cell can be imported and rendered: platform-obin's blue/green environments (obin-ai/platform-obin#213) declare configuration-source: hosted-development and publish under deployments/environments/<environment>, which the released CLI does not read.

From #1118:

  • environment import accepts an explicit configuration source and generates an environment profile deriving from authored common inputs through Core's profile inheritance; validates inherited flat inputs and blank secret placeholders; refuses symlinks, cycles and authored destination conflicts; dry-run writes nothing.
  • Module-qualified service configuration is resolved during validation and rendering: qualified public values override bare values per key, preserving imported managed-resource configuration and refusing collisions with secrets.
  • The generated cell file's source path carries isolated publication roots; a builder that refuses its plan names the service.

Checked here: merges with main cleanly, go build ./..., go test ./cmd/environment ./pkg/environments ./pkg/gitops/... green, no composition or cell name in the code; CLI reference regenerated.

🤖 Generated with Claude Code

@antoinetoussaint-byte
antoinetoussaint-byte added this pull request to the merge queue Oct 23, 2026
Merged via the queue into main with commit 430e51e Oct 23, 2026
13 of 14 checks passed
@antoinetoussaint-byte
antoinetoussaint-byte deleted the takeover/first-hosted-delivery branch October 25, 2026 17:09
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.

2 participants