|
| 1 | +## 2026-09-06 — lance-graph PR #1209 (merged `f1336b66`, branch `claude/ndarray-simd-tract-o3jfrn`) — the temporal architecture, measured against the wrong store first |
| 2 | + |
| 3 | +- **Added:** `crates/lance-graph/tests/canonical_witness_identity_probe.rs` (3 |
| 4 | + tests), `crates/lance-graph-contract/tests/v3_mint_reachability_probe.rs` (2), |
| 5 | + `crates/lance-graph/tests/test_e_old_diff_tripartite.rs` (1), four |
| 6 | + `STATUS_BOARD` rows, and the `SUPERSESSION-INDEX` regeneration. No source |
| 7 | + changes — this PR measures, it does not migrate. |
| 8 | +- **Corrected, about this session's own earlier reporting:** a first pass traced |
| 9 | + `VersionedGraph`/`NodeSchema` and reported it as the addressed architecture, |
| 10 | + concluding the spine "terminates at `u32 node_id`" and that the last link "was |
| 11 | + never built". Wrong store: `graph/cycle_sink.rs` (#911) had not been opened. |
| 12 | + The operator named the failure mode before the evidence did — *"we may be |
| 13 | + debugging the migration while accidentally treating a secondary |
| 14 | + VersionedGraph/BLASGraph representation as the architecture."* |
| 15 | +- **The V1/V3 two-rail control was operator-mandated and earned its keep |
| 16 | + immediately.** The first fixture used `NodeGuid::new` — the V1 (deprecated) |
| 17 | + constructor — so it would have proven the live rail nothing. Rewritten to mint |
| 18 | + through `mint_for(classid_read_mode(c).tail_variant, …)`. It then failed twice, |
| 19 | + both times on MY assertions, not the store: `local_key_v2` was expected to |
| 20 | + separate keys differing only in `leaf`, and must not — `leaf` is the 4th HHTL |
| 21 | + routing tier (bytes 10..12), part of the addressing PREFIX, deliberately |
| 22 | + outside the basin-local key (bytes 12..16). **Both facts are now pinned**, so |
| 23 | + "use the V3 accessor" is recorded as TWO choices and picking the wrong V3 one |
| 24 | + fails like picking the V1 one. |
| 25 | +- **Pinned false-green (D-TEMPORAL-1a):** 256 V3 addresses differing only in |
| 26 | + `leaf` → 256 distinct raw keys, 256 distinct `to_hex_v2()`, **1** distinct |
| 27 | + `identity()`. The miniature of the measured OBO incident (60 478 rows → 658 |
| 28 | + apparent identities on a byte-exact round trip). Byte fidelity does not imply |
| 29 | + address fidelity. |
| 30 | +- **Checked rather than assumed (D-V3-MINT-1):** `mint_for`'s V2/V3 arm is gated |
| 31 | + on `guid-v2-tail`, absent from `default`; the registry's V3 entries are gated |
| 32 | + on `guid-v3-tail`, present. A contradiction was one step from being reported — |
| 33 | + and does not exist: `guid-v3-tail = ["guid-v2-tail"]` (`Cargo.toml:69`). |
| 34 | + Measured `tail_variant=V3 / guid-v2-tail=true / leaf_is_live=true`. |
| 35 | +- **CodeRabbit finding, CONFIRMED and fixed (`91cfe1c1`):** the removal probe |
| 36 | + could pass without validating removal handling. `graph_seal_check` returns on |
| 37 | + its FIRST divergence while walking `to_seals`, so on `V1={A,B,D} → V2={A,B',C}` |
| 38 | + the update to `B` or the insert of `C` tripped it before the removal branch |
| 39 | + (`versioned.rs:632-637`) ran — the assertion held with that branch deleted. |
| 40 | + Added `V3={A,B'}` (a removal-ONLY transition). The isolation also SHARPENED the |
| 41 | + blind spot: for a pure removal `GraphDiff` is entirely empty AND its own |
| 42 | + `seal_status` reads `Wisdom`, while `graph_seal_check` calls the same pair |
| 43 | + `Staunen` — the two surfaces disagree. |
| 44 | +- **CodeRabbit finding, DECLINED with evidence** (reply on |
| 45 | + `#discussion_r3944816260`): "prepend new board rows above the existing |
| 46 | + `D-LNC-*` entries". The `CLAUDE.md` prepend rule names its scope — |
| 47 | + `EPIPHANIES` / `PR_ARC_INVENTORY` / `INTEGRATION_PLANS` / `AGENT_LOG`, the |
| 48 | + chronological logs. `STATUS_BOARD` is a dashboard grouped by plan and is not in |
| 49 | + that list; the last three commits to the section (`8078b69e`, `01569280`, |
| 50 | + `5fe1ffae`) all APPEND. The finding did catch a real defect, just not the one |
| 51 | + it named: the rows were originally inserted mid-table because the patch |
| 52 | + anchored on a convenient unique string. Moved to the append point. |
| 53 | +- **Own prediction falsified by CI:** the PR body claimed |
| 54 | + `SUPERSESSION-INDEX.md` needed no regeneration, reasoning a STATUS_BOARD-only |
| 55 | + change could not move it. `regenerate-and-diff` went red within a minute — |
| 56 | + `crates/` is also an index input, and the new probe calls `persist_cycle`, a |
| 57 | + ruled symbol, moving its reference count 10 → 11. Fixed in `01829f6a`; body |
| 58 | + corrected. |
| 59 | +- **Locked:** R4 with an R1 core (see `LATEST_STATE`). Historical snapshot |
| 60 | + comparison is the correctness ORACLE for the changed set — `inserted`/`updated` |
| 61 | + from `GraphDiff`, `removed` derivable from the same two seal maps `diff()` |
| 62 | + already materialises before discarding their difference. Native lance delta is |
| 63 | + an optional accelerator measured against that, never the architecture; D-LNC-5a's |
| 64 | + RED is a statement about `DatasetDelta`'s configuration needs, not about store A. |
| 65 | +- **Deferred:** `test_c_cycle_payload_roundtrip.rs`, written by the probe fleet |
| 66 | + and deliberately NOT landed — D-TEMPORAL-1 already proves the stronger V1/V3 |
| 67 | + result, and a second green test of the same claim adds review cost without |
| 68 | + adding a falsifier. |
| 69 | +- **NOT proven, and the caveat must ride with any citation:** production |
| 70 | + reachability. `LanceCycleWriter` has zero non-test callers. The architecture |
| 71 | + survived; the writer is dormant. |
| 72 | +- **Confidence:** High on the byte/address/semantic results (all locally |
| 73 | + executed, each failing at least once first). Medium on "no membrane was ever |
| 74 | + intended" — grounded in chronology, a deleted design doc and exhaustive |
| 75 | + negative greps, but no doc ever states the relationship positively, because A |
| 76 | + did not yet exist when B was built. |
| 77 | + |
1 | 78 | ## 2026-09-06 — lance-graph PR #1201 (merged `54285a42`, branch `claude/great-curie-d2ufyl`) — #1199's records, and the SPEC v1 that made it mixed |
2 | 79 |
|
3 | 80 | - **Added:** #1199's post-merge records (`PR_ARC_INVENTORY` entry + the |
|
0 commit comments