Uh oh!
There was an error while loading. Please reload this page.
chore(lint): repoint check-route-envelope.mjs's ratchet owner to #9559 - #9564
Merged
Merged
Conversation
Both `ratchet:` fields in the route-envelope gate (`rest-server.ts`, `error-response.ts`) named #7035, closed since 2026-08-10 (PR #7293, three `/meta` 501 handlers converged — never the owner of "convert this file onto the shared sendOk/sendError"). Triage executed option 1 at the triage layer: #9559 is a `tracking` card that stays open while any `ratchet:` field names it, with both legal exits (a dedicated envelope card, or the ADR-0112-D5 permanently-accepted path) written into it. Repoints both fields from #7035 to #9559, preserving the option-1 prose verbatim. Nothing else moves: no counter, threshold, or exempt entry changed — confirmed by diffing the gate's own before/after output, which is byte-identical except the citation. The eleven other `#7035` mentions in the file are historical prose (what #7035 did, past tense) and are left alone. Fixes#9461 Co-Authored-By: Claude <noreply@anthropic.com>
os-zhuang
marked this pull request as ready for review
August 18, 2026 10:44
os-zhuang
enabled auto-merge
August 18, 2026 10:45
Uh oh!
There was an error while loading. Please reload this page.
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Fixes#9461
Repoints
scripts/check-route-envelope.mjs's tworatchet:fields (rest-server.ts:245,error-response.ts:309) from the closed#7035to the live tracking card#9559, preserving the option-1 prose verbatim. Nothing else moves.Why
#7035reachedcompletedon 2026-08-10 (PR #7293, three/meta501 handlers) and wasnever the owner of "convert this file onto the shared sendOk/sendError." Triage on
#9461 executed option 1 at the triage layer:
#9559is atrackingcard that staysopen while any
ratchet:field names it, with both legal exits written into it (adedicated envelope card, or the ADR-0112 D5 permanently-accepted path).
Scope check (Zone 2 of the dispatch card)
Measured
ratchet: '#7035onorigin/mainbefore editing: exactly 2 occurrences,at
:245(rest-server.ts) and:309(error-response.ts) — matches the card'sassumption exactly. A control query (
dialectOnly, 15 hits; a made-up string, 0 hits)confirmed the grep itself works before trusting the zero/two counts.
The file's other eleven
#7035mentions are historical prose (comments describingwhat #7035 did, past tense) — left untouched per Zone 1 rule 2.
What did not move
That is the entire diff in the gate's own output — captured by running
pnpm check:route-envelopebefore and after the edit and diffing the two runs. Every otherline (
10 route module(s) audited: 7 conformant, 2 ratcheted, 1 exempt,stringError 44, siblingCode 69,responses: 2pinned-at-current-drift, the dispatcher/plugin-routesections) is byte-identical. No counter, threshold, or exempt entry changed.
Verification
Run at
d22bc2ecb(final commit;git status --porcelainempty):check:route-envelope— self-test ✓ (✓ check-route-envelope self-test passed),audit ✓ exit 0:
10 route module(s) audited: 7 conformant, 2 ratcheted, 1 exempt.node scripts/pm/dispatch-gates.mjs scripts/check-route-envelope.mjs— names exactlyone family:
pnpm check:route-envelope [lint.yml].check:nul-bytes— clean (6157 tracked text files, no raw control bytes), plus atargeted self-scan of the changed file.
No package build or test was run, and none is owed: the diff touches no package
source (
scripts/check-route-envelope.mjsis a repo-root dev-tooling script, notinside any
packages/*), so there is no affected package to build a closure for.skip-changeset
The diff releases nothing — one lint script, no package source, matching the identical
precedent on this same file (#9455,
chore(lint): record the ruled pre-auth exemption in the route-envelope gate, which used this exact label for this exact reason). Labelapplied on this PR.
Generated by Claude Code