Skip to content

Commit a77b4e9

Browse files
claude[bot]claude
andauthored
skills(pm-dispatch): a mid-task narrowing names its replacement owner and the moment (#17794)
Fixes #17729 A mid-task note that narrows a standing clause and names neither who takes the narrowed duty over nor when it acts leaves the dev holding half a contract. The measured instance was a live dual-carrier split that the seat had to repair by hand. This lands the rule in the runbook section where the dispatch-word rules live. ## The edit: one clause, riding an existing bullet by density `.claude/skills/pm-dispatch/references/dispatch-runbook.md` line 222, inside the in-flight-overlap interception group (lines 220-222). Before (23 bytes): ```text - ⛔ 不硬造 diff。 ``` After (119 bytes, cap 120): ```text - ⛔ 不硬造 diff。中途收窄标准条款须点名替换承接人与时点,缺一 dev 报 blocked 不半执行。 ``` The appended clause is 96 bytes. The file is still 241 lines: no line added, no rule deleted, no re-wrap used as payment, ceiling untouched. ## The four elements, mapped to the words | Element | Words that carry it | |:---|:---| | (a) an instruction that narrows a standing contract mid-task | 中途收窄标准条款 | | (b) names the replacement owner | 须点名替换承接人 | | (c) and the moment the replacement acts | 与时点 | | (d) a narrowing naming neither is refused by the dev (report `blocked`) rather than executed half | 缺一 dev 报 blocked 不半执行 | `标准条款` is the term the file and `.claude/agents/os-dev.md` already use for the standing clauses (see line 226 「条件性标准条款…」 and line 231 「信封断言本体是 os-dev 标准条款」), so the clause names the same object the contract does. ## Why this host Lines 220-222 are the runbook's one rule group about a note sent to a dev **mid-task**: the seat intersects each round's newly landed PRs with the in-flight file surfaces and sends a four-sentence warning. Line 222 is that group's tail rule about how the dev must not contort to comply with such a note (do not manufacture a diff). The new clause is the same channel's other failure mode — the note that narrows instead of warning — and the dev-side posture it prescribes (refuse, report `blocked`) is the same kind of rule as the one already there. It is also the only bullet in the section that can hold the clause at all. Free bytes re-measured on this branch against the 120-byte convention: line 222 has 97 free; the next largest are line 211 with 70, line 230 with 69, lines 214 and 228 with 63, line 201 with 61. The clause is 96 bytes, so no semantically closer host had room even in principle — the semantic choice and the byte budget point at the same line, and no trade-off had to be made. ## Governed verdict `node scripts/pm/check-governed-merges.mjs --test .claude/skills/pm-dispatch/references/dispatch-runbook.md` :: exit 3 ```text governed-surface predicate: 1 of 1 path(s) hit the register (5 surfaces, repo-agnostic). ⛔ GOVERNED — a human merge is the review record for this PR (#9495 regime). .claude/** ×1 — the agent instruction tree (skills, agents, hooks, settings) ``` This PR stays draft. No seat flips it ready, enqueues it, or arms auto-merge; the maintainer hand-merges after the skills seat's review. ## Gates Derived with `node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstack` (no paths passed: the script derives the change set itself from the merge base) and reconciled with `--ran` on the final diff at `902f019c`: ```text Run reconciliation — 14 derived, 14 run, 0 NOT-MEASURED, 0 UNRUN. ✓ dispatch-gates --ran: 14 derived famil(ies) accounted for — 14 run, 0 NOT-MEASURED (a DERIVED zero — all 14 recorded an exit code and none of them is 3). ``` All 14 exited 0. The ratchet's own verdict lines: ```text ✓ check-skill-line-ratchet: .claude/skills/pm-dispatch/references/dispatch-runbook.md: widest table row is 0 bytes (pin 0; headroom 0). ✓ check-skill-line-ratchet: .claude/skills/pm-dispatch/references/dispatch-runbook.md is 241 lines (ceiling 241; headroom 0). ``` `pnpm --filter @objectstack/lint run check:doc-formula-expressions` first exited 3 (PREREQUISITE NOT MET — `@objectstack/formula` and `@objectstack/lint` unbuilt in a fresh worktree; nothing measured). After `pnpm exec turbo run build --filter=@objectstack/formula --filter=@objectstack/lint --concurrency=2` under `scripts/pm/os-verify-lock.sh` (VERDICT command-exit 0), it re-ran :: exit 0. Control-character self-scan over the changed file (`grep -naP` over the control ranges): no hits. Repo-wide `pnpm lint` is CI's run; the narrowed reading here is total — `pnpm exec eslint --no-inline-config --format json` over the one changed file reports `File ignored because no matching configuration was supplied`, i.e. the eslint config matches zero of this PR's files, so this diff cannot move any untouched file's verdict. No changeset: `.claude/**` is the skip-changeset fast track — nothing in any package's `files[]` moves. ## Acceptance notes - noted, not filed: the dispatch word asked for `node scripts/pm/dispatch-gates.mjs --commands .claude/skills/pm-dispatch/references/dispatch-runbook.md` (with a path); `.claude/agents/os-dev.md` says to derive without paths so the script reads the real change set from the merge base. The os-dev clause wins per its own conflict rule, and the two agree here anyway (one path, committed). Carrier: the next runbook edit that touches the derivation sentence. ## 维护者速读(草稿) **改了什么** — `dispatch-runbook.md` 〈派发词构造细则〉第 222 行末尾加一句:中途收窄标准条款的指令必须点名替换承接人与时点,缺一由 dev 报 `blocked`,不半执行。文件仍是 241 行,不动 ceiling,不删任何规则。 **为什么改** — 实测事故:座位中途把「挂/清 `needs:contract-review`」收归自己,却没说另一半由谁、在何时做;dev 已挂了 PR 侧、停下来报了旗(正确行为),卡与 PR 的双载体半态一直活着,直到座位手工补挂。规则把这种半态从「靠 dev 临场判断」变成「派发侧必须写全,否则 dev 拒收」。 **风险与代价(含回滚)** — 风险是 dev 对一条本来还能推进的收窄指令报 `blocked`,多一轮往返;代价上限是一轮。回滚成本为零:单行单句,`git revert` 即回到原句。 **席位意见** — (留空,待 skills 座位定稿成评论) **你要做的** — 这是受管面(`.claude/**`),按 #9495 规矩由你手工合并;先看上面「四要素对照表」确认这一句确实四件事都说到了,再看措辞是否合你对这份文件语气的要求(一 bullet 一规则、⛔/⇒ 记号、无卡号)。 --- _Generated by [Claude Code](https://claude.ai/code/session_01MCLBsUgfykL74aU716rzVK)_ Co-authored-by: Claude <noreply@anthropic.com>
1 parent 6682c5d commit a77b4e9

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.claude/skills/pm-dispatch/references/dispatch-runbook.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,7 @@
219219
- 该行原文:基于合并后的代码工作,issue 引用的片段可能已变,先核对当前 main。
220220
- 在飞重叠每轮拦截:main 新落 PR 与在飞申报文件面求交,相交即发四句警告。
221221
- 四句:合 main 重跑测试矩阵、读对方 diff 重划边界、只补它没覆盖的、被完全覆盖就停下。
222-
- ⛔ 不硬造 diff。
222+
- ⛔ 不硬造 diff。中途收窄标准条款须点名替换承接人与时点,缺一 dev 报 blocked 不半执行。
223223
- 全仓 pin 清扫两句原文:① grep 错误码与错误消息全仓扫同语义 pin,一轮翻完,不只改本包。
224224
- ② 翻转后的 pin 必须承重:断言新语义的实质(行数、译文、状态码),不是旧断言已删。
225225
- 真正非法形状的拒收断言逐字保留,被守护的面永不缩水。

0 commit comments

Comments
 (0)