Skip to content

docs: RELEASE-NOTES honesty pass — real 0.7.2 delta, historical counts restored - #696

Merged
matt82198 merged 3 commits into
mainfrom
docs/release-notes-honesty
Aug 4, 2026
Merged

matt82198 merged 3 commits into
mainfrom
docs/release-notes-honesty

Conversation

@matt82198

Copy link
Copy Markdown
Owner

The finding

RELEASE-NOTES.md's 0.7.2 section was fabricated — a near-verbatim copy of the 0.7.1 section's PR list. Every PR it cited (#638, #636, #643, #645, #647) shipped in 0.7.1, not 0.7.2. The one change that actually shipped in 0.7.2 appeared nowhere in it.

Both defects were introduced by the same commit, 2f52f39f (2026-08-02, "docs: RELEASE-NOTES/README currency to 0.7.2 with gate-derived test counts (266 tests)").

Ground truth: what 0.7.2 actually was

$ git log v0.7.1..v0.7.2 --oneline
e061f2bd Merge pull request #669 from matt82198/release/0.7.2
a00deb7f release: 0.7.2
a87c3966 Merge pull request #668 from matt82198/fix/api-state-empty-section-cache
fa29fb88 fix: /api/state served a collector's empty default instead of real data

Four commits. One substantive change (#668) + release metadata.

$ git show fa29fb88 --stat
 tests/test_agent_detail_roundtrip.py |  4 ++++
 ui/CLAUDE.md                         |  1 +
 ui/handler.py                        | 15 +++++++++++----

$ git show a00deb7f --stat
 CHANGELOG.md | 7 +++++++
 STATE.md     | 2 +-
 package.json | 2 +-

The section is rewritten to describe exactly that: the /api/state cache-staleness bug (mtime-gated CollectorSource seeded with {}; serve_api_state only computed inline when the cached payload was None, and {} is not None, so the endpoint served an empty data section as live state), the config-reload leak fixed alongside it, and the release-metadata commit. Fix detail is quoted from fa29fb88's own commit body.

Ground truth: the 0.7.0 test count

The 0.7.0 section's historical count was retroactively rewritten 254 → 266:

$ git show 2f52f39f -- RELEASE-NOTES.md
-3. **Test suite accountability**: 254 tests across 3 harnesses (25 Node + 13 Shell + 216 Python) ...
+3. **Test suite accountability**: 266 tests across 3 harnesses (26 Node + 13 Shell + 227 Python) ...

254 is the at-tag truth. The section was authored at 366a7fc4 as "254 tests across 3 harnesses (25 Node + 13 Shell + 216 Python)", and PR #644 ("docs: fix version drift to 0.7.0 and correct test count to 254") had verified that number for the release. Rewriting a shipped release's numbers to match a later, larger suite falsifies the record — a release note is a historical document, not a live dashboard.

254 is restored, with the currency concern handled by a footnote instead:

[count as of the v0.7.0 tag; suite has since grown — see current README]

(Nuance, stated plainly: the 0.7.0 section was authored during the 0.7.1 cycle — RELEASE-NOTES.md at the v0.7.0 tag only reaches back to 0.5.0. 254 is nonetheless the count as verified for that release by #644, and is the value the section shipped with.)

0.7.1 cross-check

Cross-checked against git log v0.7.0..v0.7.1 --oneline. Not fabricated — every claim traces to a commit in range. Two citations were wrong and are corrected, verified via gh:

Claim Was Verified truth
merge-train Unicode #654 #654 is CLOSED. Change shipped as c6e465bf inside integration batch #667
portability-gate verdict #667 502ccfaf; #667 is the integration batch PR (integrate: 0.7.1 batch — 12 PRs in one CI run), not the fix

Complexity-reduction claims were verified against PR titles and are now cited: #637 (_run_wave_inner F/141→C/13), #664 (wave_scheduler F82→C15), #658 (SSE collector), #662 (tracker reconciliation E39→C19), #663 (do_GET D30→A2).

All 13 PR numbers appearing in the touched sections were verified with gh pr view <n> --json title,state.

Scope

RELEASE-NOTES.md only. No gate in tools/, hooks/, or .github/workflows/ references this file. secret_scan.py --stagedCLEAN, exit 0.

🤖 Generated with Claude Code

…s restored

The 0.7.2 section was fabricated. Commit 2f52f39 authored it as a near-verbatim
copy of the 0.7.1 section's PR list -- #638, #636, #643, #645, #647 all shipped in
0.7.1, none of them in v0.7.1..v0.7.2. The one change that actually shipped in
0.7.2 (the /api/state cache-staleness fix, #668) appeared nowhere in it.

Ground truth, `git log v0.7.1..v0.7.2 --oneline`, four commits:

  e061f2b Merge pull request #669 from matt82198/release/0.7.2
  a00deb7 release: 0.7.2
  a87c396 Merge pull request #668 from matt82198/fix/api-state-empty-section-cache
  fa29fb8 fix: /api/state served a collector's empty default instead of real data

The section is rewritten from that: one substantive fix plus release metadata.

The same commit also rewrote the 0.7.0 section's historical test count from 254 to
266. 254 was the at-tag truth -- the section was authored at 366a7fc as "254 tests
(25 Node + 13 Shell + 216 Python)", and PR #644 ("correct test count to 254") had
verified it for that release. Rewriting a shipped release's numbers to match a
later suite falsifies the record; 254 is restored with a dated footnote pointing
currency at the README.

0.7.1 was cross-checked against `git log v0.7.0..v0.7.1` and is substantively
accurate, but two citations were wrong and are corrected: #654 was CLOSED (the
merge-train UTF-8 change shipped as c6e465b inside batch #667), and the
portability-gate/remote-identity work is 502ccfa, not #667 itself (#667 is the
integration batch PR). Complexity-reduction claims verified and now cited
(#637, #664, #658, #662, #663).

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@matt82198 matt82198 added the merge-queue Queued for the merge-queue advancer daemon label Aug 3, 2026
@matt82198

Copy link
Copy Markdown
Owner Author

Evicted from the merge queue: batch #736 (integrate/q-1785730980) red with every member individually green

@matt82198 matt82198 added the merge-queue Queued for the merge-queue advancer daemon label Aug 3, 2026
@matt82198 matt82198 removed the queue-rejected Evicted from the merge queue (red or culprit) label Aug 4, 2026
@matt82198 matt82198 added queue-rejected Evicted from the merge queue (red or culprit) and removed merge-queue Queued for the merge-queue advancer daemon labels Aug 4, 2026
@matt82198

Copy link
Copy Markdown
Owner Author

Evicted from the merge queue: batch #766 (integrate/q-1785803317) red with every member individually green

@matt82198 matt82198 added merge-queue Queued for the merge-queue advancer daemon and removed queue-rejected Evicted from the merge queue (red or culprit) labels Aug 4, 2026
@matt82198
matt82198 merged commit 34c601b into main Aug 4, 2026
9 checks passed
@matt82198
matt82198 deleted the docs/release-notes-honesty branch August 4, 2026 04:12
@matt82198 matt82198 mentioned this pull request Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-queue Queued for the merge-queue advancer daemon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant