Compact the differential safety snapshot - #2472
Conversation
Co-Authored-By: Codex Fable 5 <noreply@anthropic.com>
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 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 |
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_744b4e06-7248-4710-8be5-03182d1a58ed) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c677e3e0dd
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…ntial-safety-note-20260830
Co-Authored-By: Codex Fable 5 <noreply@anthropic.com>
…y-note-20260830' into codex/compact-differential-safety-note-20260830
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_2c07dab3-819d-43b8-b966-8e3ab13b0299) |
CI triageCI failed on this PR. Automated classification of the 2 failed job(s):
Compared with main CI run #14542 (success). That run's conclusion is an aggregate and did not exercise Classification is evidence routing, not permission to ignore a failure. Exact quarantined Playwright identities remain governed by the flake ledger. |
Co-Authored-By: Codex Fable 5 <noreply@anthropic.com>
Summary
Verification
npm run verify:pr-local— passed: runtime and lock parity, changed-file formatting, lint, typecheck, 930 test files / 11,660 tests, repo-awareness, production build (1,998 static pages), client-secret scan, 36 RAG golden cases / 26 suites, and medication data checks.During development, use
npm run verify:cheapas the faster iteration gate before the final PR-local preflight.npm run verify:uiwhen UI, routing, styling, browser behavior, reduced-motion, or forced-colors behavior changednpm run verify:releasebefore release or handoff confidence claimsUI verification not run: the broad UI suite was not repeated because shared UI foundations did not change. The focused Chromium journey in
tests/ui-tools.spec.tspassed 1/1 and checks 320px phone and 1024px desktop/tablet behavior, no horizontal page overflow, four metrics on one row, the compact card height, removal of the old action, reduced motion, and forced colors.For retrieval, ranking, selection, chunking, source/citation rendering, or answer-contract changes,
verify:pr-localrunseval:rag:offlineautomatically. Run the offline command directly during iteration before spending a live eval.npm run eval:retrieval:quality(must stay 36/36) when retrieval, ranking, selection, chunking, or scoring behavior changed — not applicable; retrieval behavior is unchanged.npm run eval:rag -- --limit 15+npm run eval:quality -- --rag-onlywhen answer generation, the synthesis prompt, or answer post-processing changed — not applicable; answer generation is unchanged.npm run check:production-readinesswhen clinical workflow, privacy, environment, Supabase, source governance, or deployment behavior changed — not applicable; this is a presentation-only change.npm run check:deployment-readinesswhen deployment startup, hosting, or rollout behavior changed — not applicable.Risk and rollout
c677e3e0ddc93fbe31d77b068843343a094e11c3.Clinical Governance Preflight
Clinical KB Database(sjrfecxgysukkwxsowpy)Notes
npm run formatandnpm run design-system:adoption:updateleft the branch unchanged.bashto WSL for an unchanged current-main test. Prefixing Git Bash on the test process PATH made the focused file pass 24/24; the final full gate passed..next/dev/types/validator.tsinitially blocked the build. Route types were regenerated, the stale ignored artifact was removed, and the final production build passed without tracked changes.Co-Authored-By: Codex Fable 5 noreply@anthropic.com
Note
Low Risk
Presentation-only changes on the differential detail overview; clinical safety data and counts are unchanged, with full metric labels still exposed via aria-label.
Overview
Compacts the differential overview Safety Snapshot so it uses less vertical space while keeping the same safety content (summary, watch-for tags, and metric counts).
The card gets tighter padding and smaller header/typography, drops the inline status badge and the Review must-not-miss causes button (along with scroll-to-section behavior and
resolveScrollBehavior). Watch-for chips move to a single horizontal row with in-card scrolling on narrow widths. Four safety metrics stay on one row viagrid-cols-4, with smaller mobile metric text anddata-testidhooks for values and the watchlist.Tests extend unit checks for the compact grid and removed CTA, and Playwright at 320px asserts one metric row, no CTA, bounded card height, no horizontal overflow, and stable layout under reduced motion and forced colors.
Reviewed by Cursor Bugbot for commit 714785b. Configure here.