Add branch review records for Run PR sweep (#2322, #2321, #2316)

[BigSimmo]

Summary

• Records a Run PR maintenance sweep across three open PRs: Mockup: Dictionary Filter band always, Terms and A–Z under it #2322, Add three ChatGPT-style Answer result mockups #2321, Polish the dictionary catalogue Terms / Abbreviations control row #2316.
• Mockup: Dictionary Filter band always, Terms and A–Z under it #2322 and Polish the dictionary catalogue Terms / Abbreviations control row #2316 were behind main with no failing CI and no unresolved review threads — synced via a clean fast-forward update_pull_request_branch call, no conflicts.
• Add three ChatGPT-style Answer result mockups #2321 reported mergeable_state: dirty. git merge-tree confirmed a real conflict isolated to the generated docs/site-map.md (plus a clean auto-merge in src/app/mockups/mockups-layout-client.tsx). Resolved by merging origin/main in an isolated worktree, keeping --ours for site-map.md, then regenerating it with npm run sitemap:update (the pre-commit doc-sync hook ran clean), and pushing the merge commit.
• All three PRs' PR required aggregate is now green on their current heads.
• Docs-only change: three new immutable docs/branch-review-records/*.record.md files. No source, test, or config code touched.

Verification

• npm run ledger:append for each PR — immutable record files created and validated by the script itself
• Verification not run: npm run verify:pr-local — pure docs/ledger addition, no source change
• Verification not run: npm run verify:ui / verify:release — no UI or release change
• Verification not run: npm run eval:* — no RAG/answer-generation change
• Verification not run: npm run check:production-readiness — no production/clinical behaviour change

Risk and rollout

• Risk: none — additive docs-only ledger records, no production code paths touched
• Rollback: revert this PR
• Provider or production effects: None
• RAG impact: none

Clinical Governance Preflight

Not required: this PR only adds branch-review ledger records documenting a CI/merge-sync sweep. It does not touch ingestion, answer generation, retrieval/ranking, source rendering, document access, privacy, production env, or clinical output.

---

Generated by Claude Code

---

Note

Low Risk
Additive documentation-only ledger records; no production, auth, or data-handling paths are touched.

Overview
Adds three immutable branch-review ledger rows documenting a maintenance sweep of open PRs #2322, #2321, and #2316. No application, test, or config code is changed.

The records note that #2322 and #2316 were behind main with clean CI and no unresolved threads, then fast-forward synced. #2321 had a site-map.md merge conflict that was resolved by regenerating the sitemap after merging origin/main. All three heads are recorded as PR required green.

^{Reviewed by Cursor Bugbot for commit 5d4ad79. Configure here.}

[coderabbitai]

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (3)

• docs/branch-review-records/3156724bfd67ff8868ffb5e65b60aa070953ab022a9147a4bf35d732df66931e.record.md is excluded by !docs/**, !**/*.md
• docs/branch-review-records/8bc499ac7dde32f6a06543bccf35b98778ebaa785e8bc95b144c7e40763c14e2.record.md is excluded by !docs/**, !**/*.md
• docs/branch-review-records/a94911bf269d66b1dee46a927086585c035a3df3c2582244a082251553cb75a4.record.md is excluded by !docs/**, !**/*.md

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 70e6f3f0-d69e-4b82-b3da-96da204d563c

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]

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 ↗︎.

[cursor]

Approved. Cursor Bugbot completed successfully with no findings that need human review, and this docs-only ledger-record change does not require additional reviewers. No reviewers were assigned.

_{Sent by Cursor Approval Agent: Pull Request Router and Approver}