@bian-womp/spark-workbench 0.2.15 → 0.2.17

package/lib/cjs/index.cjs

@@ -1535,17 +1535,23 @@ function WorkbenchProvider({ wb, runner, registry, setRegistry, children, }) {
                 const snap = await runner.snapshotFull();
                 const remoteDef = snap?.def;
                 if (remoteDef && Array.isArray(remoteDef.nodes)) {
+                    // Mutate current def in-place to avoid emitting graphChanged and causing update loop
                     const cur = wb.export();
-                    const merged = {
-                        ...cur,
-                        nodes: cur.nodes.map((n) => {
-                            const rn = (remoteDef.nodes || []).find((m) => m.nodeId === n.nodeId);
-                            if (rn && rn.resolvedHandles)
-                                return { ...n, resolvedHandles: rn.resolvedHandles };
-                            return n;
-                        }),
-                    };
-                    await wb.load(merged);
+                    const byId = new Map((remoteDef.nodes || []).map((n) => [n.nodeId, n]));
+                    let changed = false;
+                    for (const n of cur.nodes) {
+                        const rn = byId.get(n.nodeId);
+                        if (rn && rn.resolvedHandles) {
+                            const before = JSON.stringify(n.resolvedHandles || {});
+                            const after = JSON.stringify(rn.resolvedHandles || {});
+                            if (before !== after) {
+                                n.resolvedHandles = rn.resolvedHandles;
+                                changed = true;
+                            }
+                        }
+                    }
+                    if (changed)
+                        wb.refreshValidation();
                 }
             }
             catch { }
@@ -1557,8 +1563,6 @@ function WorkbenchProvider({ wb, runner, registry, setRegistry, children, }) {
                     ...s,
                     [nodeId]: { ...s[nodeId], invalidated: true },
                 }));
-                // On any input change, refresh resolved handles (covers context-driven dynamic ports)
-                refreshResolvedHandles();
             }
             return add("runner", "value")(e);
         });
@@ -1602,9 +1606,10 @@ function WorkbenchProvider({ wb, runner, registry, setRegistry, children, }) {
                     return next;
                 });
             }
-            // After build/update, pull resolved handles from remote snapshot and merge into local def for UI
-            if (e?.reason === "graph-updated" || e?.reason === "graph-built")
+            // After build/update, pull resolved handles and merge in-place (no graphChanged)
+            if (e?.reason === "graph-updated" || e?.reason === "graph-built") {
                 refreshResolvedHandles();
+            }
             return add("runner", "invalidate")(e);
         });
         const off3b = runner.on("stats", (s) => {