code-reviewer

name	code-reviewer
description	专业的代码审查技能，支持多语言代码的全面性能分析和质量评估。包含详细的磁盘IO、网络IO、死锁风险分析，统一的接口规范和注释标准检查。用于代码Review、性能优化建议、代码质量评估等场景。

Code Reviewer

专业的多语言代码审查工具，提供全面的性能分析和代码质量评估。

适用场景

• 代码Review和代码走查
• 性能瓶颈识别和优化建议
• 接口规范性检查
• 注释完整性评估
• 并发安全风险检测
• 代码质量评分

核心能力

1. 性能深度分析

   • 时间/空间复杂度评估
   • 磁盘IO模式分析（缓冲、流式、批量）
   • 网络IO分析（超时、重试、连接池、并发）
   • 并发安全检测（死锁、race condition、锁粒度）
   • 内存管理分析（泄漏、循环引用）

2. 接口一致性检查

   • 命名规范（PascalCase、camelCase、snake_case）
   • 参数顺序和类型一致性
   • 返回值类型统一性
   • 函数签名规范性

3. 注释规范检查

   • 模块/类/函数注释完整性
   • 参数和返回值文档
   • 异常说明
   • 使用示例

4. 综合质量评估

   • 错误处理健壮性
   • 代码风格和复杂度
   • 安全性检查
   • 综合评分和等级

使用方法

标准Review流程

1. 读取待Review的代码文件
2. 分析代码结构和功能
3. 按维度进行详细检查
4. 生成结构化Review报告

报告格式

支持两种输出格式：

Markdown报告（默认）：assets/report-template.md

• 人类可读的详细报告
• 包含评分、问题列表、修复建议

JSON报告：

• 结构化数据便于后续处理
• 包含完整的评分维度和问题详情

Review维度详解

1. 接口一致性

检查命名规范、参数一致性、返回值一致性。

参考：references/code-quality-checklist.md 第1节

2. 注释规范

检查模块注释、类注释、函数注释、内联注释。

参考：references/code-quality-checklist.md 第2节

3. 性能分析

3.1 时间/空间复杂度

• 识别低效算法（O(n²)以上）
• 检测内存使用问题
• 评估递归深度风险

3.2 磁盘IO

• 检查文件读取模式（流式 vs 一次性）
• 评估批量操作优化
• 检测资源泄漏风险

参考：references/performance-analysis.md 第3节

3.3 网络IO

• 检查超时设置
• 评估重试机制
• 检测连接池使用
• 分析并发请求模式

参考：references/performance-analysis.md 第4节

3.4 并发与锁

• 死锁风险检测：嵌套锁顺序、回调锁内调用
• 锁粒度评估：全局锁 vs 细粒度锁
• Race Condition检测：非原子操作、共享数据访问
• 线程安全类型：检查非线程安全集合使用

参考：references/performance-analysis.md 第5节

3.5 内存管理

• 内存泄漏风险（全局缓存、事件监听）
• 循环引用检测
• 大对象处理评估
• 对象池使用建议

参考：references/performance-analysis.md 第6节

4. 代码质量

• 错误处理完整性
• 代码风格一致性
• 安全性检查（注入、敏感信息）
• 复杂度控制

参考：references/code-quality-checklist.md 第3-5节

评分标准

性能维度评分（满分100）

子维度	权重	A(100)	B(85)	C(70)	D(50)	F(30)
时间复杂度	25%	< O(n log n)	O(n log n)	O(n)	O(n²)	> O(n²)
空间复杂度	20%	O(1)	O(log n)	O(n)	O(n²)	> O(n²)
磁盘IO	15%	流式/异步	有缓冲	标准	频繁小IO	泄漏风险
网络IO	15%	连接池+并发	有超时重试	同步	无超时	阻塞
并发安全	15%	无锁/细粒度	正确使用	粒度粗	使用不当	死锁风险
内存管理	10%	对象池	及时释放	标准GC	循环引用	内存泄漏

质量维度评分（满分100）

子维度	权重	A(100)	F(30)
接口一致性	20%	完全一致	混乱
注释质量	20%	完整规范	无注释
错误处理	25%	完善健壮	无处理
代码风格	20%	优雅简洁	混乱
安全性	15%	无漏洞	严重漏洞

综合评级

总分 = 性能得分 × 0.5 + 质量得分 × 0.5

S级：95-100分（卓越）
A级：85-94分（优秀）
B级：70-84分（良好）
C级：60-69分（及格）
D级：40-59分（需改进）
F级：< 40分（不合格）

问题分级

级别	标识	定义	响应要求
P0	🔴 严重	功能缺陷、安全风险、明显死锁	必须立即修复
P1	🟠 重要	性能问题、代码异味、潜在风险	建议下个迭代修复
P2	🟡 一般	风格问题、注释缺失	可选修复
P3	🟢 建议	优化建议、最佳实践	参考采纳

示例输出

死锁风险报告示例

### 并发与锁分析

**评分**: 65/100 | **等级**: C

| 检查点 | 状态 | 风险等级 |
|--------|------|---------|
| 死锁风险 | ⚠️ 发现风险 | **高** |

**死锁分析报告**:

潜在死锁场景:

函数: process_order() 第45-52行 锁获取顺序: order_lock -> inventory_lock

函数: cancel_order() 第78-85行
锁获取顺序: inventory_lock -> order_lock

风险: 两个线程分别以不同顺序获取锁，可能导致死锁

建议修复方案:
1. 统一锁获取顺序（推荐：按资源ID排序后获取）
2. 使用超时锁 try_lock(timeout)
3. 考虑使用无锁数据结构替换

磁盘IO报告示例

### 磁盘IO分析

**评分**: 45/100 | **等级**: D

| 检查点 | 状态 | 说明 |
|--------|------|------|
| 缓冲/流式处理 | ❌ 未使用 | 第120行一次性读取大文件 |
| 资源释放 | ⚠️ 有风险 | 第88行文件句柄未使用with语句 |

**问题列表**:
- [P1] process_large_file() 一次性读取整个文件到内存
  - 当前: data = open(path).read()  # 可能导致OOM
  - 建议: 使用生成器逐行读取
  ```python
  with open(path) as f:
      for line in f:
          process(line)

## 多语言支持

本Skill支持以下编程语言（按分析深度排序）：

1. **Python** - 完整支持所有分析维度
2. **Go** - 完整支持，重点关注goroutine和channel
3. **Java** - 完整支持，重点关注线程和锁
4. **JavaScript/TypeScript** - 完整支持，重点关注Promise和异步
5. **C/C++** - 基础支持，重点关注内存管理
6. **Rust** - 基础支持，重点关注所有权系统
7. **其他语言** - 通用分析（复杂度、注释、接口）

## 使用约束

1. **文件大小**：建议单文件 < 5000行，超大文件可能截断分析
2. **分析时间**：复杂文件可能需要较长时间，请耐心等待
3. **假阳性**：静态分析可能产生误报，需人工确认
4. **动态行为**：无法分析运行时性能，仅能静态推断