feat(differentials): surface map and related actions - #2471
Conversation
Co-Authored-By: Codex Fable 5 <noreply@anthropic.com>
|
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 |
|
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. |
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_40c61455-9462-4b65-949d-8da79130660e) |
CI triageCI failed on this PR. Automated classification of the 2 failed job(s):
Compared with main CI run #14582 (failure). Classification is evidence routing, not permission to ignore a failure. Exact quarantined Playwright identities remain governed by the flake ledger. |
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_fb6d60d6-b6c3-4e55-b785-c9b9118ac147) |
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_2d6d5d69-0a28-4a10-8883-db6e754da15c) |
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_ebbe9aaf-7c6f-41d6-b523-20354e005f9a) |
Summary
Verification
npm run verify:pr-localnpm run verify:uiwhen UI, routing, styling, browser behavior, reduced-motion, or forced-colors behavior changednpm run verify:releasebefore release or handoff confidence claimsnpm run test:e2e -- tests/ui-tools.spec.ts --project=chromium --grep "diagnosis detail actions stay tappable and tabs stay single-line"— 1 passed, covering 320/768/1024 px, keyboard activation, overflow, reduced motion, and forced colors.npm run check:design-system-contract— passed for 1,083 production files; adoption checked for 55 components and 84 roots; design-sync checked for 55 components and 7 guidelines.git diff --checkpassed before publication.Risk and rollout
edf16bde9fd1acf7d0c832a82cb24fbe3444365d.Clinical Governance Preflight
Notes
main; inspection showed no overlap between those commits and the three changed files. GitHub is the requested integration and verification surface.Co-Authored-By: Codex Fable 5 noreply@anthropic.com
Note
Low Risk
Presentation and test-only changes on diagnosis navigation and answer status-row layout; no clinical data, retrieval, or routing logic changes.
Overview
Diagnosis detail pages drop the three-level PageHeader breadcrumb (back in the in-page header remains) and add a phone-only “Explore diagnosis” panel with Map and Related shortcuts that call the same URL-backed
changeTabflow as the desktop tab strip, including live section detail text.Answer UI tweaks add
self-starton the source-only disclosure and stale-evidence banner so mixed-height controls in the status row align without stretching. Playwright updates register the new journey inui-tools.spec.ts, assert no breadcrumb and discovery behavior (touch targets, keyboard, forced colors), relax Chromium sub-pixel height checks inui-smoke.spec.ts, and compare vertical centers when source-only and Review due share a row. The design-system adoption manifest listsui-tools.spec.tsas Breadcrumb proof.Reviewed by Cursor Bugbot for commit b3700f8. Configure here.