fix(supervisor): copy retry-prisma-generate.mjs into the image build#4157
Conversation
The database packages' `generate` scripts shell out to `scripts/retry-prisma-generate.mjs`, but the supervisor Containerfile never copied it into the builder stage, so `pnpm run generate` fails with `Cannot find module '/app/scripts/retry-prisma-generate.mjs'`. Same fix as #4156 (webapp Dockerfile); the supervisor Containerfile is the other build that runs `pnpm run generate`. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
WalkthroughThis change adds a single COPY instruction in the supervisor Containerfile's builder stage to include the scripts/retry-prisma-generate.mjs file in the built image, which was previously missing. ChangesCompact Summary
Sequence Diagram(s)Not applicable; this change is a single Dockerfile COPY instruction addition with no observable runtime flow. Estimated code review effort: 1 (trivial) Related issues: None specified. Related PRs: None specified. Suggested labels: none Suggested reviewers: none Poem A rabbit hopped through builder's stage, 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Verified locally:
Ready for review. |
What
Adds the missing
COPY scripts/retry-prisma-generate.mjsto the supervisorContainerfilebuilder stage, beforeRUN pnpm run generate.Why
The
generatescripts ininternal-packages/databaseandinternal-packages/run-ops-databaseshell out toscripts/retry-prisma-generate.mjs. The supervisor build never copied that file into the image, sopnpm run generatefailed:This is the same failure class as #4156 (webapp Dockerfile). The supervisor
Containerfileis the only other build file that runspnpm run generate— the coordinator / docker-provider / kubernetes-provider Containerfiles don't, so this completes the fix.Verification
Local
docker buildof the supervisorContainerfilebuilder target — result appended below once the build completes.🤖 Generated with Claude Code