Skip to content

Commit e669e5e

Browse files
os-justinclaude
andauthored
pm-dispatch: name the review of record for default-tier lanes, make clearing cite it, and read it in check-clause2-carriers --pair (C6) (#17349)
* feat(pm-dispatch): name the review of record for default-tier lanes; --pair reads it (C6) contract-review.md names the default-tier lanes' review of record — one comment on the PR or its card in the tier verdict's shape minus the tier line — and makes every clear cite it; SKILL.md and core-rules carry the twin lines. check-clause2-carriers --pair gains C6: on a completed pair it reads the PR's thread and the card's for that record (H51's heading + head-sha shape, plus a Reviewed-by: line) and answers 4 when none exists. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MoTv7pn338AZ71owsp19gQ * fix(pm-dispatch): key --pair's success sentence on note CODE, not note count The C6-RECORD note made renderPair print the C2-SIBLING wording; each note kind now speaks for itself, and a found record is named in the sentence. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MoTv7pn338AZ71owsp19gQ --------- Co-authored-by: Claude <noreply@anthropic.com>
1 parent 706ad0f commit e669e5e

4 files changed

Lines changed: 390 additions & 28 deletions

File tree

.claude/skills/pm-dispatch/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -638,8 +638,8 @@ PM 的工作是循环:选卡 → 认领 → 派发 → 收集 → 复核 → 报
638638
- `--tier` 嫌疑行是提示非裁定;双肢命中任一 ⇒ 无席内条款②复核 PASS 在案 ⛔ 禁止入队。
639639
- 路径肢 = diff 触及契约面 `packages/spec/src/**`,含 error-code-ledger 与 `*.zod.ts` 契约 schema。
640640
- 声明肢 = 认领评论声明 `Clause-②: yes`,与路径无关;错误的 `no` 是可审计的假申报。
641-
- 交付后复核由派发席席内完成:spec 席达档自审或派该档子任务,余席默认档自审加门禁
642-
- PASS ⇒ 同席剥标、ready、auto-merge;FAIL ⇒ 补丁轮;⛔ 免复核不放行。
641+
- 交付后复核由派发席席内完成:spec 席达档,余席默认档加门禁;记录 = 同形评论落 PR 或卡
642+
- PASS ⇒ 同席剥标并引记录、ready、auto-merge;FAIL ⇒ 补丁轮;⛔ 免复核不放行。
643643
- 真正设计分叉照旧进决策箱,席内复核 ⛔ 不替代维护者裁定。
644644
- 外部评审链降为可选事后审计,非放行前提。
645645
- `needs:contract-review`(恒英文)由 PR 创建者随可复审契约增量同笔挂:draft PR,或先到的报告。

.claude/skills/pm-dispatch/references/contract-review.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,15 +18,15 @@
1818
- 前瞻的条款②事实住卡上裁决与分诊评论、`Clause-②` 申报和 `--tier` 输出。
1919
- 开着的载体恒 = 真实待审;载体不迁移:⛔ 不迁 PR review 或 Request Changes,⛔ 不为迁移留门。
2020
- FAIL 同 PASS 剥双载体:同笔留卡上交接评论(引复审、独立性对、欠改);卡态与 assignee 不动。
21-
- 重挂前先查裁决:闸门标签缺失 ⇒ 先 grep 卡评论找复审结论。
22-
- PASS + 无标 + head 未动 = 已清标不是被剥;head 后移或无结论才重挂。
23-
- `get_reviews` 读空 ≠ 未复审。
21+
- 重挂前先查裁决:闸门标签缺失 ⇒ 先 grep 卡评论找复审结论;`get_reviews` 读空 ≠ 未复审。
22+
- PASS + 无标 + head 未动 = 已清标不是被剥;head 后移或无结论才重挂;清标缺引记录即半态。
2423

2524
## 复核归属与资格(席内)
2625

2726
- 归属该卡派发席,交付后收集复核当轮席内完成。
2827
- spec 席审低档实现者的契约增量,非自身产物;余席条款②复核 = 默认判断档自审加门禁。
29-
- 契约判断清单逐项落卡或 PR 评论,⛔ 不是散文自述。
28+
- 复核记录 = 一条评论落 PR 或卡,达档与默认档同形;散文、dev 自评、`os-dev-report` 恒不算。
29+
- 同形 = `## Contract review` 题头、所审 head sha 码段、①②③ 逐项、独立性对、PASS/FAIL 判词。
3030
- ① derived judgments 逐项:diff 引出的接受集与公开面变化逐条点名判对错。
3131
- ② semver 定级:变更级别与 changeset 声明一致。
3232
- ③ 边界旗处置:dev 挂旗与 `open_questions` 逐旗答复或升级。
@@ -35,12 +35,12 @@
3535
- 裁决载独立性对(机读):产出 diff 的身份写 `Implemented-by:`,出裁决的席位写 `Reviewed-by:`
3636
- `mode:subagent` dev 记其分支 `claude/issue-…`(子代理无自有 session);`mode:remote` dev 记 session id。
3737
- 两者同 session ⇒ 报 SELF-REVIEW,⛔ 不作独立复核;两行皆无的历史裁决恒静默。
38-
- 放行 = 清标即落地:PASS ⇒ 同席同笔剥双载体,清标同笔留 provenance 评论引该 PASS
38+
- 清标即落地:PASS ⇒ 同席同笔剥双载体;凡清标同笔留 provenance 评论,引记录 id 与所判 head
3939
- 随后按 `landing-operations.md` 走落地前检 → 转 ready → 挂 auto-merge 或入队。
4040
- 轮次报告设复审清单专节,形状与代裁清单同为强制审计。
41-
- 落地前检三条:① 席内条款②复核 PASS 在案,即卡上的复核裁决评论(spec 席达档)。
41+
- 落地前检三条:① 席内条款②复核 PASS 在案,即 PR 或卡上同形的复核记录(spec 席达档)。
4242
- ② 双载体已清,逐对机读 `node scripts/pm/check-clause2-carriers.mjs --pair PR-NUMBER`
43-
- 该命令 0 = 双肢一致且无放宽 tell,4 = 任一不成立,3 = 环境答不了;⛔ 3 不作干净。
43+
- 0 = 双肢一致、无放宽 tell、head 上有记录;4 = 任一不成立;3 = 环境答不了,⛔ 不作干净。
4444
- 放宽 tell 由 `scripts/pm/check-widening-tells.mjs` 判,`no` 撞新键/成员/导出/登记即拒,附 file:line
4545
- ③ PR 全部 check 全绿,⛔ 非 required 子集;受管面不适用,draft-only 终局不变。
4646

.claude/skills/pm-dispatch/references/core-rules.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@
120120
- 复核对 GitHub 核验 ⛔ 不对自述核验,逐项过清单并亲核形态、范围与整包价值密度。
121121
- CI 收敛读数只属于复核侧;判决三种:验收落卡、返工最多两轮、升级走决策通道。
122122
- 验收后先取 PR 路径面,一条命中受管面即分叉,⛔ 不翻正式、不入队、不挂自动合并。
123-
- 受管面 PR 留 draft 并向两个授权批准账户请审;无席内复核 PASS 的契约卡 ⛔ 禁止入队。
123+
- 受管面 PR 留 draft 并向两个授权批准账户请审;契约卡无同形复核记录 PASS ⛔ 禁止入队。
124124
- 入队资格是每一个检查全绿 ⛔ 不是必查子集;碰生成物的 PR 入队前先同步再重生成。
125125
- 唯一例外:源码自述 pushed 上按设计而红、不跑 `merge_group`、评论记明门与因,三条全立。
126126
- 跟到合并为止并每轮读队列成员资格与 `origin/main`;踢出按签名台账四分支处置。

0 commit comments

Comments
 (0)