自主代码审查

类别： 开发
作者： Beta
版本： 1.0.0
运行时： OpenClaw + Claude/GPT

功能说明

自动审查代码中的缺陷、安全问题、性能瓶颈及代码风格违规项，可作为任何代码库不知疲倦的首轮审查者。

适用场景

• 任何重大代码变更后
• 合并 Pull Request 前
• 收到代码审查请求时
• 在人工审查前主动发现问题

审查清单

🔴 致命问题（阻断合并）

• 安全漏洞（SQL 注入、XSS、认证绕过）
• 数据损坏风险（竞态条件、死锁）
• 认证/授权绕过
• 源码中硬编码敏感信息（secrets）

🟡 重要问题（建议修复）

• 性能问题（N+1 查询、低效循环）
• 错误处理缺失或不充分
• 输入验证缺失
• 资源泄漏（连接未关闭、文件未释放）

🟢 建议项（可选修复）

• 代码风格违规
• 缺少文档
• 应通过配置管理但被硬编码的值
• 过于复杂的逻辑

使用方法

# 审查单个文件
openclaw code review --file src/auth.py

# 审查差异（diff）
openclaw code review --diff "main..feature-branch"

# 全仓库审计
openclaw code review --repo ./ --exclude "node_modules,dist"

集成方式

GitHub Actions

- name: Code Review
  uses: openclaw/code-review-action@v1
  with:
    api-key: ${{ secrets.OPENCLAW_API_KEY }}

Pre-commit Hook

openclaw code review --staged --fail-on critical

输出格式

{
  "file": "src/auth.py",
  "issues": [
    {
      "severity": "critical",
      "line": 42,
      "rule": "sql-injection",
      "message": "User input directly interpolated into SQL query",
      "fix": "Use parameterized queries instead"
    }
  ],
  "score": 72,
  "summary": "1 critical, 2 important, 3 suggestions"
}

最佳实践

• 每次提交均执行审查，而不仅限于合并前
• 对关键路径，应结合人工审查共同使用
• 追踪审查历史，识别重复出现的问题
• 按项目类型定制审查规则