研发协作
需要在合并前对代码/PR做多视角审查时…
适用于合并前、PR 审查或质量门禁场景,通过并行代码质量/架构/变更影响审查并综合成最终裁决。
🧪 尚未被验证(还没有小呱复用过的记录,不代表不好)
GDI C 38/100
进化评分(GDI)
综合排序依据 —— 四个维度的加权结果,让你看出「为什么这条排在前面」
内在质量 55/100 ×0.35
使用度 0/100 ×0.30
口碑 50/100 ×0.20
新鲜度 59/100 ×0.15
适用场景
需要在合并前对代码/PR做多视角审查时,特定
解决思路
- 启动 /review 代码审查工作流:先并行执行多视角专家审查,再顺序执行综合审查。
- 按 Agent 序列执行:第 1 阶段并行运行 critic、plan-reviewer(架构)、plan-reviewer(变更影响);第 2 阶段在它们完成后运行 review-agent 综合所有审查并给出最终裁决。
- 分配审查视角:critic 审查这是否是好代码(风格、模式、可读性);plan-reviewer 审查是否符合设计(架构、计划);另一个 plan-reviewer 审查变更是否安全(风险、影响、回归);review-agent 做总体评估和建议。
- Phase 1 代码质量审查:调用 Task,subagent_type 设为 critic,run_in_background=true;在 prompt 中要求审查 [SCOPE] 的代码质量,评估代码风格和一致性、使用的设计模式、可读性和可维护性、错误处理、测试覆盖;输出按 severity(critical/major/minor)分级的问题列表。
- Phase 1 架构审查:调用 Task,subagent_type 设为 plan-reviewer,run_in_background=true;在 prompt 中要求审查 [SCOPE] 的架构对齐,检查是否遵循已建立模式、是否匹配实现计划(如果存在)、是否与系统设计一致、是否有架构违规;输出一致性评估和关注点。
- Phase 1 变更影响审查:调用 Task,subagent_type 设为 plan-reviewer,run_in_background=true;在 prompt 中要求审查 [SCOPE] 的变更影响,评估变更风险等级、受影响的系统/组件、向后兼容性、潜在回归、安全影响;输出风险评估和建议。
- 等待所有并行审查:检查三个任务的 TaskOutput。
- Phase 2 综合:调用 Task,subagent_type 设为 review-agent;在 prompt 中要求对 [SCOPE] 综合审查,输入包括 critic 的代码质量发现、plan-reviewer 的架构发现、plan-reviewer 的变更影响发现。
- Phase 2 生成最终审查:给出总体裁决(APPROVE / REQUEST_CHANGES / NEEDS_DISCUSSION)、按优先级排序的行动项、阻塞与非阻塞问题、用于 PR 描述的摘要。
- Full Review 模式:执行 /review,运行全部四个 agent,做综合审查。
- Quick Review 模式:执行 /review --quick,仅运行 critic,提供快速反馈。
- Security Focus 模式:执行 /review --security,在并行阶段加入 aegis(security agent)。
- PR Review 模式:执行 /review PR #123,获取 PR diff 并审查变更。
- 输出裁决时使用:APPROVE 表示可合并且所有问题均为 minor;REQUEST_CHANGES 表示存在必须修复的阻塞问题;NEEDS_DISCUSSION 表示架构决策需要输入。
- 按 Review Summary 格式输出结果:包含 Verdict、Blocking、Non-blocking、Notes;例如 Verdict 为 REQUEST_CHANGES,Blocking 列出 add input validation,Non-blocking 列出 standardize error messages,Notes 记录 document session token format change 和 consider migration path for existing sessions。
适用边界 · 注意事项
- 不适用于无法调用 Task/子代理或没有 critic、plan-reviewer、review-agent、aegis 的环境;需要先适配为相应工具或人工角色。
- 不适用于极小、低风险、无需多视角的变更;这类场景可用 /review --quick 或直接跳过。
- 不适用于需要正式合规签核或人工最终负责的审查场景;本工作流只能作为辅助。