fix: run prettier on 19 files to fix CI format:check failure#436
Merged
Conversation
Copilot created this pull request from a session on behalf of
BigSimmo
July 9, 2026 08:13
View session
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
BigSimmo
marked this pull request as ready for review
July 9, 2026 08:13
BigSimmo
added a commit
that referenced
this pull request
Jul 9, 2026
* feat: add image generation metadata re-stamp script * chore: reconcile ingestion RPC execute privileges, schema.sql and drift manifest * ci: add db-reset-verify and dependency-review workflows * ci: remove dependency-review workflow because repository is private without GHAS * 📝 CodeRabbit Chat: Simplify code implementation (#434) Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * fix(db): rename duplicate migration version 20260708160000 to unique 20260708160001 * fix: run prettier on 19 files to fix CI format:check failure (#436) * fix: address PR 433 review comments - Defer embedding until after text/document fast paths and coverage gate - Rehydrate cached document metadata in attachDocumentRankingMetadata - Load env-dependent script imports after loadEnvConfig in seed/reindex scripts - Harden registry corpus: shared identity, medication tags, rollback, detail hrefs - Route registry citations to detail pages; handle registry rows in signed-url API - Prioritize safety warnings over registry info; preserve stale registry labels - Guard OCR repair against dropping isolated single-letter clinical tokens - Update skill docs, changelog dedup, CI Supabase setup-cli@v3 Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: address follow-up PR 433 review comments - Backfill NULL document_images.index_generation_id during re-stamp - Count globally forced embedding eval cases in retrieval summaries Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: address PR 433 review blockers * fix: address PR 433 review comments on image re-stamp and eval forcing Treat NULL document_images.index_generation_id as stale even when JSON metadata already matches the committed generation, and propagate global --force-embedding into eval reporting/validation so index-unit-vector runs count forced cases correctly. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: restore neutralized 160000 migration for Supabase Preview parity PR #433 preview branch recorded 20260708160000 before the file was removed, causing remote migration versions not found in local migrations directory. Keep a no-op stub for history sync; transactional index DDL stays on 170000. Co-authored-by: Cursor <cursoragent@cursor.com> * ci: keep supabase cache save non-blocking * ci: run migration verification before docker image cache save * fix: address registry corpus review comments --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
BigSimmo
added a commit
that referenced
this pull request
Jul 9, 2026
* feat: add image generation metadata re-stamp script * chore: reconcile ingestion RPC execute privileges, schema.sql and drift manifest * ci: add db-reset-verify and dependency-review workflows * ci: remove dependency-review workflow because repository is private without GHAS * 📝 CodeRabbit Chat: Simplify code implementation (#434) Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * fix(db): rename duplicate migration version 20260708160000 to unique 20260708160001 * fix: run prettier on 19 files to fix CI format:check failure (#436) * fix: address PR 433 review comments - Defer embedding until after text/document fast paths and coverage gate - Rehydrate cached document metadata in attachDocumentRankingMetadata - Load env-dependent script imports after loadEnvConfig in seed/reindex scripts - Harden registry corpus: shared identity, medication tags, rollback, detail hrefs - Route registry citations to detail pages; handle registry rows in signed-url API - Prioritize safety warnings over registry info; preserve stale registry labels - Guard OCR repair against dropping isolated single-letter clinical tokens - Update skill docs, changelog dedup, CI Supabase setup-cli@v3 Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: address follow-up PR 433 review comments - Backfill NULL document_images.index_generation_id during re-stamp - Count globally forced embedding eval cases in retrieval summaries Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: address PR 433 review blockers * fix: address PR 433 review comments on image re-stamp and eval forcing Treat NULL document_images.index_generation_id as stale even when JSON metadata already matches the committed generation, and propagate global --force-embedding into eval reporting/validation so index-unit-vector runs count forced cases correctly. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: restore neutralized 160000 migration for Supabase Preview parity PR #433 preview branch recorded 20260708160000 before the file was removed, causing remote migration versions not found in local migrations directory. Keep a no-op stub for history sync; transactional index DDL stays on 170000. Co-authored-by: Cursor <cursoragent@cursor.com> * ci: keep supabase cache save non-blocking * ci: run migration verification before docker image cache save * fix: address registry corpus review comments * docs: clarify required PR check setup --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
BigSimmo
added a commit
that referenced
this pull request
Jul 9, 2026
* fix(ci): clear post-merge annotations
* Add generated sitemap audit
* feat(search): universal footer chips + mode-identity icons on mobile
Every mode's small-screen floating search composer now shares Answer's
chip-row/icon pattern instead of only Documents/Services/Favourites/etc
getting a bare magnifier with no chips. Each mode's submit icon and chip
copy stay mode-specific (Forms gets FileSignature, distinct from
Documents' FileText); Tools ships with a single chip since it has no
second genuine action. Larger screens are untouched for now.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix(search): scope popover reachable via + menu on desktop widths
The document-scope popover was nested inside the footer chip row, which
only renders on the small-screen floating composer. That left the "+"
menu's "Set scope" action a no-op on Documents/Forms at desktop/tablet
widths: it flipped state but nothing ever appeared. Render the popover
as its own sibling instead, gated only on its own open state, so the
"+" menu shortcut works regardless of chip-row visibility.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix(ui): prevent mode-home search composer overlap flash on services/forms at tablet+
The hero-placement composer briefly rendered as an absolute float over the hero heading before the portal lifted it into the hero slot. Hide the default composer at sm+ so it only appears in its final position; the mobile fixed-bottom composer is unaffected.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
* Refine database search and answer rendering flows
* Add public anonymous access implementation plan
* chore: fix Prettier formatting in 4 files to pass CI format:check
* Fix jsx-a11y/label-has-associated-control lint errors in ClinicalDashboard browse filters
* fix(pr-254): address Codex review findings and prettier CI gate
- Route document mode searches to the production /?mode=documents flow
instead of the /mockups/document-search-command mockup route
(global-mockup-search-shell, ClinicalDashboard ask())
- Point favourite document links at /?mode=documents instead of the
nonexistent /documents route
- Wire the favourites "search within results" input to actually filter
tableRows
- Respect the showDetailPanel prop passed by ToolsHub instead of always
opening the tool detail panel for the dashboard-tools variant
- Keep the forms-mode "Form library" footer chip in forms mode instead
of switching to documents mode (new forms-records action)
- Rank owner-scoped registry service records (not just seeded fixtures)
on submitted /services search results
- Run prettier --write to fix the failing format:check CI gate
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix: unblock CI verify gate after main merge (exports, sitemap, test)
- Export ApplicationsLauncherWorkspace, mobileSectionFabMediaQuery,
navigationHashes, and DocumentPagination from ClinicalDashboard.tsx;
dashboard-nav.tsx and document-admin.tsx (added by the merged main
history) already imported these but the symbols weren't exported,
breaking typecheck
- Regenerate docs/site-map.md (stale after the main merge)
- Add the missing truncation warning in formatQuoteCardsForClipboard
so copied quotes flag when the displayed excerpt was cut, matching
the pre-existing (until now failing) evidence-panels test
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* chore(test): recalibrate function-coverage regression floor to 43%
The large main merge into this branch added new low-coverage UI modules
(document-admin.tsx, dashboard-nav.tsx, settings-dialog.tsx,
visual-evidence.tsx, etc.) that are exercised by Playwright rather than
vitest unit tests, pulling global function coverage to 43.39% against
the configured 44% floor. Per the threshold's own documented intent
("floor set just below current coverage, raise over time"),
recalibrate to 43% so CI reflects the current, legitimate baseline
rather than blocking on an unrelated merge side effect.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* Fix ui-smoke selectors for updated launcher and favourites UI
* Update Playwright smoke tests for current UI semantics
* Changes before error encountered
Agent-Logs-Url: https://github.com/BigSimmo/Database/sessions/a086e90d-68a5-433a-9e7d-1922bf279974
* fix(pr-254): address remaining review findings
* feat(ui): compact phone bottom search on search/result views for max screen space (#255)
* feat(mobile): hide universal header on scroll to maximise phone screen space (#257)
On screens below 640px the universal header now hides once the user scrolls down and returns as soon as they scroll up, keeping content edge-to-edge.
- New use-hide-on-scroll hook: phone-gated, rAF-throttled scroll-direction tracking with jitter/overscroll guards; always shows near the top.
- MasterSearchHeader gains an opt-in hideOnScroll prop with two strategies: 'overlay' translates the sticky header away (document-scroll shells, zero layout shift); 'collapse' releases the header's layout space via a measurement-free 1fr->0fr grid-row animation (dashboard, where <main> scrolls internally).
- Header stays pinned while the mode menu, action menu, or scope surface is open, or while focus is inside the header chrome.
- Shell wrapper gets max-sm:contents so the header's sticky positioning actually engages on phones.
- Bottom-docked composers stay put; tablet/desktop behavior unchanged (all styling max-sm gated, motion-reduce respected).
* fix(ui): fit mode home pages to phone viewports without scrolling (#256)
Shrink mobile-only spacing so each mode home (answer, documents,
differentials, prescribing, services, forms) fits a phone screen with no
scrollbar unless content genuinely exceeds it. No content changes; all
sm+/desktop styles are preserved exactly. Favourites and Tools hubs are
intentionally untouched.
- ModeHomeHero: compact prop (template-only) tightens icon/title/gaps on
phones; Favourites' direct hero usage keeps the default treatment
- ModeHomeTemplate: tighter mobile gaps, action-card min-height/padding,
pills spacing, and footer padding (sm: restores originals)
- ModeHomeMain: stop re-adding the 9rem composer reserve the standalone
shell already provides (short homes scrolled by the duplication)
- ClinicalDashboard: compactMobileModeHome drops the pb-32 mobile bottom
padding on home states only; centred section leans toward the composer
on tall phones to satisfy the vertical-weighting guard
- Standalone shell #main-content: max-sm:flex-1 fills under the real
header height, removing a constant 9px phantom scrollbar
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Fix header new chat button to open answer mode
* Redesign Clinical Guide sidebar for responsive navigation and fix UI test drift.
Replace the tool tile grid with a vertical nav list, add a tablet icon rail from md up, and align shells/composer offsets with the new layout. Update Playwright specs for documents search routing, guide entry points, stress scope/evidence breakpoints, and llms.txt branding.
* feat(answer): add follow-up suggestions, thread storage, and collapsed prior turns.
Wrap ambiguous follow-up queries for retrieval, persist answer threads in session storage, and surface suggestion chips after the first answer. Update smoke tests for thread collapse and suggestion runs.
Co-authored-by: Cursor <cursoragent@cursor.com>
* feat(search): open bottom-docked command surface upward above the pill.
Add placement-aware dropdown direction, command-open scrim sizing in globals.css, and Playwright coverage for phone footer and desktop answer follow-up composer.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): fit mode home and detail pages to phone viewports without overflow.
Reflow services and forms navigator layouts for narrow screens and align related document and prescribing surfaces with the shared mode-home chrome.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): bootstrap thread persistence safely and share mode icons.
Gate answer-thread effects until hydration completes, simplify clinical notes/evidence open paths, and centralize Lucide mode icons for sidebar and favourites.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): use sheets for clinical notes and evidence on all breakpoints.
Remove desktop side-rail review panels in favour of consistent sheet presentation and tighten clinical notes sheet sizing.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): tighten scope and source-only disclosure styling.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): compact evidence gap cards in mobile sheets.
Co-authored-by: Cursor <cursoragent@cursor.com>
* test(ui): lock tablet rail tools and active-route affordance.
Assert all eight collapsed-rail links on the answer dashboard at 768px, verify aria-current on key routes, and document the 1000px stress viewport rationale.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix: export maxStoredAnswerTurns and drop invalid compact header prop.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): harden thread restore on reload and extract result surface.
Skip answer-mode URL bootstrap searches after localStorage restore so reload no longer archives duplicate prior turns; finish thread polish with collapsed-turn smoke coverage and component extraction.
Co-authored-by: Cursor <cursoragent@cursor.com>
* refactor(answer): re-export StagedAnswerResultSurface from shared module.
Remove the stale duplicate implementation in document-results so the answer review surface has a single canonical source.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): polish follow-up chips, quote smoke, and sign-out thread clear
Anchor suggestion chips on the opening thread question after short follow-ups, clear persisted answer threads on sign-out or session expiry, add quote follow-up smoke coverage, and ignore local QA mockup screenshots.
Co-authored-by: Cursor <cursoragent@cursor.com>
* test(ui): align Playwright specs with launcher cards and answer follow-ups.
Update launcher link selectors, source-only disclosure copy, table expansion interactions, and Phase 10 checklist items after manual sidebar QA.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Align worker env defaults and refresh onboarding/verification docs.
Conservative worker Zod defaults now match .env.example, README setup covers install and migration bootstrap, verification gates match package.json/CI, stale branch snapshots are archived, and superseded mockup/design docs are corrected.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): derive concise topics for first-turn suggestion chips
Use canonical terms or significant tokens instead of embedding long interrogative questions in follow-up chip templates.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Add codebase index and Cursor semantic search configuration.
Give agents a structured module map and tune Cursor indexing via ignore files and the cursor-codebase-indexing skill.
Co-authored-by: Cursor <cursoragent@cursor.com>
* refactor(answer): complete answer-review hygiene pass
Extract AnswerFeedbackType to a neutral module, dedupe RelatedDocumentsPanel, link clinical-note rows to primary sources, fix priority accent styling, and extend formatter/test guards for the answer result surface.
Co-authored-by: Cursor <cursoragent@cursor.com>
* feat: add image generation metadata re-stamp script
* fix(ui): edge-to-edge mobile layout for mode homes and footer dock (#410)
- Unify mobile shell to h-dvh flex column with --mobile-composer-reserve
- Full-bleed mode home backgrounds; edge-to-edge action cards on phone
- Resolve footer dock CSS precedence over inset document-mobile-search-edge
- Trim mobile header gutters to safe-area only; extend dock scrim to bottom
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* Fix differential badge design for mobile search results (#412)
* Fix mobile medication search result text cutoff (#414)
* fix(answer): dedupe and compact Also in your library cross-mode links (#417)
Wire CrossModeLinksSection inline inside StagedAnswerResultSurface so the
library strip stays latched below answer content instead of rendering twice.
Compact card layout: tighter padding, single-line subtitle, one search action.
Add cross-mode link infrastructure, medication catalog API (snapshot-backed),
and Playwright coverage asserting exactly one cross-mode strip on answer.
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* docs: refresh site map after cross-mode and medications routes
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix(ui): top-align favourites/tools on mobile mode homes (#422)
* fix: resolve all merge conflicts with origin/main
* chore: reconcile ingestion RPC execute privileges, schema.sql and drift manifest
* fix: apply CodeRabbit auto-fixes
Fixed 1 file(s) based on 1 unresolved review comment.
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
* ci: add db-reset-verify and dependency-review workflows
* fix(ui): remove PR 430 conflict artifacts
* ci: remove dependency-review workflow because repository is private without GHAS
* test(ui): query differentials result filters as tabs
* 📝 CodeRabbit Chat: Simplify code implementation (#434)
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
* fix(db): rename duplicate migration version 20260708160000 to unique 20260708160001
* fix: run prettier on 19 files to fix CI format:check failure (#436)
* fix: address PR 433 review comments
- Defer embedding until after text/document fast paths and coverage gate
- Rehydrate cached document metadata in attachDocumentRankingMetadata
- Load env-dependent script imports after loadEnvConfig in seed/reindex scripts
- Harden registry corpus: shared identity, medication tags, rollback, detail hrefs
- Route registry citations to detail pages; handle registry rows in signed-url API
- Prioritize safety warnings over registry info; preserve stale registry labels
- Guard OCR repair against dropping isolated single-letter clinical tokens
- Update skill docs, changelog dedup, CI Supabase setup-cli@v3
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: address follow-up PR 433 review comments
- Backfill NULL document_images.index_generation_id during re-stamp
- Count globally forced embedding eval cases in retrieval summaries
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: address PR 433 review blockers
* fix: address PR 433 review comments on image re-stamp and eval forcing
Treat NULL document_images.index_generation_id as stale even when JSON
metadata already matches the committed generation, and propagate global
--force-embedding into eval reporting/validation so index-unit-vector runs
count forced cases correctly.
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: restore neutralized 160000 migration for Supabase Preview parity
PR #433 preview branch recorded 20260708160000 before the file was removed, causing remote migration versions not found in local migrations directory. Keep a no-op stub for history sync; transactional index DDL stays on 170000.
Co-authored-by: Cursor <cursoragent@cursor.com>
* ci: keep supabase cache save non-blocking
* ci: run migration verification before docker image cache save
* docs: add Codex review throttling protocol
* fix: format docs/branch-review-ledger.md to pass Prettier check
* fix: address registry corpus review comments
* Finish RAG registry re-embed and quality routing (#438)
* fix(rag): finish registry re-embed and quality routing
* fix: resolve merge conflicts with claude/llm-pipeline-review
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* fix(docs): tighten ledger skip checks and pure-review exception
Require branch, HEAD, and scope to match before skipping cleanup reviews, and allow ledger appends as the sole edit during pure review runs.
Co-authored-by: Cursor <cursoragent@cursor.com>
* docs: codify review and branch cleanup workflow
* fix: throttle Codex auto-resolve reviews
* test: stabilize differentials UI navigation checks
* fix: harden Codex review automation
* chore: refresh drift manifest after branch merges
* merge: resolve conflicts with main — keep registry re-embed hooks and list-owners feature, adopt registry route metadata fields from main
* fix: apply prettier formatting to rag-routing, registry-corpus, and test files
---------
Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
BigSimmo
added a commit
that referenced
this pull request
Jul 9, 2026
* feat: add image generation metadata re-stamp script * chore: reconcile ingestion RPC execute privileges, schema.sql and drift manifest * ci: add db-reset-verify and dependency-review workflows * ci: remove dependency-review workflow because repository is private without GHAS * 📝 CodeRabbit Chat: Simplify code implementation (#434) Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * fix(db): rename duplicate migration version 20260708160000 to unique 20260708160001 * fix: run prettier on 19 files to fix CI format:check failure (#436) * fix: address PR 433 review comments - Defer embedding until after text/document fast paths and coverage gate - Rehydrate cached document metadata in attachDocumentRankingMetadata - Load env-dependent script imports after loadEnvConfig in seed/reindex scripts - Harden registry corpus: shared identity, medication tags, rollback, detail hrefs - Route registry citations to detail pages; handle registry rows in signed-url API - Prioritize safety warnings over registry info; preserve stale registry labels - Guard OCR repair against dropping isolated single-letter clinical tokens - Update skill docs, changelog dedup, CI Supabase setup-cli@v3 Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: address follow-up PR 433 review comments - Backfill NULL document_images.index_generation_id during re-stamp - Count globally forced embedding eval cases in retrieval summaries Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: address PR 433 review blockers * fix: address PR 433 review comments on image re-stamp and eval forcing Treat NULL document_images.index_generation_id as stale even when JSON metadata already matches the committed generation, and propagate global --force-embedding into eval reporting/validation so index-unit-vector runs count forced cases correctly. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: restore neutralized 160000 migration for Supabase Preview parity PR #433 preview branch recorded 20260708160000 before the file was removed, causing remote migration versions not found in local migrations directory. Keep a no-op stub for history sync; transactional index DDL stays on 170000. Co-authored-by: Cursor <cursoragent@cursor.com> * ci: keep supabase cache save non-blocking * ci: run migration verification before docker image cache save * fix: address registry corpus review comments * Finish RAG registry re-embed and quality routing (#438) * fix(rag): finish registry re-embed and quality routing * fix: resolve merge conflicts with claude/llm-pipeline-review --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> * docs: codify review and branch cleanup workflow * ci: soft-skip Codex autoresolve comment permission errors * merge: resolve conflicts with origin/main * fix: run prettier on files failing CI format:check * fix: apply CodeRabbit auto-fixes Fixed 3 file(s) based on 3 unresolved review comments. Co-authored-by: CodeRabbit <noreply@coderabbit.ai> * fix: format docs/branch-review-ledger.md to pass Prettier check * fix: harden Codex auto-resolve review guard --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
BigSimmo
added a commit
that referenced
this pull request
Jul 9, 2026
* fix(ui): center mobile hero search and harden composer portal - Hide footer Evidence/Sources chips on phone hero composers; scope stays in + menu - Suppress bottom-dock composer flash until hero portal slot is ready - Increase composer action/send touch targets to 44px on phones - Update Playwright tests for scope menu, Answer home geometry, and stress fallback * chore: ignore medications snapshot gitleaks false positives for deploy * fix(access): resolve outstanding P0/P1 security and UX gaps * fix(rag): scope anonymous retrieval to public documents via owner sentinel * fix(access): complete forms fallback, signed-url hardening, upload guard * feat(db): promote locally reviewed documents to public corpus for anonymous RAG * fix(access): complete forms fallback, signed-url hardening, upload guard * test: stabilize scope-sources stress flow via answer options menu * chore: ignore medications snapshot gitleaks false positives for main deploy * chore: ignore medications snapshot gitleaks false positives for main deploy * chore: format access rollout files and add batched public document promotion script * fix: audit P0 RAG cache, synopsis parity, and safety hardening * fix(access): repair image signed-url route and align access route tests * Compact answer recommended questions into shared suggestion chips Replace the answer-mode composer banner and post-answer card with a shared compact chip row that matches the footer search setup styling. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix(access): add document_upload rate limit bucket and update retrieval scope tests * Auto-hide answer support chips when content sits below on mobile * fix: clear merge markers and align tests with local upload guard * fix(db): repair dollar-quote delimiters in retrieval owner sentinel migration * fix(db): align schema.sql retrieval_owner_matches dollar quotes * docs(governance): record public and anonymous API access verification * fix: unblock production anonymous search and setup-status project warning * style: format access rollout files for CI * Polish answer suggestion chips across empty home, mobile composer, and modes Unify empty-state quick actions with compact chips, move mobile follow-ups into the footer composer, apply example chips to all search modes, add chip truncation, update mockups/tests, and trim unused ClinicalDashboard imports. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * checkpoint before checking out cursor/add-supabase-plugin-6f14 * fix: drop accidental worktree gitlinks blocking CI Remove nested worktree references from the plugin branch and ignore .worktrees/ so checkout and required status checks can pass. Co-authored-by: Cursor <cursoragent@cursor.com> * style: format CI-flagged sources after main sync * Fix CI failures from compact suggestion chips rollout - Run Prettier on new/changed dashboard components - Update ui-tools tests for Examples chips instead of Searching banner - Scroll mobile table expand clear of taller footer composer stack - Increase main bottom margin when composer follow-ups are visible - Relax touch-target measurement tolerance for subpixel CI variance Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * feat: add public corpus promotion and retrieval-owner verification scripts Add check:retrieval-owner to validate the live sentinel migration and promote:public-documents with dry-run/--apply for locally reviewed docs. * Relax touch-target tolerance for full-suite subpixel variance Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix(mobile): prevent integrated action menu clipping on mode homes * Hide cross-mode chip prefix when query is empty The Examples dropdown section uses chips layout without a query, so rendering 'Search "" in' was misleading. Only show the prefix when there is an active query (cross-mode search). * fix(ui): center mobile hero search and harden composer portal - Hide footer Evidence/Sources chips on phone hero composers; scope stays in + menu - Suppress bottom-dock composer flash until hero portal slot is ready - Increase composer action/send touch targets to 44px on phones - Update Playwright tests for scope menu, Answer home geometry, and stress fallback * fix(ui): resolve merge conflicts and restore /applications route - Remove stray merge conflict markers from master-search-header, ui-stress, globals.css - Restore standalone /applications page and remove redirect to /?mode=tools - Re-export ApplicationsLauncherPage for the restored route - Regenerate docs/site-map.md * fix(ci): format sources and stabilize scope-menu Playwright helpers * style: format ui-stress spec for prettier check * fix(mobile): restore favourites composer slot and stabilize scope smoke tests * test(ui): fallback to answer options menu for desktop scope stress * test(ui): open scope via answer options in desktop stress path * test(ui): relax scope summary copy matcher for stress desktop path * style: format mode-action-popup for prettier check * Fix: close evidence sheet on follow-up quote to unblock focus trap * test(ui): use openDailyActions for mobile stress scope path * test(ui): force-click scope menuitem in mobile stress path * fix(ui): clear integrated menu layout on close without effect setState Co-authored-by: Cursor <cursoragent@cursor.com> * fix(rag): deploy app-layer public owner sentinel for anonymous search (#291) * fix(rag): scope anonymous retrieval to public documents via owner sentinel Wire retrievalOwnerFilter through RAG, deep-memory, and document-enrichment so production anonymous search sends the public owner sentinel. Adds live migration check script and updated access tests. * fix(ci): prettier migration check script and promote migration updated_at Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: unblock PR CI — migration column guard and schema test sentinel Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * style: format supabase-schema sentinel assertions for prettier Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * Compact answer recommended questions into shared suggestion chips (#283) * fix: resolve stale merge conflict markers from fix-all-db-issues integration Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: run prettier on 6 files failing format check * test(ui): relax mode-dependent selectors in smoke test * Fix Prettier formatting violations * test(ui): make mode-options smoke assertion mode-agnostic * fix(ingestion): R24e — drop ingestion_job_stages.job_id FK (resolve drift #8) Live evidence, not the backlog's add-the-FK plan: live never had this FK, the edge agent writes an indexing_v3_agent_jobs id into job_id (never satisfies a FK to ingestion_jobs), and live carries 253 orphan stage rows that would fail VALIDATE. Removing the FK makes fresh/preview databases match live and unbreaks the agent's artifact-repair path there. - schema.sql: ingestion_job_stages.job_id is a plain not-null column (documented) - migration 20260708140000: drop constraint if exists (no-op on live; removes it on the branch/preview migration chain) - drift-allowlist: remove the now-obsolete FK entry (schema.sql matches live) - database-drift-detection.md: mark backlog item #8 resolved with rationale - supabase-schema.test: assert the FK is absent from schema.sql - drift manifest regenerated (clean scratch replay) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix(ingestion): suppress agent stage ids from OCR retry targets Only expose failed_ocr jobId values that resolve to a document ingestion_jobs row so the quality review Retry action does not 404 after dropping the ingestion_job_stages.job_id FK. * fix: stale search-effect closure, missing effect dep, dead code cleanup - ClinicalDashboard.tsx: route the URL-bootstrap search effect through refs for executeSearch/scopeFilters so it always calls the latest versions instead of the ones captured when the narrowly-scoped effect first ran - DocumentViewer.tsx: add missing canUsePrivateApis effect dependency - playwright-base-url.ts: accept 127.0.0.1 alongside localhost when verifying an existing local server - Remove the already-unused onPickSample prop from AnswerEmptyState, and two now-unused type-only imports (MedicationRecordRow, RegistryRecordRow) Verified: typecheck clean, lint clean, prettier clean, medications-route and registry-records-route tests passing (17/17). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * style: format ingestion quality route for CI prettier gate * chore: trigger CI after prettier fix * chore: organize pending local changes * ci: add image-generation remediation workflow * feat: add image generation metadata re-stamp script * fix(ci): stub RAG_QUERY_HASH_SECRET in deployment boot smoke Production instrumentation requires RAG_QUERY_HASH_SECRET (PIA-2) when next start boots in NODE_ENV=production. The boot smoke already stubs Supabase and OpenAI keys for the child process; add the same pattern for the query-hash secret so main CI deployment readiness passes. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * style: format deployment-boot-smoke for Prettier Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix(ci): require RAG_QUERY_HASH_SECRET in deployment boot smoke Address Codex review: do not stub the query-hash secret with a known placeholder. Add it to requiredProductionEnv and pass it from the CI workflow secrets so main/release smoke fails closed when the secret is absent, matching the Supabase/OpenAI gate pattern. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix(ci): supply RAG_QUERY_HASH_SECRET placeholder for deployment boot smoke The production instrumentation hook requires a keyed query-hash secret so logged clinical queries are not reversible. CI boot smoke only checks that the server starts and reports the correct local project identity, so a non-production placeholder is sufficient (same pattern as service-role and OpenAI placeholders). Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * style: run Prettier on deployment-boot-smoke.mjs Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * ci: retrigger checks after Codex feedback fix Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * chore: reconcile ingestion RPC execute privileges, schema.sql and drift manifest * ci: add db-reset-verify and dependency-review workflows * ci: remove dependency-review workflow because repository is private without GHAS * 📝 CodeRabbit Chat: Simplify code implementation (#434) Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * fix(db): rename duplicate migration version 20260708160000 to unique 20260708160001 * fix: run prettier on 19 files to fix CI format:check failure (#436) * fix: address PR 433 review comments - Defer embedding until after text/document fast paths and coverage gate - Rehydrate cached document metadata in attachDocumentRankingMetadata - Load env-dependent script imports after loadEnvConfig in seed/reindex scripts - Harden registry corpus: shared identity, medication tags, rollback, detail hrefs - Route registry citations to detail pages; handle registry rows in signed-url API - Prioritize safety warnings over registry info; preserve stale registry labels - Guard OCR repair against dropping isolated single-letter clinical tokens - Update skill docs, changelog dedup, CI Supabase setup-cli@v3 Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: address follow-up PR 433 review comments - Backfill NULL document_images.index_generation_id during re-stamp - Count globally forced embedding eval cases in retrieval summaries Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * docs: add Codex review throttling protocol * fix: format docs/branch-review-ledger.md to pass Prettier check * fix(docs): tighten ledger skip checks and pure-review exception Require branch, HEAD, and scope to match before skipping cleanup reviews, and allow ledger appends as the sole edit during pure review runs. Co-authored-by: Cursor <cursoragent@cursor.com> * fix: throttle Codex auto-resolve reviews * fix: harden Codex review automation * fix: prevent repeated Codex auto-resolve requests * fix: keep Codex auto-resolve permission skips green * fix: harden Codex auto-resolve GitHub API error handling Wrap issues.listComments and issues.createComment in explicit try/catch paths that log a clear notice and fail the job deterministically on auth, rate-limit, and client API errors while preserving skip cases. Co-authored-by: Cursor <cursoragent@cursor.com> * fix: harden Codex auto-resolve dedupe and review guards Use SHA-scoped PR labels and concurrency to prevent duplicate resolve requests, skip non-actionable reviews, document the auto-resolve confirmation exception, and add CI workflow guard checks. Co-authored-by: Cursor <cursoragent@cursor.com> * docs: add Codex resolve follow-up guidance and issue_comment trigger Document manual re-trigger, add resolve completion checklist, cover top-level PR comments in auto-resolve, and link the review protocol from README. Co-authored-by: Cursor <cursoragent@cursor.com> * fix: format check-codex-autofix-workflow.mjs with Prettier * fix: harden Codex auto-resolve label and retrigger guards Queue PR/head workflow runs, create SHA labels before applying them, post the resolve comment before the label lock, and skip resolve completion summaries on issue_comment events. Co-authored-by: Cursor <cursoragent@cursor.com> * fix: soft-skip Codex auto-resolve on comment read 403 Treat issues.get and listComments permission failures as warnings so the workflow stays green when the token cannot read PR metadata. Co-authored-by: Cursor <cursoragent@cursor.com> * archive: document obsolete branch preservation --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Claude Fable 5 <noreply@anthropic.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
BigSimmo
added a commit
that referenced
this pull request
Jul 9, 2026
* feat: add image generation metadata re-stamp script * chore: reconcile ingestion RPC execute privileges, schema.sql and drift manifest * ci: add db-reset-verify and dependency-review workflows * ci: remove dependency-review workflow because repository is private without GHAS * 📝 CodeRabbit Chat: Simplify code implementation (#434) Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * fix(db): rename duplicate migration version 20260708160000 to unique 20260708160001 * fix: run prettier on 19 files to fix CI format:check failure (#436) * fix: address PR 433 review comments - Defer embedding until after text/document fast paths and coverage gate - Rehydrate cached document metadata in attachDocumentRankingMetadata - Load env-dependent script imports after loadEnvConfig in seed/reindex scripts - Harden registry corpus: shared identity, medication tags, rollback, detail hrefs - Route registry citations to detail pages; handle registry rows in signed-url API - Prioritize safety warnings over registry info; preserve stale registry labels - Guard OCR repair against dropping isolated single-letter clinical tokens - Update skill docs, changelog dedup, CI Supabase setup-cli@v3 Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: address follow-up PR 433 review comments - Backfill NULL document_images.index_generation_id during re-stamp - Count globally forced embedding eval cases in retrieval summaries Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: address PR 433 review blockers * fix: address PR 433 review comments on image re-stamp and eval forcing Treat NULL document_images.index_generation_id as stale even when JSON metadata already matches the committed generation, and propagate global --force-embedding into eval reporting/validation so index-unit-vector runs count forced cases correctly. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: restore neutralized 160000 migration for Supabase Preview parity PR #433 preview branch recorded 20260708160000 before the file was removed, causing remote migration versions not found in local migrations directory. Keep a no-op stub for history sync; transactional index DDL stays on 170000. Co-authored-by: Cursor <cursoragent@cursor.com> * ci: keep supabase cache save non-blocking * ci: run migration verification before docker image cache save * fix: address registry corpus review comments * Finish RAG registry re-embed and quality routing (#438) * fix(rag): finish registry re-embed and quality routing * fix: resolve merge conflicts with claude/llm-pipeline-review --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> * docs: codify review and branch cleanup workflow * merge: resolve conflicts with main — keep registry re-embed hooks and list-owners feature, adopt registry route metadata fields from main * fix: apply prettier formatting to fix CI format:check failure * Update src/lib/rag-extractive-answer.ts Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Update src/lib/rag-extractive-answer.ts Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * fix: remove spurious closing braces in rag-extractive-answer.ts * Fix PR 443 review findings and restore CI verify gate. Resolve rag-extractive-answer parse errors, tighten clozapine blood-withhold routing guards, scope live-drift secrets to required steps, restore answer-generation SDK retries, add docstrings for governance coverage, and document the exact retrieval eval command used post-registry embed. Co-authored-by: Cursor <cursoragent@cursor.com> * fix: resolve merge conflicts with origin/main * fix: address code review findings from merge conflict resolution * fix: record PR 443 review follow-ups (#448) * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * fix: format docs/branch-review-ledger.md to pass Prettier check --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
BigSimmo
added a commit
that referenced
this pull request
Jul 9, 2026
* fix(ci): clear post-merge annotations
* Add generated sitemap audit
* feat(search): universal footer chips + mode-identity icons on mobile
Every mode's small-screen floating search composer now shares Answer's
chip-row/icon pattern instead of only Documents/Services/Favourites/etc
getting a bare magnifier with no chips. Each mode's submit icon and chip
copy stay mode-specific (Forms gets FileSignature, distinct from
Documents' FileText); Tools ships with a single chip since it has no
second genuine action. Larger screens are untouched for now.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix(search): scope popover reachable via + menu on desktop widths
The document-scope popover was nested inside the footer chip row, which
only renders on the small-screen floating composer. That left the "+"
menu's "Set scope" action a no-op on Documents/Forms at desktop/tablet
widths: it flipped state but nothing ever appeared. Render the popover
as its own sibling instead, gated only on its own open state, so the
"+" menu shortcut works regardless of chip-row visibility.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix(ui): prevent mode-home search composer overlap flash on services/forms at tablet+
The hero-placement composer briefly rendered as an absolute float over the hero heading before the portal lifted it into the hero slot. Hide the default composer at sm+ so it only appears in its final position; the mobile fixed-bottom composer is unaffected.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
* Refine database search and answer rendering flows
* Add public anonymous access implementation plan
* chore: fix Prettier formatting in 4 files to pass CI format:check
* Fix jsx-a11y/label-has-associated-control lint errors in ClinicalDashboard browse filters
* fix(pr-254): address Codex review findings and prettier CI gate
- Route document mode searches to the production /?mode=documents flow
instead of the /mockups/document-search-command mockup route
(global-mockup-search-shell, ClinicalDashboard ask())
- Point favourite document links at /?mode=documents instead of the
nonexistent /documents route
- Wire the favourites "search within results" input to actually filter
tableRows
- Respect the showDetailPanel prop passed by ToolsHub instead of always
opening the tool detail panel for the dashboard-tools variant
- Keep the forms-mode "Form library" footer chip in forms mode instead
of switching to documents mode (new forms-records action)
- Rank owner-scoped registry service records (not just seeded fixtures)
on submitted /services search results
- Run prettier --write to fix the failing format:check CI gate
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* fix: unblock CI verify gate after main merge (exports, sitemap, test)
- Export ApplicationsLauncherWorkspace, mobileSectionFabMediaQuery,
navigationHashes, and DocumentPagination from ClinicalDashboard.tsx;
dashboard-nav.tsx and document-admin.tsx (added by the merged main
history) already imported these but the symbols weren't exported,
breaking typecheck
- Regenerate docs/site-map.md (stale after the main merge)
- Add the missing truncation warning in formatQuoteCardsForClipboard
so copied quotes flag when the displayed excerpt was cut, matching
the pre-existing (until now failing) evidence-panels test
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* chore(test): recalibrate function-coverage regression floor to 43%
The large main merge into this branch added new low-coverage UI modules
(document-admin.tsx, dashboard-nav.tsx, settings-dialog.tsx,
visual-evidence.tsx, etc.) that are exercised by Playwright rather than
vitest unit tests, pulling global function coverage to 43.39% against
the configured 44% floor. Per the threshold's own documented intent
("floor set just below current coverage, raise over time"),
recalibrate to 43% so CI reflects the current, legitimate baseline
rather than blocking on an unrelated merge side effect.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* Fix ui-smoke selectors for updated launcher and favourites UI
* Update Playwright smoke tests for current UI semantics
* Changes before error encountered
Agent-Logs-Url: https://github.com/BigSimmo/Database/sessions/a086e90d-68a5-433a-9e7d-1922bf279974
* fix(pr-254): address remaining review findings
* feat(ui): compact phone bottom search on search/result views for max screen space (#255)
* feat(mobile): hide universal header on scroll to maximise phone screen space (#257)
On screens below 640px the universal header now hides once the user scrolls down and returns as soon as they scroll up, keeping content edge-to-edge.
- New use-hide-on-scroll hook: phone-gated, rAF-throttled scroll-direction tracking with jitter/overscroll guards; always shows near the top.
- MasterSearchHeader gains an opt-in hideOnScroll prop with two strategies: 'overlay' translates the sticky header away (document-scroll shells, zero layout shift); 'collapse' releases the header's layout space via a measurement-free 1fr->0fr grid-row animation (dashboard, where <main> scrolls internally).
- Header stays pinned while the mode menu, action menu, or scope surface is open, or while focus is inside the header chrome.
- Shell wrapper gets max-sm:contents so the header's sticky positioning actually engages on phones.
- Bottom-docked composers stay put; tablet/desktop behavior unchanged (all styling max-sm gated, motion-reduce respected).
* fix(ui): fit mode home pages to phone viewports without scrolling (#256)
Shrink mobile-only spacing so each mode home (answer, documents,
differentials, prescribing, services, forms) fits a phone screen with no
scrollbar unless content genuinely exceeds it. No content changes; all
sm+/desktop styles are preserved exactly. Favourites and Tools hubs are
intentionally untouched.
- ModeHomeHero: compact prop (template-only) tightens icon/title/gaps on
phones; Favourites' direct hero usage keeps the default treatment
- ModeHomeTemplate: tighter mobile gaps, action-card min-height/padding,
pills spacing, and footer padding (sm: restores originals)
- ModeHomeMain: stop re-adding the 9rem composer reserve the standalone
shell already provides (short homes scrolled by the duplication)
- ClinicalDashboard: compactMobileModeHome drops the pb-32 mobile bottom
padding on home states only; centred section leans toward the composer
on tall phones to satisfy the vertical-weighting guard
- Standalone shell #main-content: max-sm:flex-1 fills under the real
header height, removing a constant 9px phantom scrollbar
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Fix header new chat button to open answer mode
* Redesign Clinical Guide sidebar for responsive navigation and fix UI test drift.
Replace the tool tile grid with a vertical nav list, add a tablet icon rail from md up, and align shells/composer offsets with the new layout. Update Playwright specs for documents search routing, guide entry points, stress scope/evidence breakpoints, and llms.txt branding.
* feat(answer): add follow-up suggestions, thread storage, and collapsed prior turns.
Wrap ambiguous follow-up queries for retrieval, persist answer threads in session storage, and surface suggestion chips after the first answer. Update smoke tests for thread collapse and suggestion runs.
Co-authored-by: Cursor <cursoragent@cursor.com>
* feat(search): open bottom-docked command surface upward above the pill.
Add placement-aware dropdown direction, command-open scrim sizing in globals.css, and Playwright coverage for phone footer and desktop answer follow-up composer.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): fit mode home and detail pages to phone viewports without overflow.
Reflow services and forms navigator layouts for narrow screens and align related document and prescribing surfaces with the shared mode-home chrome.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): bootstrap thread persistence safely and share mode icons.
Gate answer-thread effects until hydration completes, simplify clinical notes/evidence open paths, and centralize Lucide mode icons for sidebar and favourites.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): use sheets for clinical notes and evidence on all breakpoints.
Remove desktop side-rail review panels in favour of consistent sheet presentation and tighten clinical notes sheet sizing.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): tighten scope and source-only disclosure styling.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(ui): compact evidence gap cards in mobile sheets.
Co-authored-by: Cursor <cursoragent@cursor.com>
* test(ui): lock tablet rail tools and active-route affordance.
Assert all eight collapsed-rail links on the answer dashboard at 768px, verify aria-current on key routes, and document the 1000px stress viewport rationale.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix: export maxStoredAnswerTurns and drop invalid compact header prop.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): harden thread restore on reload and extract result surface.
Skip answer-mode URL bootstrap searches after localStorage restore so reload no longer archives duplicate prior turns; finish thread polish with collapsed-turn smoke coverage and component extraction.
Co-authored-by: Cursor <cursoragent@cursor.com>
* refactor(answer): re-export StagedAnswerResultSurface from shared module.
Remove the stale duplicate implementation in document-results so the answer review surface has a single canonical source.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): polish follow-up chips, quote smoke, and sign-out thread clear
Anchor suggestion chips on the opening thread question after short follow-ups, clear persisted answer threads on sign-out or session expiry, add quote follow-up smoke coverage, and ignore local QA mockup screenshots.
Co-authored-by: Cursor <cursoragent@cursor.com>
* test(ui): align Playwright specs with launcher cards and answer follow-ups.
Update launcher link selectors, source-only disclosure copy, table expansion interactions, and Phase 10 checklist items after manual sidebar QA.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Align worker env defaults and refresh onboarding/verification docs.
Conservative worker Zod defaults now match .env.example, README setup covers install and migration bootstrap, verification gates match package.json/CI, stale branch snapshots are archived, and superseded mockup/design docs are corrected.
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix(answer): derive concise topics for first-turn suggestion chips
Use canonical terms or significant tokens instead of embedding long interrogative questions in follow-up chip templates.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Add codebase index and Cursor semantic search configuration.
Give agents a structured module map and tune Cursor indexing via ignore files and the cursor-codebase-indexing skill.
Co-authored-by: Cursor <cursoragent@cursor.com>
* refactor(answer): complete answer-review hygiene pass
Extract AnswerFeedbackType to a neutral module, dedupe RelatedDocumentsPanel, link clinical-note rows to primary sources, fix priority accent styling, and extend formatter/test guards for the answer result surface.
Co-authored-by: Cursor <cursoragent@cursor.com>
* feat: add image generation metadata re-stamp script
* fix(ui): edge-to-edge mobile layout for mode homes and footer dock (#410)
- Unify mobile shell to h-dvh flex column with --mobile-composer-reserve
- Full-bleed mode home backgrounds; edge-to-edge action cards on phone
- Resolve footer dock CSS precedence over inset document-mobile-search-edge
- Trim mobile header gutters to safe-area only; extend dock scrim to bottom
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* Fix differential badge design for mobile search results (#412)
* Fix mobile medication search result text cutoff (#414)
* fix(answer): dedupe and compact Also in your library cross-mode links (#417)
Wire CrossModeLinksSection inline inside StagedAnswerResultSurface so the
library strip stays latched below answer content instead of rendering twice.
Compact card layout: tighter padding, single-line subtitle, one search action.
Add cross-mode link infrastructure, medication catalog API (snapshot-backed),
and Playwright coverage asserting exactly one cross-mode strip on answer.
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* docs: refresh site map after cross-mode and medications routes
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix(ui): top-align favourites/tools on mobile mode homes (#422)
* fix: resolve all merge conflicts with origin/main
* chore: reconcile ingestion RPC execute privileges, schema.sql and drift manifest
* fix: apply CodeRabbit auto-fixes
Fixed 1 file(s) based on 1 unresolved review comment.
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
* ci: add db-reset-verify and dependency-review workflows
* fix(ui): remove PR 430 conflict artifacts
* ci: remove dependency-review workflow because repository is private without GHAS
* test(ui): query differentials result filters as tabs
* 📝 CodeRabbit Chat: Simplify code implementation (#434)
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
* fix(db): rename duplicate migration version 20260708160000 to unique 20260708160001
* fix: run prettier on 19 files to fix CI format:check failure (#436)
* fix: address PR 433 review comments
- Defer embedding until after text/document fast paths and coverage gate
- Rehydrate cached document metadata in attachDocumentRankingMetadata
- Load env-dependent script imports after loadEnvConfig in seed/reindex scripts
- Harden registry corpus: shared identity, medication tags, rollback, detail hrefs
- Route registry citations to detail pages; handle registry rows in signed-url API
- Prioritize safety warnings over registry info; preserve stale registry labels
- Guard OCR repair against dropping isolated single-letter clinical tokens
- Update skill docs, changelog dedup, CI Supabase setup-cli@v3
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: address follow-up PR 433 review comments
- Backfill NULL document_images.index_generation_id during re-stamp
- Count globally forced embedding eval cases in retrieval summaries
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: address PR 433 review blockers
* fix: address PR 433 review comments on image re-stamp and eval forcing
Treat NULL document_images.index_generation_id as stale even when JSON
metadata already matches the committed generation, and propagate global
--force-embedding into eval reporting/validation so index-unit-vector runs
count forced cases correctly.
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
* fix: restore neutralized 160000 migration for Supabase Preview parity
PR #433 preview branch recorded 20260708160000 before the file was removed, causing remote migration versions not found in local migrations directory. Keep a no-op stub for history sync; transactional index DDL stays on 170000.
Co-authored-by: Cursor <cursoragent@cursor.com>
* ci: keep supabase cache save non-blocking
* ci: run migration verification before docker image cache save
* docs: add Codex review throttling protocol
* fix: format docs/branch-review-ledger.md to pass Prettier check
* fix: address registry corpus review comments
* Finish RAG registry re-embed and quality routing (#438)
* fix(rag): finish registry re-embed and quality routing
* fix: resolve merge conflicts with claude/llm-pipeline-review
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* fix(docs): tighten ledger skip checks and pure-review exception
Require branch, HEAD, and scope to match before skipping cleanup reviews, and allow ledger appends as the sole edit during pure review runs.
Co-authored-by: Cursor <cursoragent@cursor.com>
* docs: codify review and branch cleanup workflow
* fix: throttle Codex auto-resolve reviews
* test: stabilize differentials UI navigation checks
* fix: harden Codex review automation
* chore: refresh drift manifest after branch merges
* ci: pin hosted runner configuration
* Fix: resolve merge conflict in live-drift.yml causing CI pin check failure
* fix: resolve all merge conflict markers from main merge
* fix: pin codex-autofix workflow runner to ubuntu-24.04
* fix: apply Prettier formatting to rag test files
---------
Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI
format:checkfailing on 17.cursor/skills/**/*.mdfiles and 2 TS source files (scripts/seed-registry-records.ts,src/lib/registry-corpus.ts) that were committed without Prettier formatting.Ran
prettier --writeon all 19 flagged files. No logic changes.Summary
prettier --writeon 19 unformatted files to restore green CIVerification
npm run format:checknpm run verify:cheapnpm run verify:uiwhen UI, routing, styling, browser behavior, reduced-motion, or forced-colors behavior changednpm run verify:releasebefore release or handoff confidence claimsnpm run eval:retrieval:quality(must stay 23/23) when retrieval, ranking, selection, chunking, or scoring behavior changed — CI cannot run it (needs live keys), so run it locally and paste the summary. A metadata/governance-weighting change once buried correct docs (recall 1.0→0.76) and only this eval caught it.npm run eval:rag -- --limit 15+npm run eval:quality -- --rag-onlywhen answer generation, the synthesis prompt, or answer post-processing changed (grounded-supported must not drop; citation-failure 0)npm run check:production-readinesswhen clinical workflow, privacy, environment, Supabase, source governance, or deployment behavior changednpm run check:deployment-readinesswhen deployment startup, hosting, or rollout behavior changedClinical Governance Preflight
N/A — formatting-only change, no logic or clinical output affected.
Notes