Repository navigation
feat: migrate hosting back to Cloudflare Workers - #83
legions-developer wants to merge 4 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe web app now has Cloudflare Worker configuration, build and deployment tasks, automated smoke checks, and deployment documentation. The README and Node.js requirement reflect the Cloudflare hosting setup. ChangesCloudflare deployment
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~25 minutes Change: Feature Merge Risk: ⚪ Minimal · up to The Cloudflare deployment path is ready to merge. The size guidance is corrected, and staging requires separately configured database and asset-storage values; no confirmed issue blocks the change. Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to The migration preserves the existing authentication and data-storage model, separates staging cache resources, and keeps production cutover explicit with Vercel available for rollback. No introduced security vulnerability was established. Deployment permissions, actual environment isolation, and authenticated cache behavior still require verification. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
Resilience and Maintainability Implications
Hardening Proposals
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 3 files. (1 skipped: 1 unsupported.)
✨ 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: 4e2432f2-e681-457f-ad8f-b1aaba75f4b3
⛔ Files ignored due to path filters (1)
yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (15)
.github/workflows/cloudflare-checks.ymlREADME.mdapps/web/.dev.vars.exampleapps/web/.gitignoreapps/web/next.config.tsapps/web/open-next.config.tsapps/web/package.jsonapps/web/public/_headersapps/web/scripts/smoke-cloudflare.mjsapps/web/src/app/layout.tsxapps/web/tsconfig.jsonapps/web/wrangler.jsoncdocs/cloudflare.mdpackage.jsonturbo.json
💤 Files with no reviewable changes (1)
- apps/web/src/app/layout.tsx
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.
This restores Cloudflare hosting using the current OpenNext Workers adapter, while preserving Neon, Better Auth, existing S3-compatible R2 assets, and browser PDF/local-invoice flows. It retains the storage changes from #41 instead of reverting that PR and discarding later application changes.
Changes:
docs/cloudflare.md.Validation:
/ingest/static/array.jsrewrite returned JavaScript successfully.Deployment requirements:
Existing issue found during validation: CJK strings are preserved in PDF text extraction but do not appear visually in the PDF. This was reproduced from the unchanged
mainPDF source atfc14a4awith the same PDF dependencies/fonts, without Next.js or Cloudflare involved. PDF source, dependencies, and fonts are unchanged by this migration; this finding should be addressed separately before claiming complete CJK rendering support.