AI Code Reviewer

AI 驱动的代码审查工具。自动分析代码质量、发现潜在 bug、生成 PR 描述、建议测试用例。支持多种编程语言。

触发词

• 代码审查
• code review
• 代码分析
• PR 描述
• 代码质量检查
• 自动 review

核心功能

1. 代码质量分析

• 代码风格检查: 命名规范、缩进、注释完整性
• 潜在 bug 检测: 空指针、资源泄漏、边界条件
• 性能优化建议: 时间复杂度、内存使用、数据库查询
• 安全漏洞扫描: SQL 注入、XSS、敏感信息泄漏

2. 智能 PR 描述生成

输入：Git diff 或代码变更
↓
AI 分析:
- 变更摘要
- 影响范围
- 测试建议
- 注意事项
↓
输出：完整的 PR 描述 (Markdown)

3. 测试用例生成

• 根据代码逻辑生成单元测试
• 覆盖边界条件和异常场景
• 支持主流测试框架 (Jest, Pytest, JUnit 等)

4. 多语言支持

配置说明

环境变量

# OpenRouter API (已有)
OPENROUTER_API_KEY=sk-or-v1-xxxxx

# GitHub Token (可选，用于自动 PR)
GITHUB_TOKEN=ghp_xxxxx

# 代码审查规则 (可选)
CODE_REVIEW_STRICTNESS=medium  # low/medium/high
ENABLE_SECURITY_CHECK=true
ENABLE_PERFORMANCE_CHECK=true

本地配置

在 TOOLS.md 中添加:

### AI Code Reviewer 配置

- 默认审查严格度：medium
- 自动修复建议：是
- 生成测试用例：是
- 支持语言：JS/TS, Python, Java, Go, Rust, PHP

使用示例

基础用法

审查这段代码

指定语言

用 Python 审查这段代码

生成 PR 描述

为这个变更生成 PR 描述

生成测试

为这个函数生成单元测试

完整审查

完整审查：检查代码质量 + 生成 PR 描述 + 生成测试

输出格式

代码审查报告

🔍 AI 代码审查报告

📁 文件：src/utils/validator.js
📊 评分：85/100 (良好)

✅ 优点:
• 函数命名清晰
• 注释完整
• 错误处理得当

⚠️ 需要改进 (3 个):
1. [性能] 第 23 行：循环内重复计算，建议提取到循环外
   建议：const limit = items.length; for (let i = 0; i \x3C limit; i++)

2. [安全] 第 45 行：用户输入未验证，存在注入风险
   建议：添加 sanitizeInput() 验证

3. [风格] 第 67 行：函数过长 (80 行)，建议拆分
   建议：拆分为 validateUser() 和 validatePermissions()

🧪 测试建议:
• 添加边界条件测试 (空数组、null、undefined)
• 添加异常输入测试
• 覆盖率目标：80%

📝 自动修复：需要我帮你修复这些问题吗？[是/否/查看建议]

PR 描述

## 🎯 变更摘要

实现了用户权限验证功能，包括角色检查和权限过滤。

## 📝 详细说明

- 新增 `validatePermissions()` 函数
- 优化了循环性能 (减少重复计算)
- 添加了输入验证防止注入攻击

## 🧪 测试

- [x] 单元测试已添加
- [x] 边界条件测试通过
- [ ] 集成测试待完成

## ⚠️ 注意事项

- 需要更新数据库迁移脚本
- 向后兼容，无需手动迁移

## 📋 检查清单

- [x] 代码审查通过
- [x] 测试覆盖率 > 80%
- [x] 文档已更新

测试用例

// 生成的测试用例
describe('validatePermissions', () => {
  test('应该允许管理员访问所有资源', () => {
    const user = { role: 'admin' };
    expect(validatePermissions(user, 'any')).toBe(true);
  });

  test('应该拒绝未授权用户访问', () => {
    const user = { role: 'guest' };
    expect(validatePermissions(user, 'admin-only')).toBe(false);
  });

  test('应该处理空用户对象', () => {
    expect(() => validatePermissions(null, 'resource'))
      .toThrow('Invalid user object');
  });

  test('应该处理边界条件：空权限列表', () => {
    const user = { role: 'user', permissions: [] };
    expect(validatePermissions(user, 'resource')).toBe(false);
  });
});

错误处理

定价策略

开发优先级

• 技能框架和文档
• 代码解析引擎
• AI 审查规则库
• PR 描述生成器
• 测试用例生成器
• GitHub 集成 (自动 PR)
• CI/CD 集成
• 自定义规则支持

依赖技能

• coding-agent (代码分析)
• github (PR 操作)

更新日志

v0.1.0 (2026-03-14)

• 初始框架创建
• 完成 SKILL.md 和 README.md
• 设计审查规则和输出格式