关于 code-review
code-review 是用于组织代码审查的技能,提供快速、深入、专项和全面审查模式,帮助开发团队分派专门的审查 Agent,并系统处理反馈。
主要功能
- 01按场景选择审查模式
- 02分派专门的审查 Agent
- 03准备 Git 上下文与差异
- 04检查代码质量与测试覆盖
- 05按结构化步骤处理反馈
适用场景
- 01快速检查临时改动
- 02在合并前审查新功能
- 03调查特定质量问题
- 04在合入 main 前进行多轮审查
安装
复制下面的命令,在终端中运行。
npx skills add https:/ / github.com/ hjewkes/ agent-skills --skill code-review安装后在 Claude Code 中输入/code-review确认已加载。
包含的文件
除 SKILL.md 外,安装时一起下载的参考资料和脚本。
references/comment-analyzer.mdcomprehensive-review-procedure.mddeep-reviewer.mdhandling-feedback.mdquick-reviewer.mdsilent-failure-hunter.mdsimplifier.mdtest-analyzer.mdtype-design-analyzer.mdscripts/review-prep详细介绍
代码审查
根据具体情况,分派合适的专门审查 Agent。选择一种模式,或根据上下文决定。
审查模式
快速审查
适用场景: 临时检查、快速健全性检查、低噪声反馈 分派: quick-reviewer Agent(置信度评分,仅报告评分不低于 80 的问题) 输出: 按严重程度分组的高置信度问题(严重:90–100,重要:80–89)
深度审查
适用场景: 完成功能后、合并前、计划执行审查循环中 分派: deep-reviewer Agent(使用模板占位符的完整检查清单) 占位符: {WHAT_WAS_IMPLEMENTED}, {PLAN_OR_REQUIREMENTS}, {BASE_SHA}, {HEAD_SHA}, {DESCRIPTION} 输出: 优势、问题(严重/重要/次要)、建议、合并结论
专项审查
适用场景: 针对特定质量维度进行分析 分派: 一个或多个专项 Agent:
| Agent | 关注领域 |
|---|---|
| comment-analyzer | 注释准确性、文档质量、注释过时问题 |
| test-analyzer | 测试覆盖质量、行为覆盖缺口、测试韧性 |
| silent-failure-hunter | 错误处理、静默失败、catch 块的具体性 |
| type-design-analyzer | 类型不变量、封装性、类型实用性 |
全面审查
适用场景: 合并到 main 前、开发分支完成时 分派: 4 轮结构化审查(TDD 合规性、仅检查差异的质量、上下文质量、浏览器审计)。各轮详情和顺序请参阅 references/comprehensive-review-procedure.md。
如何发起审查
- 使用
scripts/review-prep收集 git 上下文(自动检测基分支,输出 BASE_SHA、HEAD_SHA、差异统计、变更文件、提交日志)。使用--json获取结构化输出。 - 阅读所选 Agent 的参考文件
- 使用提示词和 review-prep 输出,启动
general-purposeTask Agent - 根据反馈采取行动:立即修复严重问题,继续之前修复重要问题
工作流集成
| 工作流 | 审查模式 |
|---|---|
| plan-execution(逐项任务) | 深度审查 |
| plan-execution(批量) | 深度审查 |
| 合并到 main 前 | 全面审查 |
| 临时审查 / 遇到阻碍时 | 快速审查 |
| 特定问题 | 专项审查 |
| GitHub PR 审查与发布 | github-pr 技能(自动化 PR 审查编排器) |
简化模式
审查后,可选择分派一轮简化处理,在不改变功能的前提下提升清晰度、一致性和可维护性。阅读 references/simplifier.md 获取完整的 Agent 提示词,然后使用该提示词启动 general-purpose Task Agent,针对已审查的文件进行处理。
处理反馈
收到代码审查反馈(来自人工或其他 Agent)时,请遵循结构化的 6 步响应模式:阅读、理解、验证、评估、回应、实施。不要敷衍附和——实施前先验证,必要时基于技术理由提出异议。完整模式、提出异议的标准、YAGNI 检查和按来源处理的方式,请参阅 references/handling-feedback.md。
警示信号
- 因为“很简单”而跳过审查
- 忽略严重问题
- 尚有未修复的重要问题就继续推进
- 按顺序逐一运行全面审查的所有轮次(第 1 轮和第 2 轮相互独立,应并行执行)
- 全面审查模式下,未对第 2 轮和第 3 轮的发现去重
Reference Files
| Agent | Reference |
|---|---|
| deep-reviewer | references/deep-reviewer.md |
| quick-reviewer | references/quick-reviewer.md |
| comment-analyzer | references/comment-analyzer.md |
| test-analyzer | references/test-analyzer.md |
| silent-failure-hunter | references/silent-failure-hunter.md |
| type-design-analyzer | references/type-design-analyzer.md |
| simplifier | references/simplifier.md |
| handling-feedback | references/handling-feedback.md |