@synergenius/flow-weaver 0.21.17 → 0.21.18

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.21.17";
+    VERSION = "0.21.18";
   }
 });
 
@@ -39494,6 +39494,45 @@ Add '@param ${fromPort}' to the workflow JSDoc and include it in the params obje
             }
           }
         }
+        const parentOf = /* @__PURE__ */ new Map();
+        for (const inst of workflow.instances) {
+          parentOf.set(inst.id, inst.parent ?? void 0);
+        }
+        const scopedParentIds = /* @__PURE__ */ new Set();
+        for (const inst of workflow.instances) {
+          const nt = instanceMap.get(inst.id);
+          if (!nt) continue;
+          const hasScopedPorts = [
+            ...Object.values(nt.inputs ?? {}),
+            ...Object.values(nt.outputs ?? {})
+          ].some((p) => p.scope);
+          if (hasScopedPorts) scopedParentIds.add(inst.id);
+        }
+        for (const conn of workflow.connections) {
+          if (conn.from.scope || conn.to.scope) continue;
+          const fromParent = parentOf.get(conn.from.node);
+          const toParent = parentOf.get(conn.to.node);
+          if (!parentOf.has(conn.from.node) || !parentOf.has(conn.to.node)) continue;
+          if (!fromParent && !toParent) continue;
+          const fromInScopedParent = fromParent && scopedParentIds.has(fromParent.id);
+          const toInScopedParent = toParent && scopedParentIds.has(toParent.id);
+          if (!fromInScopedParent && !toInScopedParent) continue;
+          const fromIsParentOfTo = toParent && toParent.id === conn.from.node;
+          const toIsParentOfFrom = fromParent && fromParent.id === conn.to.node;
+          if (fromIsParentOfTo || toIsParentOfFrom) continue;
+          const sameScope = fromParent && toParent && fromParent.id === toParent.id && fromParent.scope === toParent.scope;
+          if (!sameScope) {
+            const fromCtx = fromParent ? `${fromParent.id}.${fromParent.scope}` : "root";
+            const toCtx = toParent ? `${toParent.id}.${toParent.scope}` : "root";
+            this.errors.push({
+              type: "error",
+              code: "CROSS_SCOPE_CONNECTION",
+              message: `Connection from "${conn.from.node}.${conn.from.port}" (in ${fromCtx}) to "${conn.to.node}.${conn.to.port}" (in ${toCtx}) crosses scope boundaries. Nodes in different scopes cannot connect directly.`,
+              connection: conn,
+              location: this.getConnectionLocation(conn)
+            });
+          }
+        }
       }
       // ── H: Duplicate instance IDs ──────────────────────────────────────────
       validateDuplicateInstanceIds(workflow) {
@@ -93231,7 +93270,7 @@ function displayInstalledPackage(pkg) {
 // src/cli/index.ts
 init_logger();
 init_error_utils();
-var version2 = true ? "0.21.17" : "0.0.0-dev";
+var version2 = true ? "0.21.18" : "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.21.17";
+export declare const VERSION = "0.21.18";
 //# 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.21.17';
+export const VERSION = '0.21.18';
 //# sourceMappingURL=generated-version.js.map

package/dist/validator.js

@@ -1374,6 +1374,67 @@ export class WorkflowValidator {
                 }
             }
         }
+        // Check: non-scoped connections must not cross scope boundaries
+        // when the parent is a scoped node type (has scoped ports).
+        // A child in scope "a" cannot connect to a child in scope "b" (or root),
+        // because they execute in separate callback contexts.
+        // Only applies to per-port scoped parents (not flat grouping scopes).
+        const parentOf = new Map();
+        for (const inst of workflow.instances) {
+            parentOf.set(inst.id, inst.parent ?? undefined);
+        }
+        // Build set of node IDs that are scoped parent types (have scoped ports)
+        const scopedParentIds = new Set();
+        for (const inst of workflow.instances) {
+            const nt = instanceMap.get(inst.id);
+            if (!nt)
+                continue;
+            const hasScopedPorts = [
+                ...Object.values(nt.inputs ?? {}),
+                ...Object.values(nt.outputs ?? {}),
+            ].some((p) => p.scope);
+            if (hasScopedPorts)
+                scopedParentIds.add(inst.id);
+        }
+        for (const conn of workflow.connections) {
+            // Skip scoped connections (already validated above)
+            if (conn.from.scope || conn.to.scope)
+                continue;
+            const fromParent = parentOf.get(conn.from.node);
+            const toParent = parentOf.get(conn.to.node);
+            // Both must exist in the workflow
+            if (!parentOf.has(conn.from.node) || !parentOf.has(conn.to.node))
+                continue;
+            // If neither node is inside a scope, skip (root-to-root is fine)
+            if (!fromParent && !toParent)
+                continue;
+            // Only validate when the parent is a scoped node type (has scoped ports).
+            // Flat grouping scopes (createScope without scoped ports) are allowed to cross.
+            const fromInScopedParent = fromParent && scopedParentIds.has(fromParent.id);
+            const toInScopedParent = toParent && scopedParentIds.has(toParent.id);
+            if (!fromInScopedParent && !toInScopedParent)
+                continue;
+            // Allow connections where one side is the scoped PARENT itself (not a child)
+            const fromIsParentOfTo = toParent && toParent.id === conn.from.node;
+            const toIsParentOfFrom = fromParent && fromParent.id === conn.to.node;
+            if (fromIsParentOfTo || toIsParentOfFrom)
+                continue;
+            // Check if they share the same parent+scope
+            const sameScope = fromParent && toParent &&
+                fromParent.id === toParent.id &&
+                fromParent.scope === toParent.scope;
+            if (!sameScope) {
+                const fromCtx = fromParent ? `${fromParent.id}.${fromParent.scope}` : 'root';
+                const toCtx = toParent ? `${toParent.id}.${toParent.scope}` : 'root';
+                this.errors.push({
+                    type: 'error',
+                    code: 'CROSS_SCOPE_CONNECTION',
+                    message: `Connection from "${conn.from.node}.${conn.from.port}" (in ${fromCtx}) to "${conn.to.node}.${conn.to.port}" (in ${toCtx}) crosses scope boundaries. Nodes in different scopes cannot connect directly.`,
+                    connection: conn,
+                    location: this.getConnectionLocation(conn),
+                });
+            }
+        }
     }
     // ── H: Duplicate instance IDs ──────────────────────────────────────────
     validateDuplicateInstanceIds(workflow) {

package/package.json

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