بنقرة واحدة
code-reviewer
专业的代码审查技能,支持多语言代码的全面性能分析和质量评估。包含详细的磁盘IO、网络IO、死锁风险分析,统一的接口规范和注释标准检查。用于代码Review、性能优化建议、代码质量评估等场景。
التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.
القائمة
专业的代码审查技能,支持多语言代码的全面性能分析和质量评估。包含详细的磁盘IO、网络IO、死锁风险分析,统一的接口规范和注释标准检查。用于代码Review、性能优化建议、代码质量评估等场景。
التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.
专业的代码重构师,从代码分析开始,到代码架构的重编排。将代码的逻辑按图论解析为图,在逻辑层优化该图运行,并结束这一切。
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.
استنادا إلى تصنيف SOC المهني
| 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. **动态行为**:无法分析运行时性能,仅能静态推断