requesting-code-review

# 请求代码审查 调度 superpowers:code-reviewer 智能体以在问题蔓延前捕获它们。 **核心原则：** 尽早审查，频繁审查。 ## 何时请求审查 **强制性：** - 在智能体驱动开发的每个任务之后 - 在完成主要功能之后 - 在合并到主分支之前 **可选但有价值：** - 当遇到困难时（获取新视角） - 在重构之前（基准检查） - 在修复复杂 Bug 之后 ## 如何请求 **1. 获取 git SHA：** bash BASE_SHA=$(git rev-parse HEAD~1) # 或 origin/main HEAD_SHA=$(git rev-parse HEAD) **2. 调度 code-reviewer 智能体：** 使用 Task 工具，类型为 superpowers:code-reviewer，填写 `code-reviewer.md` 中的模板 **占位符：** - `{WHAT_SWAS_IMPLEMENTED}` - 您刚刚构建的内容 - `{PLAN_OR_REQUIREMENTS}` - 它应该做什么 - `{BASE_SHA}` - 起始提交 - `{HEAD_SHA}` - 结束提交 - `{DESCRIPTION}` - 简要总结 **3. 根据反馈采取行动：** - 立即修复严重问题 - 在继续之前修复重要问题 - 记录次要问题以供后续处理 - 如果审查者有误，请（带理由）进行反驳 ## 示例 [刚刚完成任务 2：添加验证函数] 您：在继续之前让我请求代码审查。 BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}') HEAD_SHA=$(git rev-parse HEAD) [调度 superpowers:code-reviewer 智能体] WHAT_WAS_IMPLEMENTED: 会话索引的验证和修复函数 PLAN_OR_REQUIREMENTS: 来自 docs/plans/deployment-plan.md 的任务 2 BASE_SHA: a7981ec HEAD_SHA: 3df7661 DESCRIPTION: 添加了带有 4 种问题类型的 verifyIndex() 和 repairIndex() [智能体返回]： 优势：架构清晰，真实测试 问题： 重要：缺少进度指示器 次要：报告间隔的魔术数字 (100) 评估：准备好继续 您：[修复进度指示器] [继续执行任务 3] ## 与工作流集成 **智能体驱动开发：** - 每个任务后审查 - 在问题复合前捕获 - 在进入下一个任务前修复 **执行计划：** - 每批（3 个任务）后审查 - 获取反馈，应用，继续 **临时开发：** - 合并前审查 - 受阻时审查 ## 危险信号 **绝对不要：** - 因为“很简单”而跳过审查 - 忽略严重问题 - 在未修复重要问题的情况下继续 - 与有效的技术反馈争论 **如果审查者有误：** - 用技术理由反驳 - 展示证明其有效的代码/测试 - 请求澄清 查看模板：