@bian-womp/spark-graph 0.3.72 → 0.3.73

package/lib/cjs/index.cjs

@@ -2523,11 +2523,10 @@ class HandleResolver {
  * EdgePropagator component - handles value propagation through edges
  */
 class EdgePropagator {
-    constructor(graph, eventEmitter, runContextManager, handleResolver, nodeExecutor, runtime) {
+    constructor(graph, eventEmitter, runContextManager, nodeExecutor, runtime) {
         this.graph = graph;
         this.eventEmitter = eventEmitter;
         this.runContextManager = runContextManager;
-        this.handleResolver = handleResolver;
         this.nodeExecutor = nodeExecutor;
         this.runtime = runtime;
         this.arrayInputBuckets = new Map();
@@ -2716,7 +2715,7 @@ class EdgePropagator {
         // Set input value (respecting skipPropagateValues)
         const shouldSetValue = this.shouldSetInputValue(effectiveRunContexts);
         if (shouldSetValue && valueChanged) {
-            this.setTargetInput(edge, processedValue);
+            this.runtime.setTargetInput(edge, processedValue, "applyToTarget");
         }
         else if (shouldSetValue && !valueChanged) {
             // Even if value didn't change, update timestamp if we're forcing execution
@@ -2772,13 +2771,6 @@ class EdgePropagator {
         }
         return true;
     }
-    /**
-     * Set target input value and emit event
-     */
-    setTargetInput(edge, value) {
-        this.graph.updateNodeInput(edge.target.nodeId, edge.target.handle, value);
-        this.handleResolver.scheduleRecomputeHandles(edge.target.nodeId);
-    }
     /**
      * Execute downstream if conditions are met
      */
@@ -3622,7 +3614,7 @@ class GraphRuntime {
         this.graph = new Graph(this.eventEmitter);
         this.runContextManager = new RunContextManager(this.graph, "warn");
         this.handleResolver = new HandleResolver(this.graph, this.eventEmitter, this.runContextManager, this);
-        this.edgePropagator = new EdgePropagator(this.graph, this.eventEmitter, this.runContextManager, this.handleResolver, this, this);
+        this.edgePropagator = new EdgePropagator(this.graph, this.eventEmitter, this.runContextManager, this, this);
         // Create NodeExecutor with EdgePropagator and HandleResolver
         this.nodeExecutor = new NodeExecutor(this.graph, this.eventEmitter, this.runContextManager, this.handleResolver, this, this);
         // Create RuntimeValidatorManager
@@ -3711,6 +3703,30 @@ class GraphRuntime {
     on(event, handler) {
         return this.eventEmitter.on(event, handler);
     }
+    /**
+     * Check if an event is an invalidate event that should trigger re-execution
+     */
+    isInvalidateEvent(event) {
+        if (!event || typeof event !== "object")
+            return false;
+        // Check if event has action === "invalidate"
+        const e = event;
+        return e.action === "invalidate";
+    }
+    executeNodeAutoRun(nodeId, opts) {
+        const node = this.graph.getNode(nodeId);
+        const shouldAutoRun = this.runMode === "auto" || node?.policy?.autoRun === true;
+        let runContextIdsToUse = undefined;
+        if (this.runMode === "manual") {
+            runContextIdsToUse = new Set([this.runContextManager.createRunContext(nodeId, { propagate: false })]);
+        }
+        if (shouldAutoRun && this.graph.allInboundHaveValue(nodeId)) {
+            this.execute(nodeId, {
+                runContextIds: runContextIdsToUse,
+                reason: opts?.reason ?? "executeNodeAutoRun",
+            });
+        }
+    }
     setInputs(nodeId, inputs) {
         const node = this.graph.getNode(nodeId);
         if (!node)
@@ -3766,8 +3782,6 @@ class GraphRuntime {
             const same = valuesEqual(prev, value);
             if (!same) {
                 this.graph.updateNodeInput(nodeId, handle, value);
-                // Emit value event for input updates
-                this.eventEmitter.emit("value", { nodeId, handle, value, io: "input" });
                 anyChanged = true;
             }
         }
@@ -3819,16 +3833,6 @@ class GraphRuntime {
         // Forward event to node's onExternalEvent handler for custom actions
         node.runtime.onExternalEvent?.(event, node.state);
     }
-    /**
-     * Check if an event is an invalidate event that should trigger re-execution
-     */
-    isInvalidateEvent(event) {
-        if (!event || typeof event !== "object")
-            return false;
-        // Check if event has action === "invalidate"
-        const e = event;
-        return e.action === "invalidate";
-    }
     cancelNodeRuns(nodeIds) {
         this.nodeExecutor.cancelNodeRuns(nodeIds);
     }
@@ -3856,6 +3860,7 @@ class GraphRuntime {
         this.nodeExecutor.setEnvironment(this.environment);
         for (const nodeId of this.graph.getNodeIds()) {
             this.handleResolver.scheduleRecomputeHandles(nodeId);
+            this.executeNodeAutoRun(nodeId, { reason: "setEnvironment" });
         }
     }
     setCustomNodeData(customNodeData) {
@@ -4003,19 +4008,10 @@ class GraphRuntime {
             }
         }
     }
-    executeNodeAutoRun(nodeId, opts) {
-        const node = this.graph.getNode(nodeId);
-        const shouldAutoRun = this.runMode === "auto" || node?.policy?.autoRun === true;
-        let runContextIdsToUse = undefined;
-        if (this.runMode === "manual") {
-            runContextIdsToUse = new Set([this.runContextManager.createRunContext(nodeId, { propagate: false })]);
-        }
-        if (shouldAutoRun && this.graph.allInboundHaveValue(nodeId)) {
-            this.execute(nodeId, {
-                runContextIds: runContextIdsToUse,
-                reason: opts?.reason ?? "executeNodeAutoRun",
-            });
-        }
+    setTargetInput(edge, value, reason) {
+        this.graph.updateNodeInput(edge.target.nodeId, edge.target.handle, value);
+        this.handleResolver.scheduleRecomputeHandles(edge.target.nodeId);
+        this.executeNodeAutoRun(edge.target.nodeId, { reason });
     }
     copyOutputs(fromNodeId, toNodeId, options) {
         const fromNode = this.getNodeData(fromNodeId);