[os-support-ai]

Requested by Yibo Wang · project thread

No card: maintainer-directed in the project thread (「修改 pm-dispatch/os-dev 技能」, after 「为什么等我批的现在没有推送給 os-zhuang，是技能的问题吗」).

维护者速读

• 改了什么:SKILL.md〈路径分叉〉三行就地改写(行数不变):③ 请审在席位 ACCEPT + 现 head PASS 落案的同一轮发出;>3000 行 PR 明写也走 ③ 请审;④ 轮报现读请审人,没请审就不许在聊天里说「等你批」。
• 为什么改:规则(10-29 的 pm-dispatch(review): a PR reaches a human only after the seat's ACCEPT and a same-head contract PASS: no seat assignee at the draft open, the approver request is the seat's hand-off, a PR landed owing a review is re-reviewed with findings filed as cards #27385)本来就要求席位交棒时请审 os-zhuang/hotlong,但席位没执行:feat(spec): page:header placedActions draws every placed header action, refused beside an actions list (#30018) #30028 已有 ACCEPT + 同 head PASS 却无人被请;refactor(spec): codemod split of conversions/registry.ts into per-entry modules (#29987) #29993、refactor(metadata-protocol): generate the protocol.ts section split with a deterministic codemod #29998 这类 >3000 行拆分 PR 被「Tier S 不交人」误读成不请;聊天里却报了「等你批」。
• 风险与回滚:只改技能文本 3 行,不动脚本、门禁、workflow;回滚 = revert 本 PR。
• 先看:.claude/skills/pm-dispatch/SKILL.md L187、L197、L201。
• 两分钟验证:pnpm check:pm-skill-ratchet && pnpm check:pm-skill-id-lint && pnpm check:skill-frame-sync,三条皆绿(行数 319/319,每行 ≤120 字节)。
• 需要你做的:审阅并批准本 PR(.claude/** 属 Tier S,但这是 PM 自身规则 PR,按〈路径分叉〉末行留给维护者)。

What changes

.claude/skills/pm-dispatch/SKILL.md, three lines rewritten in place (319 lines before and after, each ≤120 bytes):

• L197, step ③: the approver request (pr_request_reviewers) goes out in the same round that the seat's ACCEPT and a same-head contract PASS are both on record, not "after" at some later point.
• L187, the >3000-line rule: it already swaps ACCEPT for the four end-of-review steps with no Tier S exception; it now names step ③ explicitly, so L198's "Tier S hands off to nobody" no longer reads as covering a large diff on ungoverned paths (the split PRs).
• L201, step ④: the round report reads the requested reviewers live, and a seat never reports a PR to the maintainer in chat as awaiting approval before the request is on the PR; it sends the request first.

os-dev.md is unchanged: a dev's pr_create still carries no assignees and requests nobody. That is the rule the maintainer asked for on 2026-10-29 (「这个是技能的bug，应该是开发完了你审核通过才会推送给我审核」, 「审核人是 os-zhuang 和 hotlong」), landed in #27385, and this PR keeps it. Only the seat-side hand-off is tightened.

Why

Measured on 2026-11-04 ~10:10Z, the six PRs reported in chat as waiting for the maintainer:

PR	Seat ACCEPT on current head	PASS on current head	Requested reviewers before	os-zhuang review
#29968	yes	yes	none	APPROVED
#30004	yes	yes	none	APPROVED
#30028	yes (09:11Z)	yes	none	none
#29993 (35,227 lines)	no	no (PASS names an older head)	none	none
#29998 (24,422 lines)	no	no	none	none
#30030	no	yes	none	none

#30028 was a due hand-off with no request. The two splits are over 3000 lines and owe the same end-of-review steps, but L198's "Tier S hands off to nobody" plus a patrol (H43) that reads only register paths left them with no request and no warning. As a one-off, os-zhuang was requested on #30028, #29993, #29998 and #30030 from the project thread; #29968 and #30004 already carry an os-zhuang approval, so they wait on the seat's landing, not on a review.

Out of scope, follow-up if wanted: H43 in scripts/pm/check-half-states.mjs reads governed-register PRs only, so it cannot flag a >3000-line PR whose request is missing.

Verification

The 20 commands node scripts/pm/dispatch-gates.mjs --commands derives for this diff, run on head 262327bc, all exit 0 (pnpm --filter @objectstack/lint run check:doc-formula-expressions after turbo run build --filter=@objectstack/formula --filter=@objectstack/lint). Key lines:

• check-skill-line-ratchet: green, SKILL.md at its 319 ceiling.
• check-skill-id-lint: 110 files clean.
• check-skill-frame-sync and check-governed-prose: green.

Clause-②: no
Metadata-protocol: none

Human Minimum

• 1. Schemas start as Zod; packages/spec holds no runtime logic. (not touched)
• 2. Every name I added uses the casing for its kind. (none added)
• 3. No formatter ran; pnpm lint passes. (no formatter ran; pnpm lint not run for a three-line skill text edit)
• 4. One issue, one branch off an up-to-date main, one pull request.
• 5. A change that publishes carries a changeset, and both declaration lines are filled in here and in the changeset. (nothing publishes; skip-changeset)
• 6. packages/spec touched: its generated artifacts are regenerated and committed. (not touched)
• 7. No hand edits to content/docs/references/, packages/*/CHANGELOG.md or content/docs/releases/.
• 8. Title, body and comments are in English. (the 维护者速读 block is Chinese by the skill's own rule)
• The Before Submitting list in CONTRIBUTING.md is done.

---

Generated by Claude Code