Skip to content

chore(mockups): document-navigation mockups with verification pass - #1321

Closed
BigSimmo wants to merge 5 commits into
mainfrom
claude/dazzling-blackwell-f348d0-ckwry8
Closed

chore(mockups): document-navigation mockups with verification pass#1321
BigSimmo wants to merge 5 commits into
mainfrom
claude/dazzling-blackwell-f348d0-ckwry8

Conversation

@BigSimmo

Copy link
Copy Markdown
Owner

Overlaps #1311. That PR holds the same seven mockup routes at f833621; this branch is f833621 plus one commit finishing the verification pass that was still outstanding. Only one of these two should land — close whichever you prefer, or ask me to move the fix commit onto claude/navigation-pane-mockups-0600af and close this one.

Summary

  • Seven design-scratch mockup routes exploring section navigation for the document viewer (Overview, High-yield summary, PDF preview, Pinned evidence, Indexed source text, Tables and diagrams, Indexing details): document-navigation-pane, document-phone-title, document-phone-title-refined, document-phone-fused-directions, document-phone-zero-chrome, document-navigation-contract, document-navigation-final.
  • Regenerated docs/site-map.md, which was missing six of the seven new routes.
  • Removed an unused useState import in document-phone-title-mockups.tsx that failed --max-warnings 0.
  • Applied Prettier to five mockup files that were failing the pre-push format guard.

Mockups only — no production route, component, or behaviour is touched. src/app/mockups/** is design-scratch and exempt from the button-wiring and route-reachability gates by existing convention.

Verification

  • npm run typecheck — clean
  • npm run lint — clean (0 errors, 0 warnings)
  • npm run format — clean
  • npm run sitemap:update — regenerated, diff committed

UI verification not run: mockup-only routes with no production surface; no shared chrome, token, routing, or styling change to exercise.

Verification not run beyond the above: npm run verify:pr-local was not run in this session. Everything it would add for this diff (build, client-bundle scan, RAG fixture/manifest validation) is unreachable from mockup-only files.

Risk and rollout

  • Risk: None to production. Additive routes under /mockups, plus a regenerated generated doc.
  • Rollback: Revert the branch; no data, schema, or config is touched.
  • Provider or production effects: None.

Notes

The design exploration is not finished. Two items are still open:

  1. A document-navigation-final-review variant (a two-version comparison refining document-navigation-final) was built in an earlier session but never committed, and was lost when that container was reclaimed. It is not in this branch.
  2. A follow-up request for a universal, config-driven navigation panel modelled on TherapyCompassNav is awaiting a direction decision.

Generated by Claude Code

@coderabbitai

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: fe4b86ec-c912-4772-8f9e-d2ddeff1e2e6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@supabase

supabaseBot commented Jul 28, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@BigSimmo@claude