critical-code-reviewer

🕵️ 严苛精准的代码安全审查专家

来自资深工程师视角的严苛代码审查工具,通过对抗性思维识别安全漏洞、性能隐患与代码异味,守护代码库质量。

收藏
6.1k
安装
1.3k
版本
v0.1.0
CLS 安全性认证2026-05-09
点击查看完整报告 >

使用说明

核心用法

Critical Code Reviewer 是一款基于对抗性思维模式的代码审查指导工具。当用户使用"critically review"、"critique my code"等触发词激活时,它会以资深工程师的视角对 Python、R、JavaScript/TypeScript、SQL 及前端代码进行严苛审查。该 Skill 采用"有罪推定"(Guilty Until Proven Exceptional)的核心理念,默认每行代码都存在问题,直到被证明是高质量的。审查过程遵循严格的四级严重度体系:Blocking(阻止合并的安全漏洞和逻辑错误)、Required Changes(必须的改进项)、Strong Suggestions(优化建议)和 Noted(轻微风格问题),确保反馈结构清晰、优先级明确。

显著优点

该 Skill 的突出优势在于其系统化的缺陷检测能力。首先,Slop Detector 能精准识别代码中的"懒惰模式"——无意义的命名(data/temp/result)、显而易见的注释(// increment counter)、复制粘贴痕迹和货物崇拜编程。其次,Adversarial Lens(对抗性视角)强制审查者假设最坏场景:每个未处理的 Promise 都会在凌晨3点拒绝、每个 null 值都会出现、每个用户输入都是恶意的。这种思维模式有效覆盖了安全漏洞(XSS、SQL注入)、竞态条件和边界情况。第三,语言特定的红旗标记(Language-Specific Red Flags)针对不同语言(如 Python 的裸 except 子句、TypeScript 的 any 类型滥用、React 的依赖数组谎言)提供专业指导。最后,明确的"退出条件"确保审查诚实客观——没有阻止性问题时直接批准,不制造虚假问题。

潜在缺点与局限性

尽管功能强大,该 Skill 存在几个值得注意的局限。首要问题是来源可信度为 T3(个人开发者账号),虽经审计无代码风险,但长期维护、更新频率和社区支持弱于企业级工具。其次,审查风格极度严苛,"零容忍"(zero tolerance)的态度可能导致过度审查(false positives),将轻微的风格偏好标记为必需修改,在小团队或原型开发阶段可能造成不必要的摩擦。第三,该 Skill 仅提供文本反馈,无法自动修复代码,用户需手动实施所有建议。第四,部分建议(如"structural contempt"中的架构批评)高度依赖上下文,在缺乏完整代码库视图时可能产生误判。

适合的目标群体

此 Skill 最适合以下场景:一是安全敏感型项目,需要识别潜在的安全漏洞和边界情况;二是代码审查培训,帮助初级开发者学习专业审查标准和反模式识别;三是重构前的质量评估,在重大架构调整前识别技术债务;四是高标准的开源项目或企业级代码库维护。对于追求"够用就行"(move fast and break things)的早期创业团队,或正在进行快速原型开发的场景,此 Skill 的严苛性可能反而降低开发效率。

使用风险

使用此 Skill 时需注意以下风险:一是审查疲劳,过度严苛的反馈可能导致开发者忽视真正重要的问题("狼来了"效应);二是团队规范冲突,Skill 中内置的某些偏好(如对注释的负面看法)可能与现有团队编码标准冲突,需人工调整;三是上下文缺失导致的误判,当审查代码片段而非完整 PR 时,Skill 会明确标记限制,但用户可能忽略这些警告;四是责任边界,Skill 明确声明不执行代码修复,所有实施决策仍需开发者承担,不能替代完整的 CI/CD 安全扫描和人工审计。

安全解读

核心用法

critical-code-reviewer 是一款专为开发者设计的 AI 代码审查助手,采用"有罪推定"的对抗性思维模式,对 Python、R、JavaScript/TypeScript、SQL 及前端代码进行深度审查。

触发方式

  • 明确指令:"critically review my code"、"find issues in this PR"、"what's wrong with this code"
  • 代码粘贴:直接提交代码片段或 PR 描述

审查维度

| 维度 | 检测重点 |
|------|---------|
| 安全性 | SQL注入、XSS、类型强制攻击、未处理Promise |
| 可靠性 | 边缘案例失败、空值处理、竞态条件、静默失败 |
| 代码质量 | 命名懒惰、注释冗余、复制粘贴代码、死代码 |
| 性能 | N+1查询、未优化渲染、向量化缺失 |
| 可维护性 | 函数职责混乱、文件组织糟糕、类型安全违规 |

输出格式

采用四级严重度体系:

  • Blocking:安全漏洞、数据损坏风险、逻辑错误
  • Required Changes:代码异味、未处理边缘案例、命名糟糕
  • Strong Suggestions:次优方案、缺失测试、性能顾虑
  • Noted:轻微风格问题

显著优点

1. 对抗性思维:默认假设代码存在问题,有效规避确认偏误
2. 语言专精:针对不同语言的红旗模式(如 Python 的 bare except、JS 的 == 滥用)

3. 可执行性:不仅指出问题,还提供具体修复方案或替代模式

4. 上下文感知:支持增量审查(focus on delta),避免重复评论已解决问题

潜在局限

1. 误报风险:过度严苛可能导致对合理 trade-off 的过度批评
2. 上下文盲区:部分代码审查需完整代码库才能验证(如重复工具函数检测)

3. 框架覆盖不全:对特定领域框架(如 Django、FastAPI、Next.js 等)的深度规则有限

4. 文化摩擦:"constructively brutal" 风格可能在某些团队引发抵触

适合人群

  • 寻求突破舒适区的初级/中级开发者
  • 需要统一审查标准的团队技术负责人
  • 开源项目维护者处理外部 PR
  • 代码面试前的自我审查

常规风险

  • 过度优化:可能建议对非关键路径进行不必要的重构
  • 安全焦虑:对 any 类型、缺失 await 等标记可能引发不必要的恐慌
  • 维护成本:严格遵循所有建议可能导致开发速度显著下降

使用建议

建议将本 Skill 作为守门工具而非开发伴侣——在关键 PR、生产发布前启用,日常开发中适度采用其建议,避免完美主义瘫痪。

critical-code-reviewer 内容

手动下载zip · 4.5 kB
SKILL.mdtext/markdown
请选择文件