关于 review-code
review-code 是供其他工作流内联执行的代码审查 Skill，只报告发现，不修改或重构代码。它按固定清单检查变更范围，并以证据和严重级别呈现问题。
主要功能
- 01按指定范围审查代码变更
- 02逐项检查逻辑与错误处理
- 03检查重复代码和死代码
- 04核对命名、依赖与引用
- 05按严重级别报告问题
适用场景
- 01审查当前变更与默认分支的差异
- 02检查指定路径或 glob 的变更
- 03在代码审查流程中生成问题清单
MCP 相关
输入未说明该 Skill 与 MCP 的关系。
安装
复制下面的命令，在终端中运行。
npx skills add https:/安装后在 Claude Code 中输入/review-code确认已加载。
详细介绍
代码审查（内部）
由 review-change / product-audit 在各自的对话中编排；对于任何
Agent，都应将此文件作为路由步骤内联遵循。仅报告发现；绝不编辑，
绝不重构。
范围
调用方传入的差异或路径 / glob；默认审查当前更改与默认分支的差异。 在返回表格顶部注明范围。
检查清单（评估每一项——无一例外；不适用时必须说明）
✓ 更改路径上没有逻辑错误（追踪每个已修改函数的输入 → 输出，包括边界值） ✓ 每条错误路径都得到处理——没有被吞掉的异常、空 catch 或被静默忽略的返回码 ✓ 没有重复逻辑（更改的代码块没有重新实现现有辅助函数——如有，请指出现有函数） ✓ 没有引入无用代码（未使用的函数、参数、导入项、不可达分支） ✓ 差异中没有遗留的 TODO/FIXME/HACK 标记 ✓ 命名和文件约定符合项目文档（先阅读文档；指出违反的约定） ✓ 没有超出 SPEC 要求的新抽象（只有一种实现的接口 / 基类属于发现） ✓ 没有 SPEC 未证明合理的新依赖 ✓ 简化：是否有任何更改的代码块可以减少行数而不影响行为（引用更改前 / 后内容） ✓ 引用追踪：每个更改的符号 / API 都已进行引用搜索——未更新的调用方属于发现 ✓ SPEC 开发场景中列出的边界情况确实在代码中得到处理，而不只是出现在测试中
重要性门槛
只有当有能力的用户的结果会因此改变，或违反项目明确声明的规则时，才报告一行——须在证据旁引用被违反的规则。不属于发现的情况：注释 / 标点错误、仅格式偏差、没有引用规则的风格偏好、超出 SPEC 所列场景的假设性稳健性问题。对于格式规范的更改，预期结果是空表并附上 Decision: PASS——切勿为填充表格而添加内容。
严格按以下格式返回
REVIEW CODE — scope: <scope>
| # | Finding | Sev | Evidence | Suggested fix |
|---|---------|-----|----------|---------------|
| 1 | <what> | critical|major|minor | <file:line> | <smallest action> |
Checklist: <n> evaluated, <n> pass, <n> findings, <n> n/a (<which + why>)
Summary: <1-2 sentences>
Decision: PASS | FAIL
若存在任何未解决的严重或主要发现，则判定为 FAIL；否则判定为 PASS。次要发现不会阻止通过——它们会转交给调用方的分诊步骤。
完成条件
- 每项检查清单都通过证据（文件:行号或命令输出）进行评估，或明确标记为不适用并说明原因。
- 返回上面的固定格式代码块——除此之外不多不少，且未更改任何代码。