address comments

Author: icecrasher321

apps/sim/app/workspace/[workspaceId]/w/components/sidebar/hooks/use-folder-expand.ts

@@ -1,6 +1,9 @@
 import { useCallback } from 'react'
 import { useFolderStore } from '@/stores/folders/store'
 
+const toggleFolderExpanded = useFolderStore.getState().toggleExpanded
+const setFolderExpanded = useFolderStore.getState().setExpanded
+
 interface UseFolderExpandProps {
   folderId: string
 }
@@ -14,23 +17,21 @@ interface UseFolderExpandProps {
  */
 export function useFolderExpand({ folderId }: UseFolderExpandProps) {
   const expandedFolders = useFolderStore((state) => state.expandedFolders)
-  const toggleExpanded = useFolderStore((state) => state.toggleExpanded)
-  const setExpanded = useFolderStore((state) => state.setExpanded)
   const isExpanded = expandedFolders.has(folderId)
 
   /**
    * Toggle folder expansion state
    */
   const handleToggleExpanded = useCallback(() => {
-    toggleExpanded(folderId)
-  }, [folderId, toggleExpanded])
+    toggleFolderExpanded(folderId)
+  }, [folderId])
 
   /**
    * Expand the folder (useful when creating items inside)
    */
   const expandFolder = useCallback(() => {
-    setExpanded(folderId, true)
-  }, [folderId, setExpanded])
+    setFolderExpanded(folderId, true)
+  }, [folderId])
 
   /**
    * Handle keyboard navigation (Enter/Space)

apps/sim/hooks/use-reactive-conditions.ts

@@ -3,7 +3,7 @@ import type { CanonicalModeOverrides } from '@/lib/workflows/subblocks/visibilit
 import { buildCanonicalIndex, resolveDependencyValue } from '@/lib/workflows/subblocks/visibility'
 import type { SubBlockConfig } from '@/blocks/types'
 import { useWorkspaceCredential } from '@/hooks/queries/credentials'
-import { useSubBlockStore } from '@/stores/workflows/subblock/store'
+import { EMPTY_BLOCK_SUBBLOCK_VALUES, useSubBlockStore } from '@/stores/workflows/subblock/store'
 
 /**
  * Evaluates reactive conditions for subblocks. Always calls the same hooks
@@ -27,7 +27,8 @@ export function useReactiveConditions(
     useCallback(
       (state) => {
         if (!reactiveCond || !activeWorkflowId) return ''
-        const blockValues = state.workflowValues[activeWorkflowId]?.[blockId] ?? {}
+        const blockValues =
+          state.workflowValues[activeWorkflowId]?.[blockId] ?? EMPTY_BLOCK_SUBBLOCK_VALUES
         for (const field of reactiveCond.watchFields) {
           const val = resolveDependencyValue(
             field,