fix: Auto select model only for active provider

web-app/src/containers/DropdownModelProvider.tsx

@@ -35,25 +35,25 @@
 
 // Helper functions for localStorage
 const getLastUsedModel = (): { provider: string; model: string } | null => {
   try {
     const stored = localStorage.getItem(localStorageKey.lastUsedModel)
     return stored ? JSON.parse(stored) : null
   } catch (error) {
     console.debug('Failed to get last used model from localStorage:', error)
     return null
   }
 }
 
 const setLastUsedModel = (provider: string, model: string) => {
   try {
     localStorage.setItem(
       localStorageKey.lastUsedModel,
       JSON.stringify({ provider, model })
     )
   } catch (error) {
     console.debug('Failed to set last used model in localStorage:', error)
   }
 }
 
 const DropdownModelProvider = ({
   model,
@@ -78,43 +78,39 @@
   const [searchValue, setSearchValue] = useState('')
   const searchInputRef = useRef<HTMLInputElement>(null)
 
+  // Helper function to check if a model exists in providers
+  const checkModelExists = useCallback((providerName: string, modelId: string) => {
+    const provider = providers.find(
+      (p) => p.provider === providerName && p.active
+    )
+    return provider?.models.find((m) => m.id === modelId)
+  }, [providers])
+
   // Initialize model provider only once
   useEffect(() => {
     // Auto select model when existing thread is passed
     if (model) {
       selectModelProvider(model?.provider as string, model?.id as string)
+      if (!checkModelExists(model.provider, model.id)) {
+        selectModelProvider('', '')
+      }
     } else if (useLastUsedModel) {
       // Try to use last used model only when explicitly requested (for new chat)
       const lastUsed = getLastUsedModel()
-      if (lastUsed) {
-        // Verify the last used model still exists
-        const provider = providers.find(
-          (p) => p.provider === lastUsed.provider && p.active
-        )
-        const modelExists = provider?.models.find(
-          (m) => m.id === lastUsed.model
-        )
-
-        if (provider && modelExists) {
-          selectModelProvider(lastUsed.provider, lastUsed.model)
-        } else {
-          // Fallback to default model if last used model no longer exists
-          selectModelProvider('llamacpp', 'llama3.2:3b')
-        }
+      if (lastUsed && checkModelExists(lastUsed.provider, lastUsed.model)) {
+        selectModelProvider(lastUsed.provider, lastUsed.model)
       } else {
-        // default model, we should add from setting
-        selectModelProvider('llamacpp', 'llama3.2:3b')
+        // Fallback to default model if last used model no longer exists
+        selectModelProvider('', '')
       }
-    } else {
-      // default model for non-new-chat contexts
-      selectModelProvider('llamacpp', 'llama3.2:3b')
     }
   }, [
     model,
     selectModelProvider,
     updateCurrentThreadModel,
     providers,
     useLastUsedModel,
+    checkModelExists,
   ])
 
   // Update display model when selection changes
@@ -128,21 +124,21 @@
 
   // Reset search value when dropdown closes
   const onOpenChange = useCallback((open: boolean) => {
     setOpen(open)
     if (!open) {
       requestAnimationFrame(() => setSearchValue(''))
     } else {
       // Focus search input when opening
       setTimeout(() => {
         searchInputRef.current?.focus()
       }, 100)
     }
   }, [])
 
   // Clear search and focus input
   const onClearSearch = useCallback(() => {
     setSearchValue('')
     searchInputRef.current?.focus()
   }, [])
 
   // Create searchable items from all models
@@ -152,15 +148,17 @@
     providers.forEach((provider) => {
       if (!provider.active) return
 
       provider.models.forEach((modelItem) => {
         // Skip models that require API key but don't have one (except llamacpp)
         if (
           provider &&
           predefinedProviders.some((e) =>
             e.provider.includes(provider.provider)
-          ) && provider.provider !== 'llamacpp' && !provider.api_key?.length
+          ) &&
+          provider.provider !== 'llamacpp' &&
+          !provider.api_key?.length
         )
-            return
+          return
 
         const capabilities = modelItem.capabilities || []
         const capabilitiesString = capabilities.join(' ')