全部模板
🔎编程

审查 Pull Request 的真实风险

优先检查正确性、安全性、回归和缺失测试,而不是堆砌样式意见。

适合:工程团队

提示词模板

请以资深维护者身份审查这个 Pull Request。

改动目标
[说明 PR 应实现什么]

仓库约束
[列出架构规则、兼容性要求和安全边界]

差异 / 文件
[附上或指向改动文件]

审查优先级
1. 正确性缺陷和边界条件错误。
2. 安全、权限、隐私或数据完整性风险。
3. 向后兼容和迁移问题。
4. 并发、错误处理和部分失败行为。
5. 能捕获真实回归但目前缺失的测试。

除非格式或命名会带来具体维护风险,否则不要评论纯样式问题。每个发现都要指出准确文件和最小有效行范围,描述失败场景,并提出范围明确的修正方式。不要为了凑数量虚构问题。

最后给出:
- 合并建议:批准 / 要求修改
- 风险最高的未解决问题
- 合并前应运行的测试

使用前请替换

  • 补充仓库特有的不变量。
  • 把 PR 目标和具体实现分开提供。
  • 只要求可执行的发现。

结果质量检查

  • 每个问题都有具体失败场景。
  • 评论指向精确代码位置。
  • 不把个人偏好当成缺陷。

相关模板