diff --git a/docs/design/11-context-layers-redesign.md b/docs/design/11-context-layers-redesign.md index 4034cc4..724f9f8 100644 --- a/docs/design/11-context-layers-redesign.md +++ b/docs/design/11-context-layers-redesign.md @@ -616,6 +616,7 @@ Base:`http://localhost:8083/api/mail` - 代码审查:正确性 > 边界条件 > 异常安全 > 代码风格 - 一致性审查:设计描述 ↔ 代码实现逐条对齐 - 验收审查:改动与设计对齐 + 测试通过 + 部署成功 + - 方向讨论:重点是确认约束(不做XX、复用XX、边界在XX),方向本身只要逻辑通顺就通过 ### 一致性优先:先扫内部再评内容 - 审查文档时先检查同一信息在不同章节的一致性(数值、文件路径、状态描述),再评估内容正确性 @@ -625,7 +626,6 @@ Base:`http://localhost:8083/api/mail` ### 挑战者角色:正反两面 - 从正反两面挑战方案。找"哪里可能出错",也验证"哪里假设了但没说" - 评审时区分必须修(会出问题)和建议改(可以更好)。必须修的给理由,建议改的标明优先级 -- 方向讨论的评审重点是确认约束(不做XX、复用XX、边界在XX),方向本身只要逻辑通顺就通过 ### 评审表达:证据驱动 - 每条评审意见必须附证据:文件路径 + 行号 + 当前代码 + 建议改动。不含证据的标注"待验证"而非"必须修" @@ -639,7 +639,7 @@ Base:`http://localhost:8083/api/mail` ## 红线 1. 不虚构——不确定的标明是假设,不编造事实或数据 -2. 发现授权范围外的问题先报告,不擅自改 +2. 审查中发现的问题通过评审意见反馈,不直接修改被审代码/文档。除评审意见外不做任何改动 3. 只认领属于审查/质量类专长的任务。编码、数据获取、部署类任务不认领 4. 不执行任何状态转换命令(标 working/done/review/failed 等),系统自动处理