fix(ThinkingMode): enhance model thinking capability checks in getModel function and ThinkingModeSwitch display

Author: NekoWings

entrypoints/sidepanel/components/Chat/index.vue

@@ -95,10 +95,10 @@
               triggerStyle="ghost"
             />
             <div
-              v-if="isThinkingToggleable"
+              v-if="isThinkingToggleable && isModelSupportsThinking"
               class="h-4 w-px bg-[#E5E7EB]"
             />
-            <ThinkingModeSwitch v-if="isThinkingToggleable" />
+            <ThinkingModeSwitch v-if="isThinkingToggleable && isModelSupportsThinking" />
           </div>
           <div
             ref="sendButtonContainerRef"
@@ -128,7 +128,7 @@
 
 <script setup lang="ts">
 import { useElementBounding } from '@vueuse/core'
-import { computed, onBeforeUnmount, onMounted, ref } from 'vue'
+import { computed, onBeforeUnmount, onMounted, ref, toRefs, watch } from 'vue'
 
 import IconSendFill from '@/assets/icons/send-fill.svg?component'
 import IconStop from '@/assets/icons/stop.svg?component'
@@ -140,6 +140,7 @@ import Button from '@/components/ui/Button.vue'
 import { FileGetter } from '@/utils/file'
 import { useI18n } from '@/utils/i18n'
 import { isToggleableThinkingModel } from '@/utils/llm/thinking-models'
+import { useOllamaStatusStore } from '@/utils/pinia-store/store'
 import { setSidepanelStatus } from '@/utils/sidepanel-status'
 import { getUserConfig } from '@/utils/user-config'
 import { classNames } from '@/utils/vue/utils'
@@ -161,6 +162,8 @@ import ThinkingModeSwitch from './ThinkingModeSwitch.vue'
 const inputContainerRef = ref<HTMLDivElement>()
 const sendButtonContainerRef = ref<HTMLDivElement>()
 const { height: inputContainerHeight } = useElementBounding(inputContainerRef)
+const { modelList: ollamaModelList } = toRefs(useOllamaStatusStore())
+const { updateModelList: updateOllamaModelList } = useOllamaStatusStore()
 
 const { t } = useI18n()
 const userInput = ref('')
@@ -172,10 +175,17 @@ defineExpose({
   attachmentSelectorRef,
 })
 
+const updateModelList = async () => {
+  if (endpointType.value === 'ollama') {
+    await updateOllamaModelList()
+  }
+}
+
 const chat = await Chat.getInstance()
 const userConfig = await getUserConfig()
 const contextAttachmentStorage = chat.contextAttachmentStorage
 const currentModel = userConfig.llm.model.toRef()
+const endpointType = userConfig.llm.endpointType.toRef()
 
 initChatSideEffects()
 
@@ -192,6 +202,23 @@ const actionEventHandler = Chat.createActionEventHandler((actionEvent) => {
   }
 })
 
+const modelList = computed(() => {
+  if (endpointType.value === 'ollama') {
+    return ollamaModelList.value
+  }
+  return []
+})
+
+// Check if current model supports thinking
+const isModelSupportsThinking = computed(() => {
+  if (endpointType.value !== 'ollama') return false
+  if (!currentModel.value) return false
+  if (!modelList.value || !Array.isArray(modelList.value)) return false
+
+  const model = modelList.value.find((m) => m.model === currentModel.value)
+  return model?.supportsThinking ?? false
+})
+
 const allowAsk = computed(() => {
   return !chat.isAnswering() && userInput.value.trim().length > 0
 })
@@ -235,9 +262,15 @@ const ask = async () => {
   userInput.value = ''
 }
 
-onMounted(() => {
+// Watch for model list updates to refresh thinking capabilities (following ModelSelector pattern)
+watch([endpointType, currentModel], async () => {
+  await updateModelList()
+})
+
+onMounted(async () => {
   scrollContainerRef.value?.snapToBottom()
   setSidepanelStatus({ loaded: true })
+  updateModelList()
 })
 
 onBeforeUnmount(() => {

utils/llm/models.ts

@@ -5,6 +5,7 @@ import { getUserConfig } from '@/utils/user-config'
 import { ModelNotFoundError } from '../error'
 import { makeCustomFetch } from '../fetch'
 import { middlewares } from './middlewares'
+import { getLocalModelList } from './ollama'
 import { createOllama } from './providers/ollama'
 import { WebLLMChatLanguageModel } from './providers/web-llm/openai-compatible-chat-language-model'
 import { isToggleableThinkingModel } from './thinking-models'
@@ -44,6 +45,7 @@ export async function getModel(options: {
   onLoadingModel?: (prg: ModelLoadingProgressEvent) => void
 }) {
   const userConfig = await getUserConfig()
+  const modelList = await getLocalModelList()
   let model: LanguageModelV1
   const endpointType = userConfig.llm.endpointType.get()
   if (endpointType === 'ollama') {
@@ -53,13 +55,18 @@ export async function getModel(options: {
         if (options.autoThinking) return body
         if (typeof body !== 'string') return body
 
-        // add additional check to avoid errors, eg gamma3 does not support think argument
-        const _isToggleableThinkingModel = isToggleableThinkingModel(options.model)
+        // Models have different thinking capabilities
+        // Edge Case: Qwen3 Instruct does not support think argument even it is toggleable
+        // add additional check to avoid api error
+        const currentModel = userConfig.llm.model.get()
+
+        const supportsToggleThinking = isToggleableThinkingModel(options.model)
+        const supportsThinking = modelList.models.find((m) => m.model === currentModel)?.supportsThinking
 
         const parsedBody = JSON.parse(body)
         return JSON.stringify({
           ...parsedBody,
-          think: _isToggleableThinkingModel ? options.reasoning : undefined,
+          think: supportsThinking && supportsToggleThinking ? options.reasoning : undefined,
         })
       },
     })