一、先定边界：Codex 做预审，不替代人工评审
PR **最怕两种极端：一种是完全靠人工逐行看，效率低且容易漏掉重复问题；另一种是把模型输出当最终结论，忽略业务**和上线风险。更合理的方式，是让 Codex 做预审，把高频、结构化、可枚举的问题先扫出来，再交给评审人判断。
预审的目标不是证明代码一定正确，而是提高评审入口质量。它可以帮助团队快速发现接口字段变化、错误处理缺失、测试覆盖不足、配置文件改动、潜在空值、权限边界模糊等问题。真正涉及业务取舍、安全策略和上线节奏的结论，仍然需要人来确认。
通过灵能API统一接入后，团队可以把 Codex 预审做成固定动作。每个 PR 都先生成一份风险清单，评审人不必从零开始翻差异，而是先看模型整理出的重点，再回到代码里逐项核对。
- Codex 负责提出可疑点，不负责给出最终批准。
- 评审人负责判断业务合理性、上线风险和取舍。
- 预审结果必须能回到具体文件、具体改动和具体复核动作。
二、接入入口先统一：别让**结果来自不同配置
如果团队成员各自使用不同 *ase **L、不同模型、不同配置卡，那么同一个 PR 得到的**结果可能会差异很大。代码**需要可复现，所以第一步是统一接入入口和配置来源。
建议通过 https://www.lnsns.com/ 进入灵能API控制台，确认当前接入说明、模型可用状态和账号资源。团队文档里只保留入口、负责人和配置说明，不要写入完整 Key。这样既方便新成员接入，也能避免敏感信息在**文档中扩散。
统一入口之后，再定义**任务使用的模型路线。普通 PR 可以走标准路线，跨模块重构或关键路径变更再走深度路线。这样可以控制成本，也能让**质量和任务复杂度匹配。
- 同一类 PR 使用同一套**配置。
- 模型和入口变更要记录日期和负责人。
- **脚本里只引用变量名，不**实凭证。
三、准备**输入：只给相关差异，不要把全仓库塞进去
代码**任务最容易失控的地方，是输入范围太大。很多人会直接让 Codex 读取整个项目，然后要求它判断 PR 有没有问题。这样做不仅成本高，输出也容易发散，因为模型会把无关历史代码、未修改模块和当前差异混在一起分析。
更稳的输入方式，是围绕本次 PR 的变更集合组织材料：变更文件列表、核心 diff、相关测试文件、接口或配置说明、必要的业务**。不要追求一次读完所有上下文，而是让 Codex 先看和本次合并直接相关的内容。
PR 预审输入建议
1. 本次变更文件列表
2. 核心 diff 或变更摘要
3. 相关测试文件
4. 接口文档或 sche** 文件
5. 影响模块说明
6. 已知风险或评审关注点
不建议输入：无关历史目录、完整依赖缓存、大型构建产物、真实密钥文件如果 PR 很大，可以先拆成多个**任务。例如接口层、数据层、前端交互、测试覆盖、配置变更分别分析。拆分后每份输出更短，复核人也能按领域快速检查。
- 小 PR 可以一次预审，大 PR 应拆成多个维度。
- 输入必须排除密钥、构建产物和无关缓存。
- 每个**任务都要写清楚范围和不分析的内容。
⚙️ 四、用 CC Switch 固定**配置：让预审结果可复现
PR **需要稳定输出，因此建议在 CC Switch 中单独建立代码**配置卡。不要复用日常聊天配置，也不要和文档生成、CI 预检混在一起。**配置应写清楚用途、模型路线、输出格式、是否允许写入文件以及负责人。
配置卡名称可以采用 Lingneng-Codex-PR-Review 或 Lingneng-Codex-Review-Stan**rd。备注里明确：只用于 PR 风险扫描，默认只读，不直接修改代码，输出必须经过人工确认。如果要允许 Codex 生成修复建议，也应限制为建议片段，而不是自动覆盖文件。
配置卡备注示例
用途：PR 代码预审
权限：只读差异和相关文件
输出：风险清单、修复建议、测试补充建议
禁止：自动合并、自动覆盖核心文件、输出真实密钥
复核：必须由评审人确认
最后验证：2026-09-04- **配置卡要独立命名。
- 默认只读，写入能力需要单独开启。
- 输出格式越固定，越适合放进团队流程。
五、先跑小样本：用一个 PR 验证**模板
不要一上来就把所有 PR 都接入预审。先找一个中等复杂度的历史 PR 做样本：它最好包含接口变化、测试变化和少量业务逻辑调整，但不要涉及生产事故或大规模重构。这样的样本能帮助你判断 Codex 是否能稳定输出有价值的风险点。
小样本测试的重点不是看模型能不能夸代码写得好，而是看它能不能指出具体、可执行、可复核的问题。例如某个字段缺少默认值，某个错误分支没有测试，某个配置变更没有文档说明，某个接口返回结构可能影响前端。
小样本预审提示词
请只读取本次 PR 的变更摘要和相关测试文件。
输出以下内容：
1. 本次改动概览
2. 高风险问题
3. 中风险问题
4. 测试缺口
5. 建议补充的验证动作
6. 需要人工确认的问题
限制：不要修改文件，不要推测未出现的业务规则。- 样本 PR 要有真实差异，但范围不能过大。
- 输出必须指向具体文件或具体改动。
- 如果输出太空泛，先改模板，再扩大范围。
六、风险分级：把问题从“有点不对”变成可处理清单
模型**最怕输出一堆没有优先级的建议。评审人看到十几条“建议优化”，反而不知道先处理哪一条。PR 预审应该把问题分成高、中、低三类，并说明为什么属于这个等级。
高风险通常是可能导致线上错误、权限绕过、数据损坏、兼容性破坏的问题；中风险通常是测试不足、边界处理不完整、错误信息不清晰；低风险则是命名、注释、重复逻辑、可读性优化。分级后，团队可以先处理真正影响合并判断的问题。
风险分级模板
高风险
- 可能影响生产稳定性
- 可能造成数据错误
- 可能破坏兼容性
- 可能引入权限问题
中风险
- 测试覆盖不足
- 错误分支不完整
- 配置变更缺少说明
- 接口字段缺少边界说明
低风险
- 命名不够清晰
- 注释过期
- 重复逻辑可整理
- 文档可读性优化在灵能API接入流程里，可以把风险分级模板写进团队固定提示词。这样不同成员触发预审时，得到的结果结构一致，评审人也能更快形成阅读习惯。
- 高风险必须阻断或人工确认。
- 中风险要明确是否影响本次合并。
- 低风险不应淹没真正重要的问题。
七、修复建议要可落地：不要只写“建议优化”
如果 Codex 只输出“建议优化错误处理”“建议增加测试”，对评审帮助有限。好的修复建议应该包含问题位置、原因解释、改法方向和验证方式。它不一定要直接给出最终代码，但要让开发者知道下一步怎么做。
例如，与其写“建议增强异常处理”，不如写“在 createOrder 的库存扣减后增加失败回滚分支，并补充库存不足、支付失败、重复提交三类测试”。这类建议能直接转成任务，也便于评审人判断是否已经处理。
修复建议输出格式
问题位置：文件 函数或接口
风险说明：为什么可能出问题
建议改法：推荐补充的逻辑或检查
验证方式：需要增加或运行的测试
复核人：建议由哪个角色确认
状态：待处理 / 已处理 / 不采纳并说明原因- 每条建议都要能对应具体动作。
- 不确定的问题要标记待确认。
- 不采纳建议时要记录原因，方便后续复盘。
八、测试缺口单独列：不要混在普通建议里
PR **里，测试缺口经常被写在普通建议中，最后很容易被忽略。建议让 Codex 单独输出“测试缺口”部分，按单元测试、集成测试、回归测试、边界测试分类。这样开发者可以直接对照补用例。
测试缺口不只看有没有测试文件，还要看测试是否覆盖本次变化。例如接口新增了字段，测试只验证状态码不验证字段；权限逻辑改了，测试只覆盖正常用户不覆盖无权限用户；配置变更了，测试没有覆盖默认值和缺失值。
测试缺口输出模板
单元测试：函数边界、异常分支、默认值
接口测试：请求参数、返回字段、错误码
权限测试：未登录、无权限、跨角色访问
回归测试：旧调用方是否兼容
配置测试：变量缺失、默认配置、错误配置如果团队把这部分固定下来，PR 评审就会更稳定。评审人不必临时想“还要测什么”，而是直接看 Codex 提醒的缺口，再结合业务经验补充。
- 测试缺口必须单独成段。
- 新增字段要检查返回断言是否同步更新。
- 权限和异常路径不要只靠人工口头确认。
九、文档和接口变更：**时顺手检查知识库
代码**不应该只看代码。接口路径、请求字段、返回结构、错误码、配置变量、部署步骤一旦变化，相关文档也要同步更新。很多线上沟通问题，根源不是代码错了，而是文档仍然停留在旧版本。
可以让 Codex 在预审结果中单独输出“文档影响”。如果本次 PR 改了对外接口，就提示更新接口文档；如果改了环境变量，就提示更新部署说明；如果改了错误码，就提示更新排查手册；如果改了使用限制，就提示更新联调说明。
文档影响检查
接口变化 -> do**/api/ 对应模块
配置变化 -> do**/deploy/ 或 README
错误码变化 -> do**/run*ook/ 错误排查手册
权限变化 -> do**/security/ 权限说明
发布影响 -> release-notes/ 当期发布记录- 接口变化必须检查文档。
- 配置变化必须检查部署说明。
- 错误码变化必须检查排查手册。
十、控制成本：不是所有 PR 都需要深度**
如果每个 PR 都跑最重的**流程，成本和等待时间都会上升。更合适的方式，是按 PR 类型触发不同**深度。小改动只做轻量摘要和基础风险检查；接口、权限、数据相关 PR 做标准**；跨模块重构、支付、账务、权限等关键路径再做深度**。
通过灵能API统一查看接入和资源状态时，团队可以顺手建立 PR **策略。不要把模型选择做成个人偏好，而要和任务价值绑定。轻量**用于保持节奏，深度**用于保护关键路径。
PR **触发策略
轻量**：文案、样式、注释、小范围重命名
标准**：接口字段、业务逻辑、测试文件、配置文件
深度**：权限、支付、账务、数据迁移、跨模块重构
人工优先：生产事故结论、安全策略、合规边界- 轻量 PR 不跑长上下文深度**。
- 关键路径 PR 必须提高**等级。
- **等级变化要记录原因。
十一、建立复核闭环：模型意见要有处理状态
预审结果如果只是贴在评论区，后续很难知道哪些问题已经处理，哪些被忽略，哪些被确认不采纳。建议给每条模型意见添加处理状态：待处理、已修复、无需处理、待确认。这样 PR **才不会变成一次性输出。
复核闭环的关键，是让每条意见都能被关闭。开发者可以根据建议修改代码，评审人确认是否解决；如果不采纳，也要写明原因。下一次出现类似建议时，团队就能参考历史处理方式，而不是反复争论。
预审意见状态
待处理：需要开发者修改或补充说明
已修复：代码或测试已调整，等待评审确认
无需处理：确认不影响本次合并，并说明原因
待确认：涉及业务规则、权限边界或上线策略，需要负责人判断- 每条意见都要有状态。
- 不采纳必须说明原因。
- 待确认问题不能被普通建议覆盖。
十二、常见误区：别让预审变成新的噪声源
PR 预审如果设计不好，也会变成噪声。最常见的问题包括：输出太长、建议太泛、风险不分级、重复提醒低价值问题、缺少具体文件位置、把猜测写成结论。出现这些问题时，不要急着否定流程，先收紧输入范围和输出模板。
另一个误区，是把模型建议当成评审人的替代品。Codex 可以帮助发现遗漏，但它不知道团队所有历史取舍，也不能承担最终责任。真正有效的流程，是让它减少人工的重复劳动，而不是取消人工判断。
降噪规则
1. 每类风险最多输出 5 条重点
2. 每条建议必须包含位置、原因、建议动作
3. 不确定内容进入待确认
4. 低风险建议折叠到最后
5. 不允许输出与本次 PR 无关的泛泛建议- 输出越长，不一定越有用。
- 泛泛而谈的建议要从模板里压掉。
- 模型负责辅助发现，人工负责最终判断。
✅ 十三、收尾：把 PR 预审变成团队合并前的固定动作
Codex 接入 Claude中转站 后，PR 代码**是一个很适合长期落地的场景。它不需要模型直接控制生产流程，却能在合并前帮助团队整理风险、补齐测试、提醒文档更新，并把**过程变得更结构化。
落地顺序可以很清楚：先通过灵能API统一接入入口，再用 CC Switch 固化**配置卡，然后准备小样本 PR 验证模板，接着建立风险分级、测试缺口、文档影响和复核状态。流程跑顺后，再逐步接入更多项目和更多 PR 类型。
最终目标不是让模型替你点合并，而是让每一次合并前都多一层清晰、可复核、可追踪的检查。这样 Codex 才能从临时助手变成团队工程质量体系的一部分。
- 先做预审，不替代人工评审。
- 先固定模板，再接入更多 PR。
- 先追求可复核，再追求自动化程度。
