fix default model picking based on user config

Author: icecrasher321

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/combobox/combobox.tsx

@@ -135,7 +135,7 @@ export function ComboBox({
   const value = isPreview ? previewValue : propValue !== undefined ? propValue : storeValue
 
   // Permission-based filtering for model dropdowns
-  const { isProviderAllowed } = usePermissionConfig()
+  const { isProviderAllowed, isLoading: isPermissionLoading } = usePermissionConfig()
 
   // Evaluate static options if provided as a function
   const staticOptions = useMemo(() => {
@@ -269,16 +269,34 @@ export function ComboBox({
     setStoreInitialized(true)
   }, [])
 
-  // Set default value once store is initialized and value is undefined
+  // Check if current value is valid (exists in allowed options)
+  const isValueValid = useMemo(() => {
+    if (value === null || value === undefined) return false
+    return evaluatedOptions.some((opt) => getOptionValue(opt) === value)
+  }, [value, evaluatedOptions, getOptionValue])
+
+  // Set default value once store is initialized and permissions are loaded
+  // Also reset if current value becomes invalid (e.g., provider was blocked)
   useEffect(() => {
-    if (
-      storeInitialized &&
-      (value === null || value === undefined) &&
-      defaultOptionValue !== undefined
-    ) {
+    if (isPermissionLoading) return
+    if (!storeInitialized) return
+    if (defaultOptionValue === undefined) return
+
+    const needsDefault = value === null || value === undefined
+    const needsReset = subBlockId === 'model' && value && !isValueValid
+
+    if (needsDefault || needsReset) {
       setStoreValue(defaultOptionValue)
     }
-  }, [storeInitialized, value, defaultOptionValue, setStoreValue])
+  }, [
+    storeInitialized,
+    value,
+    defaultOptionValue,
+    setStoreValue,
+    isPermissionLoading,
+    subBlockId,
+    isValueValid,
+  ])
 
   // Clear fetched options and hydrated option when dependencies change
   useEffect(() => {

apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/search-modal/search-modal.tsx

@@ -12,6 +12,7 @@ import { getTriggersForSidebar, hasTriggerCapability } from '@/lib/workflows/tri
 import { searchItems } from '@/app/workspace/[workspaceId]/w/components/sidebar/components/search-modal/search-utils'
 import { SIDEBAR_SCROLL_EVENT } from '@/app/workspace/[workspaceId]/w/components/sidebar/sidebar'
 import { getAllBlocks } from '@/blocks'
+import { usePermissionConfig } from '@/hooks/use-permission-config'
 
 interface SearchModalProps {
   open: boolean
@@ -99,12 +100,14 @@ export function SearchModal({
   const router = useRouter()
   const workspaceId = params.workspaceId as string
   const brand = useBrandConfig()
+  const { filterBlocks } = usePermissionConfig()
 
   const blocks = useMemo(() => {
     if (!isOnWorkflowPage) return []
 
     const allBlocks = getAllBlocks()
-    const regularBlocks = allBlocks
+    const filteredAllBlocks = filterBlocks(allBlocks)
+    const regularBlocks = filteredAllBlocks
       .filter(
         (block) => block.type !== 'starter' && !block.hideFromToolbar && block.category === 'blocks'
       )
@@ -138,16 +141,17 @@ export function SearchModal({
       },
     ]
 
-    return [...regularBlocks, ...specialBlocks]
-  }, [isOnWorkflowPage])
+    return [...regularBlocks, ...filterBlocks(specialBlocks)]
+  }, [isOnWorkflowPage, filterBlocks])
 
   const triggers = useMemo(() => {
     if (!isOnWorkflowPage) return []
 
     const allTriggers = getTriggersForSidebar()
+    const filteredTriggers = filterBlocks(allTriggers)
     const priorityOrder = ['Start', 'Schedule', 'Webhook']
 
-    const sortedTriggers = allTriggers.sort((a, b) => {
+    const sortedTriggers = filteredTriggers.sort((a, b) => {
       const aIndex = priorityOrder.indexOf(a.name)
       const bIndex = priorityOrder.indexOf(b.name)
       const aHasPriority = aIndex !== -1
@@ -170,13 +174,14 @@ export function SearchModal({
         config: block,
       })
     )
-  }, [isOnWorkflowPage])
+  }, [isOnWorkflowPage, filterBlocks])
 
   const tools = useMemo(() => {
     if (!isOnWorkflowPage) return []
 
     const allBlocks = getAllBlocks()
-    return allBlocks
+    const filteredAllBlocks = filterBlocks(allBlocks)
+    return filteredAllBlocks
       .filter((block) => block.category === 'tools')
       .map(
         (block): ToolItem => ({
@@ -188,7 +193,7 @@ export function SearchModal({
           type: block.type,
         })
       )
-  }, [isOnWorkflowPage])
+  }, [isOnWorkflowPage, filterBlocks])
 
   const pages = useMemo(
     (): PageItem[] => [