From 772b5e559399e331e699d6a905c82ad640b06178 Mon Sep 17 00:00:00 2001 From: Claude Date: Tue, 4 Aug 2026 12:55:19 +0000 Subject: [PATCH] board: post-merge arc + state entry for #884 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit #884 was itself the post-merge hygiene PR for #881/#882/#883 — and it merged without its own arc entry, reproducing one level up exactly the gap it was written to close. Writing the hygiene PR does not discharge the rule for the hygiene PR. Entry added, with that noted in it. Records what #884 locked: - The C1 audit finding: phi = pearson-on-binaries and KR-20 = alpha-on-dichotomous are the SAME computations already in jc (named wrapper + caveat only, arithmetic not re-implemented); kappa is absent and is NOT a renamed ICC — the real gap, and the one that blocks D3's fusion falsifier. - Effect size means the r-family: R, R-squared, eta-squared, phi. Cohen's d explicitly out, calculated separately if ever wanted. The t-test is the significance companion to eta-squared, not a d-family route. - ADDITIVE ONLY with one carve-out: existing jc statistics keep their arithmetic, signature and semantics; the sole permitted edit to an existing file is widening reliability.rs's private helpers to pub(crate) for reuse — visibility only. - The transport-vs-engine separation carried from #883, and that the spawn inventory carries no architectural legitimacy. Board prose only. D-KIA-C1b remains Queued. Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_01K3RyLEbuNSHxxB3NTTrGki --- .claude/board/LATEST_STATE.md | 16 ++++++++++++++++ .claude/board/PR_ARC_INVENTORY.md | 14 ++++++++++++++ 2 files changed, 30 insertions(+) diff --git a/.claude/board/LATEST_STATE.md b/.claude/board/LATEST_STATE.md index 47fe07ed..cb68cb9b 100644 --- a/.claude/board/LATEST_STATE.md +++ b/.claude/board/LATEST_STATE.md @@ -1,3 +1,19 @@ +## 2026-08-04 — branch `claude/x265-x266-plans-review-h9osnl` (PR #884 MERGED `1e90cef`) — D-KIA-C1b scoped: the r-family, additive-only + +Board/plan prose only; **no code, no runtime behaviour**. Carried the post-merge arc entries for #881/#882/#883 (see the entry below) and re-scoped the statistics work removed from #883 as its own deliverable, **D-KIA-C1b** (`Queued` — scope, not code). + +**C1 audit (read-only).** `jc` is in-tree at `crates/jc/`; `reliability.rs` ships `pearson` / `spearman` / `cronbach_alpha` / `icc(ratings, IccForm)` (`Icc2_1`, `Icc3_1`), and `jirak.rs` gives C4's noise floors a local implementation to cite. Two of C2's three renames are the **same computation**, one is a real gap: + +- **φ = Pearson r on two binary variables** → `jc::reliability::pearson` already computes it. Only a *named wrapper* + the marginal-capped-ceiling caveat are new; the arithmetic is not re-implemented. +- **KR-20 = Cronbach's α on dichotomous items** → `cronbach_alpha` is the right function; naming + caveat only. +- **κ is absent entirely and is NOT a renamed ICC** — a different estimator. **This is the gap, and it blocks D3's fusion falsifier.** + +**Effect size means the r-family (operator ruling):** R, R², **η²** (*erklärte Varianz*), φ. **Cohen's d is explicitly OUT** — calculated separately if a mean-difference contrast is ever wanted. The **t-test** (t/df/p) is in scope as the *significance* companion to η², not a d-family route: effect size is read off η²/R². This supersedes the vague "Effektstärke / effect size" wording the PR opened with. + +**ADDITIVE ONLY, with exactly one carve-out.** `pearson` / `spearman` / `cronbach_alpha` / `icc` keep their **arithmetic, signature and semantics**; new estimators land in a new module beside them, and any diff changing an existing `jc` statistic is an automatic reject. **The one sanctioned edit to an existing file is visibility only:** widening `reliability.rs`'s private helpers (`mean` / `all_finite` / `average_ranks` / `pop_var`) to `pub(crate)` plus the `pub mod` line, so the new module reuses them rather than growing a second source of truth. No body change, no signature change, no incidental cleanup. + +**Process note.** #884 existed because #881/#882/#883 merged without arc entries — then merged without its own. Writing the hygiene PR does not discharge the rule for the hygiene PR itself; its entry is now in the arc. + ## 2026-08-04 — branch `claude/x265-x266-plans-review-h9osnl` (PRs #881, #882, #883 MERGED) — arc hygiene restored + the legacy actor surface quarantined **#881** post-merge arc/state for #880 and recorded that the arc had gone stale for four PRs. **#882** backfilled those four (#862/#875/#876/#879) as marked RECONSTRUCTIONS, plus the cross-PR finding that every review-found defect in three consecutive probe PRs sat in a falsifier or a label, never in a measurement. diff --git a/.claude/board/PR_ARC_INVENTORY.md b/.claude/board/PR_ARC_INVENTORY.md index 0a42253d..dbb2b227 100644 --- a/.claude/board/PR_ARC_INVENTORY.md +++ b/.claude/board/PR_ARC_INVENTORY.md @@ -33,6 +33,20 @@ > - **Docs** — knowledge files produced (immutable) > - **Confidence (YYYY-MM-DD):** — the ONLY mutable field +## 2026-08-04 — lance-graph #884 — post-merge arc entries for #881/#882/#883 + D-KIA-C1b statistics re-scope + +**Merged:** `1e90cef` (branch `claude/x265-x266-plans-review-h9osnl`, head `c6acd75`). 4 files, board/plan prose only — no code, no runtime behaviour. + +- **Added.** Arc entries for #881, #882, #883; the matching `LATEST_STATE` 2026-08-04 entry; the C1 audit result and the C1b deliverable in `.claude/plans/kanban-64k-inverted-awareness-v1.md`; the `D-KIA-C1b` row in `STATUS_BOARD.md`. +- **Locked — the C1 audit finding.** `jc` is in-tree at `crates/jc/`. Two of C2's three renames are the SAME computation and one is a genuine gap: **φ = Pearson r on two binary variables** (`jc::reliability::pearson` already computes it — a named wrapper + the marginal-capped-ceiling caveat are the only new surface, the arithmetic is NOT re-implemented); **KR-20 = Cronbach's α on dichotomous items** (`cronbach_alpha` is the right function, naming + caveat only); **κ is absent entirely and is NOT a renamed ICC** — a different estimator, and the one that blocks D3's fusion falsifier. +- **Locked — effect size means the r-family (operator, 2026-08-04).** R, R², **η²** (*erklärte Varianz*), φ. **Cohen's d is explicitly OUT of D-KIA-C1b** — calculated separately if a mean-difference contrast is ever wanted. The **t-test** (t/df/p) is in scope as the *significance* companion to η², not a d-family route: effect size is read off η²/R². Supersedes the vague "Effektstärke / effect size" wording this PR itself opened with; the correction landed in `c6acd75` before merge. +- **Locked — ADDITIVE ONLY, with exactly one carve-out.** `pearson` / `spearman` / `cronbach_alpha` / `icc` keep their **arithmetic, signature and semantics**; new estimators land in a new module beside them. Any diff changing an existing `jc` statistic is an automatic reject, independent of merit. **The one sanctioned edit to an existing file is visibility only:** widening `reliability.rs`'s private helpers (`mean` / `all_finite` / `average_ranks` / `pop_var`) to `pub(crate)` plus the `pub mod` line in `lib.rs`, so the new module reuses them instead of growing a second source of truth. No body change, no signature change, no incidental cleanup. +- **Locked — the separation carried into #883's arc entry.** `MulAdvance` / `drive_mul_advance` are legacy actor-message **wrappers**, not the canonical MUL reasoning engine; `gate_decision_i4` is independent, consumed directly by the #879 path via `shade_owner` / `run_cognitive_work_gated[_over]`, and **not deprecated**. The NARS tactic recipes and the awareness rung ladder are separate and untouched. +- **Locked — the spawn inventory carries no architectural legitimacy.** Recorded as deletion-impact evidence and a removal work-list only. +- **Deferred.** D-KIA-C1b implementation (remains **Queued** — this PR is scope, not code); D-KIA-A1's rescoped OPEN items (`run_cycle` drained-writer retry guard; missing-owner counter in `cognitive_pass`). +- **Process note.** This PR existed because #881/#882/#883 merged without their arc entries. It then merged without its OWN entry — the same gap one level up, closed by this entry. The lesson is that "write the hygiene PR" does not discharge the rule for the hygiene PR itself. +- **Confidence (2026-08-04):** working — prose only, no behaviour to regress. + ## 2026-08-04 — lance-graph #883 — legacy actor-surface quarantine + stale phase-progression comment correction **Merged:** `48d4841` (branch `claude/x265-x266-plans-review-h9osnl`, head `c65a405`). 10 files, comment/doc-only.