Skip to content

fix(codex): honor the ultra alias set for parenthesized suffixes - #15710

Merged
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.52from
HouMinXi:fix/codex-parenthesized-ultra
Oct 7, 2026
Merged

diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.52from
HouMinXi:fix/codex-parenthesized-ultra

Conversation

@HouMinXi

@HouMinXi HouMinXi commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Summary

model(ultra) was checked against CODEX_MAX_ALIAS_MODELS no matter which effort the parentheses captured. A max-only model such as gpt-6-luna therefore accepted (ultra), while the hyphenated gpt-6-luna-ultra was already rejected. Both spellings now pick the alias set from the captured effort.

Test

gpt-6-luna(ultra) and gpt-6-luna-ultra stay unresolved. gpt-6-luna(max) still splits, because luna is in the max set.

Pre-existing: the parenthesized form `model(ultra)` was validated against
CODEX_MAX_ALIAS_MODELS regardless of the captured effort, so a max-only
model like gpt-6-luna accepted `(ultra)` while the hyphenated
`gpt-6-luna-ultra` was correctly rejected. Select the alias set by the
captured effort value for both syntaxes.

Signed-off-by: Minxi Hou <houminxi@gmail.com>
@HouMinXi
HouMinXi requested a review from diegosouzapw as a code owner October 7, 2026 10:18
@diegosouzapw
diegosouzapw merged commit 81cd353 into diegosouzapw:release/v3.8.52 Oct 7, 2026
27 of 51 checks passed
diegosouzapw added a commit that referenced this pull request Oct 8, 2026
Drains 9 v3.8.52 tip unit reds: 1 production fix (combo family-fallback skipped for request-scoped refusals, regression of #13603) + 8 test alignments to intentional contract changes (#15301 #15132 #15035 #15478 #15310 #15710 #15487), each annotated. Validated on a combined board with #15902 after clean npm ci: 32 previously-red files 327/329 (remaining 2 = quality-rail-gate-membership owned by #15682, and route-namespaces fixed in #15902), typecheck/open-sse/api typecheck, lint, 22 gates.
@HouMinXi
HouMinXi deleted the fix/codex-parenthesized-ultra branch October 10, 2026 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants