[utils] Fix issues reported by the React Compiler (#43051)

Signed-off-by: Mark <markliu2013@gmail.com>
Co-authored-by: Aarón García Hervás <aaron.garcia.hervas@gmail.com>

packages/mui-utils/src/useControlled/useControlled.js

@@ -1,4 +1,5 @@
 'use client';
+// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- process.env never changes, dependency arrays are intentionally ignored
 /* eslint-disable react-hooks/rules-of-hooks, react-hooks/exhaustive-deps */
 import * as React from 'react';
 

packages/mui-utils/src/useForkRef/useForkRef.ts

@@ -20,6 +20,7 @@ export default function useForkRef<Instance>(
         setRef(ref, instance);
       });
     };
+    // TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- intentionally ignoring that the dependency array must be an array literal
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, refs);
 }

packages/mui-utils/src/useId/useId.ts

@@ -31,6 +31,7 @@ export default function useId(idOverride?: string): string | undefined {
     const reactId = maybeReactUseId();
     return idOverride ?? reactId;
   }
+  // TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler
   // eslint-disable-next-line react-hooks/rules-of-hooks -- `React.useId` is invariant at runtime.
   return useGlobalId(idOverride);
 }

packages/mui-utils/src/useOnMount/useOnMount.ts

@@ -7,6 +7,7 @@ const EMPTY = [] as unknown[];
  * A React.useEffect equivalent that runs once, when the component is mounted.
  */
 export default function useOnMount(fn: React.EffectCallback) {
+  // TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- no need to put `fn` in the dependency array
   /* eslint-disable react-hooks/exhaustive-deps */
   React.useEffect(fn, EMPTY);
   /* eslint-enable react-hooks/exhaustive-deps */