requesting-code-review · 代码评审发起
在合并前请"第二双眼睛"把关的 skill：按模板组织评审上下文（变更描述 + 需求/计划 + BASE/HEAD 两个 commit SHA），派代码评审子 agent 做不带会话历史的新鲜视角评审，返回按严重级别分级的问题清单。
评审时机：任务完成、重大功能落地、合并前是强制项；卡住、重构前、修复杂 bug 后推荐使用。Critical 立即修，Important 本轮修，Minor 记下，评审结论有误可据理反驳。
调用示例：在聊天框中说"评审一下我这次的改动再合并"。
产品详解
定位
requesting-code-review 是代码合并前的质量门：派一个不带会话历史的评审子 agent，对两次提交之间的 diff 做新鲜视角评审，在问题扩散前抓住它们。它不写代码，只输出评审发现与处置建议。
核心能力
- 精准评审上下文：按
code-reviewer.md模板组织——变更描述、需求/计划、BASE_SHA、HEAD_SHA；只给评审需要的信息，不带会话历史。 - 评审时机：强制——每任务完成后、重大功能完成、合并到主分支前；推荐——卡住时（新鲜视角）、重构前（基线检查）、修复复杂 bug 后。
- 分级处置：Critical 立即修复；Important 本轮修复；Minor 记下稍后处理。
- 可反驳：评审结论有误时可据理反驳（给出代码/测试证据），不是无条件服从。
- 反借口清单：不许"太简单就不评审"、不许忽略 Critical、不许带着未修复的 Important 继续推进。
工作流程
- 取 git SHA：
BASE_SHA（起点，如HEAD~1或与主分支的 merge-base）、HEAD_SHA（终点） - 按模板派评审子 agent：填充变更描述、需求/计划、两个 SHA
- 按严重级别处理反馈：Critical/Important 修复，Minor 记录
输入与输出
| 输入 | 说明 |
|---|---|
| 变更描述 | 这次改动做了什么 |
| 需求/计划文档 | 改动应该达成什么 |
| BASE_SHA / HEAD_SHA | 评审的起止 commit |
输出：评审发现（优点 + 按严重级别分级的问题）与处置建议（修复/记录/反驳）。
与相邻 skill 的区别
- brainstorming / executing-plans：管设计与执行阶段；本 skill 管执行完成后的评审质量门。
- executing-plans 收尾的 whole-branch review 阶段会调用本 skill 做评审。
适用场景
- 合并到主分支前的质量检查
- 大功能开发完成后的整体评审
- 想用新鲜视角发现自己的盲区
- 修复复杂 bug 后确认没有引入新问题
使用前准备
- 无需 API key；变更必须已提交到 git（需要可计算的 BASE/HEAD SHA）。
- 评审模板见 skill 目录下的
code-reviewer.md。 - 分级执行是硬规则：Critical 不修不放行，Important 不修不继续。
requesting-code-review 隶属于 Aiglade Skill 库。在 Aiglade 聊天框中用自然语言描述需求即可调用。