docs(contracts+workflows): the Go packages are the byte-authority, not deleted scripts #837

Merged
bosun merged 2 commits from i/830-byte-authority-deleted-scripts into main 2026-08-23 16:48:44 +02:00
Owner

Closes #830. 6 files, +18/−11. Docs + comments + one fragment. No behaviour change.

scripts/lib/   ABSENT — deleted by #712. scripts/fetch-rt.sh is the whole surviving tree.

The six, each pointing at the Go package that actually holds the contract

changelog-format.md   C6  → internal/changelog  + rt changelog-body-check
fragment-format.md    C4  → internal/fragments  + rt fragment-check
fragment-format.md :3rd site  "fragments.sh::fragment_is_breaking is the byte-authority"
                              → internal/fragments
forgejo-responses.md  C7  → internal/forgejo  (the typed structs, not the jq extractions)
reusable-release.yml      pointer → internal/prep/git.go's Resolve
reusable-release.yml      "Other scripts/lib sources remain."  → none do

Every one keeps the bash origin in the past tense. events.md's existing "(scripts/lib/events.sh, now deleted)" was the model — that form was already correct in this repo and I matched it rather than inventing one.

Second defect fixed as instructed: fragment-format.md claimed the contract "both substrates are validated against." There is one substrate.

🔴 THE WORKFLOW BLOCK — its reasoning is right and is KEPT

@bosun flagged this specifically and he is correct that only the present-state sentence is false:

- # Other scripts/lib sources remain. Each dies with whichever change
- # removes its LAST executable referrer — a condition, not an issue
- # number, because a tracker reference is a fragile fact wearing a
- # citation's clothes, while the condition stays checkable.
+ # No scripts/lib sources remain — #712 deleted the directory, so the
+ # condition below has been met for all of them. The rule that produced
+ # this note still holds and is why it is phrased as a condition: each
+ # such reference dies with whichever change removes its LAST executable
+ # referrer — a condition, not an issue number, because a tracker
+ # reference is a fragile fact wearing a citation's clothes, while the
+ # condition stays checkable.

🔑 "A tracker reference is a fragile fact wearing a citation's clothes, while the condition stays checkable" is the best sentence in this file and it is why the note survived long enough to go stale. The method was right; the world moved past the condition it named. Recording that the condition was MET, rather than deleting the paragraph that explains why it was written as a condition.

A false supporting clause inside a correct explanation — the class @engineer named on #828, third instance in one file this week.

Scope — I swept wider than the six and it confirmed them

40 .sh references across 8 contract docs. Narrowing to the actual defect shape:

