Add a topic-grouped mockup index to mockups/README.md - #2423
Conversation
Catalogues all 84 mockup routes by topic with a status (prototype app, chosen design, active study, redirect, or superseded draft), so the accumulated design-scratch pages under src/app/mockups/ can be scanned at a glance instead of opened one by one. Identifies six early favourites-page studies as safe-to-remove (already recorded elsewhere in this file as superseded, confirmed here to have no other route depending on them) — pending separate approval before deletion.
Warning Review limit reachedNext included review available in 7 minutes. View limit detailsLimit details: You’ve used the included review currently available. Your 102 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (10)
Comment |
This pull request has been ignored for the connected project Preview Branches by Supabase. |
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_d0ea0bb5-00e5-4723-9953-4d7a3835cebb) |
favourites-command-console, favourites-command-desk, favourites-library-view, favourites-review-console, favourites-set-board, and favourites-set-navigator were already documented in mockups/README.md as replaced by favourites-phone-perfected. Confirmed by import search that no other route or test depends on them or their shared component folder (favourites-page-mockups/), and re-confirmed the same result with check:dead-code-candidate after deepening the shallow clone -- the handful of flagged symbol names (ReviewState, IconTile, PageHeader, variants, favouriteItems) all resolve to unrelated, differently-shaped symbols elsewhere in the codebase, not real dependents. Updates mockups/README.md's mockup index and regenerates docs/site-map.md accordingly.
…5wshq' into claude/organize-remote-mockups-15wshq
CI's check:repo-awareness-snapshot caught that data/repo-awareness-snapshot.json still listed routes for the six favourites mockup pages removed in the previous commit. The local pre-commit hook doesn't run this check, only sitemap:update, so it went stale locally.
…e-mockups-15wshq # Conflicts: # data/repo-awareness-snapshot.json
…e-mockups-15wshq # Conflicts: # data/repo-awareness-snapshot.json
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_2e223092-4565-4eb6-b948-399ba4b1b19f) |
…e-mockups-15wshq # Conflicts: # data/repo-awareness-snapshot.json
Uh oh!
There was an error while loading. Please reload this page.
Summary
mockups/README.mdcataloguing all 84 routes undersrc/app/mockups/by topic, with a status for each (prototype app / chosen design / active study / redirect / superseded draft / parallel draft with no recorded winner).favourites-command-console,favourites-command-desk,favourites-library-view,favourites-review-console,favourites-set-board,favourites-set-navigator) already documented as superseded byfavourites-phone-perfected, plus their shared component foldersrc/components/favourites-page-mockups/— confirmed by an import-graph search to have no other route or test depending on them.docs/site-map.mdanddata/repo-awareness-snapshot.jsonto reflect the removed routes.src/app/mockups/**is design-scratch, exempt from the wiring/reachability gates perAGENTS.md).Verification
npm run typecheck— clean, both before and after mergingmaininto this branch.npm run check:dead-code-candidate -- --diff origin/main— run after deepening the shallow clone. The handful of flagged symbol names (ReviewState,IconTile,PageHeader,variants,favouriteItems) were individually checked against the files/tests that reference them and all resolve to unrelated, differently-shaped symbols elsewhere in the codebase (name collisions), not real dependents of the removed files.npm run formatrun and committed.verify:pr-local/verify:ui/verify:release— the changed routes 404 in production and carry no user-facing behaviour; nothing in scope for those gates changed.Risk and rollout
src/lib/rag/**or any other RAG-ranking surface are touched.Clinical Governance Preflight
This PR is flagged clinical-risk only because it touches
data/repo-awareness-snapshot.json(a generated repo-metadata snapshot listing routes/docs counts, not clinical content). None of the below items are affected by this change; all remain true as before it. The last item is not applicable — no clinical decision-support behaviour changed in this PR — and is checked to record that it was considered, per the preflight's own convention for satisfied/not-applicable items.Clinical KB Database(sjrfecxgysukkwxsowpy)Notes
mockups/README.mdfor the removal rationale, and the "One-off studies" / "Parallel draft" sections for what was deliberately left alone (no written evidence of a chosen direction).Note
Low Risk
Documentation and dev-only mockup route removal only; no production wiring, auth, or clinical decision-support changes.
Overview
Adds a topic-grouped mockup index to
mockups/README.mdwith a status key (prototype app, chosen design, active study, redirect, superseded, parallel draft) so reviewers can scan routes by area without opening every folder.docs/site-map.mdanddata/repo-awareness-snapshot.jsonare refreshed to match (mockup page count 195 → 189).Favourites cleanup: six superseded design-scratch routes (
favourites-command-console,command-desk,library-view,review-console,set-board,set-navigator) and the sharedsrc/components/favourites-page-mockups/tree are removed;favourites-phone-perfectedremains as the recorded chosen direction (2026-08-27).No production routes or clinical behaviour change — removed paths are dev-only
/mockups/*surfaces that 404 in production.Reviewed by Cursor Bugbot for commit e1e4200. Configure here.