mcpField
C
code-review

code-review

github.com/hjewkes测试与质量456 次安装36 周前更新
代码仓库

关于 code-review

code-review 是用于组织代码审查的技能,提供快速、深入、专项和全面审查模式,帮助开发团队分派专门的审查 Agent,并系统处理反馈。

主要功能

  1. 01按场景选择审查模式
  2. 02分派专门的审查 Agent
  3. 03准备 Git 上下文与差异
  4. 04检查代码质量与测试覆盖
  5. 05按结构化步骤处理反馈

适用场景

  1. 01快速检查临时改动
  2. 02在合并前审查新功能
  3. 03调查特定质量问题
  4. 04在合入 main 前进行多轮审查

安装

复制下面的命令,在终端中运行。

$npx skills add https://github.com/hjewkes/agent-skills --skill code-review

安装后在 Claude Code 中输入/code-review确认已加载。

包含的文件

除 SKILL.md 外,安装时一起下载的参考资料和脚本。

references/
comment-analyzer.md
comprehensive-review-procedure.md
deep-reviewer.md
handling-feedback.md
quick-reviewer.md
silent-failure-hunter.md
simplifier.md
test-analyzer.md
type-design-analyzer.md
scripts/
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。

如何发起审查

  1. 使用 scripts/review-prep 收集 git 上下文(自动检测基分支,输出 BASE_SHA、HEAD_SHA、差异统计、变更文件、提交日志)。使用 --json 获取结构化输出。
  2. 阅读所选 Agent 的参考文件
  3. 使用提示词和 review-prep 输出,启动 general-purpose Task Agent
  4. 根据反馈采取行动:立即修复严重问题,继续之前修复重要问题

工作流集成

工作流审查模式
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

AgentReference
deep-reviewerreferences/deep-reviewer.md
quick-reviewerreferences/quick-reviewer.md
comment-analyzerreferences/comment-analyzer.md
test-analyzerreferences/test-analyzer.md
silent-failure-hunterreferences/silent-failure-hunter.md
type-design-analyzerreferences/type-design-analyzer.md
simplifierreferences/simplifier.md
handling-feedbackreferences/handling-feedback.md

评论

支持 Markdown