@bian-womp/spark-workbench 0.2.33 → 0.2.34

package/lib/cjs/index.cjs

@@ -567,7 +567,10 @@ class LocalGraphRunner extends AbstractGraphRunner {
             this.build(payload.def);
         this.setEnvironment?.(payload.environment || {}, { merge: false });
         // Hydrate via runtime for exact restore and re-emit
-        this.runtime?.hydrate({ inputs: payload.inputs || {}, outputs: payload.outputs || {} }, { reemit: true });
+        this.runtime?.hydrate({
+            inputs: payload.inputs || {},
+            outputs: payload.outputs || {},
+        });
     }
     dispose() {
         super.dispose();
@@ -1629,7 +1632,7 @@ function toReactFlow(def, positions, registry, opts) {
     });
     const edges = def.edges.map((e) => {
         const st = opts.edgeStatus?.[e.id];
-        const isRunning = !!st?.activeRuns;
+        const isRunning = (st?.activeRuns || 0) > 0;
         const hasError = !!st?.lastError;
         const isInvalidEdge = !!opts.edgeValidation?.[e.id];
         const sourceMissing = !validHandleMap[e.source.nodeId]?.outputs.has(e.source.handle);
@@ -1666,7 +1669,7 @@ function getNodeBorderClassNames(args) {
     const hasError = !!status.lastError;
     const hasValidationError = issues.some((i) => i.level === "error");
     const hasValidationWarning = !hasValidationError && issues.length > 0;
-    const isRunning = !!status.activeRuns;
+    const isRunning = (status.activeRuns || 0) > 0;
     const isInvalid = !!status.invalidated && !isRunning && !hasError;
     // Keep border width constant to avoid layout reflow on selection toggles
     const borderWidth = "border";
@@ -2051,8 +2054,12 @@ function WorkbenchProvider({ wb, runner, registry, setRegistry, overrides, uiVer
                 const isValidRunId = runId && typeof runId === "string" && runId.length > 0;
                 setNodeStatus((prev) => {
                     const current = prev[id]?.activeRuns ?? 0;
-                    const nextActive = current - 1;
                     const currentRunIds = prev[id]?.activeRunIds ?? [];
+                    if (isValidRunId && !currentRunIds.includes(runId)) {
+                        console.warn(`[WorkbenchContext] node-done event for unknown runId: node=${id} runId=${runId}`, { event: s, currentRunIds });
+                        return prev; // Ignore stale event
+                    }
+                    const nextActive = Math.max(0, current - 1); // Prevent negative values
                     const nextRunIds = isValidRunId
                         ? currentRunIds.filter((rid) => rid !== runId)
                         : currentRunIds;
@@ -2097,7 +2104,7 @@ function WorkbenchProvider({ wb, runner, registry, setRegistry, overrides, uiVer
                     const current = prev[id]?.activeRuns ?? 0;
                     return {
                         ...prev,
-                        [id]: { ...prev[id], activeRuns: current - 1 },
+                        [id]: { ...prev[id], activeRuns: Math.max(0, current - 1) }, // Prevent negative values
                     };
                 });
             }