CX-10 Review / GitHub / PR 工作流：从 App 改动到可合并结果

本篇聚焦 App 主线里的 Review、GitHub 和 PR。

⠀

主要来源：OpenAI Codex App Review、GitHub integration、Codex Cloud / Web、CLI review 官方文档。

---

课程信息

⠀

• 作者：老金

• GitHub：https://github.com/KimYx0207

• 公众号：老金带你玩AI

• X（Twitter）：老金带你玩AI

• 个人博客：https://aiking.dev

• 预计学时：2-3小时

• 难度等级：⭐⭐ 入门级

• 更新日期：2026年5月30日

• 信息来源：OpenAI Codex App Review、GitHub integration、Codex Cloud/Web、CLI review 官方文档

• 前置要求：已完成 CX-01 安装、CX-02 桌面工作流、CX-07 Plugins

---

📚 本课学习目标

⠀

完成本课学习后，你将能够：

⠀

1. 掌握Review工作流：从Codex改代码到Review面板看diff到最终验收的完整流程

2. 读懂App Review面板：逐项检查文件列表、单文件diff、测试结果、行内评论

3. 使用/review命令：用带重点描述的/review发现特定类型问题

4. 管理GitHub Connector：授权正确的仓库范围，控制读写权限

5. 完成App到GitHub PR的全流程：Review → 运行测试 → stage → commit → push → create PR

6. 写好PR描述：包含Summary、Verification、Risk三部分

7. 判断Cloud与App分工：知道什么时候交给Cloud处理PR，什么时候留在App

8. 避免Review常见风险：不让Codex顺手重构、不跳过测试、不改主分支

---

🗺️ 学习路径导航（先看这里！）

⠀

💡 根据你的情况选择学习路径：不用全看！

⠀

路径A：快速上手（⏱️ 15分钟）

⠀

适合人群：想让Codex改代码并提交PR

⠀

只看这些章节：

⠀

✅ 第1-2部分：Review主关口 + 完整改动流程（5分钟）
✅ 第6部分：从App到GitHub PR（5分钟）
✅ 第7部分：PR前验收清单（5分钟）

---

路径B：完整学习（⏱️ 2-3小时）

⠀

适合人群：想系统掌握Review和PR工作流

⠀

学习顺序：从头到尾所有章节

---

1. App Review 是合并前的主关口

⠀

Codex 写完代码后，不要只看它的总结。你应该进入 Review：

⠀

1. 看改了哪些文件。

2. 看每个文件 diff。

3. 确认是否超出任务范围。

4. 确认测试是否运行。

5. 必要时行内评论让 Codex 修。

6. 人工决定是否 stage / commit / push。

⠀

2. 一个完整 App 改动流程

⠀

目标：修复登录页移动端按钮换行问题。
范围：只改 src/pages/login.tsx 和 src/styles/login.css。
验证：运行 npm test -- login 和 npm run lint。
交付：展示 diff 摘要，不要自动提交。

⠀

完成后检查：

⠀

检查

标准

范围

只改目标文件

行为

修复目标问题，没有顺手重构

测试

运行了相关测试

样式

和现有代码风格一致

风险

没有引入密钥、网络、权限变化

⠀

3. App Review 面板里具体看什么

⠀

Review 不是只看“Codex 说完成了”。逐项看：

⠀

区域

要看什么

文件列表

是否出现范围外文件、生成物、锁文件、配置文件

单文件 diff

是否有无关重构、删除、格式化噪音

测试结果

是否真的运行过命令，失败是否解释清楚

行内评论

能否针对具体行要求 Codex 修复

Stage / Commit

是否由你确认提交范围和 message

Push / PR

是否推到正确分支和远端

⠀

如果 Review 中发现问题，优先用行内评论或明确提示让 Codex 修，不要重新开一个含糊的大任务。

⠀

3.1 Review 的严重程度分级

⠀

级别

例子

处理

P0 必修

泄露 secret、删库脚本、绕过权限、改错业务逻辑

立即停止，人工处理

P1 必修

测试失败、范围外核心文件、破坏兼容性

让 Codex 定点修复后重新 Review

P2 建议修

命名不清、重复逻辑、缺少边界测试

看时间和风险决定是否修

P3 可后续

文案、格式、非阻塞优化

记录到后续任务

⠀

不要把所有 review comment 都当成同一优先级。Codex 很容易为了修小建议扩大 diff，真正要拦的是 P0/P1。

⠀

4. /review 的用法

⠀

在 App 或 CLI 中，/review 可以审查当前改动：

⠀

/review

⠀

更好的写法：

⠀

/review 重点检查移动端布局、CSS specificity、是否影响桌面端，以及测试缺口。

