[material-ui][private-theming] Fix issues reported by react compiler …

…in `mui-private-theming` components (#42619)
mui · Jun 11, 2024 · c7caa1d · c7caa1d
1 parent 2cc5193
commit c7caa1d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/packages/mui-private-theming/src/ThemeProvider/ThemeProvider.js b/packages/mui-private-theming/src/ThemeProvider/ThemeProvider.js
@@ -52,7 +52,8 @@ function ThemeProvider(props) {
   }
 
   const theme = React.useMemo(() => {
-    const output = outerTheme === null ? localTheme : mergeOuterLocalTheme(outerTheme, localTheme);
+    const output =
+      outerTheme === null ? { ...localTheme } : mergeOuterLocalTheme(outerTheme, localTheme);
 
     if (output != null) {
       output[nested] = outerTheme !== null;

diff --git a/packages/mui-private-theming/src/useTheme/useTheme.js b/packages/mui-private-theming/src/useTheme/useTheme.js
@@ -5,7 +5,8 @@ export default function useTheme() {
   const theme = React.useContext(ThemeContext);
 
   if (process.env.NODE_ENV !== 'production') {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
+    // 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 -- It's not required to run React.useDebugValue in production
     React.useDebugValue(theme);
   }