code-review

代码审查技能

对代码的质量、安全性和可维护性进行全面审查，并提供按严重程度评级的反馈。

何时使用

此技能在以下情况下激活：

• 用户请求“审查这段代码”、“代码审查”
• 合并拉取请求之前
• 实现某项主要功能之后
• 用户想要进行质量评估

功能说明

委托给 code-reviewer Agent（Opus 模型）进行深度分析：

1. 识别变更

   • 运行 git diff 查找已更改的文件
   • 确定审查范围（特定文件或整个 PR）

2. 审查类别

   • 安全性 - 硬编码密钥、注入风险、XSS、CSRF
   • 代码质量 - 函数大小、复杂度、嵌套深度
   • 性能 - 算法效率、N+1 查询、缓存
   • 最佳实践 - 命名、文档、错误处理
   • 可维护性 - 重复、耦合、可测试性

3. 严重程度评级

   • CRITICAL - 安全漏洞（必须在合并前修复）
   • HIGH - Bug 或重大代码坏味道（应在合并前修复）
   • MEDIUM - 次要问题（尽可能修复）
   • LOW - 风格/建议（考虑修复）

4. 具体建议

   • 每个问题的文件:行号位置
   • 具体的修复建议
   • 适用时提供代码示例

Agent 委托

Task(
  subagent_type="oh-my-claudecode:code-reviewer",
  model="opus",
  prompt="CODE REVIEW TASK

Review code changes for quality, security, and maintainability.

Scope: [git diff or specific files]

Review Checklist:
- Security vulnerabilities (OWASP Top 10)
- Code quality (complexity, duplication)
- Performance issues (N+1, inefficient algorithms)
- Best practices (naming, documentation, error handling)
- Maintainability (coupling, testability)

Output: Code review report with:
- Files reviewed count
- Issues by severity (CRITICAL, HIGH, MEDIUM, LOW)
- Specific file:line locations
- Fix recommendations
- Approval recommendation (APPROVE / REQUEST CHANGES / COMMENT)"
)

外部咨询（可选）

code-reviewer Agent 可以咨询 Claude Task Agent 进行交叉验证。

协议

1. 先形成你自己的审查 - 独立完成审查
2. 咨询以进行验证 - 通过 Claude Task Agent 交叉核对发现
3. 批判性评估 - 绝不盲目采纳外部发现
4. 优雅降级 - 如果委托不可用，绝不阻塞

何时咨询

• 安全敏感的代码变更
• 复杂的架构模式
• 不熟悉的代码库或语言
• 高风险的的生产代码

何时跳过

• 简单的重构
• 已充分理解的模式
• 时间紧迫的审查
• 小型、独立的变更

工具使用

使用 Task(subagent_type="oh-my-claudecode:code-reviewer", ...) 进行交叉验证。

输出格式

CODE REVIEW REPORT
==================

Files Reviewed: 8
Total Issues: 15

CRITICAL (0)
-----------
(none)

HIGH (3)
--------
1. src/api/auth.ts:42
   Issue: User input not sanitized before SQL query
   Risk: SQL injection vulnerability
   Fix: Use parameterized queries or ORM

2. src/components/UserProfile.tsx:89
   Issue: Password displayed in plain text in logs
   Risk: Credential exposure
   Fix: Remove password from log statements

3. src/utils/validation.ts:15
   Issue: Email regex allows invalid formats
   Risk: Accepts malformed emails
   Fix: Use proven email validation library

MEDIUM (7)
----------
...

LOW (5)
-------
...

RECOMMENDATION: REQUEST CHANGES

Critical security issues must be addressed before merge.

审查清单

code-reviewer Agent 检查：

安全性

• 无硬编码密钥（API 密钥、密码、令牌）
• 所有用户输入均已净化
• SQL/NoSQL 注入防护
• XSS 防护（转义输出）
• 状态变更操作具备 CSRF 保护
• 正确实施身份认证/授权

代码质量

• 函数 < 50 行（指导原则）
• 圈复杂度 < 10
• 无深度嵌套代码（> 4 层）
• 无重复逻辑（DRY 原则）
• 清晰、描述性的命名

性能

• 无 N+1 查询模式
• 适用时进行适当的缓存
• 高效的算法（O(n) 可行时避免 O(n²)）
• 无不必要的重渲染（React/Vue）

最佳实践

• 具备并适当的错误处理
• 在适当的级别记录日志
• 公共 API 有文档
• 关键路径有测试
• 无被注释掉的代码

批准标准

APPROVE - 无 CRITICAL 或 HIGH 问题，仅有次要改进 REQUEST CHANGES - 存在 CRITICAL 或 HIGH 问题 COMMENT - 仅有 LOW/MEDIUM 问题，无阻塞性顾虑

与其他技能配合使用

配合 Pipeline：

/pipeline review "implement user authentication"

将代码审查作为实现工作流的一部分。

配合 Ralph：

/ralph code-review then fix all issues

审查代码，获取反馈，修复直至批准。

配合 Ultrawork：

/ultrawork review all files in src/

跨多个文件并行进行代码审查。

最佳实践

• 尽早审查 - 在问题累积之前发现它们
• 经常审查 - 小而频繁的审查优于大型审查
• 优先处理 CRITICAL/HIGH - 立即修复安全问题和 Bug
• 考虑上下文 - 某些“问题”可能是有意的权衡
• 从审查中学习 - 利用反馈改进编码实践