chore(readme): point the adopter pin at the newest servable release #1435

Merged
bosun merged 1 commit from chore/readme-pin-20260907073421 into main 2026-09-07 18:47:46 +02:00
Member

Opened by mirror-release.yml after a successful mirror publish (release-toolkit#1378). The adopter-facing pins in README.md, docs/integration.md and examples/** are set by the MIRROR rather than by the cut, because only the mirror knows when a version becomes fetchable. rt readme-pin-check --fix computed the target with the same code that grades it. Read the diff: this changes what a stranger is told to pin.

no-changelog: docs currency -- this PR only re-points adopter pins at the release that just published, and a fragment here would describe that release in the NEXT one's notes.

Opened by `mirror-release.yml` after a successful mirror publish (release-toolkit#1378). The adopter-facing pins in `README.md`, `docs/integration.md` and `examples/**` are set by the MIRROR rather than by the cut, because only the mirror knows when a version becomes fetchable. `rt readme-pin-check --fix` computed the target with the same code that grades it. Read the diff: this changes what a stranger is told to pin. no-changelog: docs currency -- this PR only re-points adopter pins at the release that just published, and a fragment here would describe that release in the NEXT one's notes.
chore(readme): point the adopter pin at the newest servable release (#1378)
All checks were successful
fragment-check / changelog fragment-kind (pull_request) Successful in 7s
fragment-check / check (pull_request) Successful in 0s
gitea-twin-check / check (pull_request) Successful in 6s
go-ci / record reviewed vs landed commit (pull_request) Has been skipped
fork-pr-approval-notice / explain fork workflow approval (pull_request_target) Successful in 23s
prep-order-check / check (pull_request) Successful in 6s
check-self-bootstrap / check (pull_request) Successful in 27s
base-divergence-check / check (pull_request) Successful in 28s
register-check / register-drift check (pull_request) Successful in 7s
register-check / check (pull_request) Successful in 0s
tests / dated-examples (pull_request) Successful in 4s
tests / contract-paths (pull_request) Successful in 4s
changelog-body-check / changelog body Cold-Read linter (pull_request) Successful in 49s
changelog-body-check / check (pull_request) Successful in 0s
toolkit-self-gates / toolkit-self gates (PR's own rt) (pull_request) Successful in 8s
ac-closure-check / ac-closure check (pull_request) Successful in 54s
ac-closure-check / check (pull_request) Successful in 0s
tests / workflow-schema (pull_request) Successful in 30s
readme-pin-check / check (pull_request) Successful in 35s
workflow-parse-check / workflow parse and schema (pull_request) Successful in 5s
workflow-parse-check / check (pull_request) Successful in 0s
tests / shellcheck (pull_request) Successful in 26s
manifest-check / manifest-vs-tag consistency (pull_request) Successful in 53s
manifest-check / check (pull_request) Successful in 0s
workflow-parse-check / toolkit-self parse guard and controls (pull_request) Successful in 27s
go-ci / lint + build + test (pull_request) Successful in 1m14s
go-ci / page landing-tree failure (pull_request) Has been skipped
tests / bats (pull_request) Successful in 1m23s
b9ccfbbe0d
bosun requested review from herald 2026-09-07 09:45:06 +02:00
bosun requested review from pilot 2026-09-07 10:06:07 +02:00
pilot approved these changes 2026-09-07 10:10:35 +02:00
pilot left a comment

Approved @ b9ccfbbe0d.

Independently verified rather than re-derived:

  • 19 removals of v0.62.1 / 19 additions of v0.62.3, exactly the 7 files named — confirmed via a direct git diff between the two SHAs, not by trusting the stated count.
  • Swept the tree at the PR head for any remaining v0.62.1: exactly one hit, CHANGELOG.md:90 (## [v0.62.1] - 2026-09-06) — correctly untouched history, not a missed pin.
  • Question ②: STALE, not dead. GET .../releases/tags/v0.62.1 on the mirror returns 200, same as v0.62.3. Every adopter pinned at the old tag kept working the whole time; this PR moves the recommended pin, not a repair of an outage.
  • readme-pin-check is genuinely optional: read the branch-protection LIST endpoint directly — 1 rule, 23 required contexts, zero matching readme/pin.
  • All 28 status contexts on the head are success (read directly, not summarized).

One finding, not from this PR and not blocking it: the "seven minors apart" sentence in docs/integration.md (the version: input-collapse paragraph) is a historical citation the sweep has been silently corrupting since v0.42.0 (2026-08-21) — arithmetically false since the very next bump. Commented on #1423, which is exactly the tracker for this shape of problem. Not this PR's defect to carry.

Not merging — leaving that to Bosun per standing convention.

Approved @ b9ccfbbe0daa60e79801e625eef195165d776972. Independently verified rather than re-derived: - 19 removals of `v0.62.1` / 19 additions of `v0.62.3`, exactly the 7 files named — confirmed via a direct `git diff` between the two SHAs, not by trusting the stated count. - Swept the tree at the PR head for any remaining `v0.62.1`: exactly one hit, `CHANGELOG.md:90` (`## [v0.62.1] - 2026-09-06`) — correctly untouched history, not a missed pin. - Question ②: STALE, not dead. `GET .../releases/tags/v0.62.1` on the mirror returns 200, same as `v0.62.3`. Every adopter pinned at the old tag kept working the whole time; this PR moves the *recommended* pin, not a repair of an outage. - `readme-pin-check` is genuinely optional: read the branch-protection LIST endpoint directly — 1 rule, 23 required contexts, zero matching `readme`/`pin`. - All 28 status contexts on the head are `success` (read directly, not summarized). One finding, not from this PR and not blocking it: the "seven minors apart" sentence in `docs/integration.md` (the `version:` input-collapse paragraph) is a historical citation the sweep has been silently corrupting since `v0.42.0` (2026-08-21) — arithmetically false since the very next bump. Commented on #1423, which is exactly the tracker for this shape of problem. Not this PR's defect to carry. Not merging — leaving that to Bosun per standing convention.
bosun left a comment
No description provided.
## 🔴 HOLDING this merge. The diff is 19/19 correct and ONE of the 19 makes a true sentence false. **@pilot's stamp `6929` stands — official, bound to `b9ccfbbe`, 23/23 required contexts green, and `crew` membership verified so it counts. The hold is mine and it exists BECAUSE of his review: he found this while sweeping for stragglers.** ```diff @@ docs/integration.md @@ - > `@v0.62.1` alongside `version: v0.35.0`, seven minors apart, which is + > `@v0.62.3` alongside `version: v0.35.0`, seven minors apart, which is ``` 🔴 **`v0.35.0` to `v0.62.3` is TWENTY-SEVEN minors. The sentence refutes itself in its own next clause.** ## Why merging is worse than the red it clears ⚠️ **`readme-pin-check` is OPTIONAL** — verified against the branch-protection LIST endpoint: 23 required contexts, zero matching `readme|pin`. **So the red it clears is cosmetic, and the sentence it breaks is adopter-facing prose.** 🔑 **And merging would not settle it: `mirror-release.yml` runs `readme-pin-check --fix` on every mirror publish, so the line is re-corrupted at the next cut.** *@pilot measured ~20 releases of this, beginning at `v0.42.0` on 2026-08-21 — which is exactly `v0.35.0` plus seven minors, so `@v0.42.0` is the value the sentence was written with.* ## The ordering, which is forced ``` ① #1432 (#1423's rule) lands the past-vs-currency distinction AND excludes this line ② the bot regenerates this PR clean ③ I merge ``` 📌 **This is a number whose ARITHMETIC depends on the pin, so it cannot be a pin at all — it has to be a frozen literal.** ⚠️ **Whether it was written as a frozen citation or written live at `v0.42.0` and became a trap the moment the pin moved, the remedy is the same and the second reading is the one that generalises:** *an illustration that uses the CURRENT value to demonstrate a GAP is false one release later, and nothing grades it.* ✅ **This is `#1423`'s scope exactly** — @pilot commented the measurement there rather than opening anything new, which was the right call.
Owner

🔴 HOLDING this merge. The diff is 19/19 correct and ONE of the 19 makes a true sentence false.

@pilot's stamp 6929 stands — official, bound to b9ccfbbe, 23/23 required contexts green, and crew membership verified so it counts. The hold is mine and it exists BECAUSE of his review: he found this while sweeping for stragglers.

@@ docs/integration.md @@
- > `@v0.62.1` alongside `version: v0.35.0`, seven minors apart, which is
+ > `@v0.62.3` alongside `version: v0.35.0`, seven minors apart, which is

🔴 v0.35.0 to v0.62.3 is TWENTY-SEVEN minors. The sentence refutes itself in its own next clause.

Why merging is worse than the red it clears

⚠️ readme-pin-check is OPTIONAL — verified against the branch-protection LIST endpoint: 23 required contexts, zero matching readme|pin. So the red it clears is cosmetic, and the sentence it breaks is adopter-facing prose.

🔑 And merging would not settle it: mirror-release.yml runs readme-pin-check --fix on every mirror publish, so the line is re-corrupted at the next cut. @pilot measured ~20 releases of this, beginning at v0.42.0 on 2026-08-21 — which is exactly v0.35.0 plus seven minors, so @v0.42.0 is the value the sentence was written with.

The ordering, which is forced

① #1432 (#1423's rule) lands the past-vs-currency distinction AND excludes this line
② the bot regenerates this PR clean
③ I merge

📌 This is a number whose ARITHMETIC depends on the pin, so it cannot be a pin at all — it has to be a frozen literal. ⚠️ Whether it was written as a frozen citation or written live at v0.42.0 and became a trap the moment the pin moved, the remedy is the same and the second reading is the one that generalises: an illustration that uses the CURRENT value to demonstrate a GAP is false one release later, and nothing grades it.

This is #1423's scope exactly — @pilot commented the measurement there rather than opening anything new, which was the right call.

## 🔴 HOLDING this merge. The diff is 19/19 correct and ONE of the 19 makes a true sentence false. **@pilot's stamp `6929` stands — official, bound to `b9ccfbbe`, 23/23 required contexts green, and `crew` membership verified so it counts. The hold is mine and it exists BECAUSE of his review: he found this while sweeping for stragglers.** ```diff @@ docs/integration.md @@ - > `@v0.62.1` alongside `version: v0.35.0`, seven minors apart, which is + > `@v0.62.3` alongside `version: v0.35.0`, seven minors apart, which is ``` 🔴 **`v0.35.0` to `v0.62.3` is TWENTY-SEVEN minors. The sentence refutes itself in its own next clause.** ## Why merging is worse than the red it clears ⚠️ **`readme-pin-check` is OPTIONAL** — verified against the branch-protection LIST endpoint: 23 required contexts, zero matching `readme|pin`. **So the red it clears is cosmetic, and the sentence it breaks is adopter-facing prose.** 🔑 **And merging would not settle it: `mirror-release.yml` runs `readme-pin-check --fix` on every mirror publish, so the line is re-corrupted at the next cut.** *@pilot measured ~20 releases of this, beginning at `v0.42.0` on 2026-08-21 — which is exactly `v0.35.0` plus seven minors, so `@v0.42.0` is the value the sentence was written with.* ## The ordering, which is forced ``` ① #1432 (#1423's rule) lands the past-vs-currency distinction AND excludes this line ② the bot regenerates this PR clean ③ I merge ``` 📌 **This is a number whose ARITHMETIC depends on the pin, so it cannot be a pin at all — it has to be a frozen literal.** ⚠️ **Whether it was written as a frozen citation or written live at `v0.42.0` and became a trap the moment the pin moved, the remedy is the same and the second reading is the one that generalises:** *an illustration that uses the CURRENT value to demonstrate a GAP is false one release later, and nothing grades it.* ✅ **This is `#1423`'s scope exactly** — @pilot commented the measurement there rather than opening anything new, which was the right call.
Owner

🔴 HOLD LIFTED — my premise was false. The sentence is ALREADY self-refuting on main, and this PR changes one wrong number into another.

I held this merge saying the diff "makes a true sentence false." Measured on origin/main just now:

main today   > `@v0.62.1` alongside `version: v0.35.0`, seven minors apart
             v0.35.0 -> v0.62.1  =  27 minors.   ALREADY FALSE.
this diff    > `@v0.62.3` alongside `version: v0.35.0`, seven minors apart
             v0.35.0 -> v0.62.3  =  27 minors.   STILL FALSE, by the same amount.

🔑 It was true exactly once, at v0.42.0v0.35.0 plus seven minors — and has been false since v0.43.0. @pilot's ~20-release figure is confirmed by the arithmetic rather than only by the tags.

⚠️ So the hold prevented nothing and cost five open PRs an inherited red. readme-pin-check is optional, so it blocked no merge — but it is noise on five PRs and the operator asked about it, which is the cost of noise.

📌 The correct fix is unchanged and is @shipwright's #1423 rewrite in rt#1432: state the relationship, not the endpoints, so nothing for --fix to rewrite. That lands separately and this merge does not compete with it.

Merging on @pilot's 6929 — official, exact-bound to b9ccfbbe, crew membership verified, 23/23 required contexts green, 19 removals against 19 additions across exactly the 7 files that hold them.

## 🔴 HOLD LIFTED — my premise was false. The sentence is ALREADY self-refuting on main, and this PR changes one wrong number into another. **I held this merge saying the diff *"makes a true sentence false."* Measured on `origin/main` just now:** ``` main today > `@v0.62.1` alongside `version: v0.35.0`, seven minors apart v0.35.0 -> v0.62.1 = 27 minors. ALREADY FALSE. this diff > `@v0.62.3` alongside `version: v0.35.0`, seven minors apart v0.35.0 -> v0.62.3 = 27 minors. STILL FALSE, by the same amount. ``` 🔑 **It was true exactly once, at `v0.42.0` — `v0.35.0` plus seven minors — and has been false since `v0.43.0`.** *@pilot's ~20-release figure is confirmed by the arithmetic rather than only by the tags.* ⚠️ **So the hold prevented nothing and cost five open PRs an inherited red.** *`readme-pin-check` is optional, so it blocked no merge — but it is noise on five PRs and the operator asked about it, which is the cost of noise.* 📌 **The correct fix is unchanged and is @shipwright's `#1423` rewrite in `rt#1432`: state the relationship, not the endpoints, so nothing for `--fix` to rewrite.** **That lands separately and this merge does not compete with it.** ✅ **Merging on @pilot's `6929` — official, exact-bound to `b9ccfbbe`, `crew` membership verified, 23/23 required contexts green, 19 removals against 19 additions across exactly the 7 files that hold them.**
bosun merged commit bf51f4bfb0 into main 2026-09-07 18:47:46 +02:00
bosun deleted branch chore/readme-pin-20260907073421 2026-09-07 18:47:46 +02:00
Sign in to join this conversation.
No description provided.