mcpField
P
piv-review-changes

piv-review-changes

github.com/coleam00测试与质量722236 次安装3 周前更新
代码仓库

关于 piv-review-changes

piv-review-changes 是面向开发者的代码审查技能，会结合仓库规范检查新增和变更文件、验证疑似问题，并将结果保存为 Markdown 报告。

主要功能

  1. 01检查仓库背景与差异
  2. 02审查新增和变更文件
  3. 03检查错误、安全与性能问题
  4. 04通过专项测试验证问题
  5. 05生成含严重性和行号的报告

适用场景

  1. 01审查未提交或近期的代码变更
  2. 02合并前检查安全与逻辑问题
  3. 03生成结构化代码审查报告

安装

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

$npx skills add https://github.com/coleam00/skills --skill piv-review-changes

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

详细介绍

代码审查

对最近修改的文件进行技术代码审查。

核心原则

审查理念：

  • 简洁是终极的精致——每一行代码都应证明其存在的必要性
  • 代码被阅读的次数远多于编写的次数——应优先考虑可读性
  • 最好的代码往往是你没有写下的代码
  • 清晰的意图与精炼的表达方能成就优雅

审查内容

首先收集代码库背景信息，了解代码库的标准和惯例。

先检查：

  • CLAUDE.md
  • README.md
  • 核心模块中的关键文件
  • docs 目录中的文档标准（以及任何 .claude/references/ 文档）

充分了解后，运行以下命令：

git status
git diff HEAD
git diff --stat HEAD

然后检查新增文件列表：

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

完整阅读每个新增文件。完整阅读每个修改过的文件（而不仅是差异），以了解完整上下文。

针对每个修改过的文件或新增文件，分析以下方面：

  1. 逻辑错误

    • 差一错误
    • 条件判断错误
    • 缺少错误处理
    • 竞态条件

  2. 安全问题

    • SQL 注入漏洞
    • XSS 漏洞
    • 不安全的数据处理
    • 暴露的机密信息或 API 密钥

  3. 性能问题

    • N+1 查询
    • 低效算法
    • 内存泄漏
    • 不必要的计算

  4. 代码质量

    • 违反 DRY 原则
    • 函数过于复杂
    • 命名不佳
    • 缺少类型提示/注解

  5. 是否遵循代码库标准与现有惯例

    • 是否遵循 docs 目录中记载的标准
    • 代码检查、类型和格式标准
    • 日志记录标准
    • 测试标准

验证问题是否属实

  • 针对发现的问题运行特定测试
  • 确认类型错误确实存在
  • 结合上下文验证安全问题

输出格式

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

统计信息：

  • 修改的文件数：0
  • 新增的文件数：0
  • 删除的文件数：0
  • 新增行数：0
  • 删除行数：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]

如果未发现问题：“代码审查通过。未检测到技术问题。”

重要说明

  • 描述具体问题（注明行号，避免含糊其辞）
  • 关注真实缺陷，而非风格问题
  • 提出修复建议，而不只是指出问题
  • 将安全问题标记为严重

评论

支持 Markdown