代码审查 差异分析 质量报告:给 git diff 加一道规则检查

前言

Code review 里最累的部分往往不是看设计,而是逐行找那些反复出现的问题:密码写在代码里、日志里带了敏感字段、循环里查库、把平台专有 API 直接焊死在业务逻辑里。这些问题单看每一处都像”以后再说”,合在一起就是线上事故。

人工 review 能发现它们,但覆盖率不稳定:有时只扫了思路没逐行看,有时看完了也没留下可追踪的结论。

下面介绍的这个 Skill 走的是另一条路:它不去理解需求,而是对 git diff 做一次规则化扫描,把发现的问题按严重程度分级,输出成报告。适合用在提交前自检,也适合把审查结论落成可查询的记录。

这是什么

一个社区发布的通用 Skill,坐标 @user_2fd890c9/code-review-report,版本 2.6.1,分类 dev-programming,来源标记为 community。它的作用是读取一次 diff,用内置规则扫描常见问题,输出分级审查报告。

页面在本次抓取时显示评分 100000、下载 1889。

它解决的问题是:把”review 时记得看看有没有硬编码密码”这类口头约定,变成每次 diff 都会执行的固定动作。

核心功能

下面只列官方简介中明确写到的能力,不加推测。

1、解析 git diff。输入是一次代码变更,不是整个仓库,只关心本次改动涉及的文件和行。

2、规则扫描四个方向:

  • 硬编码密码:源码中直接出现的口令、token、密钥。
  • 不安全日志:把敏感信息写进日志的行为。
  • 性能反模式:典型写法问题,例如在循环中执行本可批量完成的操作。
  • 平台依赖:业务逻辑直接绑定某个平台专有 API,换平台就要重写。

3、输出分级审查报告,支持 markdown 和 json 两种格式。分级意味着可以直接按严重级别过滤。

4、严重级过滤。按需只看高危,或看全量。

5、密码脱敏。报告里出现密钥类内容时做遮蔽,避免审查报告本身成为泄露源。

6、默认预览不写盘。默认只展示,不改动本地文件。需要落地时再显式指定输出。

7、内置 12 条自测。用于验证规则本身是否按预期工作。

安装与启用

这一节需要说明清楚:已抓取资料中没有给出安装命令、目录约定或配置字段,官方页面在本次抓取时也未能返回正文内容。因此下面不写安装命令,避免给出一个无法核对的路径。

可以确认的信息是:

  • 该 Skill 基于通用 SKILL.md 格式,由一个 SKILL.md 描述文件承载,宿主 Agent 读取该文件后按其中说明执行。
  • 标签 requires_api_key=false,说明使用它不需要额外配置 API Key。

官方页面地址:

https://www.skillhub.cn/skills/user_2fd890c9/code-review-report

拿到 SKILL.md 之后,把它放到你的 Agent 能读取到的位置,再在对话中触发即可。至于不同宿主(Cursor、Codex CLI、Claude Code)在目录位置上的差异,资料中没有说明,这里不写。

典型用法

官方资料只给了功能描述,没有给可复现的调用示例。下面的 git 命令是通用的取 diff 方式,用于说明这个 Skill 的输入形态,不是 Skill 自己的命令。

# 取最近一次提交的完整变更,作为 Skill 的输入
git diff HEAD~1 HEAD

# 或只看暂存区内容
git diff --staged

把变更交给 Skill 之后,按用途分两种用法:

1、提交前自检:只看严重级,把必须改的问题先修掉。
2、留档:输出 markdown 报告,附在 PR 或变更说明里,让审查结论有记录可查。

json 格式适合接自动化流程,例如把扫描结果传给后续的门禁检查。

适用场景与注意

适合的场景:

  • 提交前的规则自检,尤其是改动涉及认证、日志、外部平台调用的时候。
  • 团队需要统一执行同一套检查规则,而不依赖每个人记得。
  • 想把审查结论变成可追踪的 artifact,而不是一次性对话。

需要注意的地方:

  • 它检查的是写法,不是正确性。逻辑错误、并发问题、需求偏差不在扫描范围内。规则针对四类固定模式,不做数据流分析。
  • 性能反模式和平台依赖这两类,命中标准取决于规则定义。规则偏严会带来噪音,需要配合严重级过滤使用。
  • 密码脱敏只作用于报告输出。仓库里已经提交的密钥仍然是泄露,需要单独处理。
  • 默认不写盘是好事,但接自动化时要确认输出路径,避免在循环里重复生成报告。
  • 版本 2.6.1,来源为 community,规则集合会随版本变化,长期使用建议固定版本。

结尾

它的价值不在于替代 review,而在于把四类反复出现的问题变成每次 diff 都会执行的固定动作,并把结论落成可查询的报告。如果你的代码库出现过密钥泄露或换平台返工的情况,值得按官方页面说明启用一次。

官方地址:https://www.skillhub.cn/skills/user_2fd890c9/code-review-report

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

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

小夜