Skip to content
This repository was archived by the owner on Jun 3, 2026. It is now read-only.

Add reasoning_effort field to LLMClientCompletionRequest#2171

Closed
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1747186409-add-reasoning-effort-field
Closed

Add reasoning_effort field to LLMClientCompletionRequest#2171
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1747186409-add-reasoning-effort-field

Conversation

@devin-ai-integration

Copy link
Copy Markdown

Add reasoning_effort field to LLMClientCompletionRequest

This PR adds a new field called reasoning_effort to the LLMClientCompletionRequest struct, which can be set to Low, Medium, or High. The field is implemented as an enum ReasoningEffort with these three variants.

Changes

  • Added ReasoningEffort enum with Low, Medium, and High variants
  • Added reasoning_effort: Option<ReasoningEffort> field to LLMClientCompletionRequest struct
  • Added accessor methods set_reasoning_effort() and reasoning_effort() to the struct
  • Updated OpenAI client to use this field when making API requests
  • Updated OpenAICompatibleClient to support this field (which is used by Groq client)

Testing

  • Verified changes compile with cargo check

Link to Devin run: https://app.devin.ai/sessions/6a150dd326c541b7b99bf113e9ff6c5a
Requested by: Sandeep Kumar Pani (skcd@codestory.ai)

…penAI and Groq clients

Co-Authored-By: Sandeep Kumar Pani <skcd@codestory.ai>
@devin-ai-integration

Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration

Copy link
Copy Markdown
Author

Closing due to inactivity for more than 7 days.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants