PR 上下文
!gh pr view $0 --json title,body,labels,changedFiles
变更概览
!gh pr diff $0 --stat
完整 diff
!gh pr diff $0
审查指令
基于以上 PR 信息,进行代码审查。
审查维度
- Bug 风险:空指针、未处理异常、边界条件、类型错误
- 安全问题:XSS、SQL 注入、敏感信息泄露、未授权访问
- 可维护性:命名清晰度、函数长度、重复代码、过度耦合
- 性能:不必要的渲染、内存泄漏、大循环中的重复计算
严重度标准
- 🔴 Critical:必须修复才能合并。安全漏洞、数据丢失风险、线上必现 bug。
- 🟡 Warning:建议修复。影响可维护性、潜在的边界 bug、性能隐患。
- 🔵 Suggestion:可以考虑。更优雅的写法、微小的性能优化、代码风格。
不要把所有问题都标 Critical——只有真正阻塞合并的才算。
参考知识
根据技术栈,按需读取:
- React 项目 → references/react.md
- Go 项目 → references/go.md
这些参考文档包含该技术栈的常见陷阱和团队最佳实践,审查时作为判断依据。
输出格式
严格按以下格式输出:
问题列表
| # | 文件 | 行号 | 严重度 | 问题描述 | 修复建议 |
|---|
总结
- 总体评分:X/10
- 主要风险:一句话概括最大的问题
- 亮点:值得肯定的做法
- 建议优先级:先修什么,后修什么