关于 review
面向 Claude 的代码审查 Skill,汇集多种审查视角并按需加载指南。它要求先阅读相关角色参考文档,再评估变更并按优先级报告问题和具体建议。
主要功能
- 01根据提示选择审查视角
- 02审查前加载角色指南
- 03评估正确性、安全性和性能
- 04在发现中引用文件与行号
- 05结合多个视角进行审查
适用场景
- 01检查变更中的性能问题
- 02从测试和类型系统角度评估补丁
- 03使用多个角色审查架构
- 04排列代码问题并给出修复建议
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ johnlindquist/ claude --skill review安装后在 Claude Code 中输入/review确认已加载。
包含的文件
除 SKILL.md 外,安装时一起下载的参考资料和脚本。
references/ai-reviewer.mdanders-reviewer.mdbeck-reviewer.mdbjarne-reviewer.mdbrendan-reviewer.mdcarmack-reviewer.mddean-reviewer.mddhh-reviewer.mdfowler-reviewer.mdgithub-reviewer.mdgrace-reviewer.mdguido-reviewer.mdjames-reviewer.mdlattner-reviewer.mdlinus-reviewer.mdmatz-reviewer.mdperf-reviewer.mdpitch-reviewer.mdreact-reviewer.mdrob-reviewer.mdstb-reviewer.mdunix-reviewer.md详细介绍
审查 Skill
将所有审查者角色统一为一个 Skill。当代码需要被审查时,Claude 会激活此 Skill,然后通过打开下面链接的参考文档或启动特定角色的子 Agent 来“懒加载”确切的视角。
关键工作流
必需:在进行任何代码审查之前,你必须使用 Read 工具加载相关的角色参考文件。这些参考文件包含每个审查者角色的具体审查优先级、视角和评估标准。
- 收集代码/diff 上下文以及用户的目标(缺陷、架构、性能等)。
- 强制要求:解析审查者提示(例如 "perf, react, typescript"),并在审查之前直接使用 Read 工具阅读匹配的参考文件:
- AI/ML 相关 → 首先阅读
references/ai-reviewer.md - 类型系统相关 → 首先阅读
references/anders-reviewer.md - 测试/TDD 相关 → 首先阅读
references/beck-reviewer.md - 性能/抽象 → 首先阅读
references/bjarne-reviewer.md - 创新/实用主义 → 首先阅读
references/brendan-reviewer.md - 底层性能 → 首先阅读
references/carmack-reviewer.md - 分布式系统 → 首先阅读
references/dean-reviewer.md - 约定/简洁性 → 首先阅读
references/dhh-reviewer.md - 重构/架构 → 首先阅读
references/fowler-reviewer.md - 协作/CI/CD → 首先阅读
references/github-reviewer.md - 抽象/模块化 → 首先阅读
references/grace-reviewer.md - 可读性/Python → 首先阅读
references/guido-reviewer.md - 可移植性/Java → 首先阅读
references/james-reviewer.md - 编译器/工具链 → 首先阅读
references/lattner-reviewer.md - 系统/严谨性 → 首先阅读
references/linus-reviewer.md - 开发者体验/Ruby → 首先阅读
references/matz-reviewer.md - 可观测性/追踪 → 首先阅读
references/perf-reviewer.md - React 模式 → 首先阅读
references/react-reviewer.md - Go/并发 → 首先阅读
references/rob-reviewer.md - Unix 哲学 → 首先阅读
references/unix-reviewer.md
- AI/ML 相关 → 首先阅读
- 应用审查者角色的视角,遵循已加载参考文件中他们的具体指导和优先级。
- 引用具体的文件/行,标记问题,并提供具体的建议。
切勿在未先加载相应角色参考文件的情况下尝试进行代码审查。
通用检查清单
- 在判断变更之前,理解输入/输出、依赖关系和预期行为。
- 使用允许的工具(
Read、Grep、Glob、Bash)检查实现、历史和测试。 - 评估正确性、安全性、性能、可维护性和用户影响。
- 标记缺失的测试、薄弱的文档、回归或架构漂移;提出具体的修复方案。
- 按严重程度排序总结发现,然后note 风险、问题和验证步骤。
多视角审查
当请求多个角色时(例如 "review this with Anders and React perspectives"):
- 阅读下面列表中的每个相关参考文件
- 将每个视角的优先级和关注点应用于代码
- 综合发现:突出各视角一致或冲突的地方
- 按所有视角中的严重程度对问题排序
人物参考(按需加载)
- AI 远见者 - 自适应系统、涌现行为、数据驱动设计。打开指令
- Anders Hejlsberg - 强类型、语言/工具工效学、结构化 API。打开指令
- Kent Beck - TDD 纪律、快速反馈循环、自适应设计。打开指令
- Bjarne Stroustrup - 通过抽象实现性能、类型安全、有纪律的工程。打开指令
- Brendan Eich - 快速创新、创造性问题解决、务实实验。打开指令
- John Carmack - 底层卓越、图形/性能调优、精确思维。打开指令
- Jeff Dean - 行星级系统、效率、分布式可靠性。打开指令
- DHH - 有主见的约定、开发者自主权、简单胜于繁文缛节。打开指令
- Martin Fowler - 重构就绪、演进式架构、有意图的设计。打开指令
- GitHub 一代 - 协作规范、文档、CI/CD 自动化。打开指令
- Grace Hopper & Barbara Liskov - 抽象完整性、可替换性、模块化设计。打开指令
- Guido van Rossum - 可读性、Pythonic 简洁、务实清晰。打开指令
- James Gosling - JVM 可移植性、API 稳定性、向后兼容。打开指令
- Chris Lattner - 编译器/工具链创新、语言互操作性、性能。打开指令
- Linus Torvalds - 内核级严谨、补丁纪律、直率诚实的反馈。打开指令
- Yukihiro "Matz" Matsumoto - Ruby 美学、以人为本的设计、编码之乐。打开指令
- Brendan Gregg & Liz Rice - 可观测性、追踪、数据优先的性能分析。打开指令
- React 核心维护者 - hooks、并发渲染、面向 DX 的组件模式。打开指令
- Rob Pike - Go/Unix 极简主义、并发原语、可组合工具。打开指令
- Unix 传统主义者 - 小而锋利的工具、可组合性、文本优先的自动化。打开指令
每个参考在显式打开之前都不会进入上下文,从而保持 Claude 的上下文精简,同时仍能快速访问原始、详细的审查者指导。
评论
支持 Markdown