resolver change to make all var references optional chaining

Author: icecrasher321

apps/sim/executor/variables/resolvers/block.test.ts

@@ -126,16 +126,14 @@ describe('BlockResolver', () => {
       expect(resolver.resolve('<source.items.1.id>', ctx)).toBe(2)
     })
 
-    it.concurrent('should throw error for non-existent path', () => {
+    it.concurrent('should return undefined for non-existent path', () => {
       const workflow = createTestWorkflow([{ id: 'source' }])
       const resolver = new BlockResolver(workflow)
       const ctx = createTestContext('current', {
         source: { existing: 'value' },
       })
 
-      expect(() => resolver.resolve('<source.nonexistent>', ctx)).toThrow(
-        /No value found at path "nonexistent" in block "source"/
-      )
+      expect(resolver.resolve('<source.nonexistent>', ctx)).toBeUndefined()
     })
 
     it.concurrent('should return undefined for non-existent block', () => {
@@ -971,19 +969,17 @@ describe('BlockResolver', () => {
         source: { value: undefined, other: 'exists' },
       })
 
-      expect(() => resolver.resolve('<source.value>', ctx)).toThrow()
+      expect(resolver.resolve('<source.value>', ctx)).toBeUndefined()
     })
 
-    it.concurrent('should handle deeply nested path errors', () => {
+    it.concurrent('should return undefined for deeply nested non-existent path', () => {
       const workflow = createTestWorkflow([{ id: 'source' }])
       const resolver = new BlockResolver(workflow)
       const ctx = createTestContext('current', {
         source: { level1: { level2: {} } },
       })
 
-      expect(() => resolver.resolve('<source.level1.level2.level3>', ctx)).toThrow(
-        /No value found at path "level1.level2.level3"/
-      )
+      expect(resolver.resolve('<source.level1.level2.level3>', ctx)).toBeUndefined()
     })
   })
 })

apps/sim/executor/variables/resolvers/block.ts

@@ -108,11 +108,7 @@ export class BlockResolver implements Resolver {
       }
     }
 
-    // If still undefined, throw error with original path
-    const availableKeys = output && typeof output === 'object' ? Object.keys(output) : []
-    throw new Error(
-      `No value found at path "${pathParts.join('.')}" in block "${blockName}". Available fields: ${availableKeys.join(', ')}`
-    )
+    return undefined
   }
 
   private getBlockOutput(blockId: string, context: ResolutionContext): any {

apps/sim/triggers/lemlist/utils.ts

@@ -85,17 +85,24 @@ const coreOutputs = {
     type: 'string',
     description: 'Lemlist team identifier',
   },
+} as const
+
+/**
+ * Campaign-related fields - only present when activity is part of a campaign
+ * These may be missing for first replies or activities outside campaign context
+ */
+const campaignOutputs = {
   leadId: {
     type: 'string',
-    description: 'Lead identifier',
+    description: 'Lead identifier (only present for campaign activities)',
   },
   campaignId: {
     type: 'string',
-    description: 'Campaign identifier',
+    description: 'Campaign identifier (only present for campaign activities)',
   },
   campaignName: {
     type: 'string',
-    description: 'Campaign name',
+    description: 'Campaign name (only present for campaign activities)',
   },
 } as const
 
@@ -193,6 +200,7 @@ const emailContentOutputs = {
 export function buildEmailSentOutputs(): Record<string, TriggerOutput> {
   return {
     ...coreOutputs,
+    ...campaignOutputs,
     ...leadOutputs,
     ...sequenceOutputs,
     ...senderOutputs,
@@ -206,6 +214,7 @@ export function buildEmailSentOutputs(): Record<string, TriggerOutput> {
 export function buildEmailRepliedOutputs(): Record<string, TriggerOutput> {
   return {
     ...coreOutputs,
+    ...campaignOutputs,
     ...leadOutputs,
     ...sequenceOutputs,
     ...senderOutputs,
@@ -219,6 +228,7 @@ export function buildEmailRepliedOutputs(): Record<string, TriggerOutput> {
 export function buildEmailOpenedOutputs(): Record<string, TriggerOutput> {
   return {
     ...coreOutputs,
+    ...campaignOutputs,
     ...leadOutputs,
     ...sequenceOutputs,
     ...senderOutputs,
@@ -235,6 +245,7 @@ export function buildEmailOpenedOutputs(): Record<string, TriggerOutput> {
 export function buildEmailClickedOutputs(): Record<string, TriggerOutput> {
   return {
     ...coreOutputs,
+    ...campaignOutputs,
     ...leadOutputs,
     ...sequenceOutputs,
     ...senderOutputs,
@@ -255,6 +266,7 @@ export function buildEmailClickedOutputs(): Record<string, TriggerOutput> {
 export function buildEmailBouncedOutputs(): Record<string, TriggerOutput> {
   return {
     ...coreOutputs,
+    ...campaignOutputs,
     ...leadOutputs,
     ...sequenceOutputs,
     ...senderOutputs,
@@ -275,6 +287,7 @@ export function buildEmailBouncedOutputs(): Record<string, TriggerOutput> {
 export function buildLinkedInRepliedOutputs(): Record<string, TriggerOutput> {
   return {
     ...coreOutputs,
+    ...campaignOutputs,
     ...leadOutputs,
     ...sequenceOutputs,
     text: {
@@ -290,6 +303,7 @@ export function buildLinkedInRepliedOutputs(): Record<string, TriggerOutput> {
 export function buildInterestOutputs(): Record<string, TriggerOutput> {
   return {
     ...coreOutputs,
+    ...campaignOutputs,
     ...leadOutputs,
     ...sequenceOutputs,
   } as Record<string, TriggerOutput>
@@ -302,6 +316,7 @@ export function buildInterestOutputs(): Record<string, TriggerOutput> {
 export function buildLemlistOutputs(): Record<string, TriggerOutput> {
   return {
     ...coreOutputs,
+    ...campaignOutputs,
     ...leadOutputs,
     ...sequenceOutputs,
     ...senderOutputs,