北京
分享至
很多开发者在收到PR时，第一反应是直接翻代码，看到哪里不顺眼就评论哪里。这种“凭感觉”的Code Review方式，不但效率低，还容易漏掉关键问题。今天，我分享一套可重复执行的4步PR审查流程，用真实的PR作为示例，帮你系统化地审查代码。
前置条件
- 安装Git和GitHub CLI（
gh - Node.js 18+ 和 pnpm
- 目标仓库的访问权限
先把GitHub Issue拆解成可测试的需求点，拿着这份清单去审查PR，才能有的放矢。
gh issue view 825 --repo promptdriven/pddgh pr view 1260 --repo promptdriven/pddIssue #825 的分解结果：
- 核心目标：在AI agent执行前增加checkup检查系统
- 新增
gate命令，运行预检检查 - 检查项必须支持插件式扩展
- 未通过的检查项要阻止agent执行
- 错误信息要包含检查项名称，清晰明确
- 通过
pdd.yaml进行配置
先处理机械性问题，这能帮你发现80%的明显问题。
git clone https://github.com/promptdriven/pdd.gitcd pddgit checkout DianaTao:feat/issue-825-gatepnpm installpnpm checkup # lint、类型检查、测试、构建如果pnpm checkup失败，先修复再进入下一步。不要审查无法运行的代码。
第3步：深入代码审查
现在开始真正的审查。对照第1步分解出的需求清单，逐项检查，而不是通读整个PR。
// lib/gate/checkup.ts — 核心实现export interface CheckupResult {name: string;passed: boolean;message?: string;durationMs: number;export class Gate {private checks: Check[] = [];constructor(private config: GateConfig) {this.registerDefaultChecks();async runAll(): Promise {}重点检查：检查项的注册机制是否清晰、失败时是否返回了包含检查项名称的错误信息、配置读取是否符合pdd.yaml的约定。
第4步：测试变更并修复问题
自动化测试无法覆盖所有场景。手动跑一遍关键路径，验证：
- 检查通过时，agent可以正常执行
- 检查失败时，agent被阻止，并输出明确的错误
- 新增插件的扩展是否生效
- 配置改动后，行为是否符合预期
发现问题后，直接修复或反馈给作者，然后重新运行完整检查。
这套流程看起来简单，但能帮你从“凭感觉”变成“有清单、有证据”地审查PR。你可以直接用它来替换你目前的Code Review方式。
特别声明：以上内容(如有图片或视频亦包括在内)为自媒体平台“网易号”用户上传并发布，本平台仅提供信息存储服务。
Notice: The content above (including the pictures and videos if any) is uploaded and posted by a user of NetEase Hao, which is a social media platform and only provides information storage services.
