fix(TreeView): use prespecified active and selected on mount only (#11697)

* fix(TreeView): update onSelect signature

* fix(TreeView): use prespecified active and selected on mount only

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Co-authored-by: Taylor Jones <tay1orjones@users.noreply.github.com>

Author: 3 people

packages/react/src/components/TreeView/TreeView.js

@@ -51,11 +51,12 @@ export default function TreeView({
       } else {
         setSelected(selected.filter((selectedId) => selectedId !== nodeId));
       }
+      onSelect?.(event, node);
     } else {
       setSelected([nodeId]);
       setActive(nodeId);
+      onSelect?.(event, { activeNodeId: nodeId, ...node });
     }
-    onSelect?.(event, node);
   }
 
   function handleFocusEvent(event) {
@@ -183,14 +184,17 @@ export default function TreeView({
       });
   }, [prefix]);
 
-  useEffect(() => {
-    if (preselected.length) {
-      setSelected(preselected);
-    }
-    if (prespecifiedActive) {
-      setActive(prespecifiedActive);
-    }
-  }, [preselected, prespecifiedActive]);
+  const useActiveAndSelectedOnMount = () =>
+    useEffect(() => {
+      if (preselected.length) {
+        setSelected(preselected);
+      }
+      if (prespecifiedActive) {
+        setActive(prespecifiedActive);
+      }
+    }, []);
+
+  useActiveAndSelectedOnMount();
 
   const labelId = `${treeId}__label`;
   const TreeLabel = () =>
@@ -199,6 +203,7 @@ export default function TreeView({
         {label}
       </label>
     );
+
   return (
     <>
       <TreeLabel />