| name | metago-code-review-deep |
| description | 深度代码审查技能。对代码进行多维度分级审查（Critical/Major/Minor/Info），覆盖逻辑正确性、安全漏洞、性能瓶颈、可维护性、风格一致性。基于元构批判性分析（metago-critique）和事实核查（metago-fact-check）能力，输出含修复建议的结构化审查报告。 |
| version | 1.0.0 |
| author | MetaGO |
| category | 科技开发 |
| platforms | ["cursor","claude-code","codex","trae","codebuddy","qoder","zcode","workbuddy","qclaw","ima"] |
| trigger | ["帮我做代码审查","审查这段代码","Code Review","检查代码质量","PR Review"] |
深度代码审查（metago-code-review-deep）
描述
对代码进行多维度分级审查，检测逻辑错误、安全漏洞、性能问题、可维护性缺陷、风格不一致。输出含修复建议和优先级的结构化报告。
触发条件
- 用户请求代码审查、PR Review、Code Review 时激活
- 代码提交前审查（pre-commit hook 场景）
- 与
metago-decision-lock 协同：开发者任务输出代码时强制触发
核心流程
1. 静态结构分析
- 文件组织与模块划分是否合理
- 函数/类职责是否单一
- 依赖关系是否清晰（无循环依赖）
- 代码重复度检测（DRY 原则）
2. 逻辑正确性审查
- 边界条件处理（空值、极值、越界）
- 异常路径覆盖（错误处理、资源释放）
- 并发安全（竞态条件、死锁）
- 算法正确性（时间复杂度、空间复杂度）
3. 安全漏洞审查
- 注入漏洞（SQL/命令/XSS）
- 认证授权缺陷
- 敏感信息泄露（硬编码密钥、日志泄密）
- 依赖漏洞（已知 CVE）
4. 性能审查
- 数据库查询效率（N+1、全表扫描）
- 内存泄漏风险
- 不必要的同步阻塞
- 资源未释放（文件句柄、连接池）
5. 可维护性审查
- 命名清晰度（变量、函数、类）
- 注释完整度（复杂逻辑、公共 API）
- 单一职责原则遵循度
- 开闭原则遵循度
分级标准
| 级别 | 含义 | 处理时机 |
|---|
| 🔴 Critical | 必须立即修复（安全漏洞、数据丢失、崩溃） | 阻断合并 |
| 🟠 Major | 应在合并前修复（逻辑错误、性能严重问题） | 阻断合并 |
| 🟡 Minor | 建议修复（可维护性、风格） | 可后续修复 |
| 🔵 Info | 提示性建议（优化、最佳实践） | 仅记录 |
输出格式
【深度代码审查报告】
文件：path/to/file.ext
审查范围：函数/类/模块
总体评分：XX/100
🔴 Critical（N 项）
[C-001] 行 XX: 问题描述
影响：...
修复建议：...
依据：...
🟠 Major（N 项）
[M-001] 行 XX: 问题描述
影响：...
修复建议：...
🟡 Minor（N 项）
[m-001] 行 XX: 问题描述
建议改进：...
🔵 Info（N 项）
[i-001] 建议优化：...
审查结论：✅ 通过 / ❌ 需修复后重新审查
与其他技能的协同
- 与
metago-critique 协同：审查过程应用 L1-L5 批判性分析
- 与
metago-fact-check 协同：检测虚构 API、不存在的函数（幻觉代码）
- 与
metago-decision-lock 协同：审查报告需通过决策锁校验
- 与
metago-problem-trace 协同：问题无限溯源至根本原因
- 与
metago-security-audit 协同：安全维度联动深度安全审计
- 与
metago-refactor-suggest 协同：可维护性问题联动重构建议
