Repository navigation
fix(deps): move every Go service to sdk-go v0.3.2 so the host starts inside a composition - #1095
Merged
Merged
Conversation
…inside a composition sdk-go v0.3.1 resolved a service's own endpoint from the first workspace file above the process. A host run from its checkout or from a released tag in the module cache found its own development workspace, which names the module differently from the composition, so telemetry and accounts exited at start. v0.3.2 (codefly-dev/sdk-go#68) trusts a found workspace only when it is the process's injected composition. accounts' fixture seeding moves from codefly.Fixture() to fixture.Selected(), the API sdk-go#65 moved it to. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
antoinetoussaint-byte
added a commit
that referenced
this pull request
Oct 12, 2026
…ext cutover The four conflicts were the accounts and auth-gateway go.mod/go.sum. Both intents are kept: main's move to sdk-go v0.3.2 (#1095), and this branch's core pin (core main's merge commit for core#783) and sdk-go/workcontext v0.1.0. The require lines were combined by hand and go.sum regenerated with go mod tidy; minimal version selection keeps core on the pinned commit. No contract artefact conflicted, and a full pass of the pinned codefly 0.1.185 pipeline (proto, go generate, contracts, SDK) over the merged tree changes nothing. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Moves the five Go services (
accounts,auth-gateway,policy-log,store,telemetry) from sdk-go v0.3.1 to v0.3.2.Why. With v0.3.1, a service resolves its own endpoint from the first workspace file above the process. When the host runs from its checkout, or from a released tag in the module cache, inside a larger composition, that file is the host's own development workspace, which names the module differently from the composition.
telemetryandaccountsthen exit at start (cannot find module <saas> in <saas-starter-dev>). v0.3.2 (codefly-dev/sdk-go#68, closing sdk-go#66) trusts a found workspace only when it is the composition named by the injectedCODEFLY__WORKSPACE; otherwise it reads the carrier the CLI injected. sdk-go#68 records the boot proof with this host's v0.0.94 tag: telemetry, accounts, auth-gateway and frontend reach Running, from both the checkout and the module cache.Changes
go.mod/go.sumin the five services: sdk-go v0.3.2.go mod tidydrops indirect requirements v0.3.2 no longer needs.accounts/code/fixtures/seed.go:codefly.Fixture()becomesfixture.Selected(), the API sdk-go#65 moved it to. Without it, accounts does not compile on any sdk-go after Generate ambient authorization for internal topology edges #65.Classification: a fix, at the owner (dependency bump plus the API move it requires). Nothing was reinvented.
Verified locally (GOWORK=off, Go 1.27.2): every service builds and passes
go vet ./..., andaccounts/fixturestests pass. The full suites run in CI.Not verified here: a composed boot with this exact head; sdk-go#68 proved it with the dependency overridden.
🤖 Generated with Claude Code