Skip to content

docs: ledger PR #1197 Bugbot do-not-merge review#1211

Closed
BigSimmo wants to merge 1 commit into
mainfrom
cursor/pr1197-review-ledger-d06a
Closed

docs: ledger PR #1197 Bugbot do-not-merge review#1211
BigSimmo wants to merge 1 commit into
mainfrom
cursor/pr1197-review-ledger-d06a

Conversation

@BigSimmo

Copy link
Copy Markdown
Owner

Summary

Records the completed local Bugbot + high-risk review of open PR #1197 (execute-system-audit-remediation @ 16c318474) in docs/branch-review-ledger.md.

Outcome (for #1197)

Do not merge tip 16c318474. Not landed (prlanded: still OPEN). Branch is ~468 commits behind main with merge-tree conflicts. Confirmed P1 regressions: clinical-notes trust gate removed, summaryMode test/route mismatch (runtime 400 vs expected 200), RAG pre-classifier without policy attestation, stale outstanding-issues ledger. Hosted PR policy already fails on missing RAG impact: and Clinical Governance Preflight.

Verification

  • npm run check:branch-review-ledger passed
  • No product code changes in this PR

Risk and rollout

Docs-only append-only ledger row; no runtime risk.

Open in Web Open in Cursor 

Append do-not-merge outcome for tip 16c3184 after Bugbot,
static high-risk review, and runtime summaryMode test proof.
@supabase

supabase Bot commented Jul 25, 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

coderabbitai Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 9be71a09-72c1-4709-b82b-451ee0f42744

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cursor/pr1197-review-ledger-d06a

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.

❤️ Share

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

BigSimmo added a commit that referenced this pull request Jul 25, 2026
#1224 and the #1192 landing (#1232)

* docs(ledger): batch the appends from PRs #1206, #1207, #1210, #1211 and #1224

Five open PRs each append review records to docs/branch-review-ledger.md
and nothing else. Each one has to be synced against main and carried
through a full CI cycle, and every sync risks the union driver
re-duplicating rows, so the queue cost far exceeds the fifteen lines
involved.

Rows are copied verbatim from each PR head and verified byte-identical,
so the original PRs can be closed as landed here rather than merged
individually.

* docs(ledger): record PR #1192 prlanded squash merge
@cursor

cursor Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Closing as harmless duplicate: ledger-only Bugbot row for PR #1197 is already on main. Subject PR #1197 is merged; no product follow-up needed.

@BigSimmo BigSimmo closed this Jul 25, 2026
@BigSimmo
BigSimmo deleted the cursor/pr1197-review-ledger-d06a branch July 26, 2026 03:23
Sign up for free to 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.

1 participant