🔎编程
审查 Pull Request 的真实风险
优先检查正确性、安全性、回归和缺失测试,而不是堆砌样式意见。
适合:工程团队
提示词模板
请以资深维护者身份审查这个 Pull Request。 改动目标 [说明 PR 应实现什么] 仓库约束 [列出架构规则、兼容性要求和安全边界] 差异 / 文件 [附上或指向改动文件] 审查优先级 1. 正确性缺陷和边界条件错误。 2. 安全、权限、隐私或数据完整性风险。 3. 向后兼容和迁移问题。 4. 并发、错误处理和部分失败行为。 5. 能捕获真实回归但目前缺失的测试。 除非格式或命名会带来具体维护风险,否则不要评论纯样式问题。每个发现都要指出准确文件和最小有效行范围,描述失败场景,并提出范围明确的修正方式。不要为了凑数量虚构问题。 最后给出: - 合并建议:批准 / 要求修改 - 风险最高的未解决问题 - 合并前应运行的测试
使用前请替换
- 补充仓库特有的不变量。
- 把 PR 目标和具体实现分开提供。
- 只要求可执行的发现。
结果质量检查
- 每个问题都有具体失败场景。
- 评论指向精确代码位置。
- 不把个人偏好当成缺陷。