From c8b5cdf7b225e5064e530adfba0b6cf9ffe89bfe Mon Sep 17 00:00:00 2001 From: Chris Estreich Date: Fri, 25 Apr 2025 15:24:03 -0700 Subject: [PATCH] Omit reasoning params for non-reasoning models (#2932) --- .changeset/khaki-months-float.md | 5 +++++ src/api/providers/openrouter.ts | 3 ++- 2 files changed, 7 insertions(+), 1 deletion(-) create mode 100644 .changeset/khaki-months-float.md diff --git a/.changeset/khaki-months-float.md b/.changeset/khaki-months-float.md new file mode 100644 index 0000000000..0fdb66921c --- /dev/null +++ b/.changeset/khaki-months-float.md @@ -0,0 +1,5 @@ +--- +"roo-cline": patch +--- + +Omit reasoning params for non-reasoning models diff --git a/src/api/providers/openrouter.ts b/src/api/providers/openrouter.ts index ed19c8496e..efe659e145 100644 --- a/src/api/providers/openrouter.ts +++ b/src/api/providers/openrouter.ts @@ -9,6 +9,7 @@ import { openRouterDefaultModelInfo, PROMPT_CACHING_MODELS, OPTIONAL_PROMPT_CACHING_MODELS, + REASONING_MODELS, } from "../../shared/api" import { convertToOpenAiMessages } from "../transform/openai-format" import { ApiStreamChunk } from "../transform/stream" @@ -147,7 +148,7 @@ export class OpenRouterHandler extends BaseProvider implements SingleCompletionH }), // This way, the transforms field will only be included in the parameters when openRouterUseMiddleOutTransform is true. ...((this.options.openRouterUseMiddleOutTransform ?? true) && { transforms: ["middle-out"] }), - ...(reasoningEffort && { reasoning: { effort: reasoningEffort } }), + ...(REASONING_MODELS.has(modelId) && reasoningEffort && { reasoning: { effort: reasoningEffort } }), } const stream = await this.client.chat.completions.create(completionParams)