跳到主要内容

代码审查提示词模板

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


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

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

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

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

2) 安全专项审查

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

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

代码或 diff:
[粘贴]

3) 性能审查

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

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

代码或 diff:
[粘贴]

4) 可维护性审查

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

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

代码:
[粘贴]

5) 测试缺口审查

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

功能:
[描述]

改动代码/diff:
[粘贴]

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

FAQ

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

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

如何减少泛泛而谈?

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

审查必须覆盖什么?

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