fix: refine answer recovery message and match scope - #833
Conversation
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughClinicalDashboard now always labels the main error state “Answer unavailable” and separates ChangesClinical dashboard rendering
Estimated code review effort: 2 (Simple) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Pull request was closed
…t matches (#853) * fix(rag): avoid over-expanding broad agitation management queries Keep medication-chart token expansion for dose/route/chart/options/listed signals from the rag-tree4 clinical-search change, but drop bare "table" and "management" triggers so topic lookups and pharmacological management questions stay on the agitation arousal topic query. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * docs: record PR queue babysit outcomes for #833-#852 Document which screenshot-queue PRs were closed as superseded or unsafe, and that the unique clinical-search fix continues in #853. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * style: format clinical-search agitation chart expansion Satisfy Prettier so Static PR checks pass on the consolidation PR. 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>
Summary
Answer unavailableacross answer and search paths.showUniversalAlsoMatchesrendering byactiveModeResultKindso answer/search modes render matching content without ambiguity.Verification
npx vitest run tests/ui-primitives.dom.test.tsx --reporter=dot(passed in original working tree).npx prettier --check src/components/ClinicalDashboard.tsx(passed).npm run test:focused -- --files src/components/ClinicalDashboard.tsx(ran successfully for focused harness; no matching tests in this path).npm run verify:cheap(failed in this state due missingesbuilddependency, unrelated to this patch).Summary by CodeRabbit