@bian-womp/spark-graph 0.1.10 → 0.1.12

package/lib/cjs/index.cjs

@@ -480,16 +480,20 @@ class GraphRuntime {
                 h(payload);
     }
     setInput(nodeId, handle, value) {
+        this.setInputs(nodeId, { [handle]: value });
+    }
+    setInputs(nodeId, inputs) {
         const node = this.nodes.get(nodeId);
         if (!node)
             throw new Error(`Node not found: ${nodeId}`);
-        // If this input has an inbound edge, prefer propagated runtime value over manual input
-        const hasInbound = this.edges.some((e) => e.target.nodeId === nodeId && e.target.handle === handle);
-        if (hasInbound)
-            return; // respect linked value
-        node.inputs[handle] = value;
-        // Emit value event for input updates
-        this.emit("value", { nodeId, handle, value, io: "input" });
+        for (const [handle, value] of Object.entries(inputs)) {
+            const hasInbound = this.edges.some((e) => e.target.nodeId === nodeId && e.target.handle === handle);
+            if (hasInbound)
+                continue;
+            node.inputs[handle] = value;
+            // Emit value event for input updates
+            this.emit("value", { nodeId, handle, value, io: "input" });
+        }
         if (!this.paused)
             this.scheduleInputsChanged(nodeId);
     }
@@ -561,6 +565,7 @@ class GraphRuntime {
                 },
             };
             const exec = async (attempt) => {
+                let hadError = false;
                 try {
                     await node.runtime.onInputsChanged?.(capturedInputs, ctx);
                 }
@@ -572,6 +577,9 @@ class GraphRuntime {
                             return; // ignore switched runs
                         }
                     }
+                    hadError = true;
+                    // Record last error for node
+                    node.stats.lastError = err;
                     const retry = policy.retry;
                     if (retry && attempt < (retry.attempts ?? 0)) {
                         const delay = retry.backoffMs ? retry.backoffMs(attempt) : 0;
@@ -590,6 +598,9 @@ class GraphRuntime {
                         node.stats.lastStartAt && node.stats.lastEndAt
                             ? node.stats.lastEndAt - node.stats.lastStartAt
                             : undefined;
+                    // Clear lastError upon successful completion
+                    if (!hadError)
+                        node.stats.lastError = undefined;
                     this.emit("stats", {
                         kind: "node-done",
                         nodeId,
@@ -651,7 +662,8 @@ class GraphRuntime {
         // fan-out along all edges from this output
         const outEdges = this.edges.filter((e) => e.source.nodeId === srcNodeId && e.source.handle === srcHandle);
         for (const e of outEdges) {
-            let nextVal = value;
+            // Clone per edge to isolate conversions from mutating the shared source value
+            let nextVal = structuredClone(value);
             const applyToTarget = (v) => {
                 const dstNode = this.nodes.get(e.target.nodeId);
                 if (!dstNode)
@@ -688,6 +700,8 @@ class GraphRuntime {
                     e.stats.inFlight = false;
                     e.stats.lastEndAt = Date.now();
                     e.stats.lastDurationMs = e.stats.lastEndAt - startAt;
+                    // Clear lastError on successful conversion
+                    e.stats.lastError = undefined;
                     this.emit("stats", {
                         kind: "edge-done",
                         edgeId: e.id,
@@ -1245,7 +1259,10 @@ class AbstractEngine {
         this.graphRuntime.launch();
     }
     setInput(nodeId, handle, value) {
-        this.graphRuntime.setInput(nodeId, handle, value);
+        this.setInputs(nodeId, { [handle]: value });
+    }
+    setInputs(nodeId, inputs) {
+        this.graphRuntime.setInputs(nodeId, inputs);
     }
     triggerExternal(nodeId, event) {
         this.graphRuntime.triggerExternal(nodeId, event);
@@ -1308,8 +1325,8 @@ class BatchedEngine extends AbstractEngine {
             this.timer = setInterval(() => this.flush(), this.opts.flushIntervalMs);
         }
     }
-    setInput(nodeId, handle, value) {
-        super.setInput(nodeId, handle, value);
+    setInputs(nodeId, inputs) {
+        super.setInputs(nodeId, inputs);
         this.dirtyNodes.add(nodeId);
     }
     triggerExternal(nodeId, event) {
@@ -1385,7 +1402,7 @@ class HybridEngine extends AbstractEngine {
     launch() {
         this.graphRuntime.resume();
     }
-    setInput(nodeId, handle, value) {
+    setInputs(nodeId, inputs) {
         this.updateWindow();
         this.countInWindow += 1;
         const threshold = this.opts.batchThreshold ?? 5;
@@ -1408,7 +1425,7 @@ class HybridEngine extends AbstractEngine {
                 this.flushTimer = undefined;
             }, windowMs);
         }
-        super.setInput(nodeId, handle, value);
+        super.setInputs(nodeId, inputs);
         this.dirtyNodes.add(nodeId);
         if (!this.batching)
             this.graphRuntime.__unsafe_scheduleInputsChanged(nodeId);
@@ -1434,8 +1451,8 @@ class StepEngine extends AbstractEngine {
         this.graphRuntime.pause();
     }
     launch() { }
-    setInput(nodeId, handle, value) {
-        super.setInput(nodeId, handle, value);
+    setInputs(nodeId, inputs) {
+        super.setInputs(nodeId, inputs);
         this.dirtyNodes.add(nodeId);
     }
     triggerExternal(nodeId, event) {