chore(docs): delete the in-app /docs subroute (use the official docs codebase)#1835
Conversation
…codebase) app/docs/ (account/posts/fans/base pages + their constants) is a hand-maintained copy of the API docs that duplicates and drifts from the official docs codebase (docs.recoupable.dev). chat#1834 had to hand-edit socials[].id -> social_id there and cubic immediately flagged the example/TS-interface still showing id — exactly the drift a second docs source causes. Delete app/docs/ and its now-dead-only support components (components/docs/*, used solely by these pages). No links to the /docs route exist. Documentation lives in the official docs codebase. Part of #1833. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
Next review available in: 17 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (13)
✨ 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 |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
No issues found across 13 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Auto-approved: Removes duplicate in-app docs pages and unused components. Verified no imports or links remain; no risk of breakage.
Re-trigger cubic
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 636cda66cc
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| import { codeExamples, exampleResponse, responseProperties } from "./constants"; | ||
| import { Language, languages } from "@/types/Docs"; | ||
|
|
||
| export default function AccountDocs() { |
There was a problem hiding this comment.
When a previously published URL such as /docs/account is requested after this deletion (same for the sibling /docs/posts, /docs/fans, and /docs/base pages), Next has no matching page and I checked next.config.mjs/vercel.json contain no replacement redirect, so users with bookmarks or external links fall through to the app 404 instead of the official docs. Since this change explicitly moves docs to docs.recoupable.dev, please keep a thin route or config redirect for /docs/:path*.
Useful? React with 👍 / 👎.
1 participant
Part of #1833 — flagged on chat#1834.
Why
app/docs/(the/docs/{account,posts,fans,base}pages + theirconstants.ts) is a hand-maintained copy of the API docs that duplicates and drifts from the official docs codebase (docs.recoupable.dev, the OpenAPI we reconcile against). chat#1834 had to hand-editsocials[].id→social_idinapp/docs/account/constants.ts, and cubic immediately flagged theexampleResponse/TS-interface still showingid— the exact drift a second docs source causes. Documentation lives in the official docs codebase, not this subroute.Change
app/docs/(8 files — 4 pages + constants).components/docs/(5 files —CodeBlock,LanguageSelector,ResponseTable,AuthSection,ViewAgentSection) — used only by those pages.Verified: nothing outside
app/docs/imports these; noLink/hrefto the/docsroute anywhere;tscshows no docs-related dangling imports. 13 files removed, no other code touched.🤖 Generated with Claude Code
Summary by cubic
Remove the in-app
/docsroute and doc-only components to stop duplicating the official docs and prevent drift. Aligns withrecoupable/chat#1833and fixes drift flagged inchat#1834.app/docs/*(/docs/{account,posts,fans,base}) andcomponents/docs/*used only by those pages./docslinks/imports and no dangling types (tscclean).Written for commit 636cda6. Summary will update on new commits.