实战:让 CC 审查 PR
PR 审查要优先找风险,不是让 CC 夸改动。要按严重程度排序,重点检查越界修改、漏测、兼容性和安全风险。
审查模板
PR 审查的目标不是“让 CC 夸写得不错”,而是让它帮你提前发现合并风险。输入越具体,审查越有价值。
/code-review 请审查这个 PR 或本次 diff。 优先级: 1. 可能导致线上 Bug 的问题。 2. 安全、权限、密钥、隐私风险。 3. 测试和验收遗漏。 4. 越界修改或无关重构。 5. 可读性和维护性建议。 请按严重程度输出,不要把风格建议放在最前面。
审查前给什么信息
- PR 目标:这次改动要解决什么。
- 修改范围:主要改了哪些模块。
- 验收方式:已经跑过哪些检查。
- 风险点:你自己最担心哪里。
- 不看什么:哪些历史问题不在本次范围。
请审查这个 PR。 PR 目标: 【写清楚这次 PR 要解决什么】 本次主要改动: 【列出模块或文件】 已做验收: 【列出测试、构建、截图、人工验证】 我最担心: 【权限 / 数据 / 兼容 / 性能 / UI / 其他】 限制: 1. 只审查本次 diff。 2. 不要把历史问题当成本次阻塞项。 3. 高风险问题放前面。 4. 每条问题都给依据。
好审查结果应该有
- 文件和具体位置。
- 为什么是问题。
- 可能影响什么。
- 建议怎么修。
- 是否阻塞合并。
好的 PR 审查要像风险清单,不像读后感:
| 内容 | 合格表现 |
|---|---|
| 位置 | 能定位到文件或具体改动 |
| 依据 | 说明为什么从 diff 能看出问题 |
| 后果 | 说明不修会影响什么 |
| 级别 | 阻塞、建议修复、可选建议 |
| 处理 | 给出最小修复方向 |
问题应该怎么分级
- 阻塞合并:会导致功能错误、安全风险、数据问题、构建失败。
- 建议修复:不一定马上出事故,但会影响边界场景或维护性。
- 可选建议:命名、文案、轻微风格,不影响合并。
分级时要避免两个极端:一是把所有问题都说成阻塞,导致 Review 没重点;二是把真正的权限、数据、兼容问题写成“建议优化”,导致风险被低估。
可以让 CC 复核分级:
请重新检查刚才的 PR 审查分级。 要求: 1. 阻塞项必须说明不修会造成什么具体风险。 2. 建议项必须说明为什么不阻塞合并。 3. 可选建议不能影响本次交付判断。 4. 如果分级过重或过轻,请调整。
不同 PR 的审查重点
| PR 类型 | 重点 |
|---|---|
| 前端页面 | 响应式、交互、可访问性、截图验收 |
| 后端接口 | 兼容性、错误码、权限、调用方 |
| 数据库 | 迁移、回滚、性能、数据兼容 |
| 配置部署 | 环境变量、密钥、回滚、影响范围 |
| 文档 | 链接、导航、学习路线、SEO |
| 重构 | 行为不变、测试覆盖、diff 范围 |
PR 审查不要做什么
不要让 PR 审查变成二次需求开发:
- 不要要求顺手重构历史代码。
- 不要把本次无关的技术债都塞进阻塞项。
- 不要让 CC 根据猜测补业务规则。
- 不要让 CC 直接批量修所有建议。
- 不要把风格偏好排在真实风险前面。
如果发现审查跑偏,可以这样收回来:
请收窄到本次 PR 的合并风险。 历史问题、风格偏好和无关重构先不要展开。
如果 CC 给了一堆泛泛建议
这些建议太泛泛了,请重新审查。 要求: 1. 只保留能对应到具体 diff 的问题。 2. 每条都说明为什么影响本次 PR。 3. 删除没有证据的建议。 4. 按“阻塞合并 / 建议修复 / 可选建议”分类。 5. 如果没有阻塞问题,请明确说明。
如果要让 CC 修 PR 问题
不要让它一次修所有建议,先修阻塞项。
请只处理审查中的阻塞合并问题。 限制: 1. 不要处理可选建议。 2. 不要重构无关代码。 3. 每个修复都说明对应哪条审查问题。 4. 修复后重新 /verify。 5. 最后输出还剩哪些建议未处理。
修完 PR 问题后复审
请复审刚才修复后的 PR。 要求: 1. 原阻塞问题是否已解决。 2. 修复是否引入新风险。 3. 是否需要重新跑测试或 /verify。 4. 是否还有阻塞合并的问题。 5. 不要重新扩大到可选建议。
合并前最终确认
请做 PR 合并前确认。 请输出: 1. PR 目标是否完成。 2. 阻塞问题是否清零。 3. 已经做过哪些验证。 4. 还有哪些未验证风险。 5. 是否建议合并。 6. 如果不建议合并,必须先处理什么。
适合发给 Reviewer 的总结
最后可以让 CC 帮忙整理一段给 Reviewer 看的摘要:
请把本次 PR 审查结果整理成 Reviewer 摘要。 要求: 1. 先写是否建议合并。 2. 只保留最重要的风险。 3. 标出需要 Reviewer 人工确认的文件或逻辑。 4. 不要写冗长过程。 5. 不要把可选建议放在摘要最前面。
这个摘要适合放进 PR 评论,帮助团队快速抓重点。
PR 审查收尾
请输出 PR 审查最终结论。 格式: 1. 是否建议合并:是 / 否 / 需要补充验证。 2. 阻塞项: 3. 建议修复项: 4. 可选建议: 5. 已验证内容: 6. 未验证风险:
验收结果
- 审查优先找风险。
- 结论按严重程度排序。
- 不会把 PR 审查变成重写功能。
- 能区分阻塞合并、建议修复和可选建议。