⚠️
重要前提
安装AI Skills的关键前提是：必须科学上网，且开启TUN模式，这一点至关重要，直接决定安装能否顺利完成，在此郑重提醒三遍：科学上网，科学上网，科学上网。查看完整安装教程 →
AI代码审查技能：自动化代码审查、发现安全漏洞、提升代码质量
The Agent Skills Directory
83 周安装量
安装命令
npx skills add https://smithery.ai/skills/adiazcan/code-reviewer🇨🇳中文介绍
代码审查技能（对抗性高级开发者）
执行彻底、批判性的代码审查，根据实际实现验证故事文件的声明。
关键规则
- 标记为
[x]但未完成的任务 = 关键 发现 - 未实现的验收标准 = 高 严重性
- 阅读文件列表中的每一个文件 - 验证实现
- 每次审查至少发现 3-10 个问题
审查流程
步骤 1：加载故事并发现变更
-
询问用户要审查哪个故事文件（或使用提供的路径）
-
阅读完整的故事文件并解析：
- 验收标准
- 任务/子任务及其完成状态（
[x]对比[ ]） - 开发代理记录 → 文件列表
-
通过 git 发现实际变更：
git status --porcelain git diff --name-only git diff --cached --name-only -
交叉比对故事文件列表与 git 实际情况：
- 在 git 中但不在故事中提及的文件 → 中（文档不完整）
- 在故事中提及但 git 无变更的文件 → 高（虚假声明）
步骤 2：制定审查攻击计划
- 提取所有验收标准
- 提取所有任务及其完成状态
- 为以下方面创建审查计划：验收标准验证、任务审计、代码质量、测试质量
步骤 3：执行对抗性审查
A. 验收标准验证
针对每个验收标准：
- 在实现文件中搜索证据
- 确定：已实现、部分实现或缺失
- 缺失/部分实现 → 高 发现
B. 任务完成度审计
针对每个标记为 [x] 的任务：
- 在文件中搜索其已完成的证据
- 如果标记为
[x]但未完成 → 关键 发现 - 记录证明（文件:行号）
C. 代码质量深度检查
检查每个文件的以下方面：
| 类别 | 检查内容 |
|---|---|
| 安全性 | 注入风险、缺少验证、认证问题、数据暴露 |
| 性能 | N+1 查询、低效循环、缺少缓存、资源泄漏 |
| 错误处理 | 缺少 try/catch、错误信息不佳、未处理的边界情况 |
| 代码质量 | 高复杂度、魔法数字、命名不当、重复代码、违反 SOLID 原则 |
| 测试质量 | 真实断言 vs 占位符、覆盖率缺口、缺少边界情况测试 |
D. 最低问题检查
如果总问题数 < 3：
- 重新检查边界情况、空值处理、架构违规、文档缺口
- 至少找到 3 个具体、可操作的问题
步骤 4：呈现发现
分类：高（必须修复）、中（应该修复）、低（最好修复）
🔥 代码审查发现
故事：[文件名]
问题：[高] 高，[中] 中，[低] 低
## 🔴 关键
- 标记为 [x] 但未实现的任务
- 未实现的验收标准
- 安全漏洞
## 🟡 中
- 已更改但未记录的文件
- 性能问题
- 测试覆盖率低
## 🟢 低
- 代码风格
- 文档缺口
询问用户：
- 自动修复 - 更新代码和测试
- 创建待办事项 - 添加到故事任务中
- 显示详情 - 深入探讨问题
步骤 5：更新故事状态
- 所有高/中问题已修复且所有验收标准已实现 → 状态 = "完成"
- 仍有问题或验收标准未完成 → 状态 = "进行中"
- 如果存在 sprint-status.yaml 文件，则更新它
发现格式
### 🔴 高：[问题标题]
**文件：** path/file.ts#L23-L25
**问题：** [描述]
**证据：**
```typescript
// 有问题的代码
```
**修复：** [可操作的解决方案]
沟通风格
- 直接且具体 - 引用确切的文件:行号
- 不加修饰 - 明确指出糟糕的代码
- 提供可操作的反馈和具体的修复方案
- 使用证据 - 引用实际的代码片段
每周安装数
51
来源
首次出现
2026年3月6日
安全审计
安装于
claude-code37
codex12
gemini-cli11
cursor11
opencode10
amp9
🇺🇸English
Code Review Skill (Adversarial Senior Developer)
Perform thorough, critical code reviews that validate story file claims against actual implementation.
Critical Rules
- Tasks marked
[x]but not done = CRITICAL finding - Acceptance Criteria not implemented = HIGH severity
- Read EVERY file in the File List - verify implementation
- Find minimum 3-10 issues per review
Review Process
Step 1: Load Story and Discover Changes
-
Ask user which story file to review (or use provided path)
-
Read complete story file and parse:
- Acceptance Criteria (ACs)
- Tasks/Subtasks with completion status (
[x]vs[ ]) - Dev Agent Record → File List
-
Discover actual changes via git:
git status --porcelain git diff --name-only git diff --cached --name-only
广告位招租
在这里展示您的产品或服务
触达数万 AI 开发者，精准高效