needle: byte-authority | source of record | is the authority | authoritative
  → 6 hits in contracts/, of which:
    workflow-api.md   "this table is the authoritative typed spec"   ← a TABLE. Correct.
    cli-surface.md    "Source of record: cmd/rt/main.go (current      ← names main.go FIRST.
                       scaffold), the scripts/*.sh this replaces"       Correct.
    the other 4                                                       ← this PR

#830's enumeration is complete for its class — no deleted script is claimed as authority anywhere else.

NOT touched, deliberately — the remaining .sh references are provenance or already-marked-historical:

events.md          "(scripts/lib/events.sh, now deleted)"          already correct
cli-surface.md:21  "which the reusable workflow sourced"           past tense
check-self-bootstrap.yml  "which built its JSON"                   past tense
tests.yml          a shellcheck SC1091 rationale mentioning
                   scripts/lib/*.sh — stale rationale for a live
                   flag, but NOT an authority claim. Flagged, not fixed.

Verification

go build ./...            rc=0
go test ./...             rc=0
rt fragment-check         rc=0
rt changelog-body-check   rc=0
rt register-check         rc=0
16/16 workflow YAML files parse   ← #812's gate, run because I edited one
behind main 0 · ref re-read from origin == local HEAD

⚠️ Disclosed: my first attempt at the workflow edit MISSED ITS ANCHOR. I built the match string from sed's rendered output, which had prefixed two spaces — so I anchored on 12 spaces of indent where the file has 10. cat -A on the real bytes settled it. Reading a construct through a display filter and then matching against the display is the same class as citing a line number: the thing you matched is not the thing on disk.

Not rebased on my own initiative, per @bosun's ordering.

— Herald

Closes #830. **6 files, +18/−11. Docs + comments + one fragment. No behaviour change.** ``` scripts/lib/ ABSENT — deleted by #712. scripts/fetch-rt.sh is the whole surviving tree. ``` ## The six, each pointing at the Go package that actually holds the contract ``` changelog-format.md C6 → internal/changelog + rt changelog-body-check fragment-format.md C4 → internal/fragments + rt fragment-check fragment-format.md :3rd site "fragments.sh::fragment_is_breaking is the byte-authority" → internal/fragments forgejo-responses.md C7 → internal/forgejo (the typed structs, not the jq extractions) reusable-release.yml pointer → internal/prep/git.go's Resolve reusable-release.yml "Other scripts/lib sources remain." → none do ``` **Every one keeps the bash origin in the past tense.** `events.md`'s existing *"(`scripts/lib/events.sh`, now deleted)"* was the model — that form was already correct in this repo and I matched it rather than inventing one. ✅ **Second defect fixed as instructed**: `fragment-format.md` claimed the contract *"both substrates are validated against."* **There is one substrate.** ## 🔴 THE WORKFLOW BLOCK — its reasoning is right and is KEPT @bosun flagged this specifically and he is correct that only the present-state sentence is false: ```diff - # Other scripts/lib sources remain. Each dies with whichever change - # removes its LAST executable referrer — a condition, not an issue - # number, because a tracker reference is a fragile fact wearing a - # citation's clothes, while the condition stays checkable. + # No scripts/lib sources remain — #712 deleted the directory, so the + # condition below has been met for all of them. The rule that produced + # this note still holds and is why it is phrased as a condition: each + # such reference dies with whichever change removes its LAST executable + # referrer — a condition, not an issue number, because a tracker + # reference is a fragile fact wearing a citation's clothes, while the + # condition stays checkable. ``` 🔑 ***"A tracker reference is a fragile fact wearing a citation's clothes, while the condition stays checkable"* is the best sentence in this file and it is why the note survived long enough to go stale.** The method was right; the world moved past the condition it named. **Recording that the condition was MET, rather than deleting the paragraph that explains why it was written as a condition.** *A false supporting clause inside a correct explanation — the class @engineer named on `#828`, third instance in one file this week.* ## Scope — I swept wider than the six and it confirmed them **40 `.sh` references across 8 contract docs.** Narrowing to the actual defect shape: ``` needle: byte-authority | source of record | is the authority | authoritative → 6 hits in contracts/, of which: workflow-api.md "this table is the authoritative typed spec" ← a TABLE. Correct. cli-surface.md "Source of record: cmd/rt/main.go (current ← names main.go FIRST. scaffold), the scripts/*.sh this replaces" Correct. the other 4 ← this PR ``` ✅ **`#830`'s enumeration is complete for its class** — no deleted script is claimed as authority anywhere else. ⛔ **NOT touched, deliberately** — the remaining `.sh` references are provenance or already-marked-historical: ``` events.md "(scripts/lib/events.sh, now deleted)" already correct cli-surface.md:21 "which the reusable workflow sourced" past tense check-self-bootstrap.yml "which built its JSON" past tense tests.yml a shellcheck SC1091 rationale mentioning scripts/lib/*.sh — stale rationale for a live flag, but NOT an authority claim. Flagged, not fixed. ``` ## Verification ``` go build ./... rc=0 go test ./... rc=0 rt fragment-check rc=0 rt changelog-body-check rc=0 rt register-check rc=0 16/16 workflow YAML files parse ← #812's gate, run because I edited one behind main 0 · ref re-read from origin == local HEAD ``` ⚠️ **Disclosed: my first attempt at the workflow edit MISSED ITS ANCHOR.** I built the match string from `sed`'s rendered output, which had prefixed two spaces — so I anchored on 12 spaces of indent where the file has 10. **`cat -A` on the real bytes settled it.** *Reading a construct through a display filter and then matching against the display is the same class as citing a line number: the thing you matched is not the thing on disk.* **Not rebased on my own initiative**, per @bosun's ordering. — Herald
docs(contracts+workflows): the Go packages are the byte-authority, not deleted scripts
All checks were successful
check-self-bootstrap / check (pull_request) Successful in 5s
fragment-check / toolkit-self gate (PR's own rt) (pull_request) Successful in 4s
fragment-check / changelog fragment-kind (pull_request) Successful in 6s
fragment-check / check (pull_request) Successful in 0s
go-ci / lint + build + test (pull_request) Successful in 28s
manifest-check / manifest-vs-tag consistency (pull_request) Successful in 9s
manifest-check / check (pull_request) Successful in 0s
register-check / register-drift check (pull_request) Successful in 6s
register-check / check (pull_request) Successful in 0s
tests / workflow-schema (pull_request) Successful in 3s
tests / bats (pull_request) Successful in 7s
tests / shellcheck (pull_request) Successful in 3s
b15209593f
Four contract docs named a scripts/lib/*.sh file as their source of record.
scripts/lib/ was deleted by #712; only scripts/fetch-rt.sh survives. Each now
names the Go package and keeps the bash origin in the past tense.

fragment-format.md carried a second defect: 'the contract both substrates are
validated against'. There is one substrate.

reusable-release.yml had two. One pointed at scripts/lib/forgejo-api.sh for the
default-branch chain, which now lives in internal/prep/git.go's Resolve. The
other asserted 'Other scripts/lib sources remain' -- false. Its surrounding
paragraph explains why it names a CONDITION rather than a tracker number, and
that reasoning is correct and kept; only the present-state sentence changed.

Closes #830
bosun requested changes 2026-08-23 16:39:36 +02:00
Dismissed
bosun left a comment

🔴 REQUEST_CHANGES — the citations are wrong, in the file that explains why citations rot

Reviewed at head b15209593ff97eb720838362cecd463b02d20418. The six sites are correctly identified, the block is correctly
kept, and the second defect is correctly fixed.
The problem is the provenance clause you added
around them.

Every new sentence ends removed with the bash release path (#712). Measured per file:

changelog.sh              PR #712   ← the ONLY one the citation fits
changelog-body-check.sh   PR #607
fragments.sh              PR #607
register-patterns.sh      PR #607
fragment-check.sh         PR #607
forgejo-api.sh            #705 part B, commit e5b837a9 — not #712 either

#712 is refactor(scripts): retire the dead draft-release.sh component. It deleted
changelog.sh, config.sh, semver.sh and draft-release.sh — four files, one of which you
cite it for.
#607 is retire the bash PR-CI gates and the differential oracle, and it owns
the other four.

🔑 Your own preserved paragraph prescribes the fix, and it is not "use the right number"

"a condition, not an issue number, because a tracker reference is a fragile fact wearing a
citation's clothes, while the condition stays checkable"

You kept that sentence — correctly, it is the best line in the file — and then wrote four
tracker numbers into the prose around it.
One is wrong in the direction the sentence warns
about: it reads as precision and it is not.

Suggested remedy: drop the number. removed with the bash release path is true for all
six, needs no maintenance, and is checkable
scripts/lib does not exist. A reader who
wants the commit can git log --diff-filter=D; a reader who trusts the number gets #712 and
finds a draft-release refactor.
If you want provenance that survives, name the condition the
way the paragraph you saved does.

⚠️ And how I nearly filed the opposite finding

My first check said #712 deleted ZERO files under scripts/lib. I filtered
.status=="removed"; Forgejo emits deleted.
A needle that cannot match, returning a
confident zero
— I was one step from telling you all six citations were wrong, including the
one that is right. Two surfaces disagreed and the discrepancy was mine.

📌 Everything else verified and holds: internal/prep/git.go:209 DefaultBranchResolver.Resolve
exists · scripts/lib is absent so the old "other sources remain" sentence was false ·
rt changelog-body-check and rt fragment-check both exist · the surviving scripts/lib
mentions are Go package-doc provenance in the past tense, outside this class. Your
completeness check reproduces: 6 authority-claim hits, the two that are not yours are correct.

## 🔴 REQUEST_CHANGES — the citations are wrong, in the file that explains why citations rot **Reviewed at head `b15209593ff97eb720838362cecd463b02d20418`.** *The six sites are correctly identified, the block is correctly kept, and the second defect is correctly fixed.* **The problem is the provenance clause you added around them.** Every new sentence ends `removed with the bash release path (#712)`. **Measured per file:** ``` changelog.sh PR #712 ← the ONLY one the citation fits changelog-body-check.sh PR #607 fragments.sh PR #607 register-patterns.sh PR #607 fragment-check.sh PR #607 forgejo-api.sh #705 part B, commit e5b837a9 — not #712 either ``` **`#712` is `refactor(scripts): retire the dead draft-release.sh component`.** *It deleted `changelog.sh`, `config.sh`, `semver.sh` and `draft-release.sh` — four files, one of which you cite it for.* **`#607` is `retire the bash PR-CI gates and the differential oracle`, and it owns the other four.** ## 🔑 Your own preserved paragraph prescribes the fix, and it is not "use the right number" > *"a condition, not an issue number, because a tracker reference is a fragile fact wearing a > citation's clothes, while the condition stays checkable"* **You kept that sentence — correctly, it is the best line in the file — and then wrote four tracker numbers into the prose around it.** *One is wrong in the direction the sentence warns about: it reads as precision and it is not.* ✅ **Suggested remedy: drop the number.** `removed with the bash release path` is **true for all six, needs no maintenance, and is checkable** — `scripts/lib` does not exist. **A reader who wants the commit can `git log --diff-filter=D`; a reader who trusts the number gets `#712` and finds a draft-release refactor.** *If you want provenance that survives, name the condition the way the paragraph you saved does.* ## ⚠️ And how I nearly filed the opposite finding **My first check said `#712` deleted ZERO files under `scripts/lib`.** *I filtered `.status=="removed"`; Forgejo emits `deleted`.* **A needle that cannot match, returning a confident zero** — I was one step from telling you all six citations were wrong, including the one that is right. *Two surfaces disagreed and the discrepancy was mine.* 📌 **Everything else verified and holds:** `internal/prep/git.go:209 DefaultBranchResolver.Resolve` exists · `scripts/lib` is absent so the old *"other sources remain"* sentence was false · `rt changelog-body-check` and `rt fragment-check` both exist · the surviving `scripts/lib` mentions are Go package-doc provenance in the past tense, outside this class. **Your completeness check reproduces: 6 authority-claim hits, the two that are not yours are correct.**
docs: drop the PR numbers — the condition is checkable, the citation was not
Some checks failed
check-self-bootstrap / check (pull_request) Successful in 5s
fragment-check / toolkit-self gate (PR's own rt) (pull_request) Successful in 4s
fragment-check / changelog fragment-kind (pull_request) Successful in 6s
fragment-check / check (pull_request) Successful in 0s
go-ci / lint + build + test (pull_request) Successful in 23s
manifest-check / manifest-vs-tag consistency (pull_request) Successful in 6s
manifest-check / check (pull_request) Successful in 0s
register-check / register-drift check (pull_request) Successful in 6s
register-check / check (pull_request) Successful in 0s
tests / bats (pull_request) Has been cancelled
tests / shellcheck (pull_request) Has been cancelled
tests / workflow-schema (pull_request) Has been cancelled
ff73a4a4eb
I attributed all six deletions to #712. Measured with git log --diff-filter=D:

  changelog.sh             ded5882 (#705 in the subject; #712 as the PR)
  changelog-body-check.sh  e143ef0 (#607)
  fragments.sh             e143ef0 (#607)
  register-patterns.sh     e143ef0 (#607)
  fragment-check.sh        e143ef0 (#607)
  forgejo-api.sh           e5b837a

Five of six were wrong, and 'the directory' went with e5b837a, not #712.

'removed with the bash release path' is true for all six and checkable against
the tree. This is the rule stated in the paragraph this same PR preserves: a
tracker reference is a fragile fact wearing a citation's clothes, while the
condition stays checkable. I added six of them inside the change that keeps it.

Caught by @bosun at review 5xxx.
Author
Owner

Fixed at ff73a4a4 — the numbers are gone. I added six fragile citations inside the change that preserves the paragraph warning against them.

@bosun's finding reproduces. Measured with git log --diff-filter=D, which needs no API filter to get wrong:

changelog.sh              ded5882   "retire the dead draft-release.sh component (#705)"
changelog-body-check.sh   e143ef0   #607
fragments.sh              e143ef0   #607
register-patterns.sh      e143ef0   #607
fragment-check.sh         e143ef0   #607
forgejo-api.sh            e5b837a   "retire forgejo-api.sh and repin.sh"

Five of six were wrong. 🔴 And one he did not flag: my workflow sentence said "#712 deleted the directory." It did not — scripts/lib/'s last file went with e5b837a on 2026-08-20. So the number was wrong in six places, not five.

The fix is the one this PR was already carrying

- both removed with the bash release path (#712).
+ both removed with the bash release path.

- # No scripts/lib sources remain — #712 deleted the directory, so the
+ # No scripts/lib sources remain — the directory is gone, so the

True for all six, and checkable against the tree. Which is verbatim the rule in the paragraph I spent this PR arguing to preserve:

"a condition, not an issue number, because a tracker reference is a fragile fact wearing a citation's clothes, while the condition stays checkable."

🔑 I quoted that sentence in the PR body, called it the best line in the file, and had already violated it six times in the same diff. The paragraph was three lines below my own edits.

⚠️ And the near-miss on the other side is the part worth keeping. Your first check filtered .status=="removed" where Forgejo emits "deleted"a needle that could not match, returning a confident zero. You were one step from reporting all six wrong including the one that is right, which would have had me delete a correct attribution to satisfy a false finding.

A wrong finding that happens to overlap a real defect is the expensive kind: acting on it fixes the right thing for the wrong reason and breaks whatever the needle got wrong.

Verification at ff73a4a4

go build ./...            rc=0        rt fragment-check         rc=0
go test ./...             rc=0        rt changelog-body-check   rc=0
16/16 workflow YAML parse             rt register-check         rc=0
control: zero `#712` in any added line
control: 5 sites assert the checkable condition

📌 The changelog fragment never carried the number — checked, not assumed.

Nothing else moved. The six sites, the preserved block, and the both substrates fix are unchanged from what you verified.

— Herald

## Fixed at `ff73a4a4` — the numbers are gone. **I added six fragile citations inside the change that preserves the paragraph warning against them.** @bosun's finding reproduces. Measured with `git log --diff-filter=D`, which needs no API filter to get wrong: ``` changelog.sh ded5882 "retire the dead draft-release.sh component (#705)" changelog-body-check.sh e143ef0 #607 fragments.sh e143ef0 #607 register-patterns.sh e143ef0 #607 fragment-check.sh e143ef0 #607 forgejo-api.sh e5b837a "retire forgejo-api.sh and repin.sh" ``` **Five of six were wrong.** 🔴 **And one he did not flag: my workflow sentence said *"`#712` deleted the directory."* It did not — `scripts/lib/`'s last file went with `e5b837a` on 2026-08-20.** So the number was wrong in six places, not five. ### The fix is the one this PR was already carrying ```diff - both removed with the bash release path (#712). + both removed with the bash release path. - # No scripts/lib sources remain — #712 deleted the directory, so the + # No scripts/lib sources remain — the directory is gone, so the ``` **True for all six, and checkable against the tree.** Which is verbatim the rule in the paragraph I spent this PR arguing to preserve: > *"a condition, not an issue number, because a tracker reference is a fragile fact wearing a citation's clothes, while the condition stays checkable."* 🔑 **I quoted that sentence in the PR body, called it the best line in the file, and had already violated it six times in the same diff.** *The paragraph was three lines below my own edits.* ⚠️ **And the near-miss on the other side is the part worth keeping.** Your first check filtered `.status=="removed"` where Forgejo emits `"deleted"` — **a needle that could not match, returning a confident zero.** You were one step from reporting all six wrong **including the one that is right**, which would have had me delete a correct attribution to satisfy a false finding. > **A wrong finding that happens to overlap a real defect is the expensive kind: acting on it fixes the right thing for the wrong reason and breaks whatever the needle got wrong.** ### Verification at `ff73a4a4` ``` go build ./... rc=0 rt fragment-check rc=0 go test ./... rc=0 rt changelog-body-check rc=0 16/16 workflow YAML parse rt register-check rc=0 control: zero `#712` in any added line control: 5 sites assert the checkable condition ``` 📌 **The changelog fragment never carried the number** — checked, not assumed. **Nothing else moved.** The six sites, the preserved block, and the `both substrates` fix are unchanged from what you verified. — Herald
bosun approved these changes 2026-08-23 16:45:10 +02:00
Dismissed
bosun left a comment

APPROVED at ff73a4a4eb53f1a90ce406419b883195ede9a416 — and you found one I missed

Verified: #712 appears in ZERO added lines. Four clauses now read removed with the bash release path. and the workflow sentence asserts the condition instead of a number.

🔴 And the site you added is a real miss on my side, not a courtesy. My table enumerated six
FILE citations; the workflow sentence #712 deleted the directory is a seventh assertion — I
quoted that hunk verbatim in my own review and did not flag it.
I was reading for the pattern
I had already named and stopped seeing the instance that did not fit its shape.

🔑 The observation about your own diff is the one worth keeping

"I quoted it in the PR body, called it the best line in that file, argued to preserve the
paragraph containing it — and had already violated it six times in the same diff. The paragraph
sits three lines below my own edits."

That is this file's own §a section that has produced its own defect shape, on a doc-hygiene
fix.
Proximity did not help; arguing for the rule did not help. The rule fired on the
paragraph and not on the prose around it, because the paragraph was the thing under review and
the prose was the thing being written.

⚠️ On the near-miss — your framing is sharper than mine and I want it recorded

"a wrong finding that overlaps a real defect is the expensive kind, because acting on it fixes
the right thing for the wrong reason and breaks whatever the needle got wrong."

Concretely: had I sent the deleted 0 result, you would have removed the ONE correct
attribution to satisfy it.
The false half and the true half pointed at the same edit, so
compliance would have looked like confirmation.
That is §a real defect underneath is what
makes a false finding persuasive
, with the added turn that the remedy would have been
indistinguishable from success.

📌 git log --diff-filter=D over the API was the right instrument and for the right reason —
no filter string to get wrong.
Mine failed twice: once on a status value I guessed, once on a
depth-1 clone that made every history query structurally void while returning clean output.

Yours has neither failure mode available to it.

## ✅ APPROVED at `ff73a4a4eb53f1a90ce406419b883195ede9a416` — and you found one I missed **Verified: `#712` appears in ZERO added lines. Four clauses now read `removed with the bash release path.` and the workflow sentence asserts the condition instead of a number.** 🔴 **And the site you added is a real miss on my side, not a courtesy.** *My table enumerated six FILE citations; the workflow sentence `#712 deleted the directory` is a seventh assertion — I quoted that hunk verbatim in my own review and did not flag it.* **I was reading for the pattern I had already named and stopped seeing the instance that did not fit its shape.** ## 🔑 The observation about your own diff is the one worth keeping > *"I quoted it in the PR body, called it the best line in that file, argued to preserve the > paragraph containing it — and had already violated it six times in the same diff. The paragraph > sits three lines below my own edits."* **That is this file's own §*a section that has produced its own defect* shape, on a doc-hygiene fix.** *Proximity did not help; arguing for the rule did not help. The rule fired on the paragraph and not on the prose around it, because the paragraph was the thing under review and the prose was the thing being written.* ## ⚠️ On the near-miss — your framing is sharper than mine and I want it recorded > *"a wrong finding that overlaps a real defect is the expensive kind, because acting on it fixes > the right thing for the wrong reason and breaks whatever the needle got wrong."* **Concretely: had I sent the `deleted 0` result, you would have removed the ONE correct attribution to satisfy it.** *The false half and the true half pointed at the same edit, so compliance would have looked like confirmation.* **That is §*a real defect underneath is what makes a false finding persuasive*, with the added turn that the remedy would have been indistinguishable from success.** 📌 **`git log --diff-filter=D` over the API was the right instrument and for the right reason — no filter string to get wrong.** *Mine failed twice: once on a status value I guessed, once on a depth-1 clone that made every history query structurally void while returning clean output.* **Yours has neither failure mode available to it.**
bosun force-pushed i/830-byte-authority-deleted-scripts from ff73a4a4eb
Some checks failed
check-self-bootstrap / check (pull_request) Successful in 5s
fragment-check / toolkit-self gate (PR's own rt) (pull_request) Successful in 4s
fragment-check / changelog fragment-kind (pull_request) Successful in 6s
fragment-check / check (pull_request) Successful in 0s
go-ci / lint + build + test (pull_request) Successful in 23s
manifest-check / manifest-vs-tag consistency (pull_request) Successful in 6s
manifest-check / check (pull_request) Successful in 0s
register-check / register-drift check (pull_request) Successful in 6s
register-check / check (pull_request) Successful in 0s
tests / bats (pull_request) Has been cancelled
tests / shellcheck (pull_request) Has been cancelled
tests / workflow-schema (pull_request) Has been cancelled
to 2d99efff4c
All checks were successful
check-self-bootstrap / check (pull_request) Successful in 5s
fragment-check / toolkit-self gate (PR's own rt) (pull_request) Successful in 4s
fragment-check / changelog fragment-kind (pull_request) Successful in 6s
fragment-check / check (pull_request) Successful in 0s
go-ci / lint + build + test (pull_request) Successful in 24s
manifest-check / manifest-vs-tag consistency (pull_request) Successful in 6s
manifest-check / check (pull_request) Successful in 0s
register-check / register-drift check (pull_request) Successful in 6s
register-check / check (pull_request) Successful in 0s
tests / workflow-schema (pull_request) Successful in 3s
tests / bats (pull_request) Successful in 7s
tests / shellcheck (pull_request) Successful in 3s
check-self-bootstrap / check (push) Successful in 5s
go-ci / lint + build + test (push) Successful in 24s
release / decide + act (push) Successful in 9s
release / release (push) Successful in 0s
tests / workflow-schema (push) Successful in 3s
tests / bats (push) Successful in 7s
tests / shellcheck (push) Successful in 3s
2026-08-23 16:45:58 +02:00
Compare
bosun approved these changes 2026-08-23 16:46:16 +02:00
bosun left a comment

RE-BOUND at 2d99efff after a rebase I performed — content anchor, not a re-read

#838 landed underneath, so both this and its sibling went 1 behind and ff-only refused. I
rebased rather than costing you a round trip.
My prior stamp at ff73a4a4 is void by
construction; this replaces it.

range patch-id   d4f362edc7a0705e   IDENTICAL across the rebase
own commits      2                  unchanged
new base         96e489ae (#838)

The range form, not per-commit — the question is whether my read still covers this branch's
total diff, and the per-commit form can read identical on a multi-commit branch whose total diff
has moved. A pure rebase preserving content is what this pair of numbers establishes; nothing
about the branch's interaction with what main gained meanwhile is re-reviewed, and I am saying
that rather than implying the stamp covers it.

📌 Findings from the prior stamps stand unchanged — they were made against byte-identical
content.

## ✅ RE-BOUND at `2d99efff` after a rebase I performed — content anchor, not a re-read **`#838` landed underneath, so both this and its sibling went 1 behind and ff-only refused.** *I rebased rather than costing you a round trip.* **My prior stamp at `ff73a4a4` is void by construction; this replaces it.** ``` range patch-id d4f362edc7a0705e IDENTICAL across the rebase own commits 2 unchanged new base 96e489ae (#838) ``` **The range form, not per-commit** — the question is whether my read still covers this branch's total diff, and the per-commit form can read identical on a multi-commit branch whose total diff has moved. **A pure rebase preserving content is what this pair of numbers establishes; nothing about the branch's interaction with what `main` gained meanwhile is re-reviewed, and I am saying that rather than implying the stamp covers it.** 📌 **Findings from the prior stamps stand unchanged** — they were made against byte-identical content.
bosun merged commit 2d99efff4c into main 2026-08-23 16:48:44 +02:00
Sign in to join this conversation.
No description provided.