fix(tooling): harden git hooks, stabilize pre-push baselines, and optimize agent token payload - #2397
Conversation
…imize agent token payload Harden git hooks, stabilize pre-push guards and merge-base baselines, reduce snapshot merge contention, add Windows Dev Drive cache check, and optimize agent token payload across documentation.
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
Warning Review limit reachedNext included review available in 40 minutes. View limit detailsLimit details: You’ve used the included review currently available. Your 95 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (5)
📒 Files selected for processing (6)
Comment |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Medication pin misses real home
- Restored the Medication sidebar pin to appModeHomeHref("prescribing") so it lands on the real /medications workspace instead of /?mode=prescribing.
You can send follow-ups to the cloud agent here.
Reviewed by Cursor Bugbot for commit 3e3138d. Configure here.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3e3138d06e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
The prescribing pin used the shared-home selection URL, but /medications is a real standalone workspace and is excluded from the consolidated 307 map. Route the pin through appModeHomeHref so it matches every other Medication owner.
Merge main into codex/tooling-hooks-ci-invariants, keeping /medications for the Medication sidebar pin, restoring main issues snapshot, removing premature done inbox requests, and failing check:dev-drive-cache on untrusted Dev Drive cache paths. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
CI triageCI failed on this PR. Automated classification of the 2 failed job(s):
Compared with main CI run #14060 (failure). Classification is evidence routing, not permission to ignore a failure. Exact quarantined Playwright identities remain governed by the flake ledger. |
…uests Removes inbox done requests that would close #Y090R5 and #BJ80DB before their acceptance criteria are met, then regenerates data/outstanding-issues-snapshot.json so check:outstanding-issues-snapshot passes. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Queued done requests targeted ledger rows already closed on main (e.g. #RSD9EJ), which made docs:check-links fail. Reset issues snapshot to main, drop duplicate inbox files, and refresh repo-awareness snapshot. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Resolve post-#2397 merge conflict in data/repo-awareness-snapshot.json by regenerating the snapshot after syncing origin/main. Co-authored-by: Cursor <cursoragent@cursor.com>



Summary
/medications.Verification
npm run check:dev-drive-cachenpx vitest run tests/session-start-hook.test.ts tests/sidebar-production.dom.test.tsx(18/18 passed)npm run check:outstanding-issues-snapshotnpm run verify:pr-local— CI authoritative on headaddef8aadnpm run verify:ui— Production UI critical passed on GitHub Actions run 33030586513Risk and rollout
Clinical Governance Preflight
Clinical KB Database(sjrfecxgysukkwxsowpy)Notes
Navigation-only sidebar href fix (
/medications); no new clinical decision-support output. Review threads addressed onaddef8aad.