mirror of
https://github.com/codestable/CodeStable.git
synced 2026-09-19 09:03:09 +08:00
feat(cs-code-review): raise fix-and-rereview cap from 2 to 3 rounds
Owner decision after first dogfood hit of the cap. Escalation semantics unchanged: exceeding the cap still surfaces both sides' reasoning for human arbitration and never counts as a pass. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
@@ -20,7 +20,7 @@ argument-hint: "[--range <git-range>] [scope 或 audit 目标]"
|
||||
- 审查必须由**独立视角**执行:用独立 subagent reviewer,不受当前对话里实现思路的先入之见影响;subagent 不可用时向用户说明并获准后才降级为当前上下文自审。
|
||||
- 派发 reviewer 时给自包含上下文包:改动意图、审查范围与不审内容、期望返回的分级发现清单;整合发现与最终裁决由主流程负责。
|
||||
- 每个发现附 `文件:行号`(design 审查附对应小节)、问题说明和理由;不确定的标注为疑问而不是断言。
|
||||
- blocking 的修复-复审循环**最多 2 轮**:2 轮后仍未清零或存在分歧,停下把双方理由摆给用户裁决,不继续对轮。
|
||||
- blocking 的修复-复审循环**最多 3 轮**:3 轮后仍未清零或存在分歧,停下把双方理由摆给用户裁决,不继续对轮。
|
||||
|
||||
## 审查标准
|
||||
|
||||
|
||||
@@ -31,7 +31,7 @@ epic 天然跨会话,全程维护一个 `.codestable/work/{slug}-epic.md`:
|
||||
|
||||
## 硬门槛
|
||||
|
||||
- **拆解方案必须经用户确认**(子项清单、顺序、边界)后才开始执行;交确认前先用 `cs-code-review` 的 design review 做独立审查(修复-复审最多 2 轮,超限连分歧一起上交)。执行中要增删子项或改边界,先更新文档并征得同意。
|
||||
- **拆解方案必须经用户确认**(子项清单、顺序、边界)后才开始执行;交确认前先用 `cs-code-review` 的 design review 做独立审查(修复-复审最多 3 轮,超限连分歧一起上交)。执行中要增删子项或改边界,先更新文档并征得同意。
|
||||
- 每个子项按其类型的纪律执行(cs-feat / cs-issue / cs-refactor 的门槛照常生效),完成即更新 epic 文档状态——文档与事实不一致时以仓库事实为准并修正文档。
|
||||
- 全部子项完成后**不代替用户做整体验收**:给出汇总(各子项结果、验证证据、遗留项)并停下等用户确认。
|
||||
|
||||
|
||||
@@ -21,7 +21,7 @@ argument-hint: "[功能描述]"
|
||||
|
||||
## 风险升级信号
|
||||
|
||||
出现任何一条,走设计对齐再动手:把方案要点(改什么、契约变化、取舍、影响面——影响面分**必须修改 / 需要验证 / 仍待调查**三层)写入 `.codestable/work/{slug}.md` → 用 `cs-code-review` 的 design review 做独立审查(修复-复审最多 2 轮,超限连分歧一起上交)→ 交用户确认后动手。存在会卡死方案的技术风险时,先按风险降序垂直打通主路径再铺开(穿刺协议见 `references/code-design.md`)。信号清单:
|
||||
出现任何一条,走设计对齐再动手:把方案要点(改什么、契约变化、取舍、影响面——影响面分**必须修改 / 需要验证 / 仍待调查**三层)写入 `.codestable/work/{slug}.md` → 用 `cs-code-review` 的 design review 做独立审查(修复-复审最多 3 轮,超限连分歧一起上交)→ 交用户确认后动手。存在会卡死方案的技术风险时,先按风险降序垂直打通主路径再铺开(穿刺协议见 `references/code-design.md`)。信号清单:
|
||||
|
||||
- 公开 interface、持久化 schema 或跨模块协议变化;
|
||||
- 权限、信息安全、数据迁移、并发或不可恢复副作用;
|
||||
|
||||
Reference in New Issue
Block a user