docs: record canary comparison preflight#1180
Conversation
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
|
Warning Review limit reached
Next review available in: 19 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 (1)
📝 WalkthroughWalkthroughThe open-items ledger adds a read-only scheduled comparison preflight note documenting unavailable artifacts, prior datapoints, and constraints against deriving deltas or dispatching follow-up actions. ChangesScheduled comparison preflight
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. |
…t-comparison-8e05 Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…1141) * fix(ui): implement orientation adaptive design audit recommendations * fix: remove duplicate command dropdown height cap Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * chore: note review reply permission denial GraphQL review-thread replies returned 403; resolved the fixed duplicate-height threads directly after validation. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * docs: record PR babysit sweep Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * docs: record Run PR behind-main sweep for #1141 * docs: ledger re-sync sweep for PR #1141 * fix(ui): collapse dock child safe-area padding * docs: record PR 1141 maintenance * fix(ui): avoid Sheet max-height racing caller caps Skip the default phone max-h utility when contentClassName already sets max-h, and record the PR #1140 babysit ledger row. Co-authored-by: Cursor <cursoragent@cursor.com> * test(ui): poll guide dialog focus trap across viewport sizes Co-authored-by: Cursor <cursoragent@cursor.com> * fix(ui): do not steal focus from a newly opened sheet on restore retry The Sources opener retry must not reclaim focus when another dialog already became active (mobile Guide opened from the Clinical Guide menu). Co-authored-by: Cursor <cursoragent@cursor.com> * fix(ui): retry Sheet autofocus when open races a closing sibling Co-authored-by: Cursor <cursoragent@cursor.com> * test(ui): harden Sources opener focus restore assertion after Escape Co-authored-by: Cursor <cursoragent@cursor.com> * docs(review): record babysit merges for #1140 #1171 #1180 #1158 Co-authored-by: Cursor <cursoragent@cursor.com> * fix(ui): stabilize Sources sheet autofocus on open Default UtilityDrawer sheet mode for breakpoint=all and retry sheet autofocus so the Find-a-document field keeps focus after drawer mount races. * ci: retrigger PR policy check on tip * fix(ui): stop stacked Sheet focus steal; assert Escape restore honestly Gate Sheet autofocus retries on isTopmostSheet, and remove test-side focus repair so production restore failures stay visible. --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Summary
fields=index, differentials abort/debounce, universal documents typeahead soft-timeout (750ms), shared(search-app)shell to avoid composer remount, and Answer rate-limit in-memory fallback outside production./services→/dsm) syncssearchModeduring render (no stale-mode paint) even when the query string is unchanged; extracted ClinicalDashboard lazy imports to stay under the maintainability budget.RAG impact: no retrieval behaviour change — typeahead documents domain timeout and shell URL sync only; ranking formulas and full
/api/searchretrieval path unchanged.Verification
npm run verify:pr-local— focused Vitest on touched sources (362) plus api-rate-limit / search-shell / universal / route / site-map suites green;docs:check-indexOKverify:uinot required for this pass; mode-home smoke vianpm run ensurereturned HTTP 200 for/,/services,/dsm,/documents/search,/therapy-compass,/?mode=prescribing, and/api/answer/streamreturned 200 after the rate-limit fallback fixeval:retrieval:latency/ soak / live OpenAI canary — approval-gated provider work; not needed for timeout-only typeahead changeRisk and rollout
GlobalSearchShelllayouts and prior timeout/fallback behaviourClinical Governance Preflight
[REDACTED]([REDACTED])Notes
fields=indexremains for identity-only consumers (cross-mode links).