用途:面向拉取请求(PR)的系统化代码审查,覆盖安全性、性能、可维护性、正确性与测试五个维度,并给出可操作的审查清单与命令行工具用法。
主要能力:
- 查看 PR 差异与详情:gh pr diff <PR_NUMBER>;查看单个文件变更 gh pr diff <PR_NUMBER> -- src/specific-file.ts;查看 PR 元数据与检查 gh pr view <PR_NUMBER> --json title,body,additions,deletions,changedFiles,reviews | jq .;列出变更文件 gh pr diff <PR_NUMBER> --name-only。
- 审查清单:正确性(是否符合 PR 描述、边界情况、错误路径、输入输出契约);安全(输入校验与净化、参数化查询、无硬编码密钥、新端点鉴权、路径穿越、XSS 转义);性能(N+1 查询、分页、索引、不必要重渲染、缓存);可维护性(单一职责、命名、死代码、注释解释原因、与现有模式一致);测试(新代码有测试、覆盖边界、确定性、合理 Mock)。
- 复杂度分析:可用 grep/awk 粗略统计函数长度;Python 可安装 radon(pip install radon)后运行 radon cc src/ -a -nb 与 radon mi src/ -nb。
- 提交审查:gh pr review <PR_NUMBER> --approve/--request-changes/--comment --body "...";也可通过 gh api 在指定文件行内添加评论(需提供 body、path、line、commit_id)。
使用方式:先读 PR 描述理解意图,再看 diff 实现;按安全 > 正确性 > 性能 > 风格的优先级审查;反馈要具体可执行;检查测试覆盖与遗漏的变更。
限制:未提供该技能的安装方式、依赖要求、支持的平台范围及收费信息;示例命令需本地已安装并认证 gh CLI,radon 需另行安装。