Code Review:把 PR 审查做成系统化流程的 Skill

前言

代码审查的质量很不稳定:同一份 diff,有人只看命名和格式,有人能指出 SQL 注入和竞态条件。差距通常不在经验,而在有没有一套覆盖全面的检查清单和固定的审查流程。这个 Skill 把安全、性能、正确性、可维护性、测试等维度的审查要点整理成可直接使用的 prompt 模式,让审查者按维度系统过一遍,而不是随手扫。

这是什么

Code Review 是一个基于通用 SKILL.md 格式的代码审查 Skill,由 wpank 发布在 ClawHub,坐标 @wpank/code-review,版本 1.0.0,来源 clawhub,分类 dev-programming。SkillHub 页面显示评分 100000、下载 49793、安装 1972,标签 requires_api_key=false,即不需要 API key 即可使用。

它解决的问题是:审查 PR 时缺乏统一标准、遗漏安全项、反馈零散。Skill 内置严重等级划分、结构化反馈指南、三遍审查流程和需避免的反模式,适用于审查 PR、建立审查标准或提升审查质量。

核心功能

审查维度与优先级

Skill 把审查拆成七个维度,每个维度有明确优先级:

  1. Security:漏洞、认证、数据暴露,Critical
  2. Performance:速度、内存、扩展瓶颈,High
  3. Correctness:逻辑错误、边界情况、数据完整性,High
  4. Maintainability:可读性、结构、可维护性,Medium
  5. Testing:覆盖率、测试质量、可靠性,Medium
  6. Accessibility:WCAG 合规、键盘导航、屏幕阅读器,Medium
  7. Documentation:注释、API 文档、changelog,Low

分维度检查清单

每个维度下是一组可直接逐条核对的具体检查项,例如 Security 维度包含 SQL 注入、XSS、CSRF、认证、授权、输入校验、密钥管理、依赖安全、敏感数据、限流、文件上传安全、HTTP 安全头共 12 项。Performance 维度覆盖 N+1 查询、不必要的重渲染、内存泄漏、bundle 体积、懒加载、缓存策略、数据库索引、分页、异步任务、图片优化。Correctness、Maintainability、Testing 维度同样各有 10 条左右的清单。

严重等级与结构化反馈

Skill 要求按严重等级标注问题,并提供结构化的反馈格式,避免“这个不太行”式的模糊意见。具体等级分级和反馈模板来自 Skill 正文,使用前建议通读一遍。

三遍审查流程

Skill 明确反对一遍读完试图抓全所有问题,改为三遍:

  1. 第一遍(2-5 分钟):读 PR 描述和关联 issue,扫文件列表确认改动范围合理,检查整体方案是否对症,确认没有架构漂移。
  2. 第二遍(主要时间):逐文件逐行读 diff,对照上述清单检查每个函数改动,核对每个 I/O 边界的错误处理。
  3. 第三遍(约 5 分钟):专注边界值、失败模式、并发和缺失测试。

反模式清单

Skill 内置一份审查中要避免的反模式,比如把风格问题和阻塞问题混在一起提、对每个细节都长篇大论导致重点被淹没。这部分用于校准反馈的表达方式。

安装与启用

官方页面给出两种安装方式。OpenClaw / Moltbot / Clawbot 环境:

npx clawhub@latest install code-review

或使用 skills CLI:

$ openclaw skills install @wpank/code-review

安装后 Skill 通过 SKILL.md 描述自身用途,由 Agent 在审查 PR、建立审查标准等场景下自动加载。不需要配置 API key。

典型用法

安装后直接以审查任务的形式使用,例如:

  1. 让 Agent 审查当前分支相对主干的 diff:说明“审查这次改动”,Agent 会加载该 Skill 的维度清单和流程。
  2. 建立审查标准:把 Skill 作为团队的审查基线,要求所有 PR 按七个维度逐项过。
  3. 专项复查:针对安全敏感的改动,让 Agent 只跑 Security 清单并按严重等级输出结构化反馈。

Skill 正文是纯 prompt 文本,也可以直接复制到对话里手动使用,不依赖安装环境。

适用场景与注意

适合需要系统性审查 PR、想统一团队审查口径、或希望补齐安全和性能检查项的开发者和团队。它的清单偏通用,条目是原则性的,落地到具体语言和框架时需要结合项目实际裁剪,比如 .NET 的依赖注入问题不在内置清单里,需要自己补充。

注意两点:一是 Skill 是审查指南,不能替代自动化工具,静态扫描、测试和 CI 仍要保留;二是清单覆盖面广,逐条全跑会拖慢审查节奏,建议按改动类型选维度,而不是每次都七维度全开。

结尾

这个 Skill 的价值在于把“系统化审查”从个人习惯变成可复制的清单和流程:维度、严重等级、三遍流程、反模式都写好了,装完即可用。官方地址:https://www.skillhub.cn/skills/clawhub_wpank/code-review ,上游仓库:https://clawhub.ai/wpank/code-review 。

羽毛球分组比赛记分
小程序二维码

欢迎使用《羽毛球分组比赛记分》微信小程序

小夜