fix(execution): validate child call chain instead of parent chain

waleedlatif1 · claude · waleedlatif1 · commit 27e956008f78 · 2026-02-23T15:05:14.000-08:00
Validate childCallChain (after appending current workflow ID) rather
than ctx.callChain (parent). Prevents an off-by-one where a chain at
depth 10 could still spawn an 11th workflow.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/apps/sim/executor/handlers/workflow/workflow-handler.ts b/apps/sim/executor/handlers/workflow/workflow-handler.ts
@@ -169,7 +169,7 @@ export class WorkflowBlockHandler implements BlockHandler {
       }
 
       const childCallChain = buildNextCallChain(ctx.callChain || [], workflowId)
-      const depthError = validateCallChain(ctx.callChain || [])
+      const depthError = validateCallChain(childCallChain)
       if (depthError) {
         throw new ChildWorkflowError({
           message: depthError,

Original file line number	Diff line number	Diff line change
`@@ -169,7 +169,7 @@ export class WorkflowBlockHandler implements BlockHandler {`
`169`	`169`	`}`
`170`	`170`
`171`	`171`	`const childCallChain = buildNextCallChain(ctx.callChain \|\| [], workflowId)`
`172`		`- const depthError = validateCallChain(ctx.callChain \|\| [])`
	`172`	`+ const depthError = validateCallChain(childCallChain)`
`173`	`173`	`if (depthError) {`
`174`	`174`	`throw new ChildWorkflowError({`
`175`	`175`	`message: depthError,`