审查
使用此技能来评估已完成的工作。审查绝不会编写它正在评判的变更。
这是规范的审查工作流。merge-readiness 会路由至此,而 ai-slop-cleaner 是其内部的一个可选通道。
目标
找出实际存在的问题,按严重性排序,并提供足够详细的信息以便采取行动。
工作流程
- 明确变更内容及其意图。
- 对照该意图阅读变更。
- 首先检查正确性,然后检查风险,最后检查简化空间。
- 在报告每个候选发现之前进行验证。
- 首先报告最严重的发现。
检查内容
- 正确性 — 逻辑缺陷、边界情况、错误路径、并发问题
- 风险 — 安全边界、破坏性操作、数据完整性
- 复用 — 变更本应使用的现有工具或模式
- 简化 — 可删除或合并的代码
- 覆盖率 — 未经测试即交付的行为
规则
- 分离通道:审查者不得与作者处于同一活跃上下文。
- 报告前验证。一个听起来合理但无法复现的发现是噪音。
- 诚实陈述严重性;不要为了显得全面而夸大列表。
- 当工作本身无误时,“无发现”是有效结果。
- 默认为建议——审查提供信息,而非阻塞。硬性门禁(如发布、安全、破坏性操作)保持独立且默认关闭。
输出
- 发现项,按最严重到最不严重排序,每个包含文件、行号以及具体的故障场景
- 已检查且确认无误的内容
- 任何无法评估的内容