docs(issues): reconcile 29 queued ledger requests into the canonical ledger - #2217
Conversation
…ledger Applies every pending request on main: 11 done, 12 update and 2 add from the 2026-08-21 verification sweep, plus the 4 requests already queued by earlier sessions. Requests move to docs/outstanding-issues-inbox/applied. Produced solely by npm run issues:reconcile on a dedicated fresh-base branch; no hand edits to the canonical ledger. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in:32 minutes Limit details: You’ve used the included review currently available. Your 87 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. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Comment |
This pull request has been ignored for the connected project Preview Branches by Supabase. |
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit:f4217ff304
ℹ️ 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".
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Static PR checks failed because docs:check-links simulates a full reconciliation of pending inbox requests, and 11 of this branch's own requests targeted rows that PR #2217's already-merged reconciliation independently closed or updated first (#JVYQEM double-pending-update; #ZF006G/#TAQKCN/#43SSS0/#6K9YGQ/#D6G8TC/#0EKBGC already closed with an equivalent conclusion; #97VQK5/#4TBHS8/#SZGPAH/#VTEW3W stale against a newer row state). Cancel each via `ledger-inbox.mjs cancel` rather than touching the canonical ledger or any other pending request file. Two cancellations (#4TBHS8, #SZGPAH) note that the discarded evidence (a chromium-mockups run showing 16 passed) still answers the "re-measure before closing" ask on those rows and should be re-queued fresh by a follow-up request. #VTEW3W is flagged as a genuine count conflict (13/7 vs. 21/8) needing re-measurement, not a mechanical dedup. RAG impact: none — docs-only, no src/lib/rag or retrieval-surface change. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015StJgDC2dfef8PXN9dfriw
Summary
This is the serialized ledger reconciliation for every request pending on
mainas of9f98bbf06. It applies 29 queued inbox requests todocs/outstanding-issues.mdand moves each request file todocs/outstanding-issues-inbox/applied/. The canonical ledger diff was produced solely bynpm run issues:reconcileon this dedicated fresh-base branch; there are no hand edits to the ledger, and no other file is touched.main: elevendone, twelveupdateand twoadd.#Q5JHBJPhase 6.2 completion update, and three additions covering the OpenAI zero-data-retention contradiction, the Docling table fixtures, and thePGRST106migration-history alignment failure.Net effect on the ledger: open rows fall from 71 to 61, archived rows rise to 350, and pending requests fall to zero.
Two of the applied additions are worth a reviewer's eye because they were found during the sweep rather than inherited. The first records that production carries migration version
20260820120000(migration_history_versions_rpc) which exists nowhere in this repository, and cross-references the already-queuedPGRST106request it plausibly fixes; it carries an explicit stop rule that the reconciliation belongs in the repository and nothing may be re-applied or mark-applied on production to resolve it. The second records that an automated worktree sweep deleted an in-use worktree during that session and destroyed its uncommitted work, and proposes the three preconditions a removal should satisfy before it runs.Verification
npm run verify:pr-local— recognised low-risk documentation scope. All eleven selected checks completed and none failed:check:runtime,check:installed-lock-parity,format:changed,sitemap:check,docs:check-index,docs:check-inventory,docs:check-scripts,docs:check-links,check:branch-review-ledger,check:outstanding-issuesandcheck:ledger-write-discipline. Exit code 0, captured directly rather than through a pipe.check:ledger-write-discipline—Ledger write discipline passed for 9f98bbf06d8a..HEAD.This is the decisive gate for a reconciliation: it proves the canonical ledger diff equals exactly the recorded reconciliation transaction, with no direct table-row editing.check:outstanding-issues—Ledger inbox check passed: 0 pending request(s), 438 applied.andOutstanding-issues guard passed: 411 rows (61 open, 350 archived), unique display and durable ids, collision-free allocation enabled, deprecated next-id marker ignored, no merge driver, no ids deleted from base 9f98bbf06d8a.UI verification not run: no executable product code, route, component, style or browser behaviour is touched.
Risk and rollout
check:ledger-write-disciplinerather than by review alone.git revertthis pull request. That restores the previous ledger and returns the 29 requests to pending, since request files are immutable and revert simply moves them back out ofapplied/.src/lib/rag/, no retrieval RPC, no ranking configuration, no eval harness and no golden fixture is touched.Notes
Open pull request #2207 (
claude/ledger-reconcile-stale-ui-rows) queues 22 further requests that are not onmainyet and are therefore not in this transaction. Its requests overlap this sweep: it closes#0EKBGC,#43SSS0,#6K9YGQ,#D6G8TC,#TAQKCNand#ZF006G, which this reconciliation has already resolved, and it closes#SZGPAHand#4TBHS8, which this reconciliation only updated because the Chromium mockup run needed to close them was not performed here. After #2207 lands, the next reconciliation will report those overlapping requests as un-appliable and name their UUIDs; each should be cancelled withnode scripts/ledger-inbox.mjs cancel <uuid> --reason "…"rather than forced. Its#97VQK5and#VTEW3Wupdates remain applicable, and if its author did run the mockup spec, their#SZGPAH/#4TBHS8closures carry evidence this sweep could not produce and should supersede the update text applied here.🤖 Generated with Claude Code