Review Work 是一个面向工作产物的只读审查技能,来自 klausborges/workflow-skills。它的核心用途是:对照预期行为、文档、验收标准、测试以及其他权威资料,审查计划(Plan)、实现、续审轮次或其他限定范围的产物。除非用户明确要求修复或实现,否则该技能保持只读,不会主动修改内容。
主要能力与审查模式
该技能会先确定审查范围,包括主要目标、比较基准、边界以及请求的审查强度。对于分支或差异审查,会先解析比较点,并在进行高成本分析前确认目标范围非空。只有在无法安全发现边界时才会提问。
它支持多种审查模式:
- 计划审查模式:判断计划是否连贯、可执行、范围明确且可验证,检查依赖关系、任务与验收标准的覆盖、前置条件、锁定决策矛盾、跨计划归属、审查位置、验证方式以及文档生命周期。除非计划无法安全执行,否则不会要求实现证据、文字润色、穷尽边缘情况或架构重设计。
- 实现审查模式:从具体代码、测试、文档、差异或交付产物出发,将相关计划和当前文档作为参考事实,而非主要审查对象。
- 续审模式:聚焦已接受的修复、补丁说明、变更面、未解决风险以及已确定事项,避免重复争论。
- 独立审查:用于显式交叉验证或强制性的全新上下文实现审查。在可行时先检查一手证据,再对照先前发现或作者结论,之后进行调和。
- 临时审查:将同样的证据导向方法适配到用户指定的文件集、功能、分支、提交或非代码产物。
审查方法
审查过程包括:阅读仓库说明和权威产物;按风险需要检查实际目标及其调用方、测试、文档或运行时行为;当变更涉及环境变量、密钥、端口、脚本或安装命令时,追踪受影响的开发者运行/构建路径;用代码、差异、模式、命令、第一方文档或聚焦的运行时检查验证假设;寻找正确性失败、缺失需求、不安全的生命周期行为、文档漂移、测试缺口、安全/数据丢失风险以及重大可维护性问题;在报告前调和重复或冲突的证据。
对于全新实现审查,会包含一次校准的结构简化检查,查看是否存在可避免的概念、分支、可变状态、间接层或明显更简单的设计,但只报告具体且相称的改进。
当变更跨越共享契约、持久化、安全、并发、部署或广泛使用的接口时,可使用“爆炸半径”作为可选视角,按风险比例追踪受影响的调用方和消费方,识别变更安全所依赖的一两个不变量,并通过最便宜的稳定真实表面加以证明;未达到证明程度的会报告为残余风险,默认不进行全仓库影响扫描。
默认采用平衡审查。只有在用户要求或风险与不确定性明显值得成本时,才增加对抗性深度、多视角、子代理、研究或额外轮次。子代理报告仅作为线索,主审查者需对照一手证据验证每个实质性被接受的声明,且不会递归地委托对审查的审查。没有发现数量下限,重复问题会合并,可操作集合通常保持在约 15 条以下。覆盖充分时,干净的审查结果是有效的。
强制阶段审查
对于阶段结束或直接实现审查:独立审查者必须获得全新上下文;范围是整个阶段或阶段等效实现,而不仅是最新补丁;计划定义的高影响检查点不能替代此审查;只有在用户另行授权实现后才修复发现;修复后仅重跑由变更面和残余风险所证明合理的审查覆盖。
文档与产物生命周期
对于计划支撑的实现,只检查当前计划拥有的草稿和文档增量项。无关草稿可以记录,但不阻塞收尾。文档漂移会作为发现报告,不会静默编辑;小的事实漂移可建议直接修复,产品敏感或实质性漂移则阻塞收尾,直到决定或实现。当存在阻塞性发现、未完成验证、拥有的文档工作或未完成实现时,不会提议删除计划。当已完成的计划不在活跃路线图中时,会在最终审查和文档同步后询问一次是否删除;如果活跃路线图引用它,则报告完成就绪状态,并将计划/路线图变更留给另行授权的工作流步骤;路线图收尾门控清理。绝不自动删除。
输出
输出以按严重程度排序的已验证发现开头。每条发现包含紧凑的文件/行或命令证据、为何重要、具体影响以及建议修复。区分明确缺陷、合理权衡和所有者策略决定。对于实质性审查,报告紧凑的审查台账,包括范围与验证、审查信号(高/中/低)、建议、残余风险。仅在适用时添加计划、文档、轮次、调和或下次审查字段。对于琐碎的临时检查,简洁结论加上范围与验证即可。不产生 N/A 脚手架或长记录。当没有发现留存时,会说明并陈述覆盖范围和限制。