feat(model-providers): expose max and ultra for custom Codex models - #2912
feat(model-providers): expose max and ultra for custom Codex models#2912xiaojunonode wants to merge 1 commit into
Conversation
Custom Codex models that do not match the built-in registry used a fixed effort list ending at xhigh. Expose Codex's native max and ultra levels while preserving high as the default selection. Signed-off-by: xiaojunonode <45325630+xiaojunonode@users.noreply.github.com>
33d6725 to
c6f5c00
Compare
|
| Filename | Overview |
|---|---|
| packages/model-providers/src/user-provider.ts | 将自定义 Codex fallback effort 扩展为包含 max 和 ultra,默认 effort 仍为 high,Registry 元数据的优先级未改变。 |
| packages/model-providers/src/tests/user-provider.test.ts | 更新自定义、未注册及 Registry 匹配不明确场景的预期值,覆盖新增 effort 档位。 |
Reviews (1): Last reviewed commit: "feat(model-providers): expose max and ul..." | Re-trigger Greptile
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c6f5c00204
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
@xiaojunonode 👋 这个 PR 还有 1 条 review conversation 没 resolve(packages/model-providers/src/user-provider.ts),auto-review 因此暂时跳过、没法继续审查 / 合并。 如果你已经按评论改完或回应了,请到对应 thread 上点 Resolve conversation;全部 resolve 后,下一轮 auto-review 会自动重新审查这个 PR。 |
|
⏸️ 本 PR 触发 product 维护者确认门。 为自定义 Codex provider 暴露 max/ultra effort 档位,属产品功能扩展。 需维护者在 PR 上 Approve 后方可合并。 讨论 issue 已创建,详见上方链接。 讨论 issue:#2959 |
|
@xiaojunonode 👋 这个 PR 目前与 请在本地 merge 最新的 |
zqchris
left a comment
There was a problem hiding this comment.
这个 PR 把自定义 Codex 的 effort 兜底继续往前开放,目标清楚,也正好接上 #2979 留给产品确认的那一格。合并前请先补一处基线收敛:
请先合最新 main。当前 diff 还停在 #2979 之前,仍是 ['low','medium','high','xhigh'] → 六档;而 #2979 已于 08-18 合并,main 的 CUSTOM_EFFORTS.codex 已经是五档(到 max),注释也写明了「ultra 仍仅限已登记模型」。请把本 PR 收敛成「只增加 ultra 一档」的增量改动,避免覆盖 #2979 已合入的实现和注释。
另外请在 PR 描述里写明产品选择:开放 ultra,还是维持仅限已登记模型。若开放,请说明为什么 ultra 与 max 适用同一套「兼容性由选择者承担」的口径;若维持现状,这份 PR 可以关闭,#2979 已覆盖 #2964 的主体诉求。
误把面板的“拒绝”执行成了“要求修改”,现撤回这条错误 review,改按产品拒绝结论重发。
zqchris
left a comment
There was a problem hiding this comment.
这个 PR 把自定义 Codex 的 effort 兜底继续往前开放,目标清楚,也正好接上 #2979 留给产品确认的那一格。但产品侧这次不采纳对所有自定义 Codex 开放 ultra 的方向。
请关闭本 PR,保留 #2979 的保守口径:max 作为通用兜底,ultra 只对已登记、明确声明能力的模型开放。对未登记或能力不明的自定义端点默认亮出 ultra,用户选中后可能直接得到端点不支持的请求失败,这个失败面不值得由通用 fallback 承担。
如果后续要继续推进,请另提基于显式模型能力声明或唯一匹配元数据的方案;这不是否定你把 effort 配置做成可扩展的实现,而是否定“对所有自定义端点默认开放 ultra”这一产品方向。
|
Closing as accepted. Thanks for the review feedback — model-dependent efforts should not be exposed through the generic fallback. I'll keep this as a local-only customization. |
这次改了什么
摘要
让未命中内置模型目录的自定义 Codex 模型也显示原生
max与ultra思考强度,同时保留high为默认值,不会自动增加 subagent 或 API 消耗。变更类型
feat新功能fix缺陷修复refactor/perf重构或性能优化docs/test/chore文档、测试或工程维护范围
max/ultramax与ultra;默认仍为highUI 变化
不涉及:只调整传给既有选择器的模型能力元数据,没有修改 UI 代码、布局或文案。
怎么验证的
自动验证
手工验证
未启动或重启 Desktop;本次只改模型能力元数据与单元测试。
未执行的验证
无额外手工 UI 验证。
风险
风险分类
max/ultra档位影响与回滚
high,只有用户手动选择max/ultra才提高消耗提交前检查
git commit -s,见 DCO)