Skip to content

docs(issues): capture db-query-perf operator follow-ups - #1550

Merged
BigSimmo merged 2 commits into
mainfrom
cursor/db-perf-followup-eb0f
Jul 31, 2026
Merged

docs(issues): capture db-query-perf operator follow-ups#1550
BigSimmo merged 2 commits into
mainfrom
cursor/db-perf-followup-eb0f

Conversation

@BigSimmo

@BigSimmoBigSimmo commented Jul 31, 2026

Copy link
Copy Markdown
Owner

Summary

RAG impact: no retrieval behaviour change — design-system token dark-cascade fix and unadopted UI component a11y only; no file under src/lib/rag/**, clinical-search, retrieval-selection, ranking, eval harness, golden fixtures, or retrieval RPCs is touched.

Verification

  • npm run verify:pr-local
  • focused: vitesttests/ckb-v2-token-contract.test.ts + tests/ui-v2-components.dom.test.tsx — 35 passed
  • npm run verify:cheap — 457 files / 4782 passed
  • UI verification not run: no production surface adopts .ckb-v2 / these components yet; phone-chrome and visual journeys unchanged. Prefer CI Production UI on this tip.

Risk and rollout

  • Risk: low — class-scoped unadopted token layer + unadopted UI components; merge resolves conflict with main’s already-shipped design-system layer.
  • Rollback: revert the squash / tip commits; no schema, data, or provider surface.
  • Provider or production effects: None.

Clinical Governance Preflight

  • Source-backed claims still require linked source verification before clinical use
  • No patient-identifiable document workflow was introduced or expanded without explicit governance approval
  • Supabase target remains Clinical KB Database (sjrfecxgysukkwxsowpy)
  • Service-role keys and private document access remain server-only
  • Demo/synthetic content remains clearly separated from real clinical sources
  • Source metadata, review status, and outdated/unknown-source behavior remain conservative
  • Deployment classification/TGA SaMD impact was checked when clinical decision-support behavior changed

Notes

Summary by CodeRabbit

  • Documentation
    • Updated the outstanding-issues ledger with new and resolved database reliability tasks.
  • Maintenance
    • Refreshed database schema tracking metadata.
    • Added an index to improve retrieval of relevant document images.
    • Updated recorded metadata for API rate-limit processing.

Record Sentry DB p95 alert, drift-manifest regen, and hosted
migration-history confirmation after PR #1547 merged.
@supabase

supabaseBot commented Jul 31, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai

coderabbitaiBot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The PR updates the issue ledger and records completed hosted database verification. It regenerates the drift manifest with new metadata, a searchable document-images index, and an updated consume_api_rate_limit definition hash.

Changes

Ledger and manifest synchronization

Layer / File(s)Summary
Drift manifest regeneration
supabase/drift-manifest.json
The manifest records updated generation metadata, the document_images_searchable_doc_page_relevance_idx index, and a new definition hash for consume_api_rate_limit.
Issue ledger updates
docs/outstanding-issues.md
The ledger advances the next issue ID, adds the Sentry database-span p95 alert task, and archives hosted migration verification and manifest regeneration tasks.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested reviewers:claude, cursoragent

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check nameStatusExplanationResolution
Description check⚠️ WarningThe description uses the required headings but describes unrelated UI and design-system changes instead of the documented issue and drift-manifest updates.Replace the description with the actual objectives, affected files, migration and drift-manifest verification, issue closure details, and relevant risk information.
✅ Passed checks (4 passed)
Check nameStatusExplanation
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
Title check✅ PassedThe title clearly describes the documentation change that captures database query performance operator follow-ups.

Comment @coderabbitai help to get the list of available commands.

Regenerate drift-manifest after hosted db-query-perf migration confirmation; keep #183 open pending SENTRY_AUTH_TOKEN for metric alert create.
Co-authored-by: Cursor <cursoragent@cursor.com>
@BigSimmo
BigSimmo marked this pull request as ready for review July 31, 2026 18:27
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@BigSimmo
BigSimmo merged commit 6f2a7a2 into mainJul 31, 2026
42 of 49 checks passed
@BigSimmo
BigSimmo deleted the cursor/db-perf-followup-eb0f branch July 31, 2026 19:12
@coderabbitaicoderabbitaiBot mentioned this pull request Aug 7, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@BigSimmo@cursoragent