Only update state if excalidraw "scene" actually changes.

This fixes syncing bidirectionally, but there can still be timing issues that overwrite one person's changes

ui/src/GriffyPane.svelte

@@ -35,17 +35,17 @@
 
   $: if ($state) {
     if (excalidrawAPI) {
-      // TODO: maybe check scene version to see if its changed? getSceneVersion($state.excalidrawElements)
+      // XXX: Need to do a cloneDeep here too otherwise resizing elements doesn't trigger
+      //      updateExcalidrawState because it must be editing the $state.excalidrawElements directly and so the scene # is updated internally
       excalidrawAPI.updateScene({
-        elements: $state.excalidrawElements,
+        elements: cloneDeep($state.excalidrawElements),
         //appState: $state.excalidrawState
       })
     }
   }
 
   onMount(async () => {
     console.log('mounted', excalidrawAPI, $state)
-
 	});
 
   const setExcalidrawAPI = (api) => {
@@ -86,8 +86,10 @@
   }
 
   const updateExcalidrawState = throttle((excalidrawElements, appState, files) => {
-    activeBoard.requestChanges([{type: 'set-excalidraw', excalidrawElements, excalidrawState: appState}])
-  }, 5000)
+    if (getSceneVersion($state.excalidrawElements) !== getSceneVersion(excalidrawElements)) {
+      activeBoard.requestChanges([{type: 'set-excalidraw', excalidrawElements, excalidrawState: appState}])
+    }
+  }, 3000)
 
 </script>
 <div class="board" >

ui/src/board.ts

@@ -75,8 +75,8 @@ export const boardGrammar = {
         break;
       case "set-excalidraw":
         // For some reason customData is set to undefined which breaks in syn code getValueDescription
-        const excalidrawElements = delta.excalidrawElements.map(e => pickBy(e, (v, k) => typeof(v) !== "undefined"))
-        state.excalidrawElements = cloneDeep(excalidrawElements)
+        const newExcalidrawElements = cloneDeep(delta.excalidrawElements).map(e => pickBy(e, (v, k) => typeof(v) !== "undefined"))
+        state.excalidrawElements = newExcalidrawElements
         //state.excalidrawState = delta.excalidrawState
         break;
     }