fix: unrug Railway CI - copy patches dir and add .railwayignore#12099
fix: unrug Railway CI - copy patches dir and add .railwayignore#120990xApotheosis merged 2 commits intodevelopfrom
Conversation
…gnore Two fixes for Railway deployment failures: 1. Copy patches/ directory in Dockerfile before pnpm install - pnpm requires patch files during install even with --ignore-scripts because patched dependencies are referenced in the lockfile. 2. Add .railwayignore to reduce repo snapshot size - Railway's "Failed to snapshot repository" error was caused by the repo being ~74MB of tracked files. Excludes frontend source, images, tests, and other files not needed for the public-api Docker build. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds a Railway snapshot ignore file to exclude non-essential files from snapshots and updates two Dockerfiles to copy a Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 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 |
The swap-widget Dockerfile had the same missing patches/ dir bug as public-api — pnpm install fails with ENOENT for patched dependencies. Also add swap-widget *.md exception to .railwayignore. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix: unrug public-api Railway CI - copy patches dir and add .railwayignore Two fixes for Railway deployment failures: 1. Copy patches/ directory in Dockerfile before pnpm install - pnpm requires patch files during install even with --ignore-scripts because patched dependencies are referenced in the lockfile. 2. Add .railwayignore to reduce repo snapshot size - Railway's "Failed to snapshot repository" error was caused by the repo being ~74MB of tracked files. Excludes frontend source, images, tests, and other files not needed for the public-api Docker build. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: add patches dir to swap-widget Dockerfile and update .railwayignore The swap-widget Dockerfile had the same missing patches/ dir bug as public-api — pnpm install fails with ENOENT for patched dependencies. Also add swap-widget *.md exception to .railwayignore. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Description
Fix Railway CI deployments for public-api and swap-widget services.
Two issues were causing failures:
pnpm installENOENT — pnpm reads patch files frompatches/during install (even with--ignore-scripts), but the patches directory wasn't copied into the Docker build context beforepnpm install.railwayignorereduces this by excluding files no Dockerfile build needsChanges
packages/public-api/Dockerfile— AddedCOPY patches/ ./patches/beforepnpm install(done in prior commit)packages/swap-widget/Dockerfile— Same fix (same bug existed here too).railwayignore(new) — Excludessrc/, test dirs, images, dev tooling, docs, and test files. Verified safe for all 3 Railway services (public-api, swap-widget, unchained). Critical paths likepatches/,packages/,pnpm-lock.yaml,tsconfig*.json, andpublic/generated/generatedAssetData.jsonare NOT excluded.Issue (if applicable)
N/A — Railway deploy failures since March 3rd caused by pnpm migration
Risk
Low. Changes only affect Docker build context and Railway snapshot phase. No runtime behavior changes. The public-api Dockerfile fix was already verified working in production (deployment
120427a6).Testing
Engineering
developand verify Railway deployments succeed for public-api, swap-widget, and unchainedpnpm installstep no longer fails with ENOENT for patch filesOperations
Screenshots (if applicable)
N/A
Summary by CodeRabbit
Release Notes