D-DCR-4 Σ-transport: execute the agreement the contract copy could only cite - #1160
Conversation
D-DCR-4's remaining half. `contract::sigma_propagation::ewa_sandwich` claimed in its header to be "verified empirically by crates/jc::ewa_sandwich" and quoted that pillar's numbers. The kernel is a byte-identical copy of jc's private sandwich, and nothing anywhere ran both. The duplication is forced, not sloppy: lance-graph-contract is zero-dependency by design and its manifest forbids even optional path deps, so it can neither call jc nor share its Spd2. The copy has to exist. What cannot exist by citation is the certification — a crate does not inherit a proof from a crate it is forbidden to depend on, and naming the pillar in a doc comment moves no evidence. The failure mode is quiet by construction. Two identical copies agree right up until one is edited, which is exactly when nobody is comparing them. So the gate goes on the side of the boundary allowed to see both. jc already dev-depends on the contract, so this needed zero new dependency edges: the new test runs both kernels over 1000 sampled SPD pairs and asserts bit equality. Bit, not approximate — the two are the same arithmetic in the same order on the same f64s, so any tolerance would hide precisely the drift the test exists to catch. It carries an anti-vacuity guard requiring most sampled pairs to have a non-zero off-diagonal, because diagonal inputs make the shared symmetrization vanish and would agree under a wrong implementation too. Disable-verified: drop the symmetrization from the ABI copy and the test fails. One structural check that nearly went unmade. jc is workspace-excluded, so a test there could plausibly never run; it does, via jc-proof.yml on crates/jc/**. Worth checking rather than assuming, since an excluded crate's test that CI never runs is a gate in name only. I first misread "crates/jc" in the root manifest as membership when it is in the exclude list — reading a grep hit without checking which list it landed in, for the fifth time today. The header now points at the test rather than at the crate, which is the difference between a claim and a check. Gates: jc 137/0, contract 1309/0, fmt clean, board append-only with zero deletions, index regenerated last. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QHVUi6Q9XtmKgxh6pDRayP
|
Warning Review limit reachedNext included review available in 6 minutes. View limit detailsLimit details: You’ve used the included review currently available. Your 85 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: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (5)
Comment |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_46fa5e20-d6d1-4669-9768-175afd6fc807) |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Resolves the conflict GitHub reported on #1160. Both sides prepended a dated entry to the top of LATEST_STATE.md: this branch's D-DCR-4 Sigma-transport inventory delta and main's dangling-reference regrade from #1159. The collision is positional and neither entry is wrong, so both survive -- the branch's on top as the later arrival. Merge commit rather than a rebase: another session's branch, and a merge keeps their checkout valid. Post-check: no marker left, both headings present, and the file grew. An append-only file that got shorter is always a defect, and picking one side of a prepend conflict is exactly how that happens. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DCfrD5y19cvFc4AoyydXYv
E-THE-FIX-FOR-A-REVIEW-FINDING-SHIPS-UNREVIEWED-BY-DEFAULT-1 merged about two hours ago offering a remedy and grading it: the @codex review comment is "unrelated to spend" and "not rate-limited". On #1160 that reviewer answered a review request with "You have reached your Codex usage limits for code reviews", while the other reviewer sits at its org spending cap. Both are now capped and the workaround is not free. That sentence is the exact failure mode of the entry it lives in — a property measured once, later read as standing — and it decayed inside the document written to name that decay. The gap is narrower than "claims decay": the entry applied its own rule about carrying a commit and a command to claims about the TREE, and not to a claim about an EXTERNAL SERVICE. A quota is strictly less stable than a repo, and a remedy's availability is not a property of the mechanism it remedies. The mechanism half is untouched. A push is still not a review trigger; that is quoted from the reviewer's own notice. What changed is that asking now costs quota that is exhausted, which leaves the spend decision as the only lever rather than one of two. Status line regraded in place, which is the one edit the append-only rule permits, and the supersession pointed at a new entry rather than folded in silently. Recorded operationally as well: no external reviewer can currently see any PR in this repo, so local gates are the whole verification surface until the caps lift. That is a statement about coverage, not a licence to merge faster. This commit carries real board work. That it also emits a synchronize event and may dispatch the workflows that have not yet run on this head is a side effect, not its purpose; an empty commit to kick CI would not have been acceptable. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QHVUi6Q9XtmKgxh6pDRayP
…t-pvfbs9' into claude/adaworld-substrate-harvest-pvfbs9
…rate-harvest-pvfbs9
…t-pvfbs9' into claude/adaworld-substrate-harvest-pvfbs9
|
Cross-session relay — two items for this session, plus why an unexpected merge commit appeared on this branch. I'm the 1 · Entropy / #1154 — from the mask-RISC sessionYour Their question: your census found seven I re-verified the TERNLOG half myself at source, after fetching OGAR: 2 ·
|
Closes D-DCR-4's remaining half.
contract::sigma_propagation::ewa_sandwichcarried a header claiming it was "verified empirically bycrates/jc::ewa_sandwich", quoting that pillar's numbers (PSD-preservation1.000000, 10000/10000 hops). The kernel is a byte-identical copy of jc's privatesandwich— same arithmetic, same variable names, same0.5 * (r01 + r10)symmetrization — and nothing anywhere ran both.The duplication is forced, which is the interesting part
lance-graph-contractis zero-dependency by design; its manifest forbids even optional path deps, after one killed the whole PR pipeline on 2026-07-07. So it can neither calljcnor share itsSpd2. This was never a dedup target — the copy has to exist.What cannot exist by citation is the certification. A crate does not inherit a proof from a crate it is forbidden to depend on, and naming the pillar in a doc comment moves no evidence across that boundary. And the failure mode is quiet by construction: two identical copies agree right up until one is edited, which is exactly when nobody is comparing them. So this was a drift risk, not a divergence risk.
The fix is directional
The gate cannot live in the contract (it may never depend on
jc), so it lives injc— the only side allowed to see both.jcalready dev-depends on the contract, so this needed zero new dependency edges.the_contract_copy_matches_the_certified_kernel_bit_for_bitruns both kernels over 1000 sampled SPD pairs and asserts bit equality viato_bits(). Not approximate: the two are the same arithmetic in the same order on the samef64s, so any tolerance would hide precisely the drift the test exists to catch. It carries an anti-vacuity guard requiring most sampled pairs to have a non-zero off-diagonal, because diagonal inputs make the shared symmetrization vanish and would agree under a wrong implementation too.Disable-verified: drop
0.5 * (r01 + r10)from the ABI copy → the test fails; restore → green.A structural check that nearly went unmade
jcis workspace-excluded, socargo test -p jcdoes not work and a test there could plausibly never run. It does —.github/workflows/jc-proof.ymlrunscargo test --manifest-path crates/jc/Cargo.tomloncrates/jc/**. Worth checking rather than assuming: an excluded crate's test that CI never runs is a gate in name only.I first misread
"crates/jc"in the root manifest as membership when it is in theexcludelist — reading a grep hit without checking which list it landed in, for the fifth time today. Recorded rather than quietly corrected.The header now points at the test rather than at the crate, which is the difference between a claim and a check.
Gates
jc137/0 (was 136) ·lance-graph-contract1309/0fmtclean on bothBoard:
E-A-CITATION-IS-NOT-A-DEPENDENCY-AND-A-FORCED-COPY-NEEDS-A-GATE-1,LATEST_STATEdelta,STATUS_BOARDD-DCR-4 flipped to done.🤖 Generated with Claude Code
https://claude.ai/code/session_01QHVUi6Q9XtmKgxh6pDRayP
Generated by Claude Code