From ffff38be2cfc2cf1d8b9cc6ab8c32225be27d9c3 Mon Sep 17 00:00:00 2001 From: Chris Estreich Date: Thu, 4 Dec 2025 10:52:47 -0800 Subject: [PATCH] Always enabled reasoning for models that require it (#9836) --- src/api/transform/reasoning.ts | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/src/api/transform/reasoning.ts b/src/api/transform/reasoning.ts index 07b3f80333..f494062569 100644 --- a/src/api/transform/reasoning.ts +++ b/src/api/transform/reasoning.ts @@ -52,7 +52,18 @@ export const getRooReasoning = ({ settings, }: GetModelReasoningOptions): RooReasoningParams | undefined => { // Check if model supports reasoning effort - if (!model.supportsReasoningEffort) return undefined + if (!model.supportsReasoningEffort) { + return undefined + } + + if (model.requiredReasoningEffort) { + // Honor the provided effort if it's valid, otherwise let the model choose. + if (reasoningEffort && reasoningEffort !== "disable" && reasoningEffort !== "minimal") { + return { enabled: true, effort: reasoningEffort } + } else { + return { enabled: true } + } + } // Explicit off switch from settings: always send disabled for back-compat and to // prevent automatic reasoning when the toggle is turned off.