Skip to content

fix: stage runtime-access.sql at the recipe context root (#68) - #69

Merged
antoinetoussaint-byte merged 1 commit into
mainfrom
issue-68-0-0-125-breaks-container-build-builder-runtime
Aug 23, 2026
Merged

fix: stage runtime-access.sql at the recipe context root (#68)#69
antoinetoussaint-byte merged 1 commit into
mainfrom
issue-68-0-0-125-breaks-container-build-builder-runtime

Conversation

@antoinetoussaint-byte

Copy link
Copy Markdown
Contributor

Closes#68.

Summary

  • 0.0.125's CLI-owned build emits a recipe tree whose builder/ directory holds the Dockerfile. The CLI treats that Dockerfile directory as recipe metadata and drops it from the staged build context, so the generated COPY builder/runtime-access.sql could no longer resolve its checksum and every runtime-access build failed. migrations/ survived because it is a sibling of builder/, not inside it.
  • The fix renders runtime-access.sql at the build-context root beside migrations/ — the directory the CLI does stage — and COPYs it from there. builder/ now carries only the Dockerfile (read via -f), matching the SingleImageBuildPlan convention the Go/Rust runners already rely on. The legacy in-process build path is kept consistent (same context-root layout, same shared Dockerfile).

Test plan

  • go test ./... (includes TestBootstrapImageBuildsWhenDockerOmitsTargetArchitecture, which builds a real image via BuildKit with the new COPY runtime-access.sql)
  • go vet ./...
  • TestBuildEmitsRecipeToOutputDirectory now asserts runtime-access.sql sits at the context root and is absent from builder/

🤖 Generated with Claude Code

The build recipe rendered runtime-access.sql into builder/, but the CLI
treats the Dockerfile's builder/ directory as recipe metadata and drops it
from the staged build context, so `COPY builder/runtime-access.sql` failed
to resolve. Render it at the context root beside migrations/ (which the CLI
does stage) and COPY it from there.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@antoinetoussaint-byte
antoinetoussaint-byte merged commit 35a9da8 into mainAug 23, 2026
2 checks passed
@antoinetoussaint-byte
antoinetoussaint-byte deleted the issue-68-0-0-125-breaks-container-build-builder-runtime branch August 23, 2026 22:41
Sign up for freeto 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.

0.0.125 breaks container build: builder/runtime-access.sql missing from staged build context

1 participant

@antoinetoussaint-byte