Code Reviewing Skill

触发条件

当用户提到以下关键词或场景时，自动应用此 Skill：

• "审查代码"、"code review"、"review code"
• "代码审查"、"查看代码质量"
• "检查代码"、"check code"
• "代码有问题吗"、"帮我看看代码"
• "MR 审查"、"PR 审查"

功能概述

这是一个 AI 驱动的代码审查系统，使用 9 个专门的 Agent 并行分析代码变更，覆盖以下维度：

1. 规范检查（可选）：检查 CLAUDE.md 等项目规范遵循情况
2. Bug 检测：逻辑错误、nil 指针、资源泄漏等
3. 代码质量：可读性、命名规范、结构设计等
4. 安全检查：SQL 注入、XSS、敏感信息泄漏等
5. 性能分析：N+1 查询、算法复杂度、内存问题等

核心特性

1. 最大并行性能

• 9 个 Agent 同时执行
• 任务完全独立，无依赖
• 总耗时 ≈ 单个最慢 Agent 的时间

2. 冗余机制

• 4 个维度各有 2 个 Agent 冗余
• 当 2 个 Agent 都发现同一问题 → 置信度 +20
• 减少漏报率

3. 智能规范检查

• 自动检测项目是否有 CLAUDE.md 等规范文件
• 有则检查，无则跳过
• 适应跨部门多项目场景

4. 多语言支持

• 自动识别代码语言
• 引用语言特定的检查规则
• 当前完整支持：Go
• 其他语言：Java, Python, Kotlin, Swift, TypeScript（待完善）

5. 置信度评分

• 每个问题都有置信度评分（0-100）
• 阈值 80：≥80 直接报告，<80 标记为"需要人工确认"
• 冗余加成机制提升准确性

审查流程

详见 review-dimensions.md

1. Pre-check         → 检查 git 状态、获取变更
2. Context           → 智能检测规范文件、识别语言
3. Parallel Review   → 9 个 Agent 并行审查
4. Verification      → 置信度评分、合并重复问题
5. Report            → 生成 Markdown 报告
6. Output            → 终端输出 + 可选文件保存

审查维度

Bug 检测（2×冗余）

• Agent: bug-detector-1, bug-detector-2
• 模型: Opus 4.5
• 重点: 逻辑错误、nil 指针、资源泄漏、并发安全

代码质量（2×冗余）

• Agent: code-quality-analyzer-1, code-quality-analyzer-2
• 模型: Sonnet 4.5
• 重点: 可读性、命名规范、代码结构、复杂度

安全检查（2×冗余）

• Agent: security-analyzer-1, security-analyzer-2
• 模型: Opus 4.5
• 重点: 注入攻击、XSS/CSRF、敏感信息泄漏、不安全加密

性能分析（2×冗余）

• Agent: performance-analyzer-1, performance-analyzer-2
• 模型: Sonnet 4.5
• 重点: N+1 查询、算法复杂度、内存问题、锁竞争

规范检查（可选）

• Agent: project-standards-reviewer
• 模型: Sonnet 4.5
• 重点: CLAUDE.md 规范遵循

语言特定规则

每种语言有专门的检查规则文档（language-checks/ 目录）：

• Go: go-checks.md ✅ 完整支持
• Java: java-checks.md 🚧 待完善
• Python: python-checks.md 🚧 待完善
• Kotlin: kotlin-checks.md 🚧 待完善
• Swift: swift-checks.md 🚧 待完善
• TypeScript: typescript-checks.md 🚧 待完善

置信度评分

详见 confidence-scoring.md

• 90-100: 明确的问题，必然/很可能导致错误
• 70-89: 较明显的问题，需要进一步确认
• 50-69: 潜在问题或改进点
• <50: 不报告

使用示例

示例 1：审查当前分支的变更

用户: 帮我审查一下代码
Claude: 我将使用 9 个并行 Agent 审查你的代码变更...
[执行审查流程，输出报告]

示例 2：指定审查某个文件

用户: 审查 app/regulation/internal/service/consume.go
Claude: [聚焦该文件进行审查]

示例 3：查看审查能力

用户: 代码审查能检查什么？
Claude: 我可以检查以下维度：
1. Bug 检测（逻辑错误、nil 指针等）
2. 代码质量（可读性、命名规范等）
3. 安全检查（SQL 注入、XSS 等）
4. 性能分析（N+1 查询、算法复杂度等）
5. 规范检查（CLAUDE.md 遵循情况）

输出格式

报告采用 Markdown 格式，包含：

• Summary：问题数统计、审查语言
• Issues by Dimension：按维度分类的问题列表
• Issues Requiring Manual Review：低置信度问题
• Review Metadata：Agent 执行信息

限制和注意事项

1. 只检查变更：只审查新增或修改的代码，不检查已存在代码
2. 语言支持：当前只有 Go 语言完整支持，其他语言待完善
3. 置信度阈值：默认 80，可根据需要调整
4. 审查时间：取决于代码量，通常 30-90 秒
5. 需要 git：依赖 git 环境和变更历史

与 Commands 的关系

• Command: /review - 用户显式调用
• Skill: code-reviewing - Claude 自动识别并应用

两者使用相同的审查能力，但触发方式不同。

扩展性

当前版本（v0.0.1）

• 9 个并行 Agent
• Go 语言完整支持
• 4 维度全覆盖

未来版本

• v0.0.2: 完善其他语言支持
• v0.0.3: GitLab MR inline comments
• v0.1.0: 集成 lint、format、coverage 工具

参考资料

• Agent 定义：../../agents/
• 审查维度：review-dimensions.md
• 置信度评分：confidence-scoring.md
• Go 检查规则：../language-checks/go-checks.md