mcpField
C
code-review

code-review

github.com/anthropics测试与质量33.6k9.9k 次安装1 天前更新
代码仓库

关于 code-review

用于从安全、性能、正确性和可维护性角度检查代码变更的技能。可审查粘贴的 diff、PR URL 或文件；连接器可用于获取 PR 信息并结合团队背景审查。

主要功能

  1. 01安全与漏洞检查
  2. 02性能和正确性审查
  3. 03可维护性评估
  4. 04结构化问题报告
  5. 05可选的连接服务上下文

适用场景

  1. 01审查 diff 或拉取请求
  2. 02检查变更中的安全与性能风险
  3. 03评估测试覆盖率和可维护性

安装

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

$npx skills add https://github.com/anthropics/knowledge-work-plugins --skill code-review

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

详细介绍

/code-review

如果你看到不熟悉的占位符，或需要确认已连接哪些工具，请参阅 CONNECTORS.md。

从安全性、性能、正确性和可维护性等结构化角度审查代码变更。

用法

/code-review <PR URL or file path>

审查提供的代码变更：@$1

如果未提供具体的文件或 URL，请询问需要审查的内容。

工作原理

┌─────────────────────────────────────────────────────────────────┐
│                      CODE REVIEW                                   │
├─────────────────────────────────────────────────────────────────┤
│  STANDALONE (always works)                                       │
│  ✓ Paste a diff, PR URL, or point to files                      │
│  ✓ Security audit (OWASP top 10, injection, auth)               │
│  ✓ Performance review (N+1, memory leaks, complexity)           │
│  ✓ Correctness (edge cases, error handling, race conditions)    │
│  ✓ Style (naming, structure, readability)                        │
│  ✓ Actionable suggestions with code examples                    │
├─────────────────────────────────────────────────────────────────┤
│  SUPERCHARGED (when you connect your tools)                      │
│  + Source control: Pull PR diff automatically                    │
│  + Project tracker: Link findings to tickets                     │
│  + Knowledge base: Check against team coding standards           │
└─────────────────────────────────────────────────────────────────┘

审查维度

安全性
  • SQL 注入、XSS、CSRF
  • 身份验证与授权缺陷
  • 代码中的密钥或凭据
  • 不安全的反序列化
  • 路径遍历
  • SSRF
性能
  • N+1 查询
  • 不必要的内存分配
  • 算法复杂度（热点路径中的 O(n²)）
  • 缺失的数据库索引
  • 无界查询或循环
  • 资源泄漏
正确性
  • 边界情况（空输入、null、溢出）
  • 竞态条件与并发问题
  • 错误处理与传播
  • 差一错误（Off-by-one）
  • 类型安全
可维护性
  • 命名清晰度
  • 单一职责
  • 重复代码
  • 测试覆盖率
  • 非显而易见逻辑的文档

输出

## Code Review: [PR title or file]

### Summary
[1-2 sentence overview of the changes and overall quality]

### Critical Issues
| # | File | Line | Issue | Severity |
|---|------|------|-------|----------|
| 1 | [file] | [line] | [description] | 🔴 Critical |

### Suggestions
| # | File | Line | Suggestion | Category |
|---|------|------|------------|----------|
| 1 | [file] | [line] | [description] | Performance |

### What Looks Good
- [Positive observations]

### Verdict
[Approve / Request Changes / Needs Discussion]

如果有可用的连接器

如果已连接 ~~源代码管理：

  • 从 URL 自动拉取 PR diff
  • 检查 CI 状态和测试结果

如果已连接 ~~项目跟踪工具：

  • 将审查发现与相关工单关联
  • 验证 PR 是否满足既定需求

如果已连接 ~~知识库：

  • 对照团队编码规范和风格指南检查变更

小贴士

  1. 提供上下文 — 例如“这是热点路径”或“这里处理 PII”，可以帮助我把重点放在对的地方。
  2. 说明关注点 — 例如“重点关注安全性”，可以缩小审查范围。
  3. 附上测试 — 我也会检查测试覆盖率和测试质量。

评论

支持 Markdown