Repository navigation
feat(codex): advertise Flex tier for eligible routed models - #7533
Transient-Onlooker wants to merge 1 commit into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Deterministic PR hygiene checks passed. |
⏳ DRAFT
What to do
Review readiness checklist
0/4 boxes ticked. This PR stays in draft until every box above is ticked. |
|
Closing per a maintainer product decision for the provider sweep: the Flex tier stays opt-in, so #7534 ("expose opt-in Flex tier for eligible routed models") is the one we keep and review. This PR advertises Flex by default, which that decision rules out. Thank you for both versions; the work continues in #7534. |
Summary
openai/gpt-6-lunamodel based on NanoGPT's advertisedservice_tiersupport; leave unrelated NanoGPT models unclassified./v1/modelsdiscovery and both persisted catalog writers. Withdraw generated Flex metadata when the declaration is removed.Policy and compatibility
canonicalToWire.flex = "flex",foreignCallerTiers = "verbatim", and actual caller-tier forwarding permission.openai-chatproviders,chatServiceTier: trueis needed to carry the caller-selected foreign Flex tier; model capabilities alone do not imply safe forwarding.fastModestill follows the existing global contract. Keep it Auto so a caller-selected Flex tier can pass through. Forced Fast or Default continues to override it.Changes
Verification
dev1025acd4b.service_tier: "flex"on a configured NanoGPT key.Related: #1086, #1886. This is selectable Flex metadata, separate from the previously proposed per-model tier override policy.
References: https://nano-gpt.com/blog/shipped-in-may-2026 · https://github.com/openai/codex/blob/main/codex-rs/tui/src/chatwidget/service_tiers.rs
Review readiness checklist
This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:
Required local validation passed; commands, results, and any full-suite exception are documented.
I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).
I resolved all correct Codex and CodeRabbit findings.
My PR is ready for review.