用 claude-skills 的 Code Reviewer 给 PR 做一遍 AI 代码审查【免费下载链接】claude-skills67 Specialized Skills for Full-Stack Developers. Transform Claude Code into your expert pair programmer.项目地址: https://gitcode.com/GitHub_Trending/claud/claude-skills上一个 PR 在队列里挂了三天没人审第二个 PR 里藏着一个 N1 查询合入之后线上接口直接被拖慢了一个量级——这两个场景每个做过 PR 审查流程的团队都熟。问题从来不是没人看代码而是资深工程师的注意力有限而该看的维度太多了。claude-skills 是一个面向全栈开发者的技能合集其中的Code Reviewer 代码审查技能把这套审查经验固化成了可执行的流程AI 按固定节奏完成 PR 审查识别 Bug、SQL 注入、N1 查询这类安全与性能隐患最后产出一份按优先级排好序的审查报告。说白了它是给 AI 套上了资深工程师的审查纪律。一次审查到底覆盖哪些维度技能入口是 skills/code-reviewer/SKILL.md配套 6 份深度参考文档放在 skills/code-reviewer/references/。一次扫描同时覆盖四个维度正确性逻辑错误、边界遗漏、并发竞态性能N1 查询、缺缓存的昂贵操作可维护性抽象是否过度或缺失、模块划分是否合理测试覆盖关键路径有没有测试、断言写的是行为还是实现安全检查则以OWASP Top 10为基线SQL 注入、鉴权绕过这类问题不会被漏掉。你不需要背清单清单替你干活。五步节奏把注意力花在刀刃上审查的节奏定义在 review-checklist.md 里五步走每步对应一段注意力预算上下文10%看 PR 描述问自己我能一句话说清它要解决什么问题吗。说不清就先问别硬看。结构20%改动是否遵循代码库既有模式新引入的抽象有没有必要细节40%最重的一段。N1 查询、硬编码密钥、注入风险逐行扫出来。测试20%边界情况覆盖了吗断言验证的是行为还是实现细节反馈按 Critical → Major → Minor 三级输出结构化报告。一个值得借鉴的细节发现严重安全问题时立即标注而不是留到报告末尾。这和真实团队安全漏洞优先的沟通习惯是一致的——这一步卡住了先停下来问比闷头审完再返工便宜得多。8 个维度哪些是合并前必查的8 大审查维度的完整清单同样在 review-checklist.md 中按紧迫程度可以分成两层。合并前必查安全用户输入有没有校验SQL 参数化了没有鉴权是强制的吗逻辑空值、边界条件、并发竞态处理了吗性能有没有 N1 查询昂贵操作缓存了吗测试覆盖率够吗Mock 用得恰当吗有空再查设计改动落在正确的模块了吗抽象层级合适吗命名清晰、一致、见名知意吗错误处理错误被正确捕获了吗日志有信息量吗文档公共 API 有文档吗复杂逻辑有注释吗照着这份代码质量清单自问一遍本身就是一次低成本的合并前自查不装技能也能用。一份真实审查报告长什么样产出物遵循 report-template.md 的固定模板共 7 个部分Summary一句话复述 PR 意图加上总体评价Critical Issues合并前必须修复——Bug、安全漏洞、数据丢失Major Issues应当修复——明显性能问题、设计问题、可维护性隐患Minor Issues锦上添花——命名、可读性Positive Feedback点出做得好的具体模式Questions for Author需要作者澄清的问题VerdictApprove / Request Changes / Comment 三选一每个问题按位置 → 现状 → 建议 → 影响四要素写。比如 N1 查询不会只说这里有性能问题而是标注每次请求会多打约 100 次数据库查询——影响是可量化的。严重度划分Critical安全风险、数据丢失、崩溃SQL 注入、鉴权绕过Major明显性能或可维护性问题N1 查询、上帝函数Minor风格、命名类改进审查者最常犯的 3 种表达错误feedback-examples.md 里有一组经典对照本质上是三种表达错误❌ 笼统评价这段代码很混乱 → ✅ 指出具体问题并给拆分方案这个函数同时做了校验和持久化建议拆成validateUser()和saveUser()便于单独测试❌ 模糊指令修复这个查询 → ✅ 说清机制和影响循环里取数据会产生 N1 查询用预加载一次取出即可❌ 空洞要求加测试 → ✅ 给出具体场景缺少 email 已存在场景的测试建议验证返回 409 及错误信息三条黄金法则具体、可执行、建设性。技能还内置了分歧处理原则——如果作者已注释说明某个非常规选择的理由先认可再提替代方案有 linter 的情况下绝不在风格问题上阻塞合并。两条命令装完自然语言触发安装走插件市场两条命令/plugin marketplace add jeffallan/claude-skills /plugin install fullstack-dev-skillsjeffallan装完用自然语言触发即可帮我审查这个 Pull Request检查这段代码的安全和性能问题review code找一下 N1 查询和硬编码密钥常见触发词包括code review、PR review、pull request、code quality没自动激活就点名使用 Code Reviewer 审查这段代码。验证步骤在 QUICKSTART.md 里。单点太窄时选一条路线走到底Code Reviewer 的定位是广域快扫深度专项需要组合拳67 个技能的完整清单与决策树在 SKILLS_GUIDE.md项目级工作流见 docs/WORKFLOW_COMMANDS.md安全加固Secure Code Guardian → Security Reviewer → Test MasterBug 排查Debugging Wizard → 框架专家 → Test Master → Code Reviewer 收尾测试补强审查报告发现测试缺口直接交给 Test Master 补齐AI 审查解决的是清单不会忘、维度不会漏的问题但它不是终审。报告里的 Critical 项逐条核实后再合并Minor 项按团队规范取舍——最终判断权始终在写代码的人手里。【免费下载链接】claude-skills67 Specialized Skills for Full-Stack Developers. Transform Claude Code into your expert pair programmer.项目地址: https://gitcode.com/GitHub_Trending/claud/claude-skills创作声明：本文部分内容由AI辅助生成（AIGC），仅供参考 
长河君 · 2026/10/17 1:54:19 · 阅读 1.2k 用 claude-skills 的 Code Reviewer 给 PR 做一遍 AI 代码审查
用 claude-skills 的 Code Reviewer 给 PR 做一遍 AI 代码审查【免费下载链接】claude-skills67 Specialized Skills for Full-Stack Developers. Transform Claude Code into your expert pair programmer.项目地址: https://gitcode.com/GitHub_Trending/claud/claude-skills上一个 PR 在队列里挂了三天没人审第二个 PR 里藏着一个 N1 查询合入之后线上接口直接被拖慢了一个量级——这两个场景每个做过 PR 审查流程的团队都熟。问题从来不是没人看代码而是资深工程师的注意力有限而该看的维度太多了。claude-skills 是一个面向全栈开发者的技能合集其中的Code Reviewer 代码审查技能把这套审查经验固化成了可执行的流程AI 按固定节奏完成 PR 审查识别 Bug、SQL 注入、N1 查询这类安全与性能隐患最后产出一份按优先级排好序的审查报告。说白了它是给 AI 套上了资深工程师的审查纪律。一次审查到底覆盖哪些维度技能入口是 skills/code-reviewer/SKILL.md配套 6 份深度参考文档放在 skills/code-reviewer/references/。一次扫描同时覆盖四个维度正确性逻辑错误、边界遗漏、并发竞态性能N1 查询、缺缓存的昂贵操作可维护性抽象是否过度或缺失、模块划分是否合理测试覆盖关键路径有没有测试、断言写的是行为还是实现安全检查则以OWASP Top 10为基线SQL 注入、鉴权绕过这类问题不会被漏掉。你不需要背清单清单替你干活。五步节奏把注意力花在刀刃上审查的节奏定义在 review-checklist.md 里五步走每步对应一段注意力预算上下文10%看 PR 描述问自己我能一句话说清它要解决什么问题吗。说不清就先问别硬看。结构20%改动是否遵循代码库既有模式新引入的抽象有没有必要细节40%最重的一段。N1 查询、硬编码密钥、注入风险逐行扫出来。测试20%边界情况覆盖了吗断言验证的是行为还是实现细节反馈按 Critical → Major → Minor 三级输出结构化报告。一个值得借鉴的细节发现严重安全问题时立即标注而不是留到报告末尾。这和真实团队安全漏洞优先的沟通习惯是一致的——这一步卡住了先停下来问比闷头审完再返工便宜得多。8 个维度哪些是合并前必查的8 大审查维度的完整清单同样在 review-checklist.md 中按紧迫程度可以分成两层。合并前必查安全用户输入有没有校验SQL 参数化了没有鉴权是强制的吗逻辑空值、边界条件、并发竞态处理了吗性能有没有 N1 查询昂贵操作缓存了吗测试覆盖率够吗Mock 用得恰当吗有空再查设计改动落在正确的模块了吗抽象层级合适吗命名清晰、一致、见名知意吗错误处理错误被正确捕获了吗日志有信息量吗文档公共 API 有文档吗复杂逻辑有注释吗照着这份代码质量清单自问一遍本身就是一次低成本的合并前自查不装技能也能用。一份真实审查报告长什么样产出物遵循 report-template.md 的固定模板共 7 个部分Summary一句话复述 PR 意图加上总体评价Critical Issues合并前必须修复——Bug、安全漏洞、数据丢失Major Issues应当修复——明显性能问题、设计问题、可维护性隐患Minor Issues锦上添花——命名、可读性Positive Feedback点出做得好的具体模式Questions for Author需要作者澄清的问题VerdictApprove / Request Changes / Comment 三选一每个问题按位置 → 现状 → 建议 → 影响四要素写。比如 N1 查询不会只说这里有性能问题而是标注每次请求会多打约 100 次数据库查询——影响是可量化的。严重度划分Critical安全风险、数据丢失、崩溃SQL 注入、鉴权绕过Major明显性能或可维护性问题N1 查询、上帝函数Minor风格、命名类改进审查者最常犯的 3 种表达错误feedback-examples.md 里有一组经典对照本质上是三种表达错误❌ 笼统评价这段代码很混乱 → ✅ 指出具体问题并给拆分方案这个函数同时做了校验和持久化建议拆成validateUser()和saveUser()便于单独测试❌ 模糊指令修复这个查询 → ✅ 说清机制和影响循环里取数据会产生 N1 查询用预加载一次取出即可❌ 空洞要求加测试 → ✅ 给出具体场景缺少 email 已存在场景的测试建议验证返回 409 及错误信息三条黄金法则具体、可执行、建设性。技能还内置了分歧处理原则——如果作者已注释说明某个非常规选择的理由先认可再提替代方案有 linter 的情况下绝不在风格问题上阻塞合并。两条命令装完自然语言触发安装走插件市场两条命令/plugin marketplace add jeffallan/claude-skills /plugin install fullstack-dev-skillsjeffallan装完用自然语言触发即可帮我审查这个 Pull Request检查这段代码的安全和性能问题review code找一下 N1 查询和硬编码密钥常见触发词包括code review、PR review、pull request、code quality没自动激活就点名使用 Code Reviewer 审查这段代码。验证步骤在 QUICKSTART.md 里。单点太窄时选一条路线走到底Code Reviewer 的定位是广域快扫深度专项需要组合拳67 个技能的完整清单与决策树在 SKILLS_GUIDE.md项目级工作流见 docs/WORKFLOW_COMMANDS.md安全加固Secure Code Guardian → Security Reviewer → Test MasterBug 排查Debugging Wizard → 框架专家 → Test Master → Code Reviewer 收尾测试补强审查报告发现测试缺口直接交给 Test Master 补齐AI 审查解决的是清单不会忘、维度不会漏的问题但它不是终审。报告里的 Critical 项逐条核实后再合并Minor 项按团队规范取舍——最终判断权始终在写代码的人手里。【免费下载链接】claude-skills67 Specialized Skills for Full-Stack Developers. Transform Claude Code into your expert pair programmer.项目地址: https://gitcode.com/GitHub_Trending/claud/claude-skills创作声明：本文部分内容由AI辅助生成（AIGC），仅供参考