@synergenius/flow-weaver 0.20.6 → 0.20.7

package/dist/cli/flow-weaver.mjs

@@ -9671,7 +9671,7 @@ var VERSION;
 var init_generated_version = __esm({
   "src/generated-version.ts"() {
     "use strict";
-    VERSION = "0.20.6";
+    VERSION = "0.20.7";
   }
 });
 
@@ -10970,7 +10970,21 @@ function generateControlFlowWithExecutionContext(workflow, nodeTypes, isAsync2,
       region.failureNodes.delete(instanceId);
     });
   });
-  const nodesPromotedFromBranches = /* @__PURE__ */ new Set();
+  const nodesInBothBranches = /* @__PURE__ */ new Set();
+  branchRegions.forEach((region) => {
+    region.successNodes.forEach((nodeId) => {
+      if (region.failureNodes.has(nodeId)) {
+        nodesInBothBranches.add(nodeId);
+      }
+    });
+  });
+  branchRegions.forEach((region) => {
+    nodesInBothBranches.forEach((nodeId) => {
+      region.successNodes.delete(nodeId);
+      region.failureNodes.delete(nodeId);
+    });
+  });
+  const nodesPromotedFromBranches = new Set(nodesInBothBranches);
   branchRegions.forEach((region, branchNodeId) => {
     const allBranchNodes = /* @__PURE__ */ new Set([...region.successNodes, ...region.failureNodes]);
     allBranchNodes.forEach((nodeId) => {
@@ -11134,7 +11148,7 @@ function generateControlFlowWithExecutionContext(workflow, nodeTypes, isAsync2,
             }
           });
           if (stepSourceConditions.length > 0) {
-            const condition = stepSourceConditions.join(" && ");
+            const condition = stepSourceConditions.length === 1 ? stepSourceConditions[0] : stepSourceConditions.join(" || ");
             lines.push(`  if (${condition}) {`);
             chainIndent = "    ";
             chainNeedsClose = true;
@@ -11174,7 +11188,7 @@ function generateControlFlowWithExecutionContext(workflow, nodeTypes, isAsync2,
           }
         });
         if (stepSourceConditions.length > 0) {
-          const condition = stepSourceConditions.join(" && ");
+          const condition = stepSourceConditions.length === 1 ? stepSourceConditions[0] : stepSourceConditions.join(" || ");
           lines.push(`  if (${condition}) {`);
           branchIndent = "    ";
           branchNeedsClose = true;
@@ -46770,7 +46784,7 @@ function aggregateResults(
 
 /**
  * @flowWeaver workflow
- * @node iterator forEachItem [position: -90 0] [color: "purple"] [icon: "repeat"]
+ * @node iterator forEachItem [position: -90 0] [color: "purple"] [icon: "repeat"] [suppress: "DESIGN_SCOPE_NO_FAILURE_EXIT"]
  * @node processor processItem iterator.processItem [color: "blue"] [icon: "settings"]
  * @node aggregator aggregateResults [position: 270 0] [color: "teal"] [icon: "inventory"]
  * @position Start -450 0
@@ -46785,7 +46799,6 @@ function aggregateResults(
  * @connect iterator.results -> Exit.results
  * @connect iterator.results -> aggregator.results
  * @connect iterator.onSuccess -> aggregator.execute
- * @connect iterator.onFailure -> aggregator.execute
  * @connect aggregator.successCount -> Exit.successCount
  * @connect aggregator.failedCount -> Exit.failedCount
  * @connect aggregator.onSuccess -> Exit.onSuccess
@@ -47252,7 +47265,7 @@ interface LLMProvider {
   async chat(messages) {
     const lastMessage = messages[messages.length - 1];
     return {
-      content: \`[Mock response to: \${lastMessage.content.slice(0, 50)}...]\`,
+      content: \`[Mock response to: \${(lastMessage?.content ?? '').slice(0, 50)}...]\`,
       toolCalls: [],
       finishReason: 'stop',
       usage: { promptTokens: 10, completionTokens: 20 },
@@ -107083,7 +107096,7 @@ function displayInstalledPackage(pkg) {
 // src/cli/index.ts
 init_logger();
 init_error_utils();
-var version2 = true ? "0.20.6" : "0.0.0-dev";
+var version2 = true ? "0.20.7" : "0.0.0-dev";
 var program2 = new Command();
 program2.name("flow-weaver").description("Flow Weaver Annotations - Compile and validate workflow files").option("-v, --version", "Output the current version").option("--no-color", "Disable colors").option("--color", "Force colors").on("option:version", () => {
   logger.banner(version2);

package/dist/cli/templates/shared/llm-types.d.ts

@@ -9,7 +9,7 @@ export declare const LLM_CORE_TYPES = "interface LLMMessage {\n  role: 'system'
 /** Simplified LLM types — for templates that don't need tool calling */
 export declare const LLM_SIMPLE_TYPES = "interface LLMMessage {\n  role: 'system' | 'user' | 'assistant' | 'tool';\n  content: string;\n  toolCallId?: string;\n}\n\ninterface LLMToolCall {\n  id: string;\n  name: string;\n  arguments: Record<string, unknown>;\n}\n\ninterface LLMResponse {\n  content: string | null;\n  toolCalls: LLMToolCall[];\n  finishReason: 'stop' | 'tool_calls' | 'length' | 'error';\n  usage?: { promptTokens: number; completionTokens: number };\n}\n\ninterface LLMProvider {\n  chat(messages: LLMMessage[], options?: { systemPrompt?: string; model?: string; temperature?: number; maxTokens?: number }): Promise<LLMResponse>;\n}";
 /** Mock provider factory code */
-export declare const LLM_MOCK_PROVIDER = "const createMockProvider = (): LLMProvider => ({\n  async chat(messages) {\n    const lastMessage = messages[messages.length - 1];\n    return {\n      content: `[Mock response to: ${lastMessage.content.slice(0, 50)}...]`,\n      toolCalls: [],\n      finishReason: 'stop',\n      usage: { promptTokens: 10, completionTokens: 20 },\n    };\n  },\n});\n\nconst llmProvider: LLMProvider = (globalThis as unknown as { __fw_llm_provider__?: LLMProvider }).__fw_llm_provider__ ?? createMockProvider();";
+export declare const LLM_MOCK_PROVIDER = "const createMockProvider = (): LLMProvider => ({\n  async chat(messages) {\n    const lastMessage = messages[messages.length - 1];\n    return {\n      content: `[Mock response to: ${(lastMessage?.content ?? '').slice(0, 50)}...]`,\n      toolCalls: [],\n      finishReason: 'stop',\n      usage: { promptTokens: 10, completionTokens: 20 },\n    };\n  },\n});\n\nconst llmProvider: LLMProvider = (globalThis as unknown as { __fw_llm_provider__?: LLMProvider }).__fw_llm_provider__ ?? createMockProvider();";
 /** Mock provider with tool calling support (for ai-agent) */
 export declare const LLM_MOCK_PROVIDER_WITH_TOOLS = "const createMockProvider = (): LLMProvider => ({\n  async chat(messages, options) {\n    const last = messages[messages.length - 1];\n    if (options?.tools && last.content.toLowerCase().includes('search')) {\n      return {\n        content: null,\n        toolCalls: [\n          {\n            id: 'call_' + Date.now(),\n            name: 'search',\n            arguments: { query: last.content },\n          },\n        ],\n        finishReason: 'tool_calls',\n        usage: { promptTokens: 15, completionTokens: 30 },\n      };\n    }\n\n    return {\n      content: '[Mock answer] ' + last.content,\n      toolCalls: [],\n      finishReason: 'stop',\n      usage: { promptTokens: 10, completionTokens: 20 },\n    };\n  },\n});\n\nconst llmProvider: LLMProvider = (globalThis as unknown as { __fw_llm_provider__?: LLMProvider }).__fw_llm_provider__ ?? createMockProvider();";
 //# sourceMappingURL=llm-types.d.ts.map

package/dist/cli/templates/shared/llm-types.js

@@ -68,7 +68,7 @@ export const LLM_MOCK_PROVIDER = `const createMockProvider = (): LLMProvider =>
   async chat(messages) {
     const lastMessage = messages[messages.length - 1];
     return {
-      content: \`[Mock response to: \${lastMessage.content.slice(0, 50)}...]\`,
+      content: \`[Mock response to: \${(lastMessage?.content ?? '').slice(0, 50)}...]\`,
       toolCalls: [],
       finishReason: 'stop',
       usage: { promptTokens: 10, completionTokens: 20 },

package/dist/cli/templates/workflows/foreach.js

@@ -101,7 +101,7 @@ function aggregateResults(
 
 /**
  * @flowWeaver workflow
- * @node iterator forEachItem [position: -90 0] [color: "purple"] [icon: "repeat"]
+ * @node iterator forEachItem [position: -90 0] [color: "purple"] [icon: "repeat"] [suppress: "DESIGN_SCOPE_NO_FAILURE_EXIT"]
  * @node processor processItem iterator.processItem [color: "blue"] [icon: "settings"]
  * @node aggregator aggregateResults [position: 270 0] [color: "teal"] [icon: "inventory"]
  * @position Start -450 0
@@ -116,7 +116,6 @@ function aggregateResults(
  * @connect iterator.results -> Exit.results
  * @connect iterator.results -> aggregator.results
  * @connect iterator.onSuccess -> aggregator.execute
- * @connect iterator.onFailure -> aggregator.execute
  * @connect aggregator.successCount -> Exit.successCount
  * @connect aggregator.failedCount -> Exit.failedCount
  * @connect aggregator.onSuccess -> Exit.onSuccess

package/dist/generated-version.d.ts

@@ -1,2 +1,2 @@
-export declare const VERSION = "0.20.6";
+export declare const VERSION = "0.20.7";
 //# sourceMappingURL=generated-version.d.ts.map

package/dist/generated-version.js

@@ -1,3 +1,3 @@
 // Auto-generated by scripts/generate-version.ts — do not edit manually
-export const VERSION = '0.20.6';
+export const VERSION = '0.20.7';
 //# sourceMappingURL=generated-version.js.map

package/dist/generator/unified.js

@@ -207,9 +207,27 @@ export function generateControlFlowWithExecutionContext(workflow, nodeTypes, isA
             region.failureNodes.delete(instanceId);
         });
     });
+    // Promote nodes that appear in BOTH success and failure regions of the same
+    // branching node. These nodes execute regardless of which branch is taken,
+    // so they must not be nested inside either branch (which would cause duplicate
+    // variable declarations and cancelled events for nodes that actually run).
+    const nodesInBothBranches = new Set();
+    branchRegions.forEach((region) => {
+        region.successNodes.forEach((nodeId) => {
+            if (region.failureNodes.has(nodeId)) {
+                nodesInBothBranches.add(nodeId);
+            }
+        });
+    });
+    branchRegions.forEach((region) => {
+        nodesInBothBranches.forEach((nodeId) => {
+            region.successNodes.delete(nodeId);
+            region.failureNodes.delete(nodeId);
+        });
+    });
     // Promote nodes that have DATA dependencies on nodes outside their branch.
     // Without this, STEP-nesting places the node before its data providers are generated.
-    const nodesPromotedFromBranches = new Set();
+    const nodesPromotedFromBranches = new Set(nodesInBothBranches);
     branchRegions.forEach((region, branchNodeId) => {
         const allBranchNodes = new Set([...region.successNodes, ...region.failureNodes]);
         allBranchNodes.forEach((nodeId) => {
@@ -383,7 +401,9 @@ export function generateControlFlowWithExecutionContext(workflow, nodeTypes, isA
                         }
                     });
                     if (stepSourceConditions.length > 0) {
-                        const condition = stepSourceConditions.join(' && ');
+                        const condition = stepSourceConditions.length === 1
+                            ? stepSourceConditions[0]
+                            : stepSourceConditions.join(' || ');
                         lines.push(`  if (${condition}) {`);
                         chainIndent = '    ';
                         chainNeedsClose = true;
@@ -410,7 +430,9 @@ export function generateControlFlowWithExecutionContext(workflow, nodeTypes, isA
                     }
                 });
                 if (stepSourceConditions.length > 0) {
-                    const condition = stepSourceConditions.join(' && ');
+                    const condition = stepSourceConditions.length === 1
+                        ? stepSourceConditions[0]
+                        : stepSourceConditions.join(' || ');
                     lines.push(`  if (${condition}) {`);
                     branchIndent = '    ';
                     branchNeedsClose = true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@synergenius/flow-weaver",
-  "version": "0.20.6",
+  "version": "0.20.7",
   "description": "Deterministic workflow compiler for AI agents. Compiles to standalone TypeScript, no runtime dependencies.",
   "private": false,
   "type": "module",