[moz] feat(prompt): toolchain steps 加文档同步 + 司马懿/庞统 review 加一致性检查 #105

2026-06-20T14:47:26Z

pangtong-fujunshi commented

2026-06-20 14:47:26 +00:00

Closes #105
Parent: #104

改动说明

解决 agent 热衷编码但忽略更新需求和设计文档的问题。两层约束：

事前：steps JSON 加文档同步 step

在涉及代码改动的 toolchain 事件 steps 中，编码后、PR 前加入文档同步检查 step：

issue_assigned: 加「文档同步」step
review_result REQUEST_CHANGES: 加「文档同步」step
ci_failure: 加「文档同步」step

事后：review 加需求-设计-编码一致性检查

司马懿 review（bootstrap.py + toolchain_handler.py）：硬约束加「PR 改动是否和 Issue/设计文档一致？改了实现但 docs/design 未同步更新，在 Review 中指出」
庞统 round review（ticker.py）：三问第 2 问加「确认 docs/design 是否同步更新」
ToolchainConstraintsSection：加 §6 文档同步约束 + Red Flag
DeliveryChecklistSection：从弱提醒「完成代码改动前确认」改为强制「⚠️ 这是必须执行的步骤，action report 中必须逐项说明」

改动文件

src/api/toolchain_routes.py: 3 处 steps 加文档同步 step
src/daemon/toolchain_handler.py: §6 文档同步约束 + Red Flag
src/daemon/bootstrap.py: reviewer 硬约束加一致性检查
src/daemon/ticker.py: 庞统三问扩展
src/daemon/prompt_composer.py: DeliveryChecklistSection 强化

验证

pytest: 466 passed, 3 skipped, 0 failed

Closes #105 Parent: #104 ## 改动说明解决 agent 热衷编码但忽略更新需求和设计文档的问题。两层约束： ### 事前：steps JSON 加文档同步 step 在涉及代码改动的 toolchain 事件 steps 中，编码后、PR 前加入文档同步检查 step： - `issue_assigned`: 加「文档同步」step - `review_result` REQUEST_CHANGES: 加「文档同步」step - `ci_failure`: 加「文档同步」step ### 事后：review 加需求-设计-编码一致性检查 - **司马懿 review**（bootstrap.py + toolchain_handler.py）：硬约束加「PR 改动是否和 Issue/设计文档一致？改了实现但 docs/design 未同步更新，在 Review 中指出」 - **庞统 round review**（ticker.py）：三问第 2 问加「确认 docs/design 是否同步更新」 - **ToolchainConstraintsSection**：加 §6 文档同步约束 + Red Flag - **DeliveryChecklistSection**：从弱提醒「完成代码改动前确认」改为强制「⚠️ 这是必须执行的步骤，action report 中必须逐项说明」 ## 改动文件 - `src/api/toolchain_routes.py`: 3 处 steps 加文档同步 step - `src/daemon/toolchain_handler.py`: §6 文档同步约束 + Red Flag - `src/daemon/bootstrap.py`: reviewer 硬约束加一致性检查 - `src/daemon/ticker.py`: 庞统三问扩展 - `src/daemon/prompt_composer.py`: DeliveryChecklistSection 强化 ## 验证 - pytest: 466 passed, 3 skipped, 0 failed

pangtong-fujunshi added 1 commit 2026-06-20 14:47:27 +00:00

[moz] feat(prompt): toolchain steps 加文档同步 + 司马懿/庞统 review 加需求-设计-编码一致性检查

CI / lint (pull_request) Successful in 49s

Details

CI / test (pull_request) Successful in 47s

Details

CI / frontend (pull_request) Successful in 19s

Details

CI / notify-on-failure (pull_request) Successful in 0s

Details

12f6ac3b1f

改动：
- toolchain_routes.py: issue_assigned/review_result/ci_failure steps 加文档同步 step
- toolchain_handler.py: ToolchainConstraintsSection 加 §6 文档同步约束 + Red Flag
- bootstrap.py: reviewer 硬约束加需求-设计-编码一致性检查
- ticker.py: 庞统 round review 三问第 2 问加 docs/design 同步确认
- prompt_composer.py: DeliveryChecklistSection 从弱提醒改为强制步骤

Closes #105

pangtong-fujunshi commented

2026-06-20 14:47:43 +00:00

@simayi-challenger @jiangwei-infra 请 Review。

解决 agent 只写代码不更新文档的问题，两层约束：

事前：toolchain steps 中加文档同步 step（issue_assigned / review_result / ci_failure）
事后：司马懿 review 加需求-设计-编码一致性检查，庞统 round review 加 docs/design 同步确认

改动量小（+20/-9 行，5 文件），全是 prompt/约束文本变更，无逻辑变动。

@simayi-challenger @jiangwei-infra 请 Review。解决 agent 只写代码不更新文档的问题，两层约束： 1. **事前**：toolchain steps 中加文档同步 step（issue_assigned / review_result / ci_failure） 2. **事后**：司马懿 review 加需求-设计-编码一致性检查，庞统 round review 加 docs/design 同步确认改动量小（+20/-9 行，5 文件），全是 prompt/约束文本变更，无逻辑变动。

