Compact Answer source and safety support UI - #2474
Conversation
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
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 |
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_0046a67c-b73c-4dd6-a228-48c8c91722ca) |
Summary
Verification
npm run verify:pr-localtests/claude-cloud-profile.test.tsharness returning exit 127; this PR does not touch that test or its provisioner.npm 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 36/36) when retrieval, ranking, selection, chunking, or scoring behavior changednpm run eval:rag -- --limit 15+npm run eval:quality -- --rag-onlywhen answer generation, the synthesis prompt, or answer post-processing changednpm run check:production-readinesswhen clinical workflow, privacy, environment, Supabase, source governance, or deployment behavior changednpm run check:production-readiness:ci— READY; structural privacy requirements 11/11 passed.npm run check:deployment-readinesswhen deployment startup, hosting, or rollout behavior changedAdditional local evidence:
node scripts/run-vitest.mjs run --reporter=dot tests/answer-source-marks.dom.test.tsx— 13/13 passed.npm run check:design-system-contract— passed; 55 components, 84 adoption roots, and 7 design-sync guidelines checked.npm run build— passed; compilation, TypeScript, 1,998 routes, and client bundle secret scan completed.npm run eval:rag:offline— 26 files and 628 tests passed; 36 golden cases validated.npm run eval:rag:adversarial:offline— 25/25 passed across 24 synthetic cases.Risk and rollout
Clinical Governance Preflight
Clinical KB Database(sjrfecxgysukkwxsowpy)Notes
origin/mainbefore final verification.Note
Low Risk
Presentation and control placement only; no retrieval, auth, or data-path changes—residual risk is layout/regression on physical Safari/PWA outside Chromium coverage.
Overview
Compact answer source and safety UI on the live clinical answer surface: source-currency warnings sit in one row with neutral utilities, while safety stays a single priority strip.
NaturalLanguageAnswernow renders Source-only and Review due (RetrievalStateBanner) side by side inanswer-source-status-rowwhen applicable, with stableuseId-backed disclosure controls and a responsive tweak hiding “· verify passages” on very narrow widths. Stale synthesized answers can show review-due in that row via newanswerState/onOpenStateSourceprops; the live turn hides inline copy (showCopyAction={false}) because copy moved up-stack.AnswerCardgains optionalretrievalStatePlacement("header"|"content") so the dashboard can render stale status beside Source-only without duplicating the header banner; ready answers force"content", degraded cards use"content"only forstale_evidence.Answer utilities are split from the safety card: new
AnswerUtilityActionshosts Copy with sources, Evidence gaps, and Report a problem in the quietchatMicroActionrow.AnswerSupportSummaryCardis trimmed to the safety/priority row only; it mounts whenpriorityis set, not for gaps/feedback alone.RetrievalStateBannertypography and hit targets align with compact-meta (smaller text,min-h-compact-meta). Cross-mode “Also in your library” moves above follow-up suggestions to close the evidence/safety stack before the next conversational turn.Design-sync (
AnswerCardDTS props), adoption manifest, and tests/smoke specs follow the new hierarchy and behaviors.Reviewed by Cursor Bugbot for commit 888649d. Configure here.