Repository navigation
Ledger revision (d) against #1592–#1634 - #1636
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Warning Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption. 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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Add Carrot credits or activate Agent usage billing to use Autofix |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 47e4ec5ac0
ℹ️ 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".
…ange reads attest -> read_keys scans both key columns of every row (addressed.rs:100-117, 7-10 ms for 100,002 keys); the 3k count is only the per-query range reads (SideIo.rows_scanned). K13, C46 and the ADMISSION recommendation now name both costs. Codex review on #1636. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013z2e1YXkm6o6HNrDn5BPWA
ed17fa7 to
3c97d92
Compare
…ange reads attest -> read_keys scans both key columns of every row (addressed.rs:100-117, 7-10 ms for 100,002 keys); the 3k count is only the per-query range reads (SideIo.rows_scanned). K13, C46 and the ADMISSION recommendation now name both costs. Codex review on #1636. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013z2e1YXkm6o6HNrDn5BPWA
|
Rebased onto
Fixed here the same way #1627 named its re-run ( Generated by Claude Code |
Maps 43 merged PRs onto the recovery ledger (revision c ended at #1591). Read through ten Haiku substrate-mapper maps, checked against main abfa9e2. Closes C39-C52; merges the gather ruling and ISS-ADMISSION-PREP-BINDING into one ADMISSION item with Arm B as its measured example; narrows the slot budget to nesting depth after call_owned; records stale STATUS rows, git branch refs in the default build and two cards without frontmatter. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013z2e1YXkm6o6HNrDn5BPWA
…ange reads attest -> read_keys scans both key columns of every row (addressed.rs:100-117, 7-10 ms for 100,002 keys); the 3k count is only the per-query range reads (SideIo.rows_scanned). K13, C46 and the ADMISSION recommendation now name both costs. Codex review on #1636. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013z2e1YXkm6o6HNrDn5BPWA
3c97d92 to
a8b72ee
Compare
|
Rebased onto Generated by Claude Code |
Research and board only; no code changed.
.claude/research/2026-10-recovery/LEDGER-REVISION-2026-10-20d.mdcontinues the recovery ledger from revision (c), which ended at #1591. It covers 43 merged PRs (#1592–#1634), read through ten Haikusubstrate-mappermaps. Every file:line it cites was then read onmain(abfa9e2).Closed (C39–C52)
call_ownedremoves the relalg width limit: anOrorAndof any width now runs in 3 slots (C44).Merged into one item
Narrowed
call_owned, the 8-slot budget limits only nesting depth, not width.New open items
isa.rs's opcode reading of 46..49 behind a declared legacy reading.mainthroughlance-graph-r2il.required_role = Noneboth skip RBAC.Withdrawn from my earlier session report
commithas an admin bypass.Operator decisions still open: ADMISSION, D-RA-2, WIRE-50, ACTION-GATE, FLOAT-REFS, O30.
The board entry, the regenerated entries index and the regenerated supersession index (no change) are in the same commit.
Generated by Claude Code