docs(pm-dispatch): re-key platform-readings' GraphQL-only list now the ccr routes are counted - #18396
Merged
os-elon-musk merged 2 commits intoSep 16, 2026
Conversation
…e counted The draft flip and the auto-merge mount both have live ccr REST routes (rest-channel.md rows 46 and 50, and this file's own undraft row), so neither belongs on the GraphQL-only list and neither is a reason to stand a seat down in a rate-limit window. Point the membership row at rest-channel.md's 不可迁移 section, which the 2026-09-01 content-ownership ruling already makes the single holder of per-operation channel mapping, and re-key the red-window row to say the two go through all window. The same three lines fold in the measured mechanism behind an empty auto_merge echo: all checks green plus mergeable_state clean is consumed straight into the queue with no record stored, so NULL is the normal echo of success. 466 / 466 lines, in place, each rewritten line under the 120-byte cap. Claude-Session: https://claude.ai/code/session_01Bz6hxDBqK62NP2W1LATvnt Co-authored-by: Claude <noreply@anthropic.com>
…ring it false A fact table does not carry a row whose job is to say the row above it is false. The red-window landing row is retired by the same ccr fact this branch lands, so it is re-keyed in place: a queue repo's landing still goes through auto-merge, and in a red window that mount goes over the ccr REST route; its direct-merge half stays. The row below drops its 「⇒ 上条无退路不成立」 tail and keeps the rest. 466 / 466 lines, in place, both rewritten rows at 118 and 89 bytes. Claude-Session: https://claude.ai/code/session_01Bz6hxDBqK62NP2W1LATvnt Co-authored-by: Claude <noreply@anthropic.com>
Collaborator
Author
Contract reviewServed-tier: ① Derived judgments
② Semver level
③ Boundary flags
Implemented-by: VERDICT: PASS Generated by Claude Code |
This was referenced Sep 16, 2026
os-elon-musk
marked this pull request as ready for review
September 16, 2026 09:11
This was referenced Sep 16, 2026
os-elon-musk
deleted the
claude/issue-18361-platform-readings-graphql-only-list
branch
September 16, 2026 09:35
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.
Fixes #18361
references/platform-readings.mdonly, in place at 466 / 466 lines (ratchet ceiling, headroom 0). Four rows re-keyed by judgment; no row added, none removed, no re-wrap, every rewritten line under the gate's 120-byte cap.Three rows landed in the first commit (
b0c55121). The fourth arrived with the seat's review (comment 5694684277 on #18361), which widened this claim's file surface by one row: a fact table does not carry a row whose job is to say the row above it is false, so the row above is re-keyed in place instead — second commita7026a35, no force-push, no rebase.Rows are located by CONTENT, not by the card's line numbers: the card was filed against
1411cf2cand the file has landed twice since (b57671ed#18359 at 04:18Z,05f1cdd6#18363 at 06:40Z), so :134 / :143 / :57 have drifted.Row 1 — the GraphQL-only membership row (line 134 on this branch)
Landing point: the row directly under 「GraphQL 池 5000/时,只留给没有 REST 对应物的几件。」
before (verbatim, 113 B):
after (verbatim, 118 B):
Row 2 — the queue-repo landing row (line 142), widened into scope by the seat's review
Landing point: the row directly under 「红窗调度:守候只给上面那几件 GraphQL-only 的,⛔ 其余一切不为配额空等。」
before (verbatim, 107 B):
after (verbatim, 118 B):
The row's premise — that arming is GraphQL-only — is the same one the card retires, so the row is retired with it. What survives is what is still true: a queue repo's landing still goes through auto-merge (the queue is not bypassed), the mount in a red window rides the ccr REST route, and the direct-merge half keeps its REST endpoint.
Row 3 — the red-window stand-down row (line 143)
Landing point: the row directly under row 2.
before (verbatim, 70 B):
after (verbatim, 89 B):
The first commit ended this row with 「⇒ 上条无退路不成立」, a clause whose only job was to void the row above. The review rejected that shape — correctly: a reader of a fact table should not have to hold two rows in mind to get one fact. With row 2 re-keyed, the clause has nothing to do and is gone.
Row 4 — the echo row (line 57), carrying comment 5693336603's mechanism
Landing point: the row directly under 「squash 的 committer date 是入队时刻,快进不改 ⇒ 落地时刻读 merged_at/merged,不读 git log。」
before (verbatim, 114 B):
after (verbatim, 115 B):
The byte WAS there, so the note is carried here rather than left on the card: the mechanism (all checks green plus
mergeable_state: cleanis consumed straight into the queue and stores no record, so an empty echo is the normal echo of SUCCESS) replaces two clauses the row no longer has to spend bytes on.空回显不等于未挂上is subsumed by the stronger空回显是成功常态, and⛔ 不拿它当任何方向的证据follows from回显两向不可靠in the same sentence. The note's third half — never re-arm on a NULL alone — is NOT restated: the file already carries it at 「⛔ 永不据它判没挂上而重挂 —— 重挂踢队重排。」, and a second copy is what this card exists to remove.The card's question: mirror rest-channel's 「不可迁移 —— 只有这三件」, or point at it
Point at it. Three reasons, in descending authority:
scripts/pm/check-skill-line-ratchet.mjs's ceiling comment forrest-channel.md) reads, verbatim and untranslated: 「C 同批:退役 L58 对红窗常设规则的复述 —— 内容归属裁定:红窗规则由platform-readings.md配额段独家持有,rest-channel.md只留指路」. The split it draws is symmetric: red-window POLICY is this file's, per-operation CHANNEL MAPPING is rest-channel's. Which operations have no REST twin is channel mapping.What is NOT delegated is the JUDGMENT the card asked for: the two operations that left the list are named here, in the negative, because a seat that remembers the old list needs the deletion stated, not merely absent. The three that stay are not re-listed.
⛔
rest-channel.mdis not touched by this PR. Its rows 46 and 50 are already correct.Acceptance notes
GitHub access is not enabled for this session) or by REST core, so its prohibition still has a referent.enqueuedevent on this repo's/issues/N/timeline, the queue ref answering BUILD rather than membership, andauto_mergecleared on enqueue — conflicts with rows 10–12 of this same file, which make the timeline event the decisive membership reading. Out of this card's scope, untouched here. Row 4's rewrite was written to be consistent with it (a consumed request stores no record), not against it.Gates
Targeted set for a
.claude/skills/**docs-only diff, re-derived from the actual change at the new head withnode scripts/pm/dispatch-gates.mjs --commands(no paths) and reconciled with--ran: 16 derived / 16 run / 0 NOT-MEASURED / 0 UNRUN, the same 16 as the first round. Exit codes captured to a file before reading, never across a pipe. Ratchet at the new head: 466 lines, ceiling 466, headroom 0. Per-command results are in the report comments on #18361.skip-changeset: nothing published moves —.claude/**is not in any package'sfiles[], it is the fast track, not a measurement.维护者速读(草稿)
改了什么 —— 只动
references/platform-readings.md四行,行数 466 → 466。一是 GraphQL-only 清单那行:不再自己列五件,改成指向rest-channel.md〈不可迁移〉的三件,并写明 draft 翻转与 auto-merge 挂载有 ccr 路、不在其内。二是「走队列的仓落地必经 auto-merge ⇒ 红窗里无退路」那行:改成落地仍必经 auto-merge、红窗里照样走 ccr REST 挂载,直合仓那半保留。三是红窗那行:两件在限流窗里照常走 ccr REST,别为它们守候。四是回显那行:补进「全绿加 clean 即被吞进队列、不存记录,所以空回显是成功的常态」这条机制。为什么改 —— 锁 1 之后 ccr 路是这两件操作的唯一通道,而清单还把它们记作没有 REST 对应物。后果不是措辞问题:读到红窗那几行的席位会白等一个根本不用等的窗口。清单归属有 2026-09-01 的裁定在先(红窗规则归本表独家,通道归属归
rest-channel.md),所以这次是把重复的那份退役成指路,不是再抄一遍。第二行是席位复核时扩进来的:原先我只在下一行写「上条无退路不成立」,事实表不该有一行专门宣布上一行为假,现在把上一行本身改对了。风险与代价(含回滚) —— 风险有两处。① 席位要判某个操作是不是 GraphQL-only 时,多读一个文件(
rest-channel.md,82 行、零配额);换来的是清单只有一份、不会再各自漂移。② 这四行都在同一段里,措辞收紧后相邻行的语气略有重叠(红窗里走 ccr REST 这件事在两行都出现一次),是刻意的:一行讲落地路径,一行讲要不要守候。回滚代价极低:四行就地改写,两个提交都可git revert,棘轮读数不变。席位意见 ——
你要做的 —— 若同意「指路而不镜像」这一判断,按 references 档在席复核后落地即可;若更希望本表继续自列三件(以红窗里少读一个文件换一份可能再漂的副本),请在此说明,我改回列举形。
Generated by Claude Code