Skip to content

fix(environment): a configuration-source profile may carry the module envelope - #1154

Merged
antoinetoussaint-byte merged 1 commit into
mainfrom
fix/configuration-source-carries-the-module-envelope
Oct 25, 2026
Merged

antoinetoussaint-byte merged 1 commit into
mainfrom
fix/configuration-source-carries-the-module-envelope

Conversation

@antoinetoussaint-byte

Copy link
Copy Markdown
Contributor

Why

Since #1151, a cell's module envelope is resolved through its configuration profile's derivation, so a shared hosted profile states it once (configurations/hosted/module-envelope.json). Importing an environment onto that profile then refused: configuration-source supports only flat .env inputs and profile derivation metadata: module-envelope.json.

That refusal hits every new hosted cell, and every re-import of an existing one. Found importing obin-dev, development's new platform cell.

What

The source-layer check accepts the module envelope by its exact name, fixture.EnvelopeFile (now exported), which is the name the envelope reader resolves. Anything else stays refused.

Classification: fix. It follows up my own #1151, which moved the envelope into the profile without teaching import about it.

Docs: docs/commands.md (environment import, configuration source).

Tests

TestProfileImportAcceptsTheSourceProfilesModuleEnvelope checks that the exact name is accepted and a near name is refused. go test ./cmd/environment ./pkg/run/fixture passes.

🤖 Generated with Claude Code

… envelope

Since #1151 a cell's module envelope is resolved through its configuration
profile's derivation, so a shared hosted profile states it once
(configurations/hosted/module-envelope.json). Importing an environment onto
that profile then refused: "configuration-source supports only flat .env
inputs and profile derivation metadata: module-envelope.json" — every new
hosted cell, and any re-import of an existing one. The source-layer check
now accepts the module envelope by its exact name (fixture.EnvelopeFile,
the name the reader resolves); anything else stays refused.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@antoinetoussaint-byte
antoinetoussaint-byte added this pull request to the merge queue Oct 25, 2026
Merged via the queue into main with commit 95198c5 Oct 25, 2026
14 checks passed
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