Skip to content

feat: rename Instant to None and add XHigh reasoning effort level#99

Open
razorback16 wants to merge 2 commits into
lazy-hq:mainfrom
razorback16:feat/instant-reasoning-effort
Open

feat: rename Instant to None and add XHigh reasoning effort level#99
razorback16 wants to merge 2 commits into
lazy-hq:mainfrom
razorback16:feat/instant-reasoning-effort

Conversation

@razorback16

@razorback16 razorback16 commented Mar 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Rename ReasoningEffort::Instant to ReasoningEffort::None to align with OpenAI's API naming
  • Add ReasoningEffort::XHigh variant for extra-high reasoning effort
  • Fix serde serialization for XHigh (xhigh not x_high)

Provider mappings

ReasoningEffort OpenAI Responses OpenAI Chat Completions Anthropic
None none "none" Thinking disabled
Low low "low" 25% of max_tokens
Medium medium "medium" 50% of max_tokens
High high "high" 75% of max_tokens
XHigh xhigh "xhigh" 90% of max_tokens

Test plan

  • All 98 unit tests pass
  • Verified None against OpenAI gpt-5.4 API
  • Verified XHigh against OpenAI gpt-5.4 API

Add ReasoningEffort::Instant variant for near-realtime responses.
Map to None for OpenAI, "instant" for Chat Completions, and disabled
thinking for Anthropic. Fix Low reasoning effort to map to "low"
instead of "minimal" for OpenAI.
@razorback16
razorback16 force-pushed the feat/instant-reasoning-effort branch from e875bee to 912d5a2 Compare March 1, 2026 18:13
Comment thread src/core/language_model/mod.rs Outdated
@razorback16 razorback16 changed the title feat: add Instant reasoning effort level feat: rename Instant to None and add XHigh reasoning effort level Mar 16, 2026
@razorback16

Copy link
Copy Markdown
Contributor Author

Hi @miridih-bgyu @rijkvanzanten any update on merging this pull request?

@rijkvanzanten

Copy link
Copy Markdown
Contributor

@razorback16 I'm not directly involved in the project nor have maintainer rights here, so I'm afraid I can't help you with that..

@gaki2

gaki2 commented May 5, 2026

Copy link
Copy Markdown
Contributor

Hi @ishaksebsib Can you review this PR?

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.

4 participants