Fix Dockerfile: add missing COPY line for plugins/fusion-plugin-linear-import#1940
Conversation
…ckage.json The Dockerfile's pre-install COPY block was missing a line for plugins/fusion-plugin-linear-import, which was added as a workspace member in dfb6e52 but never got a matching COPY line. Without it, that plugin's package.json (and its @types/node / @types/react devDependencies) is never staged before 'pnpm install --frozen-lockfile', causing the later 'pnpm build' step to fail with TS2688. Fixes Runfusion#1939
Greptile SummaryThis PR fixes the Docker build setup for the Linear import plugin. The main change is:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (1): Last reviewed commit: "fix: add missing COPY line for plugins/f..." | Re-trigger Greptile |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdds a single COPY instruction in the Dockerfile's builder stage to stage plugins/fusion-plugin-linear-import/package.json into the build context, ensuring it is present before pnpm install runs. ChangesDockerfile build context fix
Estimated code review effort: 1 (Trivial) | ~2 minutes Related issues: Fixes Dockerfile missing COPY line for plugins/fusion-plugin-linear-import causing TS2688 build failure ( Suggested labels: bug, docker Suggested reviewers: None 🐰 A tiny line, a copy true, 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
Fixes #1939.
Adds the single missing
COPY plugins/fusion-plugin-linear-import/package.json ./plugins/fusion-plugin-linear-import/package.jsonline to the Dockerfile's pre-install COPY block.Root cause
Commit dfb6e52 added
plugins/fusion-plugin-linear-importas a new pnpm workspace member, but didn't add a matching Dockerfile COPY line for its package.json. As a result that member's devDependencies (@types/node,@types/react) are never installed inside the image, and the laterpnpm buildstep fails with TS2688 once the compiler reaches that plugin.Fix
One line, alphabetically placed alongside the other
plugins/fusion-plugin-*COPY lines.Summary by CodeRabbit