| name | devbooks-reviewer |
| description | devbooks-reviewer：以 Reviewer 角色做可读性/一致性/依赖健康/坏味道审查，只输出审查意见与可执行建议，不讨论业务正确性。用户说"帮我做代码评审/review 可维护性/坏味道/依赖风险/一致性建议"，或在 DevBooks apply 阶段以 reviewer 执行时使用。 |
| recommended_experts | ["System Architect","Security Expert"] |
| allowed-tools | ["Glob","Grep","Read","Bash"] |
DevBooks：代码评审（Reviewer）
渐进披露
基础层（必读）
目标：明确本 Skill 的核心产出与使用范围。
输入：用户目标、现有文档、变更包上下文或项目路径。
输出：可执行产物、下一步指引或记录路径。
边界：不替代其他角色职责，不触碰 tests/。
证据：引用产出物路径或执行记录。
进阶层（可选）
适用：需要细化策略、边界或风险提示时补充。
扩展层（可选）
适用：需要与外部系统或可选工具协同时补充。
推荐 MCP 能力类型
- 代码检索（code-search）
- 引用追踪（reference-tracking）
- 影响分析（impact-analysis）
工作流位置感知（Workflow Position Awareness）
核心原则：Code Review 在 Test Owner 阶段 2 验证之后执行，是归档前的最后一个评审步骤。
我在整体工作流中的位置
proposal → design → test-owner(阶段1) → coder → test-owner(阶段2) → [Code Review] → archive
↓
可读性/一致性/依赖审查
Code Review 的职责边界
| 允许 | 禁止 |
|---|
| 审查代码可读性/一致性 | ❌ 修改代码文件 |
| 设置 verification.md Status = Done | ❌ 讨论业务正确性（那是 Test Owner 的事） |
| 提出改进建议 | ❌ 勾选 AC 覆盖矩阵（那是 Test Owner 的事） |
前置条件
前置：配置发现（协议无关）
<truth-root>：当前真理目录根<change-root>：变更包目录根
执行前必须按以下顺序查找配置（找到后停止）：
.devbooks/config.yaml（如存在）→ 解析并使用其中的映射dev-playbooks/project.md（如存在）→ Dev-Playbooks 协议，使用默认映射project.md（如存在）→ template 协议，使用默认映射- 若仍无法确定 → 停止并询问用户
关键约束：
- 如果配置中指定了
agents_doc（规则文档），必须先阅读该文档再执行任何操作
- 禁止猜测目录根
- 禁止跳过规则文档阅读
审查维度
1. 可读性审查
- 命名一致性（PascalCase/camelCase）
- 函数长度和复杂度
- 注释质量和必要性
- 代码格式化
2. 依赖健康审查
- 分层约束遵守（参见
<truth-root>/architecture/c4.md）
- 循环依赖检测
- 内部模块封装（禁止深度导入 *Internal 文件）
- 依赖方向正确性
3. 资源管理审查
必须检查的资源泄漏模式：
| 检查项 | 违规模式 | 正确模式 |
|---|
| 订阅未取消 | event.on(...) 无对应 off() | 注册到 DisposableStore |
| 定时器未清理 | setInterval() 无 clearInterval() | 在 dispose() 中清理 |
| 监听器未移除 | addEventListener() 无 removeEventListener() | 使用 AbortController |
| 流未关闭 | createReadStream() 无 close() | 使用 try-finally 或 using |
| 连接未释放 | connect() 无 disconnect() | 使用连接池或 dispose 模式 |
DisposableStore 模式检查：
private disposable = new DisposableStore();
dispose() {
this.cleanup();
}
private readonly _disposables = new DisposableStore();
override dispose() {
this._disposables.dispose();
super.dispose();
}
资源管理检查清单：
4. 类型安全审查
5. 坏味道检测
参见：references/坏味道速查表.md
6. 测试质量审查
执行方式
- 先阅读并遵守：
~/.claude/skills/_shared/references/AI行为规范.md（可验证性 + 结构质量守门）。
- 阅读资源管理指南：
references/资源管理审查清单.md。
- 严格按完整提示词输出评审意见：
references/代码评审提示词.md。
上下文感知
本 Skill 在执行前自动检测上下文，选择合适的审查范围。
检测规则参考：skills/_shared/上下文检测模板.md
检测流程
- 检测变更包是否存在
- 检测是否有代码变更（git diff）
- 检测热点文件（基于变更历史与复杂度分析）
本 Skill 支持的模式
| 模式 | 触发条件 | 行为 |
|---|
| 变更包审查 | 提供 change-id | 审查该变更包相关的代码变更 |
| 文件审查 | 提供具体文件路径 | 审查指定文件 |
| 热点优先审查 | 检测到热点文件变更 | 优先审查高风险热点 |
检测输出示例
检测结果：
- 变更包状态：存在
- 代码变更：12 个文件
- 热点文件：3 个（需重点关注）
- 运行模式：变更包审查 + 热点优先
下一步推荐
参考：skills/_shared/工作流下一步.md
完成 code-review 后，下一步取决于具体情况：
| 条件 | 下一个 Skill | 原因 |
|---|
| 有 spec deltas | devbooks-archiver | 归档前合并规格到真理 |
| 无 spec deltas | 归档完成 | 无需其他 skill |
| 发现重大问题 | 交回 devbooks-coder | 归档前修复问题 |
Reviewer 专属权限：设置 verification.md Status
只有 Reviewer 可以将 verification.md 的 Status 设为 Done。
Review 通过后，Reviewer 必须执行：
- 打开
<change-root>/<change-id>/verification.md
- 将
- Status: Ready 改为 - Status: Done
- 这是归档的前置条件（
change-check.sh --mode archive 会检查）
输出模板
完成 code-review 后，输出：
## 推荐的下一步
**下一步：`devbooks-archiver`**（如果有 spec deltas）
或
**归档完成**（如果无 spec deltas）
原因：代码评审已完成。下一步是[合并 spec deltas 到真理 / 完成归档]。
### 如何调用（如果有 spec deltas）
运行 devbooks-archiver skill 处理变更
