Repository navigation
board: catch up merged PRs #1388–#1588 (arc, status, plans, debt, open items) - #1589
Conversation
…n items) 115 merged PRs had no PR_ARC_INVENTORY entry; each now has one, written from its description and file list and inserted in date order. Hygiene-only PRs (#1527, #1528, #1563, #1568, #1583) get none, per the termination clause. - STATUS_BOARD: catch-up rows for D-ids named by merged PRs with no row (D-CMR-RESET, D-FLOW-COG, D-SMD-0..6, D-V3-CLASSVIEW-NODRIFT-0). - INTEGRATION_PLANS: one catch-up entry for the 14 plans of 2026-10-12..16 and the 59 imported plans never named in the index. - TECH_DEBT: TD-CANON-HIGH-AUDIT-RESIDUE-1 (#1430's five sites, two re-read in code) and TD-CLIPPY-ALL-TARGETS-NOT-CLEAN-1 (#1437's measurement). - Entry 2026-10-16-board-catchup-open-items.md: open, owed and unbuilt items those PRs name, each pointing at its owner doc, plus the ones closed since. - LATEST_STATE and AGENT_LOG entries; entries index regenerated; SUPERSESSION-INDEX unchanged. Board only; no code. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011BMJnDBAfzcuQh5HeT39Rh
|
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. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 14758cd6f4
ℹ️ 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".
| ## 2026-10-08 — lance-graph PR #1399 (merged `061c57e9`, branch `claude/epistemicstate5-ce64-bits-3oe0dq`, head `4e2cd963`, 2 commits) — Board: #1398 merge hygiene | ||
|
|
||
| - **Added:** Board only, for merged #1398 (D-PUZZLE-ATTN-0): a `PR_ARC_INVENTORY.md` entry, a `LATEST_STATE.md` entry, STATUS_BOARD row D-PUZZLE-ATTN-0 set to Shipped (#1398), and a one-line edit to `entries/2026-10-07-puzzle-attn-0-entropy-attention-crosswords.md`. No code. | ||
| - **Locked:** Under the CLAUDE.md termination clause it needs no board entry of its own. |
There was a problem hiding this comment.
Omit entries for hygiene-only PRs
This entry contradicts its own statement that PR #1399 needs no entry under the termination clause in CLAUDE.md:360-378. The catch-up similarly adds entries for pure board-hygiene PRs such as #1394, #1533, #1535, and #1547, recreating the recursive ledger growth that the clause explicitly prevents; exclude these PRs from the catch-up rather than counting them among the 115 missing entries.
Useful? React with 👍 / 👎.
|
|
||
| ## TD-CLIPPY-ALL-TARGETS-NOT-CLEAN-1 (2026-10-16, measured in #1437 on 2026-10-09) — OPEN | ||
|
|
||
| `cargo clippy --workspace --all-targets --keep-going -D warnings` is not clean |
There was a problem hiding this comment.
Put Clippy lint flags after
--
Anyone following the instruction to re-run this measurement gets error: unexpected argument '-D' found, so the documented command cannot measure the current warning count. cargo clippy --help gives the usage as cargo clippy [OPTIONS] [--] [<ARGS>...] and specifically says deny flags are used with cargo clippy --; write the command as cargo clippy --workspace --all-targets --keep-going -- -D warnings.
Useful? React with 👍 / 👎.
Board only, no code. These are the board updates the hygiene rule requires for merged PRs, which those PRs never made.
What changes
PR_ARC_INVENTORY.md(+807): 115 merged PRs between dir-sim: Exchange recipients through ingest, snapshot, overlay, diff, plan and uniqueness #1388 and sap: optional CATS code fields bind three-valued through validity planes #1588 had no entry. Each now has one, written from its description and file list and inserted in date order. A script refuses duplicates and refuses to shorten the file. Every header was matched against the PR metadata (merge sha, head, branch, commit count). Hygiene-only PRs (board: PR arc and latest state for #1505–#1524 #1527, board: correct five #1527 arc entries from the full PR bodies #1528, board: #1562 merged (D-WAL) — PR arc entry and LATEST_STATE #1563, board: #1566 and #1567 merged #1568, board: record #1582 merge #1583) get no entry, per the termination clause.STATUS_BOARD.md: a catch-up section for D-ids that merged PRs name but that had no row: D-CMR-RESET, D-FLOW-COG, D-SMD-0..6 and D-V3-CLASSVIEW-NODRIFT-0.INTEGRATION_PLANS.md: one catch-up entry covering two groups of plans that were never named in the index:bad63c33), listed but not read.TECH_DEBT.md(both rows are tech debt):TD-CANON-HIGH-AUDIT-RESIDUE-1: the five classid sites that classid docs and helpers: say canon-high where the code is canon-high #1430 left for a decision.ClassGrantsandFacetSchema::of_classidwere re-read in code and are unchanged.TD-CLIPPY-ALL-TARGETS-NOT-CLEAN-1: Rust 1.99 + lance 13 / lancedb 0.40; D-SCF-CARE-PAIR-0 follow-up (L1–L8) #1437's dated measurement that--all-targetsclippy is not clean.2026-10-16-board-catchup-open-items.md:LATEST_STATE.md,AGENT_LOG.md: an entry each. The entries index is regenerated;SUPERSESSION-INDEX.mdregenerates unchanged.Gather-shaped items
The entries and the plan index mark items that were ruled a violation on 2026-10-16: data-indexed reads,
EqU32Via,SemijoinandForeignlanes, and the "preimage" hop. That covers #1479'sAddr::Via{fk,key}, the council-v3 §0 preimage definition and the D-SAG-4 two-hop step. This PR records them and removes nothing. Removing them is a separate decision.Checks
append_only_gate.py origin/main: OK (8 files checked; none shrank).entries_index.py --check: current, 359 rows.citation_decay.py --since origin/main: no new decay.supersession_index.py: output identical to main.🤖 Generated with Claude Code
https://claude.ai/code/session_011BMJnDBAfzcuQh5HeT39Rh
Generated by Claude Code