docs: confirm the predicted CI counts, and re-record the head_sha trap
Acceptance 8 in its strongest form. The ledger predicted the post-round-2 job totals from the census — 3,766 for Test (crdt) and 3,487 for Test (ubuntu/luajit) — and run 30706324644 @71a1ebdreported exactly those. Predicting the count before the run and matching it says more than reconciling one afterwards: a vacuous leg would have landed near the non-crdt job's figure. Also records two readings that would otherwise be misread: The macOS legs report 3,474, thirteen fewer than ubuntu's 3,487. That is cfg-compilation, not lost coverage — the Linux-gated process tests (setsid, the `bash -m` job-control corroboration) are not built there rather than skipped. And the head_sha trap bit again. Round 1's run (30705916037 @6519bc3) was CANCELLED, not green: round 2's push superseded it, which is the concurrency group working exactly as its comment in ci.yml describes. A `gh pr checks` summary polled around that moment reported the PREVIOUS run's results — with entirely plausible timings — and round 1 was briefly reported as passing on that basis. This ledger already carried the lesson from #178 ("verified by head_sha, not by the check summary"). It recurs because the wrong answer looks exactly like the right one, so it is now recorded against this lane too. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
parent
71a1ebd4b7
commit
9f28816ce2
|
|
@ -179,13 +179,31 @@ in the primary checkout on the laptop, not a worktree.
|
||||||
**The dark count is the all-targets comparison, 3,746 − 3,467 = 279.**
|
**The dark count is the all-targets comparison, 3,746 − 3,467 = 279.**
|
||||||
A reviewer who subtracts job totals gets a wrong number that looks
|
A reviewer who subtracts job totals gets a wrong number that looks
|
||||||
entirely plausible.
|
entirely plausible.
|
||||||
- **Those two figures are already superseded** and are kept only to
|
- **Those two figures are superseded** and are kept only to explain
|
||||||
explain the trap. Review round 1 added a pmacs-protocol step to
|
the trap. Round 1 added a pmacs-protocol step to `crdt-test` and
|
||||||
`crdt-test`, and round 2 added two capability tests to that crate,
|
round 2 added two capability tests to that crate. The current
|
||||||
so the expected totals are now **3,487** (3,467 + 1 + 19) and
|
totals were **predicted from the census and then confirmed exactly**
|
||||||
**3,766** (3,746 + 1 + 19). The root-package census is untouched at
|
by run `30706324644` @ `71a1ebd`: `Test (crdt)` **3,766**
|
||||||
3,467 / 3,746 — the new tests live in a sibling crate, which is
|
(3,746 + 1 doc + 19 protocol) and `Test (ubuntu/luajit)` **3,487**
|
||||||
exactly the region `scripts/feature-census` cannot see.
|
(3,467 + 1 + 19). Predicting the count *before* the run and matching
|
||||||
|
it is a stronger reading of acceptance 8 than reconciling afterwards.
|
||||||
|
- The root-package census is untouched at 3,467 / 3,746 — the new
|
||||||
|
tests live in a sibling crate, exactly the region
|
||||||
|
`scripts/feature-census` cannot see.
|
||||||
|
- **The macOS legs report 3,474, thirteen fewer than ubuntu's 3,487**,
|
||||||
|
and that is expected: the Linux-gated process tests
|
||||||
|
(`setsid`, the `bash -m` job-control corroboration) are
|
||||||
|
`cfg`-compiled out rather than skipped. Do not read it as macOS
|
||||||
|
coverage loss.
|
||||||
|
- **Verify CI by `head_sha`, never by the check summary — it bit again
|
||||||
|
here.** Round 1's run (`30705916037` @ `6519bc3`) was **cancelled**,
|
||||||
|
not green: round 2's push superseded it, which is the concurrency
|
||||||
|
group working as designed. A `gh pr checks` summary polled around that
|
||||||
|
moment reported the *previous* run's results, with plausible timings,
|
||||||
|
and was briefly reported as round 1 passing. The ledger already
|
||||||
|
carried this lesson from #178 ("twelve checks green on head `1b44c69`
|
||||||
|
— verified by `head_sha`, not by the check summary"); it recurs
|
||||||
|
because the wrong answer looks exactly like the right one.
|
||||||
|
|
||||||
- **Root cause, unchanged:** `.github/workflows/ci.yml` never enabled
|
- **Root cause, unchanged:** `.github/workflows/ci.yml` never enabled
|
||||||
the `crdt` feature anywhere. Every `#[cfg(feature = "crdt")]` test was
|
the `crdt` feature anywhere. Every `#[cfg(feature = "crdt")]` test was
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue