fix(ollama): show full thinking levels for live-discovered models in /think menu

zhangqueping · zhangqueping · commit 8c4f49c75803 · 2026-06-17T10:42:37.000+08:00
When an Ollama model is live-discovered (not in configured catalog), the provider's resolveThinkingProfile received reasoning:undefined and returned the non-reasoning profile with only ['off']. This made the /think inline keyboard show no useful options. Change the ternary to an explicit false check: reasoning===false still yields the off-only profile, but undefined (live-discovered) now defaults to the reasoning profile so /think shows realistic choices for newly discovered models. Fixes #93835
diff --git a/extensions/ollama/provider-policy-api.ts b/extensions/ollama/provider-policy-api.ts
@@ -56,5 +56,12 @@ export function resolveThinkingProfile({
 }: {
   reasoning?: boolean;
 }): ProviderThinkingProfile {
-  return reasoning ? OLLAMA_REASONING_THINKING_PROFILE : OLLAMA_NON_REASONING_THINKING_PROFILE;
+  // When reasoning is not explicitly configured (live-discovered models),
+  // default to the reasoning profile because Ollama models typically
+  // support thinking capabilities even when unknown to the catalog.
+  // Explicit reasoning:false from config still yields the off-only profile.
+  if (reasoning === false) {
+    return OLLAMA_NON_REASONING_THINKING_PROFILE;
+  }
+  return OLLAMA_REASONING_THINKING_PROFILE;
 }
