[objectstack-fleet]

domain:skills#2 · zhuangjianguo · session_01QsR2jT34CXs5fCn3BDirZw

Requested by Steedos · project thread

Fixes #31580

Clause-②: no
Metadata-protocol: none

维护者速读

• 改了什么:pm-dispatch 执行席职责加一条每轮巡检(核本席所持终局四件套 PR 的四件)并点名读数器;落地步骤加一句「check-governed-merges 判定读全文,四件套照 SKILL.md」。
• 为什么改:规则本来就有,但席位截断读了判定、巡检也从不回看自己挂着的 PR,四个 Tier H PR 因此几小时不在您的待审列表里。
• 风险与回滚:只改两份席位文档,行数零净增;回滚即 revert 本 PR。
• 先看:execution-duties.md 第 21–22 行(新巡检与读数器);landing-operations.md 第 57 行(读全文条)。
• 两分钟验证:pnpm check:pm-skill-ratchet,预期 exit 0、两文件仍在上限 188 与 102。
• 需要你做的:无,席位落地(Tier S,待契约复审档记录)。

What changes

• .claude/skills/pm-dispatch/references/execution-duties.md L21–22, beside the soft cap on held Tier H PRs (L20): every round, check the four-piece terminal on the current head of each PR the seat holds (seat record, pm:awaiting-merge, review request, final 维护者速读) and complete a missing piece in the same round; the reader is a local node scripts/pm/check-half-states.mjs (H43/H48, which read governed Tier H PRs only; an over-3000-line PR with no governed path the seat reads itself).
• .claude/skills/pm-dispatch/references/landing-operations.md L57: the check-governed-merges.mjs --pr N step now says to read the verdict whole, never through tail/head, and to take the four pieces from SKILL.md (the verdict lists them only on the unlifted over-3000-line branch, renderSizeLines).

Home: execution-duties.md holds the executing seat's own duties and already counts the PRs this line patrols (L20 soft cap). seat-lifecycle.md's standby list says each item lives in its main section and is not copied there, so a new duty does not belong in that pointer list.

Line payment (both files stay at their ceilings, 188 and 102; every line is 120 bytes or less, nothing re-wrapped):

