← ClaudeAtlas

reviewlisted

用于对稳定 diff 做 scope、correctness、docs、entropy 和 risk 的结构性评审。触发条件:implement/diagnose 后准备进入 ready 证明,或用户要求 review/sanity check。不要在 WIP、失败未解释或只需要 fresh evidence 时使用;review 不能声明 ready。
YSAA1/harness-workflow · ★ 0 · Code & Development · score 60
Install: claude install-skill YSAA1/harness-workflow
# 工作流结构评审 `review` 用 **Spec / scope / diff / docs / entropy / risk** 六把尺检查当前结果,并默认采用 **adversarial review posture**:把实现视为未证明可信,先构造可能失败路径,再寻找代码、文档、测试或 evidence 反证。它抓正确性、范围、设计风险、缺失测试和文档漂移;fresh evidence sufficiency 与 ready judgement 归 `verify`。 ## 路由快照 - **Use when**: 有意义改动已经稳定,需要在 ready 前判断范围、正确性、文档、熵和风险。 - **Do not use when**: 工作仍在 WIP、失败正在发生、或唯一问题是缺 fresh evidence。 - **Route to**: 无 blocking finding 转 `verify`;有结构问题转 `implement`;失败不明转 `diagnose`。 ## 目的 review 必须同时检查"做超了"和"做欠了"。它可以判断"没有结构性 blocker",但不能作为最终 ready gate。通过后仍路由到 `verify`;若已有 fresh evidence,则走 `verify` fast-path 做短重检和 claim mapping。 默认原则:meaningful diff 必须先尝试 **隔离 reviewer**,失败、不可用或成本明显不成比例时才 fallback。隔离优先级: 1. read-only subagent / independent reviewer。 2. Codex 环境中的 `codex exec review`。 3. Codex 环境中的 `codex exec` + review packet + reviewer prompt。 4. packet-based fallback,由当前 agent 按同一 prompt 做弱隔离审查。 fallback 不是静默自审;输出必须记录机制、失败原因或跳过原因。Tiny / trivial diff 可以直接 fallback,但必须说明为什么不属于 meaningful diff。 ## 何时使用 ### 触发信号 - `implement` 完成一个 slice、即将转下一阶段。 - `diagnose` 修复稳定,要回主流程。 - 用户说「review 一下」「这样可以吗」「能不能合并」「commit 前检查」「sanity check」。 - 准备调用 `verify` 之前的结构性把关。 - 多 agent 协作时上一个 agent 刚交付,本 agent 接手前。 ### 不要使用 - 工作还在 WIP 中、active slice 未稳定:继续 `implement`。 - 工作完全没有稳定到可审状态:继续 `implement`。 - 用户要架构建议:转 `brainstorm`。 - 失败正在发生:转 `diagnose`。 ### 路由规则 | 状态 | 下一步 | | --- | --- | | review 通过或仅有 evidence gap | `verify` | | review 找出 Critical/Important findings | `implement` | | review 发现 Spec 漂移 | `brainstorm` 或 `plan