fix(model-providers): 自定义 Codex 模型放开 max 档 effort - #2979
Conversation
CUSTOM_EFFORTS.codex 与 claude-code 对齐,补上 max 档;默认档保持 high,存量行为不变。Registry 可确认能力的模型仍优先继承目录声明, ultra 仍仅限已登记模型(如 GPT-5.6 Sol)。 未登记模型(如 glm-5.3)此前只能落到四档兜底,UI 中选不到 max。 第三方 Responses 兼容端点是否接受 max 只有端点方/用户知道:大多数 端点接受,不接受的会明确拒绝,用户改选即可,兼容性由选择者承担。 Closes makecindy#2964 Signed-off-by: XuZhanXin <zhanxin.xu@kikitrade.com>
|
| Filename | Overview |
|---|---|
| packages/model-providers/src/user-provider.ts | 将自定义 Codex 模型的 fallback effort 列表扩展至 max;现有类型、解析和 Codex 传递链路均支持该值。 |
| packages/model-providers/src/tests/user-provider.test.ts | 更新四类 fallback/未登记模型场景的预期档位,覆盖新增的 max。 |
Reviews (1): Last reviewed commit: "fix(model-providers): 自定义 Codex 模型 effor..." | Re-trigger Greptile
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 376a0047b9
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@zhanxin-xu 👋 这个 PR 还有 1 条 review conversation 没 resolve(packages/model-providers/src/user-provider.ts),auto-review 因此暂时跳过、没法继续审查 / 合并。 如果你已经按评论改完或回应了,请到对应 thread 上点 Resolve conversation;全部 resolve 后,下一轮 auto-review 会自动重新审查这个 PR。 |
MagicLizi
left a comment
There was a problem hiding this comment.
审查通过:17 行纯功能开放(codex custom efforts 加 max),测试覆盖充分,设计意图已在代码注释文档化,无安全/性能/兼容性问题。
|
自定义 Codex 模型开到五档 effort 是个贴心的补全——用户连了第三方 Responses 端点,不用再因为选项比官方少而多想。设计决策的注释写得清楚,测试也对齐了。🎯 |
这次改了什么
摘要
自定义供应商 Codex 模型的 effort 兜底档位从四档(low/medium/high/xhigh)放开到五档(+
max),与 claude-code 的自定义档位对齐。此前CUSTOM_EFFORTS.codex写死四档,glm-5.3这类未命中 Registry 的自定义模型在 UI 中选不到 max;协议与引擎层(turn/start透传 effort)本就支持,只是被目录档位门控。产品口径:第三方 Responses 兼容端点是否接受
max只有端点方/用户知道——大多数端点接受,不接受的会明确拒绝,用户改选即可,兼容性由选择者承担。默认档保持high,存量用户无感。变更类型
fix缺陷修复范围
ultra、待产品确认)互为替代方案——若维护者采纳逐模型声明或 feat(model-providers): expose max and ultra for custom Codex models #2912 的六档方案,本 PR 可直接关闭packages/model-providers中CUSTOM_EFFORTS.codex补max档及注释更新、user-provider.test.ts对应断言更新ultra档(仍仅限 Registry 已登记模型如 GPT-5.6 Sol)、逐模型声明机制(见 issue 中分析评论的完整方案)、anthropic-responses-bridge的 effort 收敛(那是 CC 引擎对内置 x.ai/codex 端点的既有行为,与 codex 引擎自定义供应商路径无关)UI 变化
efforts驱动自动多出 max。怎么验证的
自动验证
手工验证
不涉及(无运行时行为分支:档位列表由静态表投影,单测已覆盖)。
未执行的验证
max端到端透传断言、GLM Coding Plan 真实端点的xhigh/max实测:本机为受限网络环境,未搭起 fake upstream;issue 分析评论中的验收建议 3/8 留给维护者按需复核(引擎透传链路已有内置 GPT-5.6 Solmax/ultra档位在生产使用佐证)。风险
风险分类
max,请求会被上游拒绝(明确报错,非静默);用户改选xhigh即可,默认档与存量行为不变。影响与回滚
ultra不放行。提交前检查
pnpm check:dco通过)