@synergenius/flow-weaver 0.21.17 → 0.21.19

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.19";
   }
 });
 
@@ -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.19" : "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.19";
 //# 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.19';
 //# sourceMappingURL=generated-version.js.map

package/dist/npm-packages.d.ts

@@ -32,6 +32,7 @@ export type TNpmNodeType = {
 };
 /**
  * Get list of packages that have TypeScript declarations (.d.ts files).
+ * Only includes direct dependencies from package.json (not transitive or dev).
  * Excludes @types/* packages as they are type augmentations.
  *
  * @param workdir - Directory to start searching from

package/dist/npm-packages.js

@@ -11,64 +11,34 @@ import { extractFunctionLikes } from './function-like.js';
 import { inferDataTypeFromTS } from './type-mappings.js';
 import { getSharedProject } from './shared-project.js';
 /**
- * Find all node_modules directories starting from fromDir and walking up.
+ * Read direct dependency names from the closest package.json.
+ * Only includes `dependencies` (not devDependencies, peerDependencies, etc.)
+ * since those are the packages available at runtime in workflows.
  */
-function findNodeModulesDirs(fromDir) {
-    const dirs = [];
-    let current = path.resolve(fromDir);
+function readDirectDependencies(workdir) {
+    let current = path.resolve(workdir);
     const root = path.parse(current).root;
     while (current !== root) {
-        const candidate = path.join(current, 'node_modules');
-        if (fs.existsSync(candidate) && fs.statSync(candidate).isDirectory()) {
-            dirs.push(candidate);
+        const pkgJsonPath = path.join(current, 'package.json');
+        if (fs.existsSync(pkgJsonPath)) {
+            try {
+                const pkgJson = JSON.parse(fs.readFileSync(pkgJsonPath, 'utf-8'));
+                return Object.keys(pkgJson.dependencies ?? {});
+            }
+            catch {
+                // malformed package.json, keep walking
+            }
         }
         const parent = path.dirname(current);
         if (parent === current)
             break;
         current = parent;
     }
-    return dirs;
-}
-/**
- * List all packages in a node_modules directory (including scoped packages).
- */
-function listPackagesInNodeModules(nmDir) {
-    const packages = [];
-    if (!fs.existsSync(nmDir))
-        return packages;
-    try {
-        const entries = fs.readdirSync(nmDir, { withFileTypes: true });
-        for (const entry of entries) {
-            if (!entry.isDirectory())
-                continue;
-            // Handle scoped packages (@scope/pkg)
-            if (entry.name.startsWith('@')) {
-                const scopeDir = path.join(nmDir, entry.name);
-                try {
-                    const scopedEntries = fs.readdirSync(scopeDir, { withFileTypes: true });
-                    for (const scopedEntry of scopedEntries) {
-                        if (scopedEntry.isDirectory()) {
-                            packages.push(`${entry.name}/${scopedEntry.name}`);
-                        }
-                    }
-                }
-                catch {
-                    // Ignore permission errors
-                }
-            }
-            else {
-                // Regular package
-                packages.push(entry.name);
-            }
-        }
-    }
-    catch {
-        // Ignore permission errors
-    }
-    return packages;
+    return [];
 }
 /**
  * Get list of packages that have TypeScript declarations (.d.ts files).
+ * Only includes direct dependencies from package.json (not transitive or dev).
  * Excludes @types/* packages as they are type augmentations.
  *
  * @param workdir - Directory to start searching from
@@ -76,40 +46,26 @@ function listPackagesInNodeModules(nmDir) {
  * @returns Object with packages array, each containing name and typesPath
  */
 export function getTypedPackages(workdir, nodeModulesOverride) {
-    const nodeModulesDirs = nodeModulesOverride
-        ? [nodeModulesOverride]
-        : findNodeModulesDirs(workdir);
+    const directDeps = readDirectDependencies(workdir);
     const typed = [];
-    const seenPackages = new Set();
-    for (const nmDir of nodeModulesDirs) {
-        const packages = listPackagesInNodeModules(nmDir);
-        for (const pkg of packages) {
-            // Skip @types/* packages - they are type augmentations, not actual packages
-            if (pkg.startsWith('@types/'))
-                continue;
-            // Skip if already seen (from a closer node_modules)
-            if (seenPackages.has(pkg))
-                continue;
-            seenPackages.add(pkg);
-            // Check if package has types
-            const typesPath = resolvePackageTypesPath(pkg, workdir, nodeModulesOverride);
-            if (typesPath) {
-                typed.push({ name: pkg, typesPath });
-            }
+    for (const pkg of directDeps) {
+        if (pkg.startsWith('@types/'))
+            continue;
+        const typesPath = resolvePackageTypesPath(pkg, workdir, nodeModulesOverride);
+        if (typesPath) {
+            typed.push({ name: pkg, typesPath });
         }
     }
     return { packages: typed };
 }
-/**
- * Capitalize first letter of a string.
- */
+const PRIMITIVE_TYPES = new Set(['string', 'number', 'boolean', 'any', 'unknown', 'never']);
 function capitalize(str) {
     return str.charAt(0).toUpperCase() + str.slice(1);
 }
 /**
  * Infer node type from a function declaration in a .d.ts file.
  */
-function inferNodeTypeFromDtsFunction(fn, packageName, _dtsPath) {
+function inferNodeTypeFromDtsFunction(fn, packageName) {
     const fnName = fn.getName();
     if (!fnName)
         return null;
@@ -149,8 +105,7 @@ function inferNodeTypeFromDtsFunction(fn, packageName, _dtsPath) {
     }
     const unwrappedText = returnType.getText();
     if (unwrappedText !== 'void' && unwrappedText !== 'undefined') {
-        const primitiveTypes = new Set(['string', 'number', 'boolean', 'any', 'unknown', 'never']);
-        const isPrimitive = primitiveTypes.has(unwrappedText);
+        const isPrimitive = PRIMITIVE_TYPES.has(unwrappedText);
         const isArray = unwrappedText.endsWith('[]') || unwrappedText.startsWith('Array<');
         const properties = returnType.getProperties();
         const isObjectLike = !isPrimitive && !isArray && returnType.isObject() && properties.length > 0;
@@ -239,7 +194,7 @@ export function getPackageExports(packageName, workdir, nodeModulesOverride) {
             if (!fnName || seenFunctionNames.has(fnName))
                 continue;
             seenFunctionNames.add(fnName);
-            const nodeType = inferNodeTypeFromDtsFunction(fn, packageName, typesPath);
+            const nodeType = inferNodeTypeFromDtsFunction(fn, packageName);
             if (nodeType) {
                 nodeTypes.push(nodeType);
             }

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.19",
   "description": "Deterministic workflow compiler for AI agents. Compiles to standalone TypeScript, no runtime dependencies.",
   "private": false,
   "type": "module",