跳到主要内容

代码审查提示词模板

要可执行的反馈:bug、风险、缺失测试


1) PR 审查(bug + 测试 + 风险)​

你是一名资深工程师,请做代码审查。

输入:
- 目标: [GOAL]
- 已验证方式: [COMMANDS]
- Diff(或变更文件):
[粘贴 DIFF]

输出:
- 按严重程度排序的问题清单
- 缺失的测试
- 潜在回归点
- 建议的后续动作

AI 审查会输出一份结构化反馈:需要把报告中的文件名、行号和代码段与你的本地仓库及最新 diff 逐项核对。对于每一项问题,用可复现的方式描述触发条件(输入数据、调用路径或并发场景),并说明下游系统或终端用户可能观察到的影响;若缺少必要上下文,明确标注为“待确认”。

记住:AI 的反馈是建议与线索,不是最终验收依据。你需要区分三类内容——已验证的检查与结果、建议补充的测试覆盖,以及尚未通过代码或实验确认的判断。想系统了解目标、调用链、验证质量与风险边界的完整工作流,请阅读 AI Code Review 工作流程。

2) 安全专项审查​

请从安全角度审查以下代码。

重点:
- 鉴权/授权
- 注入风险
- 密钥与敏感数据
- 输入校验
- 不安全默认值

代码或 diff:
[粘贴]

3) 性能审查​

请从性能角度审查这次改动。

重点:
- 算法复杂度
- N+1 查询 / 重复调用
- 前端不必要的 re-render
- 缓存机会

代码或 diff:
[粘贴]

4) 可维护性审查​

请从可维护性角度审查这个模块。

重点:
- 命名与可读性
- 关注点分离
- 错误处理与可观测性
- public API 设计

代码:
[粘贴]

5) 测试缺口审查​

基于这次功能改动,列出应该补的测试用例。

功能:
[描述]

改动代码/diff:
[粘贴]

输出:
- 按优先级排列的测试清单
- 每个用例为什么重要

FAQ​

提供什么材料能得到更好的审查?​

给 diff/变更文件 + 目标 + 已验证方式。

如何减少泛泛而谈?​

要求按严重程度排序,并给出文件/行号级别的问题与建议。

审查必须覆盖什么?​

行为回归风险、安全问题与缺失测试。