fix(app): return 404 for unknown top-level routes - #1130
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Warning Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption. 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 |
|
Closing as obsolete: this PR changes the deleted apps/app surface and its TanStack/API adapter contract. The hosted app has been retired. Preserving this branch and its history. |
Summary
Scope
Exactly five files: the organization TanStack adapter, its focused test, the app slug route, the app not-found component, and its focused route test. No app retirement, editorial copy, layout, canonical identity, Vercel dashboard, or unrelated cleanup is included.
Validation
The full app suite retains its documented unrelated baseline diagnostics: eight localStorage harness failures and one OAuth source-expectation failure. No changed-scope diagnostic was introduced.
Production gate
After protected merge, match the merge commit to READY production and verify arbitrary top-level unknown lightfast.ai paths return HTTP 404 with robots noindex, valid organization slugs remain valid, blog 404 behavior remains unchanged, and homepage identity/canonical/OG/discovery/assets remain healthy. Send supported Google and Bing indexing notifications only after those checks pass.