mcpField
R
review

review

github.com/johnlindquist测试与质量2637 次安装45 周前更新
代码仓库

关于 review

面向 Claude 的代码审查 Skill,汇集多种审查视角并按需加载指南。它要求先阅读相关角色参考文档,再评估变更并按优先级报告问题和具体建议。

主要功能

  1. 01根据提示选择审查视角
  2. 02审查前加载角色指南
  3. 03评估正确性、安全性和性能
  4. 04在发现中引用文件与行号
  5. 05结合多个视角进行审查

适用场景

  1. 01检查变更中的性能问题
  2. 02从测试和类型系统角度评估补丁
  3. 03使用多个角色审查架构
  4. 04排列代码问题并给出修复建议

安装

复制下面的命令,在终端中运行。

$npx skills add https://github.com/johnlindquist/claude --skill review

安装后在 Claude Code 中输入/review确认已加载。

包含的文件

除 SKILL.md 外,安装时一起下载的参考资料和脚本。

references/
ai-reviewer.md
anders-reviewer.md
beck-reviewer.md
bjarne-reviewer.md
brendan-reviewer.md
carmack-reviewer.md
dean-reviewer.md
dhh-reviewer.md
fowler-reviewer.md
github-reviewer.md
grace-reviewer.md
guido-reviewer.md
james-reviewer.md
lattner-reviewer.md
linus-reviewer.md
matz-reviewer.md
perf-reviewer.md
pitch-reviewer.md
react-reviewer.md
rob-reviewer.md
stb-reviewer.md
unix-reviewer.md

详细介绍

审查 Skill

将所有审查者角色统一为一个 Skill。当代码需要被审查时,Claude 会激活此 Skill,然后通过打开下面链接的参考文档或启动特定角色的子 Agent 来“懒加载”确切的视角。

关键工作流

必需:在进行任何代码审查之前,你必须使用 Read 工具加载相关的角色参考文件。这些参考文件包含每个审查者角色的具体审查优先级、视角和评估标准。

  1. 收集代码/diff 上下文以及用户的目标(缺陷、架构、性能等)。
  2. 强制要求:解析审查者提示(例如 "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
  3. 应用审查者角色的视角,遵循已加载参考文件中他们的具体指导和优先级。
  4. 引用具体的文件/行,标记问题,并提供具体的建议。

切勿在未先加载相应角色参考文件的情况下尝试进行代码审查。

通用检查清单

  • 在判断变更之前,理解输入/输出、依赖关系和预期行为。
  • 使用允许的工具(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