docs: RELEASE-NOTES honesty pass — real 0.7.2 delta, historical counts restored - #696
Merged
Merged
Conversation
…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>
This was referenced Aug 3, 2026
Merged
Owner
Author
|
Evicted from the merge queue: batch #736 (integrate/q-1785730980) red with every member individually green |
This was referenced Aug 4, 2026
Owner
Author
|
Evicted from the merge queue: batch #766 (integrate/q-1785803317) red with every member individually green |
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
Four commits. One substantive change (#668) + release metadata.
The section is rewritten to describe exactly that: the
/api/statecache-staleness bug (mtime-gatedCollectorSourceseeded with{};serve_api_stateonly computed inline when the cached payload wasNone, and{}is notNone, so the endpoint served an emptydatasection as live state), the config-reload leak fixed alongside it, and the release-metadata commit. Fix detail is quoted fromfa29fb88'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:254 is the at-tag truth. The section was authored at
366a7fc4as "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:
(Nuance, stated plainly: the 0.7.0 section was authored during the 0.7.1 cycle —
RELEASE-NOTES.mdat thev0.7.0tag 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 viagh:#654c6e465bfinside integration batch #667#667502ccfaf; #667 is the integration batch PR (integrate: 0.7.1 batch — 12 PRs in one CI run), not the fixComplexity-reduction claims were verified against PR titles and are now cited: #637 (
_run_wave_innerF/141→C/13), #664 (wave_schedulerF82→C15), #658 (SSE collector), #662 (tracker reconciliation E39→C19), #663 (do_GETD30→A2).All 13 PR numbers appearing in the touched sections were verified with
gh pr view <n> --json title,state.Scope
RELEASE-NOTES.mdonly. No gate intools/,hooks/, or.github/workflows/references this file.secret_scan.py --staged→CLEAN, exit 0.🤖 Generated with Claude Code