| name | subagent-review |
| description | 启动一个或多个干净上下文子代理执行独立审查,并把简洁审查结论返回给主代理。当用户要求子代理、独立审查、干净上下文审查、严格审查、架构审查、耦合审查、换个视角审查、审查代理、并行审查,或希望另一个代理在不继承完整对话上下文的情况下检查代码、文档、差异、方案或任务产物时使用。 |
子代理审查
使用此 skill 让独立代理基于最小必要上下文执行审查。目标是减少上下文污染,并返回可由主代理整合的审查发现。
工作流程
-
定义审查目标:
- 明确需要审查的具体文件、差异、产物、命令输出、PR 摘要或设计章节。
- 条件允许时,优先提供原始产物,而不是转述摘要。
- 排除无关对话历史、先验结论、预设修复方案和猜测性诊断。
-
构造干净上下文:
- 只包含执行审查所必需的信息:用户目标、相关路径、精确差异或文件、验收标准和已知约束。
- 如果需要审查代码仓库,提供工作目录,并要求子代理只检查相关路径。
- 如果审查应保持只读,必须明确说明。
- 不要传递密钥、
.env、凭证、令牌或无关私有上下文。
-
启动子代理:
- 只有在子代理工具可用,且用户要求子代理或委托审查时,才使用
multi_agent_v1.spawn_agent。
- 如果当前环境没有子代理工具,说明无法启动干净上下文子代理,并询问用户是继续普通本地审查还是停止。
- 设置
fork_context: false,让子代理从干净上下文开始。
- 只读代码库或产物审查优先使用
agent_type: "explorer"。
- 只有当用户明确希望子代理修改文件时,才使用
agent_type: "worker"。
- 除非用户要求,或存在明确的任务原因,不要覆盖模型设置。
-
要求返回审查型输出:
- 先列问题,按严重程度排序。
- 能定位时,包含文件路径和行号。
- 包含可复现证据或推理依据。
- 没有可执行问题时,明确写出“未发现问题”。
- 摘要要简短,并与问题列表分开。
- 当用户要求严格审查、架构审查或耦合审查时,必须明确要求子代理检查架构是否干净、模块边界是否清晰、是否存在不必要耦合、是否存在过大的文件,以及是否有单个源代码文件超过 500 行。
-
将结果返回给主代理:
- 等待并收集子代理审查结果后,再向用户报告审查结论。
- 子代理运行期间,只继续处理不依赖审查结果且不重叠的工作。
- 忠实总结问题,不夸大置信度。
- 标记主代理尚未独立验证的内容。
- 子代理不再需要时,关闭已完成的子代理。
提示词模板
使用类似下面的提示词,只填写相关字段:
你正在基于干净上下文执行独立审查。
目标:
{用户希望审查什么}
审查对象:
{具体文件、差异、产物路径、命令输出或范围}
上下文:
{判断正确性所需的最小背景}
约束:
- 除非明确说明,否则本次审查保持只读。
- 除非为了验证问题必须查看,否则不要检查无关文件。
- 重点关注缺陷、回归、缺失测试、错误假设和用户可感知风险。
- 不要依赖任何先前对话;只使用本提示词和引用产物。
严格审查补充项,仅在用户要求时加入:
- 检查架构是否干净,职责是否分离。
- 标记无关模块、分层、界面、业务逻辑、基础设施或数据访问之间的不必要耦合。
- 标记任何超过 500 行的单个源代码文件为可执行问题,除非用户明确豁免生成文件或第三方文件。
- 优先指出存在耦合的具体文件或模块,以及能够降低风险的拆分边界。
返回:
- 先列问题,按严重程度排序。
- 每个问题尽量包含路径和行号。
- 如果没有可执行问题,请明确说明。
- 最后简要列出剩余风险或测试缺口。
上下文检查清单
启动子代理前,确认提示词包含:
- 精确的审查问题。
- 所需的最小路径或产物。
- 预期输出格式。
- 只读或可修改权限。
- 相关验收标准。
- 是否要求严格架构、耦合和 500 行文件审查。
启动子代理前,确认提示词不包含:
- 完整父线程历史。
- 主代理预设答案或期望结论。
- 无关实现细节。
- 密钥或环境凭证。