fix(seo): restore crawler metadata rendering - #491
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
No issues found across 2 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Tip: cubic can generate docs of your entire codebase and keep them up to date. Try it here.
Re-trigger cubic
Description
Googlebot requests served from Vercel's cached shell lose page metadata after a React metadata-boundary resume mismatch. Update Next.js and its four companion packages from 16.2.6 to 16.3.6, which contains the upstream metadata-resume fixes (vercel/next.js#96364 and vercel/next.js#96367). Application code and API contracts are unchanged.
Type of change
How Has This Been Tested?
The Vercel preview passes all six repeated browser/Googlebot/Twitterbot report requests, including Googlebot cache HIT with complete metadata and no resume-error digest. Real Clerk development sign-in passes. Both mobile auth and mobile getSession return 401 anonymously and 200 with the existing test session; getSession reports isAuthenticated=true. Liveness and database readiness return 200. All CI gates pass, including 2104 unit tests (4 skipped) and 292 browser tests (2 skipped). The isolated local build used a static sitemap fallback; deployed preview checks use the existing staging database.
Checklist
Notes for reviewers
Merge deploys through the existing Vercel production pipeline after CI and preview validation. VPS cutover, DNS changes and database migration are outside this release.