SuperAPI AI 工具集

piv-review-changes

编程与开发

查看项目

更新于 10-15 15:18

技能介绍

用途

piv-review-changes 用于对最近变更的文件执行技术代码审查,重点发现 bug、安全问题以及规范符合性问题,并输出一份审查报告。它适合在提交代码之前使用,作为提交前的一道质量检查。

主要能力

该技能围绕以下方面进行审查:

  • 逻辑错误:包括差一错误(off-by-one)、条件判断错误、缺失错误处理、竞态条件。
  • 安全问题:包括 SQL 注入漏洞、XSS 漏洞、不安全的数据处理、暴露的密钥或 API key。
  • 性能问题:包括 N+1 查询、低效算法、内存泄漏、不必要的计算。
  • 代码质量:包括违反 DRY 原则、函数过于复杂、命名不佳、缺少类型提示/注解。
  • 规范与模式符合性:包括是否符合 docs 目录中记录的规范、lint/类型/格式标准、日志标准、测试标准,以及是否遵循代码库既有模式。

使用方式

技能要求先收集代码库上下文,以理解代码库的标准与模式。首先检查:

  • CLAUDE.md
  • README.md
  • 核心模块中的关键文件
  • docs 目录中记录的规范(以及任何 .claude/references/ 文档)

在充分理解之后,运行以下命令:

git status git diff HEAD git diff --stat HEAD

然后检查新增文件列表:

git ls-files --others --exclude-standard

对每个新增文件完整阅读,对每个变更文件也完整阅读(而不仅仅是看 diff),以理解完整上下文。

对每个变更或新增文件,按上述主要能力中的类别进行分析。

在确认问题时,技能要求验证问题是否真实存在:为发现的问题运行特定测试、确认类型错误是否成立、结合上下文验证安全顾虑。

输出格式

将新文件保存到 .claude/code-reviews/[appropriate-name].md。

报告包含统计信息:

Stats:

Files Modified: 0

Files Added: 0

Files Deleted: 0

New lines: 0

Deleted lines: 0

对每个发现的问题,按以下格式记录:

severity: critical|high|medium|low file: path/to/file.py line: 42 issue: [one-line description] detail: [explanation of why this is a problem] suggestion: [how to fix it]

如果没有发现问题,则输出:"Code review passed. No technical issues detected."

审查理念

  • 简洁是终极的复杂——每一行代码都应证明其存在的价值。
  • 代码被阅读的次数远多于被编写的次数——应针对可读性进行优化。
  • 最好的代码往往是你没有写出来的代码。
  • 优雅源于意图的清晰与表达的精炼。

重要要求

  • 要具体(给出行号,而不是模糊的抱怨)。
  • 关注真实 bug,而不是风格问题。
  • 提出修复建议,而不只是抱怨。
  • 将安全问题标记为 CRITICAL。

适用场景

适用于在提交代码之前对最近变更进行技术审查,尤其适合需要结合代码库既有规范与模式来检查新增和修改文件的场景。

限制

该技能的具体安装方式、收费情况、支持平台等信息暂未提供。