Optimize Cursor codebase indexing ignore rules#1171
Conversation
Exclude ~30MB of tracked lockfiles, snapshot JSON, PDFs/PNGs, and archive docs from the semantic index while keeping them @Files-reachable, and allow .env.example through .cursorignore for setup context. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
📝 WalkthroughWalkthroughChangesThe PR expands Cursor workspace and semantic-index ignore rules to exclude generated outputs, secrets, local tooling, worktrees, large data/assets, archives, manifests, and Playwright artifacts while preserving Cursor exclusion configuration
Estimated code review effort: 1 (Trivial) | ~3 minutes Suggested labels: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
…ing-optimize-7a2b Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
…ing-optimize-7a2b
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).