From 19c4b9796d2178f4ef9e1eb598a438d640b660e9 Mon Sep 17 00:00:00 2001 From: Ayush Ojha Date: Fri, 30 Jan 2026 01:20:54 -0800 Subject: [PATCH] fix: remove Ollama from reasoning tag provider check Ollama's OpenAI-compatible API handles reasoning natively via the `reasoning` field in streaming chunks, so there is no need for / tag enforcement. Including "ollama" in isReasoningTagProvider() caused enforceFinalTag to be set for all Ollama models, discarding all output since they don't emit tags. Fixes #2279 Co-Authored-By: Ayush Ojha --- src/utils/provider-utils.ts | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/utils/provider-utils.ts b/src/utils/provider-utils.ts index e29ffe1ab..5a5baffbc 100644 --- a/src/utils/provider-utils.ts +++ b/src/utils/provider-utils.ts @@ -12,11 +12,7 @@ export function isReasoningTagProvider(provider: string | undefined | null): boo const normalized = provider.trim().toLowerCase(); // Check for exact matches or known prefixes/substrings for reasoning providers - if ( - normalized === "ollama" || - normalized === "google-gemini-cli" || - normalized === "google-generative-ai" - ) { + if (normalized === "google-gemini-cli" || normalized === "google-generative-ai") { return true; }