Skip to content

feat: allow 'max' reasoning effort#261

Merged
zjwu0522 merged 1 commit into
pin-all-versionsfrom
feat/reasoning-effort-xhigh-max
Jun 12, 2026
Merged

feat: allow 'max' reasoning effort#261
zjwu0522 merged 1 commit into
pin-all-versionsfrom
feat/reasoning-effort-xhigh-max

Conversation

@zjwu0522

Copy link
Copy Markdown
Collaborator

What

  • Register openai/gpt-5.5 (public OpenAI endpoint) in ModelConfig.
  • Allow xhigh and max as --reasoning-effort choices.

Why

We need to benchmark gpt-5.5 at xhigh and Claude Opus 4.x at max reasoning effort. The CLI previously capped the choices at high.

Notes

reasoning_effort is already passed straight through to LiteLLM (completion_kwargs["reasoning_effort"]), so no agent-side changes are required — this PR only widens the CLI allow-list and adds the public model entry. gpt-5's Responses API is handled internally by LiteLLM; no explicit Responses-API code is needed.

🤖 Generated with Claude Code

@xyliugo xyliugo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

gpt-5.5 and the 'xhigh' choice already landed on pin-all-versions (via #254).
This adds the remaining 'max' level so Claude Opus 4.x can run at max
reasoning effort. reasoning_effort is passed straight through to LiteLLM, so
no agent-side changes are required.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@zjwu0522 zjwu0522 force-pushed the feat/reasoning-effort-xhigh-max branch from f3f7155 to db5bc7e Compare June 12, 2026 09:49
@zjwu0522 zjwu0522 changed the title feat: add public gpt-5.5 model and xhigh/max reasoning effort feat: allow 'max' reasoning effort Jun 12, 2026
@zjwu0522 zjwu0522 merged commit 8067263 into pin-all-versions Jun 12, 2026
2 checks passed
@zjwu0522 zjwu0522 deleted the feat/reasoning-effort-xhigh-max branch June 12, 2026 09:50
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