windmill-components 1.504.3 → 1.504.4

package/package/components/FlowBuilder.svelte

@@ -779,15 +779,14 @@ const flowHasChanged = $derived(flowPreviewContent?.flowHasChanged());
 						disabled={customUi?.topBar?.editableSummary == false}
 						bind:value={flowStore.val.summary}
 					/>
-
 					<UndoRedo
 						undoProps={{ disabled: $history.index === 0 }}
 						redoProps={{ disabled: $history.index === $history.history.length - 1 }}
 						on:undo={() => {
 							const currentModules = flowStore.val?.value?.modules
-							console.log('undo before', flowStore.val, JSON.stringify(flowStore.val, null, 2))
+							// console.log('undo before', flowStore.val, JSON.stringify(flowStore.val, null, 2))
 							flowStore.val = undo(history, flowStore.val)
-							console.log('undo after', flowStore.val, JSON.stringify(flowStore.val, null, 2))
+							// console.log('undo after', flowStore.val, JSON.stringify(flowStore.val, null, 2))
 
 							const newModules = flowStore.val?.value?.modules
 							const restoredModules = newModules?.filter(

package/package/components/graph/FlowGraphV2.svelte

@@ -262,7 +262,7 @@ let eventHandler = {
         onHideJobStatus?.();
     }
 };
-let lastModules = structuredClone($state.snapshot(modules));
+let lastModules = $state.snapshot(modules);
 let moduleCounter = $state(0);
 function onModulesChange2(modules) {
     if (!deepEqual(modules, lastModules)) {

package/package/history.svelte.js

@@ -8,7 +8,8 @@ export function undo(history, now) {
         let chistory = get(history);
         if (chistory.index == 0)
             return now;
-        if (!deepEqual(chistory.history[chistory.index], now)) {
+        if (chistory.index == chistory.history.length - 1 &&
+            !deepEqual(chistory.history[chistory.index], now)) {
             push(history, now, true);
         }
         else {
@@ -20,10 +21,12 @@ export function undo(history, now) {
         let nhistory = get(history);
         let r = nhistory.history[nhistory.index];
         if (!r) {
+            console.log('c');
             console.log('undo failed', nhistory, now);
             return now;
         }
         else {
+            console.log('d');
             return r;
         }
     }

package/package/utils.js

@@ -1202,12 +1202,11 @@ export function scroll_into_view_if_needed_polyfill(elem, centerIfNeeded = true)
     observer.observe(elem);
     return observer; // return for testing
 }
+// Structured clone raises an error on $state values
+// $state.snapshot clones everything but prints warnings for some values (e.g. functions)
+import _clone from 'clone';
 export function clone(t) {
-    if (typeof t === 'function')
-        throw new Error('Cannot clone a function');
-    if (typeof t === 'object')
-        return stateSnapshot(t);
-    return t;
+    return _clone(t);
 }
 export const editorPositionMap = {};
 export function parseS3Object(s3Object) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "windmill-components",
-	"version": "1.504.3",
+	"version": "1.504.4",
 	"scripts": {
 		"dev": "vite dev",
 		"build": "vite build",
@@ -103,6 +103,7 @@
 		"chart.js": "^4.4.6",
 		"chartjs-adapter-date-fns": "^3.0.0",
 		"chartjs-plugin-zoom": "^2.0.0",
+		"clone": "^2.1.2",
 		"d3-zoom": "^3.0.0",
 		"date-fns": "^2.30.0",
 		"diff": "^7.0.0",