Repository navigation
fix(web): pin Next.js back to 16.3.5 to restore /flows and /docs (prod 500s) - #129
khaliqgant wants to merge 1 commit into
Conversation
Reverts 646f9cc (Upgrade Next.js to 16.3.8, #126). Since that upgrade deployed, /flows and every /docs/<slug> page return 500 in production. Under 16.3.8 every prerendered page reports x-nextjs-cache: MISS: the static-assets incremental cache from @opennextjs/cloudflare 1.20.1 is no longer hit, so the Worker re-renders pages at request time. Pages that cannot render in the Workers runtime (docs MDX, /flows) fail with 500; the rest only look healthy because they happen to survive a live render. The #126 PR preview, which contains nothing but the version bump, already shows the same 500s. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
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 ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe ChangesWeb dependency update
Priority: ⬆️ High Estimated code review effort: 2 (Simple) | ~5 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to No actionable merge-blocking issue is established by this review. Confirm that the preview serves Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 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. A rabbit checks the package pin, Comment |
|
Preview deployed!
This is a Cloudflare Workers preview version of this PR's build. |
Production incident
https://agentrelay.com/flowsand everyhttps://agentrelay.com/docs/<slug>page (e.g./docs/introduction) have returned500 Internal Server Errorsince #126 (Next.js 16.3.5 → 16.3.8) deployed at 2026-10-02 04:40 UTC.This PR reverts 646f9cc. Only
web/package.jsonandpackage-lock.jsonchange, back to exactly what was deployed and healthy on 2026-10-01.Evidence
/flowsand/docs/introduction(https://8b743c8b-agentrelay-web.agent-workforce.workers.dev). So do the previews of test(flows): size the push-guard git tests' timeout to their real work #127 and Remove /chief page #128, which were built on top of it.x-nextjs-cache: MISS, including pages that return 200 (/,/blog,/about). The static-assets incremental cache is no longer hit, so the Worker renders at request time.web/open-next.config.tsand thePopulate static-assets cachestep indeploy.ymlboth document that a runtime render of docs pages fails in the Workers runtime.origin-web.agentrelay.comdirectly, so the router Worker is not involved./route-cache/<KIND>/<hash>/$<path>) opennextjs/opennextjs-aws#1269 ("Next 16.3.8: build-time prerendered entries are never read").Not yet captured: the runtime exception from the Worker logs. The cache-miss mechanism is inferred from response headers and the upstream report.
How to verify before merging
On this PR's preview URL, both of these should return 200:
/flows/docs/introductionFollow-ups (not in this PR)
@khaliqgant — needs the two reviews plus a human merge to deploy.
🤖 Generated with Claude Code
Note
Cursor Bugbot is generating a summary for commit 186d503. Configure here.
Summary by cubic
Pins Next.js back to 16.3.5 to restore /flows and /docs pages, which have returned 500 in production since the 16.3.8 upgrade.
With 16.3.8 the OpenNext adapter stops hitting the static-assets cache (every prerendered page answers
x-nextjs-cache: MISS), so pages that can't render in the Workers runtime fail at request time. The diff is only the version pin inweb/package.jsonandpackage-lock.json, reverting to the exact healthy build from 2026-10-01.@opennextjs/cloudflarereads the new prerender cache keys.Written for commit 186d503. Summary will update on new commits.
Agent Relay sessions
claudesessionfe983435· opened viagh pr create· last active 2026-10-02