@synergenius/flow-weaver 0.22.5 → 0.22.6

package/dist/cli/flow-weaver.mjs

@@ -9886,7 +9886,7 @@ var VERSION;
 var init_generated_version = __esm({
   "src/generated-version.ts"() {
     "use strict";
-    VERSION = "0.22.5";
+    VERSION = "0.22.6";
   }
 });
 
@@ -10791,16 +10791,35 @@ function buildNodeArgumentsWithContext(opts) {
   } else if (skipPorts?.has("execute")) {
     args.push(`${safeId}_execute`);
   } else if (executeConnections.length > 0) {
-    const conn = executeConnections[0];
-    const sourceNode = conn.from.node;
-    const sourcePort = conn.from.port;
     const varName = `${safeId}_execute`;
-    const sourceIdx = isStartNode(sourceNode) ? "startIdx" : `${toValidIdentifier(sourceNode)}Idx`;
-    const isConstSource = isStartNode(sourceNode) || sourceNode === instanceParent;
-    const nonNullAssert = isConstSource ? "" : "!";
-    lines.push(
-      `${indent}const ${varName} = ${getCall}({ id: '${sourceNode}', portName: '${sourcePort}', executionIndex: ${sourceIdx}${nonNullAssert} }) as boolean;`
-    );
+    if (executeConnections.length === 1) {
+      const conn = executeConnections[0];
+      const sourceNode = conn.from.node;
+      const sourcePort = conn.from.port;
+      const sourceIdx = isStartNode(sourceNode) ? "startIdx" : `${toValidIdentifier(sourceNode)}Idx`;
+      const isConstSource = isStartNode(sourceNode) || sourceNode === instanceParent;
+      if (isConstSource) {
+        lines.push(
+          `${indent}const ${varName} = ${getCall}({ id: '${sourceNode}', portName: '${sourcePort}', executionIndex: ${sourceIdx} }) as boolean;`
+        );
+      } else {
+        lines.push(
+          `${indent}const ${varName} = ${sourceIdx} !== undefined ? ${getCall}({ id: '${sourceNode}', portName: '${sourcePort}', executionIndex: ${sourceIdx} }) as boolean : false;`
+        );
+      }
+    } else {
+      const parts = executeConnections.map((conn) => {
+        const sourceNode = conn.from.node;
+        const sourcePort = conn.from.port;
+        const sourceIdx = isStartNode(sourceNode) ? "startIdx" : `${toValidIdentifier(sourceNode)}Idx`;
+        const isConstSource = isStartNode(sourceNode) || sourceNode === instanceParent;
+        if (isConstSource) {
+          return `(${getCall}({ id: '${sourceNode}', portName: '${sourcePort}', executionIndex: ${sourceIdx} }) as boolean)`;
+        }
+        return `(${sourceIdx} !== undefined ? ${getCall}({ id: '${sourceNode}', portName: '${sourcePort}', executionIndex: ${sourceIdx} }) as boolean : false)`;
+      });
+      lines.push(`${indent}const ${varName} = ${parts.join(" || ")};`);
+    }
     if (emitInputEvents) {
       lines.push(
         `${indent}${setCall}({ id: '${id}', portName: 'execute', executionIndex: ${safeId}Idx, nodeTypeName: '${effectiveNodeTypeName}' }, ${varName});`
@@ -94920,7 +94939,7 @@ var {
 // src/cli/index.ts
 init_logger();
 init_error_utils();
-var version2 = true ? "0.22.5" : "0.0.0-dev";
+var version2 = true ? "0.22.6" : "0.0.0-dev";
 var program2 = new Command();
 program2.name("fw").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/generated-version.d.ts

@@ -1,2 +1,2 @@
-export declare const VERSION = "0.22.5";
+export declare const VERSION = "0.22.6";
 //# 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.22.5';
+export const VERSION = '0.22.6';
 //# sourceMappingURL=generated-version.js.map

package/dist/generator/code-utils.js

@@ -172,16 +172,36 @@ export function buildNodeArgumentsWithContext(opts) {
         args.push(`${safeId}_execute`);
     }
     else if (executeConnections.length > 0) {
-        // Execute port has a connection - use it
-        const conn = executeConnections[0];
-        const sourceNode = conn.from.node;
-        const sourcePort = conn.from.port;
+        // Execute port has connections - use them
         const varName = `${safeId}_execute`;
-        // startIdx is const so no ! needed; parent scope node is also const; other node indices are let so need !
-        const sourceIdx = isStartNode(sourceNode) ? 'startIdx' : `${toValidIdentifier(sourceNode)}Idx`;
-        const isConstSource = isStartNode(sourceNode) || sourceNode === instanceParent;
-        const nonNullAssert = isConstSource ? '' : '!';
-        lines.push(`${indent}const ${varName} = ${getCall}({ id: '${sourceNode}', portName: '${sourcePort}', executionIndex: ${sourceIdx}${nonNullAssert} }) as boolean;`);
+        if (executeConnections.length === 1) {
+            const conn = executeConnections[0];
+            const sourceNode = conn.from.node;
+            const sourcePort = conn.from.port;
+            const sourceIdx = isStartNode(sourceNode) ? 'startIdx' : `${toValidIdentifier(sourceNode)}Idx`;
+            const isConstSource = isStartNode(sourceNode) || sourceNode === instanceParent;
+            if (isConstSource) {
+                lines.push(`${indent}const ${varName} = ${getCall}({ id: '${sourceNode}', portName: '${sourcePort}', executionIndex: ${sourceIdx} }) as boolean;`);
+            }
+            else {
+                // Non-const source may be undefined (CANCELLED branch) — guard with false default
+                lines.push(`${indent}const ${varName} = ${sourceIdx} !== undefined ? ${getCall}({ id: '${sourceNode}', portName: '${sourcePort}', executionIndex: ${sourceIdx} }) as boolean : false;`);
+            }
+        }
+        else {
+            // Multiple execute connections — coalesce with ||, each guarded
+            const parts = executeConnections.map((conn) => {
+                const sourceNode = conn.from.node;
+                const sourcePort = conn.from.port;
+                const sourceIdx = isStartNode(sourceNode) ? 'startIdx' : `${toValidIdentifier(sourceNode)}Idx`;
+                const isConstSource = isStartNode(sourceNode) || sourceNode === instanceParent;
+                if (isConstSource) {
+                    return `(${getCall}({ id: '${sourceNode}', portName: '${sourcePort}', executionIndex: ${sourceIdx} }) as boolean)`;
+                }
+                return `(${sourceIdx} !== undefined ? ${getCall}({ id: '${sourceNode}', portName: '${sourcePort}', executionIndex: ${sourceIdx} }) as boolean : false)`;
+            });
+            lines.push(`${indent}const ${varName} = ${parts.join(' || ')};`);
+        }
         // Emit VARIABLE_SET for execute input port
         if (emitInputEvents) {
             lines.push(`${indent}${setCall}({ id: '${id}', portName: 'execute', executionIndex: ${safeId}Idx, nodeTypeName: '${effectiveNodeTypeName}' }, ${varName});`);

package/dist/generator/unified.js

@@ -814,7 +814,9 @@ isAsync = false) {
         if (!instance)
             return;
         const safeId = toValidIdentifier(instanceId);
-        // Add execution index for this skipped node so the event has a valid reference
+        // Use const (block-scoped) intentionally — the outer `let ${safeId}Idx`
+        // stays undefined, which signals to downstream guards that this node was
+        // CANCELLED and its data ports should not be read.
         lines.push(`${indent}const ${safeId}Idx = ${ctxVar}.addExecution('${instanceId}');`);
         // Set STEP port variables so downstream nodes reading onSuccess/onFailure
         // from this cancelled node don't crash with "Variable not found".

package/package.json

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