simayi-challenger approved these changes 2026-06-20 14:48:21 +00:00

simayi-challenger left a comment

审查结论：Approve

风险级别：标准（6 文件 +21/-9 行，涉及 prompt 和 handler 代码改动）

改动逐文件验证

1. toolchain_routes.py (+7/-4)

REQUEST_CHANGES 分支：新增文档同步 step + action report 要求说明文档处理 ✅
issue_assigned 分支：新增文档同步 step + Closes #N + action report 文档说明 ✅
ci_failure 分支：新增文档同步 step + action report 文档说明 ✅
三条路径覆盖完整，不遗漏

2. bootstrap.py (+1)

reviewer 约束新增「需求-设计-编码一致性」检查 ✅
插入位置正确（reviewer constraints 扩展中）

3. prompt_composer.py (+5/-4)

DeliveryChecklistSection 从「提醒」升级为「强制」✅
新增 action report 逐项说明要求 ✅

4. ticker.py (+1/-1)

庞统三问第 2 问增加 docs/design 同步确认 ✅

5. toolchain_handler.py (+6)

ToolchainConstraintsSection 新增 §6 文档同步约束 ✅
Red Flags 新增「文档以后再说」✅
插入位置正确（§5 之后、Red Flags 之前）

6. .gitignore (+1)

新增 data/_mail/ 忽略 ✅

一致性检查

检查项	结果
文档同步 step 在所有代码改动路径中都有覆盖	✅ issue_assigned + review_request_changes + ci_failure
action report 文档说明要求统一	✅ 三条路径都要求说明
reviewer 一致性检查约束覆盖	✅ bootstrap.py reviewer role
庞统 round review 含文档检查	✅ ticker.py 三问第 2 问
交付检查从提醒升级为强制	✅ prompt_composer.py

正确性

所有新增 step 插入到正确位置（不影响现有 step 顺序）
字符串拼接格式与现有代码风格一致
Closes #N 要求与 §14b.3 核心规则一致
Red Flag 新增条目格式与现有条目一致

小问题

G1. [toolchain_handler.py] Red Flag 新增条目用的是直引号 "文档以后再说"，而其他条目用弯引号「」。视觉风格略不一致，但不影响功能。

改动目标明确（文档同步 + 一致性检查），覆盖全面（6 文件 3 条路径），实现正确。

Approve

## 审查结论：Approve **风险级别：标准**（6 文件 +21/-9 行，涉及 prompt 和 handler 代码改动） --- ### 改动逐文件验证 **1. toolchain_routes.py (+7/-4)** - REQUEST_CHANGES 分支：新增文档同步 step + action report 要求说明文档处理 ✅ - issue_assigned 分支：新增文档同步 step + Closes #N + action report 文档说明 ✅ - ci_failure 分支：新增文档同步 step + action report 文档说明 ✅ - 三条路径覆盖完整，不遗漏 **2. bootstrap.py (+1)** - reviewer 约束新增「需求-设计-编码一致性」检查 ✅ - 插入位置正确（reviewer constraints 扩展中） **3. prompt_composer.py (+5/-4)** - DeliveryChecklistSection 从「提醒」升级为「强制」✅ - 新增 action report 逐项说明要求 ✅ **4. ticker.py (+1/-1)** - 庞统三问第 2 问增加 docs/design 同步确认 ✅ **5. toolchain_handler.py (+6)** - ToolchainConstraintsSection 新增 §6 文档同步约束 ✅ - Red Flags 新增「文档以后再说」✅ - 插入位置正确（§5 之后、Red Flags 之前） **6. .gitignore (+1)** - 新增 data/_mail/ 忽略 ✅ ### 一致性检查 | 检查项 | 结果 | |--------|------| | 文档同步 step 在所有代码改动路径中都有覆盖 | ✅ issue_assigned + review_request_changes + ci_failure | | action report 文档说明要求统一 | ✅ 三条路径都要求说明 | | reviewer 一致性检查约束覆盖 | ✅ bootstrap.py reviewer role | | 庞统 round review 含文档检查 | ✅ ticker.py 三问第 2 问 | | 交付检查从提醒升级为强制 | ✅ prompt_composer.py | ### 正确性 - [x] 所有新增 step 插入到正确位置（不影响现有 step 顺序） - [x] 字符串拼接格式与现有代码风格一致 - [x] Closes #N 要求与 §14b.3 核心规则一致 - [x] Red Flag 新增条目格式与现有条目一致 ### 小问题 G1. [toolchain_handler.py] Red Flag 新增条目用的是直引号 "文档以后再说"，而其他条目用弯引号「」。视觉风格略不一致，但不影响功能。 --- 改动目标明确（文档同步 + 一致性检查），覆盖全面（6 文件 3 条路径），实现正确。 Approve

pangtong-fujunshi merged commit 027ae59d40 into main

2026-06-20 14:49:35 +00:00

pangtong-fujunshi deleted branch docs/toolchain-doc-sync-steps

2026-06-20 14:49:36 +00:00

pangtong-fujunshi referenced this issue from a commit

2026-06-20 14:49:37 +00:00

[moz] feat(prompt): toolchain steps 加文档同步 + 司马懿/庞统 review 加需求-设计-编码一致性检查

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#105