深度代码审查

日常开发中，单靠人眼做 Code Review 往往容易遗漏边界条件、并发竞态或潜在的 SQL 注入等深层缺陷，而走马观花式的检查又难以统一评估问题的严重程度。metago-code-review-deep 正是为了弥补这一自动化辅助的空白，通过结构化流程输出可执行的修复方案与优先级建议。

核心审查维度

该技能将审查工作拆解为五个递进的维度，确保代码质量没有死角：

• 静态结构分析：检查模块划分与文件组织是否合理，是否存在循环依赖，并基于 DRY 原则识别重复代码。
• 逻辑正确性审查：重点排查边界条件（如空值、极值）的处理，确认异常路径覆盖是否完整，并评估并发场景下的竞态与死锁风险。
• 安全漏洞审查：扫描 SQL/命令注入、认证授权缺陷以及硬编码密钥等敏感信息泄露问题，同时排查依赖中的已知 CVE。
• 性能审查：定位数据库查询中的 N+1 反模式或全表扫描，识别内存泄漏风险与未释放的资源句柄。
• 可维护性审查：评估命名清晰度与注释完整度，检查代码对单一职责原则（SRP）与开闭原则（OCP）的遵循情况。

输出与分级机制

所有审查结论都会汇总为一份结构化报告。每个问题都会带上明确的严重级别，方便开发者决定处理时机：

• 🔴 Critical：如安全漏洞、数据丢失风险或崩溃路径，这类问题必须立即修复并阻断合并。
• 🟠 Major：如核心逻辑错误或严重的性能瓶颈，同样需要在合并前完成修复。
• 🟡 Minor：可维护性或风格层面的问题，建议修复但可延后。
• 🔵 Info：提示性建议，仅记录不阻塞流程。

适用边界与注意事项

在投入工作流之前，需要明确该技能的运行边界：

• 只输出报告，不改代码：技能仅生成审查结论与修复建议，不会自动修改仓库文件。
• 无法替代人工复核：作为自动化辅助工具，重大缺陷仍需人工介入判断。
• 不适用于运行时场景：该技能不执行外部网络请求，无法替代动态调试、压力测试或真实环境下的安全渗透。

遇到超长文件或技术栈不明的输入，技能会主动提示分段提交或先确认语言框架，避免基于错误假设给出误导性结论。