把PR审查从凭经验变成按清单：claude-skills 的 Code Reviewer

把PR审查从凭经验变成按清单claude-skills 的 Code Reviewer【免费下载链接】claude-skills67 Specialized Skills for Full-Stack Developers. Transform Claude Code into your expert pair programmer.项目地址: https://gitcode.com/GitHub_Trending/claud/claude-skillsclaude-skills 是面向全栈开发者的 67 个 AI 编程技能合集其中的 Code Reviewer 技能能让 Claude Code 像资深工程师一样审查 Pull Request一次扫描覆盖 Bug、SQL 注入、N1 查询等问题并输出带严重度分级的 AI 审查报告。本文按阶段拆解这套 code review 工作流并讲清安装与触发方式。为什么审查会漏掉风险你可能遇到过PR 提上去两天才等到第一次 review反馈只有风格不好而一个 SQL 注入风险被漏了过去。这不是态度问题是结构性问题。审查维度太多——设计、逻辑、安全、性能、测试、命名资深工程师一次也难全覆盖加上资深工程师时间有限反馈必然滞后。说白了需要把资深工程师的审查方式固化成可执行流程。Code Reviewer 把审查节奏、检查清单、报告格式预先固定PR 来了都走同一套动作。完整定义见 SKILL.md。四个阶段从PR描述到最终报告很多人以为 AI 审查就是一句看看代码这个技能不是。它强制自己按四个阶段推进还配了时间预算。先是上下文不先看代码读 PR 描述问意图能不能一句话说清。说不清就让作者澄清——不理解为什么改审查等于白做。接着看结构改动是否遵循代码库已有模式、新抽象是否必要时间占比约 20%。然后是细节最重占 40%N1 查询、硬编码密钥、注入风险逐项过。这里有个细节发现严重安全问题立即标注不等报告写完和真实团队安全优先的习惯一致。最后看测试边界覆盖没有、断言的是行为还是实现占比 20%。完整清单在 review-checklist.md。它到底查什么四组能力不装技能问代码审查该查什么多数人答不出三四项以上。技能把清单归成四组正确性与设计改动是否落在正确模块抽象层级是否合适空值、边界、并发竞态有没有处理安全与性能输入校验、SQL 参数化、鉴权是否强制、N1 与缓存安全基线是 OWASP Top 10测试质量边界覆盖够不够、Mock 用得是否恰当、断言的是行为还是实现规范与文档命名是否清晰、公共 API 有无文档、错误日志有无信息量对照这份清单自查本身就是一次低成本的质量审计不装技能也能做。审查报告长什么样7 个部分但审查人只关心一个位置结尾的结论。报告就是按这个思路搭的——从 PR 摘要加总体评价到三级问题、具体表扬、给作者的问题最后落在 Approve / Request Changes / Comment 三选一上。每个问题按位置→现状→建议→影响四要素写。比如 Critical 项45 行存在 SQL 注入风险查询用了字符串拼接建议改参数化查询影响是数据泄露Major 项循环取数产生 N1 查询每次请求多约 100 次数据库查询建议预加载一次取出。影响量化了严重程度一眼能判断。三级严重度Critical 是安全风险、数据丢失、崩溃合并前必须修Major 是明显性能或可维护性问题Minor 是风格命名可取舍。完整模板见 report-template.md。安装与触发两条命令加一句话安装推荐插件市场方式两行命令/plugin marketplace add jeffallan/claude-skills /plugin install fullstack-dev-skillsfullstack-dev-skills装完用自然语言触发即可比如检查这段代码的安全和性能问题技能会自动激活触发词包括 code review、PR review、pull request、code quality没激活就点名使用 Code Reviewer 审查这段代码。更多安装方式见 QUICKSTART.md。反馈措辞也有纪律具体、可执行、建设性。这段代码很混乱要写成这个函数同时做校验和持久化建议拆成 validateUser() 和 saveUser()。还有两条规则作者已注释解释某个非常规选择先认可理由再提替代有 linter 就绝不为风格问题阻塞合并。边界在哪它能做什么不能做什么先说定位这是广域快扫一次覆盖正确性、性能、可维护性、测试覆盖但不替代深度专项。 安全加固走 Secure Code Guardian → Security Reviewer → Test Master排查 Bug 走 Debugging Wizard → 框架专家 → Test Master最后用 Code Reviewer 复检报告发现测试缺口交给 Test Master 补齐。完整清单与决策树见 SKILLS_GUIDE.md。注意Critical 项安全漏洞、数据丢失合并前务必逐条核实AI 审查是第二双眼睛判断权在人。下一次 PR 合并前先让 Code Reviewer 跑一遍、对照报告逐条过团队还可以把这套报告格式直接沉淀成 PR 模板——审查不再取决于资深工程师今天心情如何。【免费下载链接】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），仅供参考