软件技术分享

关于电子游戏的方方面面

Codex：在 Cursor 里用 Codex／Agent 做 code review 清单

作者

Agent 报完成且测试绿时，先整支 diff，再查测例／auth／密钥／范围；Cursor 用 /agent-review，Codex 用 /review。

要点

  • Cursor 本地：Agent 做完后跑 Agent Review（输入 /agent-review，或 Source Control 对比主分支；也可提交后自动审）；推前还可本地跑 /review-bugbot、/review-security（AI code review）。证据强度：高。
  • Cursor 学习路径强调：小语义提交；用测试／类型检查／lint 做可验证目标，别把“看起来对”当合并条件（Reviewing and testing）。证据强度：高。
  • 第三方复盘清单优先顺序：整支 diff（含分支点以来提交）→ 测例是否被削弱 → 不可逆／破坏性变更 → auth／校验是否删 → 密钥与注入 → 依赖幻觉 → 范围蔓延（IntentiAI）。证据强度：中高（实践指南，非官方产品文档）。
  • 安全控制可压缩进同一清单：限制工具面、审指令文件、依赖门禁、扫“删掉的护栏”、小 PR、合并前阻断高置信回归（Skylos 12 controls）。证据强度：中高。
  • Codex 侧：本地 /review（对 base／未提交／某次 commit）；GitHub 上 @codex review，可选 @codex security review；仓库用 AGENTS.md 的 ## Code Review Rules 定制（Code review、GitHub）。证据强度：高。产品按钮现名以客户端为准（待核）。
  • AI review 不替代人：清机械风险，架构／产品／部落知识仍归人类；不可逆变更（迁移、删列、force-push、密钥）必须人终审。证据强度：高（Cursor／官方口径）＋实践强调。

正文

场景

Agent 会话收尾：“已修好登录，测试全绿。”你扫一眼它写的总结：三五个文件名，一句“加了校验”。点开 diff 才发现：删了两个失败断言，webhook 验签函数整段没了，还顺手改了 CI 超时。

绿不等于安全。先把整支变更摆到眼前，再让 Cursor Agent Review 或 Codex /review 当第二遍眼睛——清单按“事后难回滚”排序。

可跟做步骤／可抄工作流

证据强度：Cursor／Codex 入口＝官方高；清单条目＝IntentiAI＋Skylos 编译（中高）；团队细则写进 AGENTS／BUGBOT＝可跟做。

1. 先把整支 diff 摆出来（别读 agent 总结）

分支点以来全部看，不看“工作区此刻还没提交的那点”：

git fetch origin
git log --oneline origin/main..HEAD
git diff --stat origin/main...HEAD
git diff origin/main...HEAD -- . ':(exclude)*.lock'
git status --short

先看 --stat。修一个 bug 却动了 40 个文件、CI workflow 和 lockfile，本身就是发现（IntentiAI）。

2. Cursor 里：本地 Agent Review →（可选）Bugbot／Security

Agent 任务结束后：

  1. 在 agent 输入框敲 /agent-review；或打开 Source Control，对主分支跑 Agent Review；也可开“每次 commit 后自动审”。
  2. 推送前需要再严一层：本地 skill /review-bugbot、/review-security。
  3. PR 上用 Bugbot；团队不变量写进 .cursor/BUGBOT.md。看 resolution rate，再决定要不要加评论类别（AI code review）。

同一系统写代码、同一系统审，比外挂 bot 少丢上下文。Bugbot 仍是辅助：人负责产品风险与“我们这边怎么建”。

3. Codex 里：/review 与 @codex review

  • 本地：在 ChatGPT／Codex／IDE 扩展 composer 输入 /review，选 Review against a base branch 或 Review uncommitted changes；它读选定 diff，报优先发现，不改工作树（Code review）。
  • GitHub PR：评论 @codex review；要更偏安全面再 @codex security review（Security Review 为 research preview，待核你的账号是否开放）。
  • 定制：在离代码最近的 AGENTS.md 加 ## Code Review Rules，写后果型约束与 safe path；格式／lint 留给 CI（GitHub）。

Cursor 写、Codex 审，或反过来，都可以；关键是人对照 diff 验收发现，别把 bot 评论当合并许可。

4. 可抄审查清单（按回滚成本排序）

复制到 PR 模板或自检笔记。打勾再点 Merge：

  1. 整支 diff 在眼前：分支点以来的 commits＋--stat；排除纯 lock 噪声后再读。
  2. 测例是否被削弱：删测试文件、skip／.only、断言变松、期望值改成“新输出”、覆盖率门槛下调。Agent 优化目标常是绿跑（IntentiAI）。
  3. 破坏性／不可逆：大表加默认列、非 concurrent 索引、改列类型、删仍有人读的列；脚本里的 DROP／无 WHERE 的 DELETE／git push --force。
  4. Auth／校验是否被删：新路由无鉴权；webhook 不验签；为过 403 删掉的 permission／rate limit／validation——先搜删除行。
  5. 密钥与注入面：凭证进代码／fixture；未信任输入拼进 SQL／HTML／shell／eval／外发 URL；Actions 把 issue／PR 正文插进脚本或 AI 步骤。
  6. 依赖：新 import 是否真在 registry；幻觉包名；跨 major 升级；install 脚本。
  7. 范围蔓延：是否做了你没要的 CI／基建／无关模块；过大就拆。
  8. 配置与护栏：AGENTS.md／MCP／hooks／workflow 权限变更单独审（Skylos）。
  9. 人类终审门：触及支付、权限、数据删除、生产配置、密钥轮换——必须人点头；AI 通过≠可合并。

5. 让提交可审：小语义提交＋可验证目标

Cursor 建议：迭代时可以乱；定稿后让 agent 把历史收成小语义提交，每条一个逻辑变更。可验证目标写进任务：

  • Tests 抓行为回归
  • Type check 抓结构错误
  • Lint 抓风格与模式违规

缺这三样，agent 更容易交“看起来对”的补丁（Reviewing and testing）。安全敏感改动再要求负向测例：未登录、越权、跨租户、坏签名（Skylos 第 9 条思路）。

真实用户诀窍／反例

  • 诀窍：总结当目录，diff 当正文。 Agent 摘要方便导航；对账仍靠 --stat 与删除行。
  • 诀窍：先搜 - 行里的 auth／assert。 削弱常藏在删减里。
  • 诀窍：Cursor 与 Codex 入口记三条。 /agent-review、/review、@codex review；其余按钮名以当前客户端为准。
  • 诀窍：规则写后果，不写口号。 AGENTS.md／.cursor/BUGBOT.md 写“禁止按转化过滤对照 cohort”这类可判定句。
  • 反例：测试绿就点 Merge。 绿可能来自删断言。
  • 反例：把 AI 评论当唯一审查。 写代码的 agent 不宜当唯一审稿人。
  • 反例：巨型“顺便清理”PR。 护栏删除最容易埋在大 diff 里。
  • 反例：只看新增、不看删除。 租户过滤被拿掉时，按 ID 取数的 happy path 仍能过。

适用边界

  • 本稿覆盖：在 Cursor／Codex 工作流里做人机接力的 code review 清单；适合个人与小团队日推。
  • 不覆盖：企业 SSO、Bugbot 计费档位、Security Review 合同条款、某语言专用 SAST 配置。
  • UI 文案、/agent-review／/review 菜单项、GitHub 自动审开关路径若产品改版，以 Cursor 指南 与 Codex Code review 现行页为准（待核）。
  • IntentiAI／Skylos 条目是实践编译，不是 Cursor／OpenAI 官方保证。

【来源与编译】