code-review

代码审查

Agent Skills™ 的一部分，由 googleadsagent.ai™ 提供

描述

代码审查通过清单驱动的评估、按严重程度分类的问题发现以及强制的问题解决跟踪，实施结构化的合并前质量门禁。Agent 会在批准变更前，根据一组可配置的质量维度审查每项差异，确保无论审查者是否疲劳或时间是否紧迫，都能维持一致的标准。

与临时的审查意见不同，此技能会生成标准化的审查文档，并按严重程度对问题发现进行分类：严重（合并前必须修复）、高（合并前应修复）、中（后续修复）和低（可选改进）。每项问题发现均包含文件、行范围、类别、描述和具体的建议修复方案。审查文档会成为永久记录的一部分。

预审查清单会在深入分析开始前，捕捉常见疏漏：缺少测试、未提交的文件、linter 错误、类型错误和文档缺失。只有清单检查通过后，Agent 才会继续对逻辑、架构、安全性和性能进行语义审查。

适用场景

• 拉取请求或差异已准备好进行审查
• 用户要求对代码变更提供反馈
• 将任何分支合并到 main 之前
• 子 Agent 完成任务后（第 2 阶段审查）
• 代码经过重构，需要验证
• 新贡献者的代码需要入门级审查

工作原理

graph TD
    A[Receive Diff] --> B[Pre-Review Checklist]
    B --> C{Checklist Passes?}
    C -->|No| D[Return with Blockers]
    C -->|Yes| E[Semantic Analysis]
    E --> F[Classify Findings by Severity]
    F --> G[Generate Review Document]
    G --> H{Critical Findings?}
    H -->|Yes| I[Request Changes]
    H -->|No| J{High Findings?}
    J -->|Yes| K[Approve with Reservations]
    J -->|No| L[Approve]

此工作流会对后续步骤进行门禁控制：清单会立即捕捉机械性问题，而语义分析则评估设计、正确性和可维护性。严重程度分类可确保严重问题阻止合并，而次要改进不会阻碍合并。

实现

pre_review_checklist:
  - name: "Tests exist for changed code"
    command: "check_test_coverage_delta"
    severity: "critical"
  - name: "No linter errors introduced"
    command: "run_linter --diff-only"
    severity: "critical"
  - name: "Type checking passes"
    command: "run_typecheck"
    severity: "critical"
  - name: "No secrets in diff"
    command: "scan_secrets --diff"
    severity: "critical"
  - name: "Documentation updated"
    command: "check_doc_staleness"
    severity: "medium"

severity_levels:
  critical:
    label: "🔴 Critical"
    action: "Must fix before merge"
    examples: ["Security vulnerability", "Data loss risk", "Broken tests"]
  high:
    label: "🟠 High"
    action: "Should fix before merge"
    examples: ["Missing error handling", "Performance regression", "API contract violation"]
  medium:
    label: "🟡 Medium"
    action: "Fix in follow-up PR"
    examples: ["Code duplication", "Unclear naming", "Missing edge case test"]
  low:
    label: "🔵 Low"
    action: "Optional improvement"
    examples: ["Style preference", "Minor refactor opportunity", "Comment improvement"]

review_dimensions:
  - correctness: "Does the code do what it claims?"
  - security: "Are inputs validated? Are secrets protected?"
  - performance: "Are there N+1 queries, unnecessary re-renders, or blocking calls?"
  - maintainability: "Can another developer understand this in 6 months?"
  - testing: "Are edge cases covered? Are tests deterministic?"
  - architecture: "Does this follow established patterns? Is coupling appropriate?"

最佳实践

• 在花时间进行语义审查前，先运行预审查清单
• 仅提出可执行的问题发现，除非风格问题十分明显，否则避免吹毛求疵
• 始终为每项问题发现提供建议修复方案
• 将严重阻碍项与锦上添花的改进分开
• 审查测试变更时，应与审查生产代码一样仔细
• 不要只指出缺陷，也要肯定良好的模式和设计决策

平台兼容性

平台	支持情况	备注
Cursor	完整	通过 Shell 工具读取差异
VS Code	完整	Git diff 集成
Windsurf	完整	PR 审查工作流
Claude Code	完整	可访问 git diff 和 gh pr
Cline	完整	基于差异的审查
aider	部分	仅限文件级审查

相关技能

• 测试驱动开发 - RED-GREEN-REFACTOR 规范，确保审查的代码有相应的测试覆盖
• 系统化调试 - 代码审查发现缺陷时采用的根因分析方法
• Agent 安全扫描 - 自动检测漏洞，补充代码审查中的安全性维度

关键词

code-review quality-gate pre-merge-checklist severity-classification pull-request-review security-review performance-review review-document

---

© 2026 googleadsagent.ai™ | Agent Skills™ | MIT License