• Deleted execution-duties.md old L180 (an edited .changeset/*.md takes no skip-changeset; confirmation is the dispatch seat's record). Survivors: .claude/agents/os-dev.md:300–301, landing-operations.md:14–16.
• Deleted execution-duties.md old L185 (mcp_calls naming a write tool means the report is refused). Survivors: .claude/agents/os-dev.md:374, execution-duties.md:134–136.
• Dropped from landing-operations.md L57 the two tier clauses. Survivors: .claude/skills/pm-dispatch/SKILL.md:203 (two governed tiers), SKILL.md:187 (over 3000 lines takes the Tier H terminal).

Tests

No code changed. Reader measured on main 6b83ec2:

• node scripts/pm/check-half-states.mjs (live sweep, about 6.5 min): finished with exit 0, 143 cards and 35 open PRs swept, H43 judged 3 governed open PRs and H48 judged 1, with no H43 or H48 finding today (the four PRs on the card were already fixed).
• The H48 sentence for a Tier H PR with a seat ACCEPT and no label, from the exported h48GovernedVerdictWithoutBrief on a fixture: open and GOVERNED on Tier H (1 changed file(s) on the register: skills/**×1), carrying a seat's **ACCEPT** verdict (newest one 2026-11-08T09:01:00Z), and no pm:awaiting-merge label. … Remedy: add pm:awaiting-merge. The same PR with the label returns null.

Acceptance notes

• H43 and H48 read governed PRs only. A PR on the terminal only because it is over 3000 changed lines with no governed path is not covered by either row; for that one the seat reads its own PR's labels and requested_reviewers.
• The live sweep reads the whole board. It needs no workflow (ruling 🔗 Broken links detected in documentation #208 on [Decision] 仪器与吞吐的取舍：哪些闸门该硬、达档复核保留到什么程度 #19491 stands), but it is a several-minute read, not a one-PR read.

Verification

Derived gates on 0466fc5 (17 derived + 1 roster), re-run on f62ee42 after the review rework

All exit 0: check-closing-keyword-parity (plain and --self-test), check-comment-mask-corpus, check-harness-current --self-test, lint check:doc-formula-expressions, check:agent-test-spelling, check:cross-package-test-inputs, check:doc-authoring, check:driver-memory-census, check:gitlink-declared, check:nul-bytes, check:pm-governed-merges, check:pm-skill-id-lint, check:pm-skill-ratchet, check:refd-timer-probe, check:skill-frame-sync, check:watch-hint-literal, check:pm-settings-deny-roster. dispatch-gates --ran: 17 derived families accounted for, 17 run, 0 NOT-MEASURED. Rework head f62ee42 (contract review 6724388379 F1+F2): the same 18 commands all exit 0, --ran 17/17, 0 NOT-MEASURED; edited lines 118 B and 116 B, files still 188 and 102.

---

Generated by Claude Code

[objectstack-fleet]

domain:skills#2 · zhuangjianguo · session_01QsR2jT34CXs5fCn3BDirZw

Contract-review-claim: session_01QsR2jT34CXs5fCn3BDirZw · head 857e5e4 · 2026-11-08T16:54Z

[objectstack-fleet]

domain:skills#2 · zhuangjianguo · session_01QsR2jT34CXs5fCn3BDirZw

Contract review

Served-tier: CONTRACT_REVIEW_TIER
Head-sha: 857e5e49638ac14abe61af958cc7fab21d23aea9
Local-runs: none
Scope: .claude/skills/pm-dispatch/references/execution-duties.md, .claude/skills/pm-dispatch/references/landing-operations.md + none

Read, never reviewed, to judge the two hits files: scripts/pm/check-governed-merges.mjs (renderTestVerdict L1847–1924, renderSizeLines L1925–1954, renderExceptionLines L1827, GOVERNED_TIERS L1136–1149, emitVerdict L3640–3659), scripts/pm/check-half-states.mjs (H43 L9924–10330, H48 L11384–11620, H69 L11622–11760), scripts/pm/check-skill-line-ratchet.mjs (ceilings L1238, L1433), .claude/skills/pm-dispatch/SKILL.md L181–203, .claude/agents/os-dev.md L296–304 and L372–376, references/review-checklist.md L15–29, references/seat-lifecycle.md L88–96, references/platform-readings.md L422 — all on the PR head tree.

① Derived judgments

• Accept-set: unchanged — the diff touches no gate, workflow, script, test or fixture; two .claude/** prose files only. Right.
• Public surface: none — no package files[] path is in the diff. Right.
• Governed register: both paths are .claude/**, Tier S; the PR lands on this record and the queue guard (Governed Surface Queue Guard: success). Right.
• Line ratchet: execution-duties.md 188 → 188 (ceiling 188), landing-operations.md 102 → 102 (ceiling 102); the three edited lines measure 120, 119 and 119 bytes, longest line in either file 120 B. Lint · PM dispatch-gates: success. Right.
• Skill id lint: no tracker number in the three new lines. Right.
• Deletion survivors, each read on the head tree: old execution-duties.md L180 → .claude/agents/os-dev.md:300–301 and landing-operations.md:14–16 (hold); old L185 → os-dev.md:374 ("a write tool in the list = this report is refused") and execution-duties.md:134–136 (hold, see nit N1); landing-operations.md L57 tier clauses → SKILL.md:203 (two tiers) and SKILL.md:187 (over 3000 lines, generated output included, takes the four-piece terminal with ③). Right.
• Check-runs on the head at record time: every non-skipped run success (Lint · PM dispatch-gates, Lint · docs & code guards, Lint · CI, release & engine gates, Lint · repo & tooling gates, PR-body guards, Check Changeset, Check PR Size, Governed Surface Queue Guard, the type-check, lint and test families); none red, none pending. The gates agree with the diff; the findings below are design, not gate, questions.
• Serial constraint from the dispatch: PR feat(pm): fold a claim or a verdict into one label-write relay stroke #31583 edits landing-operations.md L51 and execution-duties.md L70–72; this diff's hunks (L21–22, old L180, old L185, L57) are disjoint. Right.

② Semver level

skip-changeset label, no changeset file; .claude/** is unpublished (os-dev.md:298 fast lane). Check Changeset: success. Clause-②: no on the PR body matches the dispatch and the diff — no spec, schema or published surface moves. Right.

③ Boundary flags

• Dev deviation "two lines, not one": accepted — no 120-byte line holds the four pieces and a reader; paid by two deletions in the same file.
• Dev deviation "branch is claude/project-thread-5ahh62": the Claim names it as the card thread's measured platform exception. Accepted.
• open_questions: none in the report.
• PR Acceptance note "H43 and H48 read governed PRs only; an over-3000-line PR with no governed path is not covered": the tree says none of it — escalated to finding F2 (should-fix).
• PR Acceptance note "the live sweep reads the whole board, several minutes": accepted — the card itself offered this reader, ruling on the retired schedule stands (L22 says 本地), and platform-readings.md:422 already warns that the script runs whole-repo I/O. See nit N3.

④ Design judgment

Scenario accepted against: the card's incident — a seat holding Tier H PRs after ACCEPT with no pm:awaiting-merge, no review request and no final brief for hours — and the card's two asks: a per-round patrol line with a cheap reader, and a landing clause to read the check-governed-merges verdict whole.

F1 — must-fix — landing-operations.md:57, clause 「末行即终局清单」. The line's operative half (read the verdict whole, never through tail/head) holds and is the card's ask. Its justification does not hold against the code it cites. In renderTestVerdict (check-governed-merges.mjs:1847–1924) the four-piece checklist sentence — "ACCEPT on the card, pm:awaiting-merge on the PR, the final 维护者速读, review requested from GOVERNED_APPROVERS" — is emitted only by renderSizeLines in the SIZE limb's over-3000-and-unlifted branch (L1945–1953). For a GOVERNED Tier H PR under 3000 changed lines — the class the card was filed about — the Tier H block ends with ⚖️ landing tier: H(人合) — the maintainer's hand, or an authorized APPROVED review (GOVERNED_APPROVERS) and then the owning seat lands it (GOVERNED_TIERS.H.landing, L1140), which names no four pieces; then the matched surfaces, the fork lines and size: N changed line(s) ≤ 3000 — under the human-merge threshold (generated files included in the count). print after it, and renderExceptionLines after that. So the last line of the verdict a seat reads for the card's own PRs is a size clearance, not a terminal checklist, and whenever an exception line prints the checklist is not last on the SIZE limb either. The skill line therefore teaches a seat to find the checklist where the tool never prints it on the governed limb; the card's premise sentence ("the verdict ends with …") was carried into the tree unverified. Fix in place within 120 B: keep 「判定读全文 ⛔ 不 tail/head」 and replace 「末行即终局清单」 with a pointer to where the four pieces live (SKILL.md ①–④ / L186–187), or state the SIZE-limb bound — e.g. 「…判定读全文 ⛔ 不 tail/head;四件套照 SKILL.md。」 measures 118 B. Any fix moves the head; this record covers 857e5e49 alone.

F2 — should-fix — execution-duties.md:22, the reader's population is unstated. L21 correctly patrols every 「终局四件套 PR」, which by SKILL.md:186–187 is both a Tier H path hit and a PR over 3000 changed lines regardless of paths. L22 names check-half-states.mjs H43/H48 as the reader of the four pieces with no bound. Both rows judge governedPathsIn slices only and stand down on Tier S (h48SpeaksAbout L11698–11703; h43GovernedReviewRequestGap L10266–10270, governedLandsOnRecord L10190–10195): an over-3000-line PR with no governed path is in neither row's population, and H43 fires only once a same-head ## Contract review PASS sits on the PR's own thread (h43HandoffReading L10226–10237 — a PASS on the card alone reads BEFORE and is quiet). H48's sentence does match L22's words: a seat **ACCEPT** with no label or no ## 维护者速读 comment is reported, and a labelled, briefed PR returns null (L11542–11600). What L22 says is true of the rows; what it omits leaves a seat believing its over-3000-line held PRs are swept when they are not — only the PR body says otherwise, and the body does not load into any seat. Fix in place within 120 B: bound the reader to 受管 Tier H and name the self-read (labels + requested_reviewers) for the size terminal — e.g. 「四件读数器:本地 node scripts/pm/check-half-states.mjs H43/H48 只读受管 H;超行无受管 PR 自读。」 measures 116 B — or drop the row names and say what the reader does not cover.

Holds, no finding: L21 — 「按现 head」 is the seat's own duty and matches the ① record-pinned-to-head rule (SKILL.md:188) and H69's head-dated reading; 「缺件同轮补」 matches the card's ask; the line sits beside the L20 soft cap that already counts these PRs. L57's dropped tier clauses survive word for word in SKILL.md:203 and SKILL.md:187 (generated output included).

Nits (none moves the verdict; fixed only on the prose faces, or left as accepted deviation / follow-up):

• N1 — old execution-duties.md L185: the REVIEWER's refusal imperative (「⇒ 拒收,⛔ 不带注放行」) now survives only from the dev-template side (os-dev.md:374) and the dispatch-order side (execution-duties.md:134–136); review-checklist.md carries no mcp_calls item. The report contract still states the consequence, so correctness holds; follow-up: one mcp_calls item in review-checklist.md when its ratchet has room.
• N2 — old L180: survives as the check-empty-changeset red the landing seat handles (landing-operations.md:14–16), the dev rule (os-dev.md:300–301) and review-checklist.md:19–22. Holds.
• N3 — L22 names a whole-board sweep (dev measured about 6.5 min) as the every-round reader; check-half-states.mjs accepts no --pr or row filter (argv: --help, --probe, --format, --mailbox-keys, --provenance, --self-test). Accepted deviation: the card named this reader itself.
• N4 — placement under 〈候选与批次〉 beside L20 rather than a patrol section; seat-lifecycle.md:92–93 routes standby items to their main sections, so defensible.

Implemented-by: claude/project-thread-5ahh62
Reviewed-by: session_01QsR2jT34CXs5fCn3BDirZw

VERDICT: FAIL

[objectstack-fleet]

domain:skills#2 · zhuangjianguo · session_01QsR2jT34CXs5fCn3BDirZw

Contract review

Served-tier: CONTRACT_REVIEW_TIER
Head-sha: f62ee42552bc274871eeea66b0853492bedb41fa
Local-runs: none
Scope: .claude/skills/pm-dispatch/references/execution-duties.md, .claude/skills/pm-dispatch/references/landing-operations.md + none

Delta review of the rework after record 6724388379 (FAIL on 857e5e49, findings F1 must-fix and F2 should-fix). The delta 857e5e49..f62ee425 is one commit, +2/−2, touching only the two lines those findings named; the net diff against main is still 2 files, +3/−3. Everything the first record judged and did not fault — the deletion survivors, the ratchet payment, the semver level, the L21 duty, the dev deviations — is unchanged in the tree and is re-stated below as it stands on this head, not re-argued. Read, never reviewed, on this head: scripts/pm/check-governed-merges.mjs (renderTestVerdict L1847–1924, renderSizeLines L1945–1953), scripts/pm/check-half-states.mjs (governedLandsOnRecord L10190–10195, h43GovernedReviewRequestGap L10266–10270, h48SpeaksAbout L11698–11703), .claude/skills/pm-dispatch/SKILL.md L181–203.

① Derived judgments

• Accept-set: unchanged — no gate, workflow, script, test or fixture in the diff; two .claude/** prose files only. Right.
• Public surface: none. Right.
• Governed register: both paths .claude/**, Tier S; lands on this record and the queue guard (Governed Surface Queue Guard on this head: success). Right.
• Line ratchet on this head: execution-duties.md 188 (ceiling 188), landing-operations.md 102 (ceiling 102); longest line in either file 120 B; the reworked lines measure 116 B (L22) and 118 B (L57), L21 unchanged at 119 B. Right.
• Skill id lint: no tracker number in L21, L22 or L57. Right.
• Deletion survivors: unchanged from 857e5e49 and still on this tree — old execution-duties.md L180 → os-dev.md:300–301, landing-operations.md:14–16; old L185 → os-dev.md:374, execution-duties.md:134–136; L57 tier clauses → SKILL.md:203, SKILL.md:187. Right.
• Check-runs on f62ee425, read after every run completed: every non-skipped run success — Lint · PM dispatch-gates, Lint · docs & code guards, Lint · CI, release & engine gates, Lint · repo & tooling gates, Lint & Repo Gates, Lint · ESLint, the three scoped lint jobs, the type-check family, Test Core, Test UI, Dogfood Regression Gate, PR-body guards, Check PR Size, Governed Surface Queue Guard, Auto Label, filter. One PR-body guards row reads cancelled: a superseded run of the same job, whose re-run on this head is the success row. Check Changeset is skipped on this head (the skip-changeset label is already on the PR). None red, none pending. Right.
• Serial constraint: PR feat(pm): fold a claim or a verdict into one label-write relay stroke #31583's hunks (landing-operations.md L51, execution-duties.md L70–72) remain disjoint from this diff. Right.

② Semver level

skip-changeset label, no changeset file; .claude/** is unpublished (os-dev.md:298). Clause-②: no on the replaced PR body matches the dispatch and the diff. Unchanged by the delta. Right.

③ Boundary flags

• The replaced PR body now carries the two bounds in its "What changes" section (H43/H48 read governed Tier H only; the verdict lists the four pieces only on the unlifted over-3000 branch) and the tree now carries them too — the body no longer says anything the skill text does not. Closed.
• Dev deviations "two lines, not one" and the card-thread branch name: unchanged, accepted as before.
• open_questions: none. The PR body's Acceptance notes (reader population; whole-board sweep cost) stand as disclosed; the first is now in the tree (F2), the second remains an accepted deviation (the card named this reader itself).

④ Design judgment

F1 — closed. landing-operations.md:57 now reads 「转 ready/入队前跑 check-governed-merges.mjs --pr N,判定读全文 ⛔ 不 tail/head;四件套照 SKILL.md。」 The false clause 「末行即终局清单」 is gone; the operative half (read the verdict whole, never through tail/head) is kept; the four pieces are now pointed at their one home, SKILL.md ①–④ (L188–189, L197, L201, with L186–187 saying which PRs take them), which is exactly where check-governed-merges.mjs itself does NOT print them on the governed limb (renderTestVerdict L1847–1924 prints the checklist sentence only inside renderSizeLines L1945–1953). Nothing in the new sentence claims a property of the tool's output. Holds.

F2 — closed. execution-duties.md:22 now reads 「四件读数器:本地 node scripts/pm/check-half-states.mjs H43/H48 只读受管 H;超行无受管 PR 自读。」 Checked against the rows: both judge governedPathsIn slices only and stand down when governedLandsOnRecord is true (Tier S), so "只读受管 H" is the rows' exact population (H43 L10266–10270, H48 L11698–11703); an over-3000-line PR with no governed path is in neither population, and the line now hands it to the seat's own read, which L21 already owes. The earlier per-row claims ("H43 行报未请审、H48 行报缺标或缺速读") were dropped rather than kept beside the bound; that loses nothing a seat needs, because L21 names the four pieces and the rows' sentences name their own remedies. Holds.

Nits (none moves the verdict; fixed only on the prose faces, or left as accepted deviation / follow-up):

• N5 — L22's second clause names the common excluded case only. The bound 「只读受管 H」 also excludes an over-3000-line PR whose governed paths are ALL Tier S (SKILL.md:187: 「⛔ 无 Tier S 例外」 — it takes the four-piece terminal, and H43/H48 stand down on it via governedLandsOnRecord). The first clause already puts that PR outside the reader, so the seat's own read follows from the sentence as written; a later edit could say 「余者自读」 instead of 「超行无受管 PR 自读」 (shorter, and complete). Accepted deviation.
• N1–N4 from record 6724388379 stand unchanged (reviewer-side mcp_calls item absent from review-checklist.md; old L180 survivors hold; whole-board sweep cost; placement beside L20). None touched by the delta.

Implemented-by: claude/project-thread-5ahh62
Reviewed-by: session_01QsR2jT34CXs5fCn3BDirZw

VERDICT: PASS

[objectstack-fleet]

domain:skills#2 · zhuangjianguo · session_01QsR2jT34CXs5fCn3BDirZw

维护者速读

• 改了什么：席位每轮巡查多一步：对自己替你压着的 Tier H 或超 3000 行 PR，按现 head 核对四件套（席位记录、pm:awaiting-merge、请你审、终版速读），缺哪件当轮补上。读数器写明了：本地跑 check-half-states.mjs，看 H43/H48，这两行只覆盖受管 Tier H；超 3000 行、不碰受管面的 PR 由席位自己读。另外，落地时 check-governed-merges 的判定必须读全文，不准用 tail/head 截断。
• 为什么改：devx 席压着的四个 Tier H PR 既没挂 pm:awaiting-merge 也没请审，其中两个在你的收件箱外待了约六小时，直到你自己发现。你问「如何避免类似的问题」，然后说「开 skills卡」。
• 风险与回滚：只改两个规则文件，没加门禁、workflow 或脚本，也没恢复定时巡查（🔗 Broken links detected in documentation #208 号裁决不变）。行数不变，为腾位置删的几行原文在别处仍在。回滚即 revert 本 PR。
• 先看：execution-duties.md L21–22，landing-operations.md L57。
• 两分钟验证：pnpm check:pm-skill-ratchet 退出 0。
• 需要你做的：无。这是 Tier S，契约复核通过后由席位自己落地。
• 席位意见：ACCEPT。车道的隔离复核器首轮判 FAIL（记录 6724388379）：原句「末行即终局清单」对受管 Tier H PR 不成立，读数器的范围也没写清。开发已修，增量复核 PASS（记录 6724651237，Head f62ee425）。

席位核对

Head-sha: f62ee42552bc274871eeea66b0853492bedb41fa

1. Shape: draft, base main, body carries Fixes #31580, skip-changeset. 2 files, line-neutral: execution-duties 188/188, landing-operations 102/102; no line over 120 B. It merges clean with main after feat(pm): fold a claim or a verdict into one label-write relay stroke #31583, and the merged tree stays 188/102. Tier S (.claude/**).
2. Premise measured: the four-piece terminal is stated on main (SKILL.md L183–196). No patrol line had a seat re-check its held PRs. The review measured the tool: check-governed-merges prints the four pieces only on the unlifted over-3000 branch (renderSizeLines). H43 and H48 read governed Tier H slices only.
3. Gates: contract review at CONTRACT_REVIEW_TIER. Round 1 was FAIL 6724388379 on 857e5e4 (F1 must-fix, F2 should-fix). The delta, PASS 6724651237 on this head, closes both; nits remain. Dev gates: 17 derived, 17 exit 0, --ran 17/17 at this head. Check-runs on this head: 22 success, 21 skipped. One PR-body guards run was cancelled when it was superseded, and its re-run succeeded. K = 5.
4. Accepted deviations: the patrol is two lines (duty + reader), because no 120 B line holds both. Nit N5 (an over-3000 PR whose governed paths are all Tier S also falls to the seat's own read) is already covered by 「只读受管 H」.
5. Out of scope / Skills-docs: review nit N1: review-checklist.md has no mcp_calls item; the refusal still lives at os-dev.md:374 and execution-duties.md:134–136. Metadata-protocol: none, so no Skills-docs: line is owed.
6. Landing: Tier S. pr_ready + automerge_enable on this head; the queue guard reads record 6724651237.