Menu
专业的代码审查技能，支持多语言代码的全面性能分析和质量评估。包含详细的磁盘IO、网络IO、死锁风险分析，统一的接口规范和注释标准检查。用于代码Review、性能优化建议、代码质量评估等场景。
Instalar com Codex ou Claude Copie este prompt, cole no Codex, Claude ou outro assistente e deixe que ele revise a página da skill e instale para você.
Baseado na classificação ocupacional SOC
专业的代码重构师，从代码分析开始，到代码架构的重编排。将代码的逻辑按图论解析为图，在逻辑层优化该图运行，并结束这一切。
Solve, review, optimize, generate, and test algorithm problems with constraint-driven reasoning. Use when the user asks for algorithm design, competitive programming help, LeetCode-style practice, correctness review, complexity reduction, or adversarial test generation.
Design, review, evolve, and document API contracts with stable schemas, examples, error models, versioning, and compatibility rules.
| name | code-reviewer |
| description | 专业的代码审查技能，支持多语言代码的全面性能分析和质量评估。包含详细的磁盘IO、网络IO、死锁风险分析，统一的接口规范和注释标准检查。用于代码Review、性能优化建议、代码质量评估等场景。 |
专业的多语言代码审查工具，提供全面的性能分析和代码质量评估。
性能深度分析
接口一致性检查
注释规范检查
综合质量评估
1. 读取待Review的代码文件
2. 分析代码结构和功能
3. 按维度进行详细检查
4. 生成结构化Review报告
支持两种输出格式：
Markdown报告（默认）：assets/report-template.md
JSON报告：
检查命名规范、参数一致性、返回值一致性。
参考：references/code-quality-checklist.md 第1节
检查模块注释、类注释、函数注释、内联注释。
参考：references/code-quality-checklist.md 第2节
参考：references/performance-analysis.md 第3节
参考：references/performance-analysis.md 第4节
参考：references/performance-analysis.md 第5节
参考：references/performance-analysis.md 第6节
参考：references/code-quality-checklist.md 第3-5节
| 子维度 | 权重 | 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 | 循环引用 | 内存泄漏 |
| 子维度 | 权重 | 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分析
**评分**: 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. **动态行为**：无法分析运行时性能，仅能静态推断