⠀

/review 适合发现问题，但最终是否接受仍要看 Review 面板和测试结果。

⠀

5. GitHub Connector 的角色

⠀

GitHub Connector / integration 适合：

⠀

• 读取 issue。

• 读取 PR 上下文。

• 查 CI 状态。

• 评论或更新 PR（取决于权限）。

• 从 App / Cloud 之间接力任务。

⠀

授权前要确认：

⠀

• 访问哪些仓库。

• 是否有写权限。

• 是否能评论 PR。

• 是否能创建分支或提交。

⠀

6. 从 App 到 GitHub PR

⠀

典型流程：

⠀

App thread
  -> Codex 修改本地文件
  -> Review 面板检查
  -> 运行测试
  -> git status / diff
  -> stage
  -> commit
  -> push branch
  -> create PR

⠀

提交信息建议让 Codex 先草拟，但由你确认：

⠀

根据当前 diff 写一个简短 commit message。不要执行 git commit。

⠀

6.1 提交前让 Codex 做“反向解释”

⠀

在真正提交前，让 Codex 解释 diff 而不是继续写：

⠀

请基于当前 diff 反向解释：每个文件为什么必须改、对应哪个需求、验证证据是什么、还有什么没覆盖。不要修改文件。

⠀

如果它解释不清某个文件为什么改，通常说明这个文件要么是范围外改动，要么需要你重新审。

⠀

7. PR 前验收清单

⠀

创建 PR 前确认：

⠀

项

标准

分支

不是直接在 main 上提交

Diff

没有范围外改动

测试

至少运行相关测试或说明不能运行原因

描述

Summary / Verification / Risk 写清楚

Secret

没有 .env、token、账号信息

连接器权限

GitHub 写权限只用于目标 repo / PR

⠀

8. 什么时候用 Cloud / Web 处理 PR

⠀

适合 Cloud / Web：

⠀

• 远程仓库任务，不需要本机环境。

• 长时间跑的修复。

• PR 评论和 CI 反复修。

• 你不想占用本机 App 线程。

⠀

不适合：

⠀

• 依赖本机私有服务。

• 需要本地 GUI / 本机文件。

• 需要你频繁交互判断的小任务。

⠀

9. CLI review 是辅助

⠀

CLI 可以在 CI 或终端中做补充：

⠀

codex review
codex review --help

⠀

适合：

⠀

• CI 中自动审查 PR。

• 终端里快速检查当前 diff。

• 和 App Review 结果交叉验证。

⠀

不要用 CLI review 取代 App Review 面板。

⠀

10. PR 描述模板

⠀

## Summary
- 修复登录页移动端按钮换行
- 限制按钮最小宽度，避免文本溢出

## Verification
- npm test -- login
- npm run lint

## Risk
- 仅影响登录页样式
- 已检查桌面断点

⠀

11. 常见风险

⠀

风险

防法

Codex 顺手重构

任务里写清范围

未运行测试

明确验证命令

改了主分支

使用 worktree / feature branch

PR 太大

拆任务

CI 失败自动乱修

限定失败项和文件范围

⠀

常见问题

⠀

Q1：Codex 可以自动提交吗？

⠀

技术上可以，但学习阶段不建议。先让它展示 diff 和 commit message，再由你确认。

⠀

Q2：App Review 和 GitHub PR Review 哪个重要？

⠀

都重要。App Review 是本地合并前检查；GitHub PR Review 是团队协作关口。

⠀

Q3：Cloud 修 PR 后怎么回到 App？

⠀

看当前 App / Cloud 的 handoff 能力和 Git 分支状态。可靠做法是拉取远端分支，在 App 中继续 Review。

---

📝 总结与检查清单

⠀

完成本课后，请确认以下所有项：

⠀

⠀

如果以上全部勾选，恭喜你掌握Codex Review与PR工作流！

---

附录

⠀

A. PR描述模板

⠀

## Summary
- [改动概述]

## Verification
- [验证命令]

## Risk
- [风险评估]

⠀

B. Review面板检查清单

⠀

区域

要看什么

文件列表

是否有范围外文件

单文件diff

是否有无关改动

测试结果

是否真的运行了命令

行内评论

能否针对性修复

Stage/Commit

是否由你确认

⠀

C. 推荐学习资源

⠀

• Codex App Review 官方文档：https://developers.openai.com/codex/app/review

• 本系列上一篇：CX-09 Automations

• 本系列下一篇：CX-11 Web / Cloud 辅助

---

课程制作：老金
最后更新：2026年5月30日
许可：本课程采用CC BY-NC-SA 4.0许可

---

下一步

⠀

下一篇：CX-11 Web / Cloud：App 的云端辅助路径。