@bian-womp/spark-graph 0.3.3 → 0.3.5

package/lib/cjs/index.cjs

@@ -1528,10 +1528,16 @@ class NodeExecutor {
             invalidateDownstream: () => {
                 this.edgePropagator.invalidateDownstream(nodeId);
             },
-            execute: () => {
+            execute: (resolve, opts) => {
                 if (this.graph.allInboundHaveValue(nodeId)) {
-                    // Preserve run-context IDs when scheduling from execution context
-                    this.execute(nodeId, runContextIds);
+                    let runContextIdsToUse = this.runtime.getRunMode() === "auto" ? undefined : runContextIds;
+                    if (this.runtime.getRunMode() === "manual" &&
+                        (!runContextIds || runContextIds.size === 0)) {
+                        runContextIdsToUse = new Set([
+                            this.runContextManager.createRunContext(nodeId, resolve, opts),
+                        ]);
+                    }
+                    this.execute(nodeId, runContextIdsToUse);
                 }
             },
             getInput: (handle) => inputs[handle],
@@ -1554,12 +1560,21 @@ class NodeExecutor {
             console.warn("NodeExecutor.execute: no runMode, skipping execution");
             return;
         }
+        // In manual mode, require runContextIds unless autoRun policy is set
         if (runMode === "manual" && (!runContextIds || runContextIds.size === 0)) {
-            console.warn("NodeExecutor.execute: no runContextIds provided, skipping execution");
-            return;
+            // If autoRun is true, auto-generate a run context (similar to createExecutionContext pattern)
+            if (node.policy?.autoRun === true) {
+                runContextIds = new Set([
+                    this.runContextManager.createRunContext(nodeId),
+                ]);
+            }
+            else {
+                console.warn("NodeExecutor.execute: no runContextIds provided in manual mode, skipping execution");
+                return;
+            }
         }
         if (runMode === "auto" && runContextIds && runContextIds.size > 0) {
-            console.warn("NodeExecutor.execute: runContextIds provided in auto-mode, ignoring");
+            console.warn("NodeExecutor.execute: runContextIds provided in auto mode, ignoring");
             runContextIds = undefined;
         }
         // Early validation for auto-mode paused state
@@ -2103,9 +2118,12 @@ class GraphRuntime {
             }
         }
         // In auto mode, input updates can trigger execution; in manual mode they never should.
+        // However, if autoRun policy is set, nodes run automatically even in manual mode.
         if (anyChanged) {
             this.handleResolver.scheduleRecomputeHandles(nodeId);
-            if (this.runMode === "auto" && this.graph.allInboundHaveValue(nodeId)) {
+            const node = this.graph.getNode(nodeId);
+            const shouldAutoRun = this.runMode === "auto" || node?.policy?.autoRun === true;
+            if (shouldAutoRun && this.graph.allInboundHaveValue(nodeId)) {
                 this.execute(nodeId);
             }
         }
@@ -2379,6 +2397,14 @@ class GraphRuntime {
                 }
                 else {
                     existing.params = n.params;
+                    // Re-merge policy when params change (params.policy can override descriptor/category policy)
+                    const desc = registry.nodes.get(existing.typeId);
+                    const cat = registry.categories.get(desc?.categoryId ?? "");
+                    existing.policy = {
+                        ...cat?.policy,
+                        ...desc?.policy,
+                        ...n.params?.policy,
+                    };
                     if (!existing.stats) {
                         existing.stats = {
                             runs: 0,
@@ -3236,6 +3262,7 @@ function setupBasicGraphRegistry(id) {
         categoryId: "compute",
         inputs: { Value: "base.float" },
         outputs: { Result: "base.float" },
+        policy: { autoRun: true },
         impl: (ins) => ({ Result: Number(ins.Value) }),
     });
     registry.registerNode({
@@ -3243,6 +3270,7 @@ function setupBasicGraphRegistry(id) {
         categoryId: "compute",
         inputs: { Value: "base.string" },
         outputs: { Result: "base.string" },
+        policy: { autoRun: true },
         impl: (ins) => ({ Result: String(ins.Value) }),
     });
     registry.registerNode({
@@ -3250,6 +3278,7 @@ function setupBasicGraphRegistry(id) {
         categoryId: "compute",
         inputs: { Value: "base.bool" },
         outputs: { Result: "base.bool" },
+        policy: { autoRun: true },
         impl: (ins) => ({ Result: Boolean(ins.Value) }),
     });
     registry.registerNode({
@@ -3257,6 +3286,7 @@ function setupBasicGraphRegistry(id) {
         categoryId: "compute",
         inputs: { Value: "base.object" },
         outputs: { Result: "base.object" },
+        policy: { autoRun: true },
         impl: (ins) => ({ Result: ins.Value }),
     });
     registry.registerNode({
@@ -3264,6 +3294,7 @@ function setupBasicGraphRegistry(id) {
         categoryId: "compute",
         inputs: { Value: "base.vec3" },
         outputs: { Result: "base.vec3" },
+        policy: { autoRun: true },
         impl: (ins) => ({ Result: ins.Value }),
     });
     // JSON parser node: base.stringToObject