用途
该技能用于对代码变更进行代码审查,遵循资深开发者的标准,关注质量、安全与最佳实践。它面向需要在合并前系统检查代码的场景,帮助发现 SOLID 原则违规、安全隐患、性能问题、TypeScript 类型问题以及测试覆盖不足。
主要能力
审查过程分为三个阶段:
-
收集上下文:如果提供了 PR 编号,运行
gh pr view $ARGUMENTS --json files,body;如果提供了文件路径,则读取该文件;如果什么都没提供,则运行git diff查看未暂存的变更。 -
分析变更:检查 SOLID 原则违规、查找安全问题、识别性能隐患、审查 TypeScript 类型、检查测试覆盖率。
-
生成报告:输出结构化的审查报告。
审查清单
代码质量
- 清晰、描述性的命名
- 函数不超过 50 行
- 无代码重复
- 正确的错误处理
TypeScript
- 无 any 类型
- 显式返回类型
- 正确的空值处理
React 模式
- 遵循 Hooks 规则
- 正确的依赖数组
- 无不必要的重渲染
安全
- 输入校验
- 无 XSS 漏洞
- 无暴露的密钥
使用方式
该技能通过参数 $ARGUMENTS 接收输入,可传入 PR 编号或文件路径。若未提供参数,则默认审查未暂存的 git 变更。输出格式为 Markdown 报告,包含以下部分:
- Code Review Summary
- Overview:变更的简要描述
- Strengths:做得好的地方
- Issues:分为 Critical(合并前必须修复)和 Suggestions(建议改进)
- Security:任何安全关注点
适用场景
适用于在代码合并前进行系统化审查,尤其适合使用 React 和 TypeScript 的项目。可用于审查 PR、单个文件或本地未提交的变更。
限制
资料中未提供关于该技能的具体安装方式、依赖环境、收费模式或平台支持的信息。审查效果可能依赖于底层 AI 模型的能力。