diff --git a/docs/design/01-four-phase-loop.md b/docs/design/01-four-phase-loop.md index d3a325a..69168c4 100644 --- a/docs/design/01-four-phase-loop.md +++ b/docs/design/01-four-phase-loop.md @@ -2,7 +2,7 @@ **序号**: #01 **名称**: 四相循环 — PRD Phase 1~4 完整实现方案 -**版本**: v1.1(附录扩展:完整讨论过程记录) +**版本**: v1.2(修复司马懿评审 3 个设计问题 + 采纳 4 条建议) **基于**: PRD-v3.0 §4 四相架构 + architecture-v3.0.md **作者**: 庞统(副军师)🐦 **日期**: 2026-05-29 @@ -550,3 +550,31 @@ AGENTS.md 中的角色从"质量总监:代码评审、多空辩论、最终验 ### A.10 待确认事项 无。本方案为完整设计方案,待司马懿评审后决定下一步。 + +### A.11 司马懿评审记录(v1.2 修复) + +**评审结论**:通过,需修复 3 个设计问题。 + +**3 个设计问题修复**: + +| # | 问题 | 修复方案 | 位置 | +|---|------|---------|------| +| 1 | @mention 通知丢失(Agent busy 时 spawn 失败,mention 永远收不到) | 新增 `mention_queue` 表 + 重试机制 + 通知状态追踪 | §3.4 | +| 2 | 一轮结束检测不区分 done/failed/cancelled | 传给庞统的状态摘要包含维度信息 {done, failed, cancelled},prompt 增加失败处理指引 | §4.1 | +| 3 | 司马懿 review 和庞统 review 执行顺序未定义 | tick 流程明确 5 步顺序,一轮结束检测在调度之后 | §4.4 | + +**4 条建议采纳**: + +| # | 建议 | 采纳情况 | +|---|------|---------| +| 4 | Spawn Prompt 补充 Boids 四条具体行为描述 | ✅ 已映射为 Separation/Alignment/Cohesion/Boundary 四条准则 | +| 5 | 代码改动清单缺少 MVP 优先级标注 | ✅ 每行加 MVP 列(✅ 必须 / ❌ 后续) | +| 6 | 风险表缺少 3 个关键风险(无限循环/assignee 缺失/token 消耗) | ✅ 新增 3 条风险 + 缓解方案 | +| 7 | 司马懿 review skill 设计方向 | ✅ 记录为参考,司马懿独立设计不阻塞 #01 | + +**v1.2 新增章节**: +- §4.4 一轮结束后的执行顺序 +- §4.5 防止庞统 review 无限循环 +- §3.4 扩展 @mention 通知可靠性保障 + +**Rebuttal 备注无**——3 个设计问题全部接受,无争议。