器库yueser.com

检查 PR 是否夹带范围外改动

收录 2026-11-01

把「顺手改的」挑出来,别让评审为无关改动买单。

下面是一个 PR 的描述和改动文件列表。请:1)区分「实现目标所需」和「顺手改的」两类改动;2)指出哪些顺手改动会影响行为(不只是格式/命名);3)给每个无关改动的处理建议(拆出去 / 保留但单独说明 / 撤回);4)指出评审时最容易被跳过、但风险最高的一处;5)给一句可以直接回在 PR 里的评论。不要评价代码风格。
本站原创

中文原文

使用建议

评审 PR 时先看「和这个目标无关的文件」。

可以这样改

  • 要求它先复述需求与约束,确认无误再写代码。
  • 让它给出最小改动 + 回滚方式,而不是整文件重写。

交付物(这次应拿到什么)

  • 可运行的最小改动(含直接可用的代码块)
  • 影响面清单:改了哪些文件、哪些调用点要跟着改

常见错法

  • 一次让它改十个文件,最后分不清哪处改动导致了新问题。
  • 只贴报错不贴代码和版本,拿到无法落地的修法。

输出验收(怎么判断能用)

  • 改动可运行、可回滚,且没顺手改无关代码。
  • 边界条件(空值、超长、并发)有说明或测试。

适用场景与注意

  • 适合:让它先复述需求和约束,再给最小可运行改动。
  • 注意:贴报错时同时给版本号与最小复现,否则它容易给出看似合理但不适配的写法。

同方向的其他提示词