SKILL.md
请求代码审查
分派代码审查者子代理，在问题扩大前捕捉它们。审查者会获得为评估精确构造的上下文——绝不是你的会话历史。这会让审查者专注于工作产物，而不是你的思考过程，同时保留你自己的上下文以便继续工作。
核心原则： 尽早审查，经常审查。
何时请求审查
强制：
- 在 subagent-driven development 中每个任务之后
- 完成主要功能后
- 合并到 main 之前
可选但有价值：
- 卡住时（获得新视角）
- 重构前（基线检查）
- 修复复杂 bug 后
如何请求
1. 获取 git SHAs：
BASE_SHA=$(git rev-parse HEAD~1) # or origin/main
HEAD_SHA=$(git rev-parse HEAD)2. 分派代码审查者子代理：
使用 general-purpose 类型的 Task 工具，填写 code-reviewer.md 中的模板
占位符：
{DESCRIPTION}- 你构建内容的简短总结{PLANORREQUIREMENTS}- 它应该做什么{BASE_SHA}- 起始 commit{HEAD_SHA}- 结束 commit
3. 处理反馈：
- 立即修复 Critical 问题
- 继续前修复 Important 问题
- 记录 Minor 问题以便之后处理
- 如果审查者错了，进行反驳（带理由）
示例
[刚完成 Task 2：添加验证函数]
你：继续之前，我先请求代码审查。
BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
HEAD_SHA=$(git rev-parse HEAD)
[分派代码审查者子代理]
DESCRIPTION: 添加了 verifyIndex() 和 repairIndex()，包含 4 种问题类型
PLAN_OR_REQUIREMENTS: docs/superpowers/plans/deployment-plan.md 中的 Task 2
BASE_SHA: a7981ec
HEAD_SHA: 3df7661
[子代理返回]:
Strengths: 架构干净，测试真实
Issues:
Important: 缺少进度指示器
Minor: 报告间隔使用魔法数字 (100)
Assessment: 可以继续
你：[修复进度指示器]
[继续 Task 3]与工作流集成
Subagent-Driven Development：
- 每个任务后都审查
- 在问题累积前捕捉它们
- 移动到下一个任务前先修复
Executing Plans：
- 每个任务后或自然检查点审查
- 获取反馈、应用、继续
Ad-Hoc Development：
- 合并前审查
- 卡住时审查
危险信号
绝不要：
- 因为 "it's simple" 就跳过审查
- 忽略 Critical 问题
- 带着未修复的 Important 问题继续
- 与有效的技术反馈争辩
如果审查者错了：
- 用技术推理反驳
- 展示能证明它可用的代码/测试
- 请求澄清
模板见：requesting-code-review/code-reviewer.md