fix(n8n-routes): omit second arg to generateWorkflowDraft when no triggerContext

Shaw · claude · Shaw · commit 9043683bf47c · 2026-05-03T10:08:39.000-07:00
Passing `undefined` as the second argument changed the call signature
visible to test spies, causing toHaveBeenCalledWith to fail. When there
is no bridgeConversationId the function is now called with one argument only.

Co-Authored-By: Claude Sonnet 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/packages/app-core/src/api/n8n-routes.ts b/packages/app-core/src/api/n8n-routes.ts
@@ -1372,10 +1372,9 @@ async function handleGenerateWorkflow(ctx: N8nRouteContext): Promise<boolean> {
       )
     : undefined;
 
-  const draft = await service.generateWorkflowDraft?.(
-    prompt,
-    triggerContext ? { triggerContext } : undefined,
-  );
+  const draft = triggerContext
+    ? await service.generateWorkflowDraft?.(prompt, { triggerContext })
+    : await service.generateWorkflowDraft?.(prompt);
   if (name?.trim()) {
     (draft as Record<string, unknown>).name = name.trim();
   }

Original file line number	Diff line number	Diff line change
`@@ -1372,10 +1372,9 @@ async function handleGenerateWorkflow(ctx: N8nRouteContext): Promise<boolean> {`
`1372`	`1372`	`)`
`1373`	`1373`	`: undefined;`
`1374`	`1374`
`1375`		`- const draft = await service.generateWorkflowDraft?.(`
`1376`		`- prompt,`
`1377`		`- triggerContext ? { triggerContext } : undefined,`
`1378`		`- );`
	`1375`	`+ const draft = triggerContext`
	`1376`	`+ ? await service.generateWorkflowDraft?.(prompt, { triggerContext })`
	`1377`	`+ : await service.generateWorkflowDraft?.(prompt);`
`1379`	`1378`	`if (name?.trim()) {`
`1380`	`1379`	`(draft as Record<string, unknown>).name = name.trim();`
`1381`	`1380`	`}`