一、代码审查Skill的设计目标

在团队协作开发中，代码审查是保障代码质量的核心环节，但传统人工审查面临着时间成本高、标准不统一、容易遗漏细节等诸多挑战。通过创建代码审查Skill，可以将常规化的审查工作自动化，帮助团队更高效地发现潜在问题。

代码审查Skill的设计目标包括以下几个方面：

• 自动化常规代码审查：自动检测常见的代码问题，如逻辑错误、性能瓶颈、安全漏洞等，减少人工逐行检查的重复劳动。
• 减少人工审查负担：让审查者从繁琐的格式化检查中解放出来，专注于高层次的架构设计和业务逻辑合理性评估。
• 标准化审查标准和流程：确保所有Pull Request都按照统一的维度、标准和严重程度分级进行审查，消除审查者个人偏好导致的评价标准差异。
• 快速发现潜在问题：在代码合并之前及时捕获问题，降低缺陷流入生产环境的概率，缩短反馈循环。

二、Skill的prompt设计

prompt是代码审查Skill的核心。一个好的审查prompt需要明确定义审查维度、严重程度分级、输出格式以及团队特定的编码规约引用。以下是一个完整的代码审查Skill定义示例：

审查维度定义

审查维度决定了Skill的关注范围。在实际配置中，建议根据团队和项目的具体情况，选择以下维度的子集：

维度	说明	检查要点
功能正确性	代码逻辑是否按预期工作	边界条件、状态转换、异常流程、算法正确性
性能问题	是否存在性能瓶颈	不必要的循环、内存泄漏、数据库查询优化、缓存策略
安全漏洞	潜在的安全风险	SQL注入、XSS、CSRF、敏感信息泄露、输入验证
代码风格	是否符合编码规范	命名约定、缩进、注释规范、代码组织结构
可维护性	代码是否易于理解	函数长度、职责单一原则、测试覆盖率、文档完整性

严重程度分级

为了让审查结果更加直观，需要定义问题的严重程度分级。不同的团队可以根据实际情况调整分级标准：

级别	标签	定义	处理要求
P0	阻塞	导致功能错误、安全漏洞或性能严重退化	必须修复后才能合并
P1	重要	可能导致潜在问题或明显降低代码质量	建议修复，可讨论替代方案
P2	建议	风格改进、可读性提升、最佳实践推荐	可选择性采纳，不阻塞合并

三、集成Git diff获取变更内容

代码审查的前提是获取准确的代码变更内容。Skill通过集成Git diff能力来实现这一目标，常见的方式包括：

方式一：与Git MCP配合

通过Git MCP（Model Context Protocol）工具直接获取当前分支的diff内容。这种方式无需手动准备文件，Skill可以自动调用Git命令来获取变更集。例如，在prompt中使用MCP工具获取当前分支与目标分支之间的差异：

方式二：读取diff文件

通过{{FILE:diff.txt}}通配符读取预先生成的diff文件。这种方式更加灵活，可以在调用Skill之前对diff进行预处理，例如过滤掉非关键文件、合并多个commit的变更等：

限定审查范围

在prompt中可以明确指定需要重点关注的目录或文件类型，从而缩小审查范围、提升审查精度：

四、审查结果格式化输出

审查结果的呈现方式直接影响开发者的处理效率。一个好的格式化输出应该让问题一目了然，方便开发者快速定位和处理。

严重程度标签

使用emoji标签配合文字，让严重程度在视觉上立刻区分：

• 🔴 阻塞：必须修复的严重问题，不修复则不应合并
• 🟡 重要：应该修复的中等问题，建议在合并前处理
• 🔵 建议：可选的改进建议，不影响合并决策

按文件分类组织

审查结果按文件路径分组，每个文件下列出该文件中发现的所有问题。这种组织方式便于开发者按文件逐一处理：

可操作的修复方案

每个审查发现的问题都应该附带具体的修复建议，而不是仅仅指出问题。这样可以极大降低开发者的修复成本：

五、团队编码规约绑定

每个团队都有自己的编码约定和最佳实践。将这些规约绑定到代码审查Skill中，可以确保审查结果与团队的实际情况保持一致。

在CLAUDE.md中定义编码规范

CLAUDE.md是项目的知识库文件，可以在其中定义团队的编码规约。审查Skill会自动读取CLAUDE.md中的规范内容并应用到审查过程中：

引用外部规约文件

除了CLAUDE.md，审查Skill还可以通过{{FILE:CODING_STANDARDS.md}}的方式引用独立的编码规约文件。这种方式适合规约内容较多、需要单独维护的场景：

自动适配项目代码风格

更高级的做法是让Skill自动分析项目中已有的代码风格——读取项目中已有的配置文件（如.eslintrc、.prettierrc、tsconfig.json等），从中推断出团队的实际编码约定，并以此为标准进行审查：

六、不同语言的审查要点配置

不同的编程语言和框架有不同的最佳实践和常见陷阱。代码审查Skill应该针对具体的技术栈配置相应的审查要点。

语言/框架	重点审查维度	常见问题
TypeScript/JavaScript	类型安全、异步处理、闭包陷阱	any滥用、Promise未处理、内存泄漏
React	组件设计、Hooks规则、性能优化	缺少依赖数组、不必要的重渲染、状态提升不当
Python	异常处理、性能、导入规范	全局变量、循环引用、未关闭的资源
Go	错误处理、并发安全、接口设计	错误被忽略、goroutine泄漏、锁使用不当
Java/Spring	依赖注入、事务管理、异常体系	空指针、事务边界过大、N+1查询

七、Skill的调试与迭代优化

代码审查Skill并非一次配置就一劳永逸。在实际使用过程中，需要根据反馈不断调试和优化。

调试技巧

• 审查样本测试：用已知有问题的代码变更来测试Skill是否能正确发现问题
• 误报分析：记录Skill报告的false positive（误报），分析原因并调整prompt
• 漏报分析：记录遗漏的重要问题，补充对应的审查规则
• 逐步扩大范围：先在单个项目上验证效果，再推广到更多项目

迭代优化路线图