Skip to content

feat(codex): extend GPT-5.6 reasoning and fallback support#1519

Open
AaronZ345 wants to merge 4 commits into
chenhg5:mainfrom
AaronZ345:agent/codex-gpt56-reasoning-efforts
Open

feat(codex): extend GPT-5.6 reasoning and fallback support#1519
AaronZ345 wants to merge 4 commits into
chenhg5:mainfrom
AaronZ345:agent/codex-gpt56-reasoning-efforts

Conversation

@AaronZ345

@AaronZ345 AaronZ345 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • accept none, minimal, max, and ultra as Codex reasoning efforts
  • build /reasoning and /effort usage text from the active agent instead of hardcoding four levels
  • add current GPT-5.x models to the hardcoded fallback used only when catalog, cache, and API discovery return nothing
  • document the eight canonical Codex reasoning levels in config.example.toml

Relation to #1547

#1547 already fixed GPT-5.x discovery from /v1/models by replacing the stale allowlist with isCodexChatModel. This PR leaves that filter unchanged.

The remaining model change here is limited to defaultCodexModels(), the final offline fallback. The duplicate GPT-5.6 API test has also been removed; the upstream codex_model_test.go suite now owns API-filter coverage.

The reasoning changes are separate from model discovery. Codex CLI 0.144 accepts the newer levels, but cc-connect previously normalized values beyond xhigh to an empty string and showed a stale four-level usage message.

Validation

  • go test ./agent/codex -count=1
  • go test ./core -run "TestCmdReasoning_UsageListsAgentEfforts|TestCmdReasoning_SwitchesEffortAndResetsSession" -count=1
  • go test -tags no_web ./cmd/cc-connect ./core ./agent/codex -count=1
  • git diff --check

@AaronZ345
AaronZ345 requested a review from chenhg5 as a code owner July 10, 2026 03:34
@AaronZ345
AaronZ345 force-pushed the agent/codex-gpt56-reasoning-efforts branch 2 times, most recently from 2fa860e to 7f4907b Compare July 14, 2026 03:03
@AaronZ345

Copy link
Copy Markdown
Contributor Author

Hi @chenhg5, I’ve rebased this PR onto the latest main and all five CI checks are green. Could you please take another look and merge it when you have a chance? Thanks!

@AaronZ345
AaronZ345 force-pushed the agent/codex-gpt56-reasoning-efforts branch from 7f4907b to c282a92 Compare July 14, 2026 14:41
@AaronZ345 AaronZ345 changed the title feat(codex): support GPT-5.6 reasoning levels feat(codex): extend GPT-5.6 reasoning and fallback support Jul 16, 2026
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.

1 participant