Fix start block

Author: Sg312

apps/sim/hooks/use-collaborative-workflow.ts

@@ -765,10 +765,41 @@ export function useCollaborativeWorkflow() {
       // Create subBlocks from the block configuration
       if (blockConfig.subBlocks) {
         blockConfig.subBlocks.forEach((subBlock) => {
+          // Resolve initial value using the same logic as the store
+          let initialValue: unknown = null
+          
+          // Handle function-based values
+          if (typeof subBlock.value === 'function') {
+            try {
+              initialValue = subBlock.value({})
+            } catch (error) {
+              logger.warn('Failed to resolve dynamic sub-block default value', {
+                subBlockId: subBlock.id,
+                error: error instanceof Error ? error.message : String(error),
+              })
+            }
+          } else if (subBlock.defaultValue !== undefined) {
+            initialValue = subBlock.defaultValue
+          } else if (subBlock.type === 'input-format') {
+            // Initialize with a default field structure instead of empty array
+            initialValue = [
+              {
+                id: crypto.randomUUID(),
+                name: '',
+                type: 'string',
+                value: '',
+                collapsed: false,
+              },
+            ]
+          } else if (subBlock.type === 'table') {
+            // Special handling for table types
+            initialValue = []
+          }
+          
           subBlocks[subBlock.id] = {
             id: subBlock.id,
             type: subBlock.type,
-            value: subBlock.defaultValue ?? null,
+            value: initialValue,
           }
         })
       }
@@ -821,6 +852,17 @@ export function useCollaborativeWorkflow() {
         workflowStore.addEdge(autoConnectEdge)
       }
 
+      // Explicitly seed the subblock store to ensure values are available
+      // This ensures the UI can read the initial values immediately
+      if (activeWorkflowId && blockConfig.subBlocks) {
+        blockConfig.subBlocks.forEach((subBlock) => {
+          const subBlockValue = subBlocks[subBlock.id]?.value
+          if (subBlockValue !== undefined) {
+            useSubBlockStore.getState().setValue(id, subBlock.id, subBlockValue)
+          }
+        })
+      }
+
       // Record for undo AFTER adding (pass the autoConnectEdge explicitly)
       undoRedo.recordAddBlock(id, autoConnectEdge)
 

apps/sim/lib/workflows/defaults.ts

@@ -37,8 +37,21 @@ function resolveInitialValue(subBlock: SubBlockConfig): unknown {
     return cloneDefaultValue(subBlock.defaultValue)
   }
 
-  // Ensure structured fields are initialized with empty collections by default
-  if (subBlock.type === 'input-format' || subBlock.type === 'table') {
+  // Ensure structured fields are initialized properly
+  if (subBlock.type === 'input-format') {
+    // Initialize with a default field structure instead of empty array
+    return [
+      {
+        id: crypto.randomUUID(),
+        name: '',
+        type: 'string',
+        value: '',
+        collapsed: false,
+      },
+    ]
+  }
+
+  if (subBlock.type === 'table') {
     return []
   }
 

apps/sim/lib/workflows/trigger-utils.ts

@@ -153,6 +153,13 @@ export function getAllTriggerBlocks(): TriggerInfo[] {
  * Check if a block has trigger capability (contains trigger mode subblocks)
  */
 export function hasTriggerCapability(block: BlockConfig): boolean {
+  // Pure trigger blocks (category === 'triggers') should NOT have trigger mode enabled
+  // They ARE triggers, not blocks that can optionally become triggers
+  if (block.category === 'triggers') {
+    return false
+  }
+  
+  // Only non-trigger blocks that have trigger capability should enable trigger mode
   return (
     (block.triggers?.enabled === true && block.triggers.available.length > 0) ||
     block.subBlocks.some((subBlock) => subBlock.mode === 'trigger')

apps/sim/stores/workflows/workflow/store.ts

@@ -4,6 +4,7 @@ import { devtools } from 'zustand/middleware'
 import { createLogger } from '@/lib/logs/console/logger'
 import { getBlockOutputs } from '@/lib/workflows/block-outputs'
 import { getBlock } from '@/blocks'
+import type { SubBlockConfig } from '@/blocks/types'
 import { useWorkflowRegistry } from '@/stores/workflows/registry/store'
 import { useSubBlockStore } from '@/stores/workflows/subblock/store'
 import {
@@ -21,6 +22,73 @@ import { generateLoopBlocks, generateParallelBlocks } from '@/stores/workflows/w
 
 const logger = createLogger('WorkflowStore')
 
+/**
+ * Creates a deep clone of an initial sub-block value to avoid shared references.
+ *
+ * @param value - The value to clone.
+ * @returns A cloned value suitable for initializing sub-block state.
+ */
+function cloneInitialSubblockValue(value: unknown): unknown {
+  if (Array.isArray(value)) {
+    return value.map((item) => cloneInitialSubblockValue(item))
+  }
+
+  if (value && typeof value === 'object') {
+    return Object.entries(value as Record<string, unknown>).reduce<Record<string, unknown>>(
+      (acc, [key, entry]) => {
+        acc[key] = cloneInitialSubblockValue(entry)
+        return acc
+      },
+      {}
+    )
+  }
+
+  return value ?? null
+}
+
+/**
+ * Resolves the initial value for a sub-block based on its configuration.
+ *
+ * @param config - The sub-block configuration.
+ * @returns The resolved initial value or null when no defaults are defined.
+ */
+function resolveInitialSubblockValue(config: SubBlockConfig): unknown {
+  if (typeof config.value === 'function') {
+    try {
+      const resolved = config.value({})
+      return cloneInitialSubblockValue(resolved)
+    } catch (error) {
+      logger.warn('Failed to resolve dynamic sub-block default value', {
+        subBlockId: config.id,
+        error: error instanceof Error ? error.message : String(error),
+      })
+    }
+  }
+
+  if (config.defaultValue !== undefined) {
+    return cloneInitialSubblockValue(config.defaultValue)
+  }
+
+  if (config.type === 'input-format') {
+    // Initialize with a default field structure instead of empty array
+    return [
+      {
+        id: crypto.randomUUID(),
+        name: '',
+        type: 'string',
+        value: '',
+        collapsed: false,
+      },
+    ]
+  }
+
+  if (config.type === 'table') {
+    return []
+  }
+
+  return null
+}
+
 const initialState = {
   blocks: {},
   edges: [],
@@ -106,12 +174,40 @@ export const useWorkflowStore = create<WorkflowStore>()(
         }
 
         const subBlocks: Record<string, SubBlockState> = {}
+        const subBlockStore = useSubBlockStore.getState()
+        const activeWorkflowId = useWorkflowRegistry.getState().activeWorkflowId
+
         blockConfig.subBlocks.forEach((subBlock) => {
           const subBlockId = subBlock.id
+          const initialValue = resolveInitialSubblockValue(subBlock)
+          const normalizedValue =
+            initialValue !== undefined && initialValue !== null ? initialValue : null
+
           subBlocks[subBlockId] = {
             id: subBlockId,
             type: subBlock.type,
-            value: null,
+            value: normalizedValue as SubBlockState['value'],
+          }
+
+          // Always initialize the subblock store value, even if it's null or empty array
+          // This ensures the editor panel can read the value correctly
+          if (activeWorkflowId) {
+            try {
+              // Use the initial value if available, otherwise use null
+              const valueToStore = initialValue !== undefined ? cloneInitialSubblockValue(initialValue) : null
+              subBlockStore.setValue(id, subBlockId, valueToStore)
+            } catch (error) {
+              logger.warn('Failed to seed sub-block store value during block creation', {
+                blockId: id,
+                subBlockId,
+                error: error instanceof Error ? error.message : String(error),
+              })
+            }
+          } else {
+            logger.warn('Cannot seed sub-block store value: activeWorkflowId not available', {
+              blockId: id,
+              subBlockId,
+            })
           }
         })