Fix infinite loop in LearningMap useEffect by removing unstable function deps

Copilot · mikebarkmin · Copilot · commit 82433bf73bc0 · 2025-10-13T21:48:07.000Z
Co-authored-by: mikebarkmin &lt;2592379+mikebarkmin@users.noreply.github.com&gt;
diff --git a/packages/learningmap/src/LearningMap.tsx b/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,

Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,7 @@ export function LearningMap({`
`124`	`124`	`root.dispatchEvent(new CustomEvent("change", { detail: minimalState }));`
`125`	`125`	`}`
`126`	`126`	`}`
`127`		`- }, [nodes, onChange, getViewport, getRoadmapState]);`
	`127`	`+ }, [nodes, onChange]);`
`128`	`128`
`129`	`129`	`const defaultEdgeOptions = {`
`130`	`130`	`animated: false,`