refactor(mode-home): short titles and single-line hero subtitles#966
Conversation
Make every mode-home hero read consistently: the title is the mode name (Therapy, Factsheets, Specifiers, Formulation, Services, Medication) and the subtitle is one high-yield line instead of a sentence that wraps to two lines on a phone. Titles already short (Tools, Forms, DSM-5, Favourites, Differentials) keep their name; their subtitles are trimmed where still long. Update the UI/DOM specs that assert the previous hero copy. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gb9jucaNMEQDmRKXEN7SRV
|
Warning Review limit reached
Next review available in: 24 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (16)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
CI triageCI failed on this PR. Automated classification of the 2 failed job(s):
Compared with main CI run #3859 (success). Classification is evidence routing, not permission to ignore a failure. Exact quarantined Playwright identities remain governed by the flake ledger. |
The renamed "Medication" hero title is a substring of the answer section's sr-only "Medication matches" heading, so the page-scoped getByRole heading lookup hit two elements under Playwright strict mode. Scope the lookup to the mode-home container and match exactly. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gb9jucaNMEQDmRKXEN7SRV
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Summary
title/subtitleprops passed toModeHomeHero/ModeHomeTemplate; no component structure or CSS change. UI/DOM specs that asserted the old copy are updated in lockstep.Verification
npm run verify:pr-localnpm run typecheck✅,npm run lint✅, fullvitest run✅ (2971 passed; the single failure istests/pdf-extraction-budget.test.ts, an unrelated Python-OCR child-process timing test that needs the OCR stack — untouched by this change).verify:ui/Playwright could not launch in this environment — the sandbox has Chromium build 1194 installed but the pinned Playwright expects build 1228 (chrome-headless-shell-...-1228), and browser download is disabled. Every failure wasbrowserType.launch: Executable doesn't exist, not an assertion. The affected specs (ui-accessibility,ui-route-coverage,ui-specifiers,ui-formulation,ui-tools) had their hero-heading assertions updated to the new titles and should be run in CI.Risk and rollout
Clinical Governance Preflight
Not applicable — no ingestion, answer generation, search/ranking, source rendering, document access, privacy, production env, or clinical-output surface is touched. Change is presentational hero copy only.
Notes
🤖 Generated with Claude Code
Generated by Claude Code