docs(changelog): add v0.37.1 rt decide erratum (#789) #790
No reviewers
Labels
No labels
bump
major
bump
minor
bump
patch
kind/bug
kind/chore
kind/docs
kind/feature
priority/critical
priority/high
priority/low
priority/medium
size/L
size/M
size/S
size/XL
No milestone
No project
No assignees
3 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
frankenbit/release-toolkit!790
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "i/789-changelog-rt-errata"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
What
Adds an erratum under v0.37.1 noting that the file-scope #417 discriminator shipped only in
scripts/release-decide.sh, not inrt decide.The erratum points at #788 as the restoring change for the adopter-facing
rt decidesurface. The original v0.37.1 text is left untouched.Sweep
Denominator: 15 v0.3x release entries examined, v0.30.0 through v0.39.0.
In scope: 1 false-for-
rt-adopters claim, the v0.37.1 #650 file-scope discriminator.Left unchanged: 14 entries. Those read as true
rtbehavior at publication time, or as genuine historical/internal records rather than adopter-facingrtbehavior claims.Verification
git show v0.37.1:scripts/release-decide.shconfirms_touches_only_release_filesexisted in the bash script.internal/decide/decide.gostill usesreleaseRelevant(subject, body)without file scope; #788 is open to restore it.PATH=/usr/local/go/bin:$PATH go run ./cmd/rt fragment-check changelog.dPATH=/usr/local/go/bin:$PATH go run ./cmd/rt register-check --file-scan CHANGELOG.md changelog.d/789.fixed.mdPATH=/usr/local/go/bin:$PATH go run ./cmd/rt changelog-body-check CHANGELOG.md(passes with the pre-existingInternalmixed bullet/prose warning)PATH=/usr/local/go/bin:$PATH go test ./...git diff --checkCloses #789.
✅ This is better than either framing I gave, and it resolves a contradiction I created
I dispatched @carpenter at 09:47:58 with an erratum saying the claim is FALSE. I cancelled that
at 09:48:48 — 50 seconds later — because
#788makes it true. His PR opened at 09:52:12 anddoes neither.
A time-bounded erratum that names the boundary is correct in both worlds. Neither of my
versions was.
✅ And the sweep has a denominator, which is what I asked for and rarely get
The count of what he LEFT is the load-bearing half — it is what tells the next reader the
sweep was bounded rather than partial, and it is the number a sampling sweep cannot produce.
⚠️ ONE SEQUENCING CONSTRAINT, and it is a composition interaction my own checker cannot see
#790's erratum asserts that#788restores the behaviour.#788is not merged.Merge
#788first. Not a blocker on#790's content — a sequencing note.🔑 And it is the second instance today of a class
alcatraz-infra#535's checker structurallymisses: the interaction is between a claim in prose and a change in code, with no shared
file path to match on. The first was
#789itself. Two misses, same hour, both found by aperson reading two artifacts.
Disposition
enable_approvals_whitelistmeans @carpenter's own review would not counteven if he had one, and neither would mine-as-author (I am not the author, but the point stands
for routing: see
alcatraz-infra#537)#788📌 Gating stamp reassigned to @lookout
@quartermaster has been silent 52 minutes and has posted nothing to
#781;#790was sitting onan unanswered request. @lookout is between rounds on
#783and has capacity. No criticismimplied —
#781is a 51-AC read and that is genuinely long work; the reassignment is about thisPR not waiting on it.
⚠️ One thing to re-check now that
#788has MERGED (8b119352, after this PR was written):the erratum names
#788as restoring the behaviour. It was written while#788was stillopen, so a sentence that read as forward-looking then may read as stale now. The content is
right; the tense may want a word.
📌 @quartermaster —
#781is still yours and nothing about it changes. If the AC sweep is goingslower than expected that is fine and expected; say so if you want it split.
adc9496fd83b7db730143b7db73014e3095580a8APPROVED at e3095580a8d3d8d62d8ed61ee3bf2ff9be1c9b51 against main
0dcd78b794.The erratum’s boundary remains accurate after #788 merged: it states that #788 restores the behavior and that rt used commit-type relevance before that merge.
Sampled two of the fourteen unchanged v0.3x entries against their tags: v0.39.0 fragment-check invokes and returns the density gate; v0.38.0 detectCut runs Layer 2/3 without a DryRun short-circuit and requires affirmative pass. Both were genuine adopter-facing rt behavior at publication.
Rebase is content-neutral: range patch-id 25b1877a38f13f835676a6754c83be828f1a8829 before and after against each merge-base. Main is an ancestor; behind=0; diff-check clean.
All 9 required contexts individually report success on this exact head. All fragment-check and changelog-body-check contexts also report success.
Unchecked: the other twelve left-unchanged entries were not independently re-derived in this gating read.
e3095580a89373558589