fix: add rs-scripts to Docker build context#3455
Conversation
rs-scripts was added as a workspace member but not included in the Dockerfile COPY directives. This caused cargo chef prepare to fail with "No such file or directory" when trying to read packages/rs-scripts/Cargo.toml during metadata extraction. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Review GateCommit:
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Dockerfile was updated to include Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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 |
Issue Being Fixed
Docker builds fail at the
build-plannerstage with:Root Cause
packages/rs-scriptswas added as a workspace member inCargo.toml(PR #3391) but was not included in the Dockerfile'sCOPY --parentsdirectives. The three Rust build stages (build-planner, build-drive-abci, build-rs-dapi) all copy the workspaceCargo.tomlwhich referencesrs-scripts, but the package directory itself was missing from the Docker context.What was changed
Added
packages/rs-scripts \to all threeCOPY --parentsblocks in the Dockerfile (lines ~411, ~517, ~866).Breaking Changes
None.
🤖 Generated with Claude Code
Summary by CodeRabbit