skill-review

技能审查技能

流程

调用：/review-skill <skill-name>，或在发现过时模式时使用此技能

生产环境证据：better-auth 审计（2025-11-08）——发现 6 个严重问题，包括导入不存在的 API；删除了 665 行错误代码，并实现了 v2.0.0

---

9 阶段审计

1. 审查前准备：安装技能，检查版本和日期，测试发现功能
2. 标准：验证 YAML、关键词、第三人称风格和目录结构
3. 官方文档：通过 WebFetch/Context7 验证 API 模式、GitHub 更新、npm 版本和生产环境仓库
4. 代码示例：验证导入项是否存在、API 签名是否匹配、架构是否一致、模板是否可用
5. 跨文件一致性：对比 SKILL.md 与 README.md，确认捆绑资源与文件匹配
6. 依赖项：运行 ./scripts/check-versions.sh，检查破坏性变更，验证“Last Verified”
7. 分类：按严重程度分类（🔴 严重 / 🟡 高 / 🟠 中 / 🟢 低），并提供证据（GitHub/文档/npm）
8. 修复：自动修复明确的问题；遇到架构问题时询问用户；更新所有文件并提升版本
9. 验证：测试发现功能，确认模板可用、没有矛盾，并附带变更日志提交

自动化（通过 ./scripts/review-skill.sh）：YAML 语法、软件包版本、失效链接、TODO、文件组织、陈旧内容

手动（AI）：API 方法与文档对比、GitHub 问题、生产环境对比、代码正确性、架构一致性

---

严重程度分类

🔴 严重：不存在的 API/导入项、无效配置、缺少依赖项

🟡 高：示例相互矛盾、模式不一致、主要版本过时

🟠 中：次要版本陈旧（超过 90 天）、缺少文档章节、错误处理不完整

🟢 低：拼写错误、格式问题、缺少可选元数据

修复决策

自动修复：问题明确（可从文档中找到正确导入项）、证据清晰、不会影响架构

询问用户：存在多种有效方案、涉及破坏性变更或架构选择

版本提升

• 主要版本（v1→v2）：API 模式发生变化
• 次要版本（v1.0→v1.1）：新增功能，向后兼容
• 补丁版本（v1.0.0→v1.0.1）：仅修复错误

---

示例：better-auth 审计（2025-11-08）

🔴 严重 #1：从 'better-auth/adapters/d1' 导入不存在的 d1Adapter

• 证据：官方文档展示了 drizzleAdapter，GitHub 中没有 d1Adapter 导出项，4 个生产环境仓库使用 Drizzle/Kysely
• 修复：替换为从 'better-auth/adapters/drizzle' 导入 drizzleAdapter

结果：删除 3 个文件（已过时），新建 3 个文件（正确模式），增加 1,266 行，v1.0→v2.0，用时 3.5 小时

---

预防的问题（10 项）

1. 虚假的 API 适配器 - 不存在的导入项
2. 过时的 API 方法 - 签名已更改
3. 架构不一致 - 表名不同
4. 过时的脚本 - 已弃用的方法
5. 版本滞后 - 软件包超过 90 天未更新
6. 相互矛盾的示例 - 存在多种冲突模式
7. 失效链接 - 返回 404 的 URL
8. YAML 错误 - 无效的 frontmatter
9. 缺少关键词 - 不易被发现
10. 捆绑资源不完整 - 列出的文件并不存在

---

捆绑资源

规划：~/.claude/skills/../planning/SKILL_REVIEW_PROCESS.md 或仓库中的 planning/SKILL_REVIEW_PROCESS.md（完整的 9 阶段指南）

脚本：仓库根目录中的 scripts/review-skill.sh（自动验证）

命令：仓库根目录中的 commands/review-skill.md（斜杠命令，通过符号链接关联到 ~/.claude/commands/）

参考资料：references/audit-report-template.md（输出模板）

---

最后验证：2026-01-09 | 版本：1.0.1