romtuck/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: remove Ollama from reasoning tag provider check

Browse Source 
Ollama's OpenAI-compatible API handles reasoning natively via the
`reasoning` field in streaming chunks, so there is no need for
<think>/<final> tag enforcement. Including "ollama" in
isReasoningTagProvider() caused enforceFinalTag to be set for all
Ollama models, discarding all output since they don't emit <final>
tags.

Fixes #2279

Co-Authored-By: Ayush Ojha <ayushozha@outlook.com>
This commit is contained in:
Ayush Ojha 2026-01-30 01:20:54 -08:00
parent 6af205a13a
commit 19c4b9796d
1 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/utils/provider-utils.ts
View File

@ -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;
}