Merge pull request #19 from openpatch/copilot/fix-max-update-depth-error

Fix infinite loop in LearningMap component caused by unstable useEffect dependencies

Author: mikebarkmin

packages/learningmap/src/LearningMap.tsx

@@ -124,7 +124,7 @@ export function LearningMap({
         root.dispatchEvent(new CustomEvent("change", { detail: minimalState }));
       }
     }
-  }, [nodes, onChange, getViewport, getRoadmapState]);
+  }, [nodes, onChange]);
 
   const defaultEdgeOptions = {
     animated: false,

packages/learningmap/src/autoLayoutElk.ts

@@ -319,18 +319,10 @@ export const useEditorStore = create<EditorState>()(
         setSelectedNodeId: (selectedNodeId) =>
           set({
             selectedNodeId,
-            nodes: get().nodes.map((n) => ({
-              ...n,
-              selected: n.id === selectedNodeId,
-            })),
           }),
         setSelectedNodeIds: (selectedNodeIds) =>
           set({
             selectedNodeIds,
-            nodes: get().nodes.map((n) => ({
-              ...n,
-              selected: selectedNodeIds.includes(n.id),
-            })),
           }),
         setSelectedEdge: (selectedEdge) => set({ selectedEdge }),
         setNextNodeId: (nextNodeId) => set({ nextNodeId }),