Skip to content

fix(docker): add missing codegen schemas to astro-builder stage#7994

Merged
h0lybyte merged 2 commits intodevfrom
trunk/docker-build-fix-1773548646
Mar 15, 2026
Merged

fix(docker): add missing codegen schemas to astro-builder stage#7994
h0lybyte merged 2 commits intodevfrom
trunk/docker-build-fix-1773548646

Conversation

@h0lybyte
Copy link
Member

Summary

  • Add COPY packages/data/codegen/generated/ packages/data/codegen/generated/ to the astro-builder Docker stage
  • The astro-kbve content collections import generated schemas (itemdb-schema, mapdb-schema, etc.) from packages/data/codegen/generated/. This directory was missing from the Docker build context, causing astro sync to fail with Failed to load url ../../../../../../packages/data/codegen/generated/itemdb-schema, which produced no build output and surfaced as a COPY --from=astro-builder /app/dist/apps/astro-kbve: not found error

Test plan

  • Verify axum-kbve:container:ci Docker build succeeds with the codegen schemas available
  • Verify axum-kbve-e2e e2e tests pass end-to-end

The astro-kbve content collections import generated schemas from
packages/data/codegen/generated/ (itemdb-schema, mapdb-schema, etc.).
This directory was not copied into the Docker build context, causing
astro sync to fail with a missing module error and producing no build
output — which then surfaced as a COPY --from not-found error.
@github-actions
Copy link
Contributor

github-actions bot commented Mar 15, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Proactively add packages/data/codegen/generated/ COPY to the
astro-builder stage in all service Dockerfiles (discordsh, cryptothrone,
herbmail, memes, irc-gateway, mc) so they don't break when content
collections start importing generated schemas.
@h0lybyte h0lybyte merged commit 09ce3cc into dev Mar 15, 2026
5 checks passed
@h0lybyte h0lybyte deleted the trunk/docker-build-fix-1773548646 branch March 15, 2026 04:38
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