@mui/internal-code-infra 0.0.3-canary.95 → 0.0.4-canary.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/internal-code-infra",
-  "version": "0.0.3-canary.95",
+  "version": "0.0.4-canary.0",
   "description": "Infra scripts and configs to be used across MUI repos.",
   "type": "module",
   "license": "MIT",
@@ -59,8 +59,8 @@
     "@babel/preset-react": "^7.28.5",
     "@babel/preset-typescript": "^7.28.5",
     "@eslint/compat": "^2.0.2",
-    "@eslint/js": "^9.39.2",
-    "@eslint/json": "^0.14.0",
+    "@eslint/js": "^10.0.1",
+    "@eslint/json": "^1.0.1",
     "@inquirer/confirm": "^6.0.4",
     "@inquirer/select": "^5.0.4",
     "@napi-rs/keyring": "^1.2.0",
@@ -68,8 +68,8 @@
     "@octokit/oauth-methods": "^6.0.2",
     "@octokit/rest": "^22.0.1",
     "@pnpm/find-workspace-dir": "^1000.1.3",
-    "@typescript-eslint/types": "^8.56.0",
-    "@typescript-eslint/utils": "^8.56.0",
+    "@typescript-eslint/types": "^8.56.1",
+    "@typescript-eslint/utils": "^8.56.1",
     "@vitest/eslint-plugin": "^1.6.9",
     "babel-plugin-optimize-clsx": "^2.6.2",
     "babel-plugin-react-compiler": "^1.0.0",
@@ -110,17 +110,17 @@
     "resolve-pkg-maps": "^1.0.0",
     "semver": "^7.7.4",
     "stylelint-config-standard": "^40.0.0",
-    "typescript-eslint": "^8.56.0",
+    "typescript-eslint": "^8.56.1",
     "unified": "^11.0.5",
     "yargs": "^18.0.0",
-    "@mui/internal-babel-plugin-minify-errors": "2.0.8-canary.22",
+    "@mui/internal-babel-plugin-display-name": "1.0.4-canary.13",
     "@mui/internal-babel-plugin-resolve-imports": "2.0.7-canary.32",
-    "@mui/internal-babel-plugin-display-name": "1.0.4-canary.13"
+    "@mui/internal-babel-plugin-minify-errors": "2.0.8-canary.22"
   },
   "peerDependencies": {
     "@next/eslint-plugin-next": "*",
     "@typescript/native-preview": ">=7.0.0-dev.0",
-    "eslint": "^9.0.0",
+    "eslint": "^9.0.0 || ^10.0.0",
     "prettier": "^3.0.0",
     "typescript": "^5.0.0"
   },
@@ -143,13 +143,13 @@
     "@types/estree-jsx": "1.0.5",
     "@types/regexp.escape": "2.0.0",
     "@types/yargs": "17.0.35",
-    "@typescript-eslint/parser": "8.56.0",
-    "@typescript-eslint/rule-tester": "8.56.0",
-    "eslint": "9.39.2",
+    "@typescript-eslint/parser": "8.56.1",
+    "@typescript-eslint/rule-tester": "8.56.1",
+    "eslint": "10.0.2",
     "get-port": "7.1.0",
     "prettier": "3.8.1",
     "serve": "14.2.5",
-    "typescript-eslint": "8.56.0"
+    "typescript-eslint": "8.56.1"
   },
   "files": [
     "bin",
@@ -161,7 +161,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitSha": "74232307111caed6f4d181e941df5c358f57ed28",
+  "gitSha": "91b038cb6c568b63684a7be0146dd12747e76060",
   "scripts": {
     "build": "tsgo -p tsconfig.build.json",
     "typescript": "tsgo -p tsconfig.json",

package/src/eslint/baseConfig.mjs

@@ -1,4 +1,4 @@
-import { includeIgnoreFile } from '@eslint/compat';
+import { includeIgnoreFile, fixupConfigRules } from '@eslint/compat';
 import eslintJs from '@eslint/js';
 import { defineConfig } from 'eslint/config';
 import prettier from 'eslint-config-prettier/flat';
@@ -52,13 +52,13 @@ export function createBaseConfig({
       files: [`**/*${EXTENSION_TS}`],
       extends: defineConfig([
         eslintJs.configs.recommended,
-        importPlugin.flatConfigs.recommended,
-        importPlugin.flatConfigs.react,
-        jsxA11yPlugin.flatConfigs.recommended,
-        reactPlugin.configs.flat.recommended,
-        reactHooks.configs.flat.recommended,
+        // Fix ESLint 10 compatibility for plugins that use deprecated context methods
+        ...fixupConfigRules([importPlugin.flatConfigs.recommended, importPlugin.flatConfigs.react]),
+        ...fixupConfigRules(jsxA11yPlugin.flatConfigs.recommended),
+        ...fixupConfigRules(reactPlugin.configs.flat.recommended),
+        ...fixupConfigRules(reactHooks.configs.flat.recommended),
         tseslint.configs.recommended,
-        importPlugin.flatConfigs.typescript,
+        ...fixupConfigRules(importPlugin.flatConfigs.typescript),
         enableReactCompiler ? reactCompilerPluginConfigs.recommended : {},
         compatPlugin.configs['flat/recommended'],
         {
@@ -104,6 +104,18 @@ export function createBaseConfig({
             '@typescript-eslint/no-empty-object-type': 'off',
           },
         },
+        // New ESLint 10 rules - turned off initially to ease migration
+        {
+          name: 'ESLint 10 new rules',
+          rules: {
+            // Requires attaching caught errors as `cause` when re-throwing
+            'preserve-caught-error': 'off',
+            // Disallows assignments that are never used
+            'no-useless-assignment': 'off',
+            // Disallows unused vars without explicit init (use @typescript-eslint/no-unused-vars instead)
+            'no-unassigned-vars': 'off',
+          },
+        },
       ]),
     },
     {

package/src/eslint/mui/rules/disallow-active-elements-as-key-event-target.test.mjs

@@ -22,7 +22,6 @@ ruleTester.run('disallow-active-element-as-key-event-target', rule, {
         {
           message:
             "Don't use document.activeElement as a target for keyboard events. Prefer the actual element.",
-          type: 'MemberExpression',
         },
       ],
     },
@@ -32,7 +31,6 @@ ruleTester.run('disallow-active-element-as-key-event-target', rule, {
         {
           message:
             "Don't use document.activeElement as a target for keyboard events. Prefer the actual element.",
-          type: 'MemberExpression',
         },
       ],
     },
@@ -42,7 +40,6 @@ ruleTester.run('disallow-active-element-as-key-event-target', rule, {
         {
           message:
             "Don't use document.activeElement as a target for keyboard events. Prefer the actual element.",
-          type: 'MemberExpression',
         },
       ],
     },
@@ -52,7 +49,6 @@ ruleTester.run('disallow-active-element-as-key-event-target', rule, {
         {
           message:
             "Don't use document.activeElement as a target for keyboard events. Prefer the actual element.",
-          type: 'MemberExpression',
         },
       ],
     },
@@ -63,7 +59,6 @@ ruleTester.run('disallow-active-element-as-key-event-target', rule, {
         {
           message:
             "Don't use document.activeElement as a target for keyboard events. Prefer the actual element.",
-          type: 'TSNonNullExpression',
         },
       ],
     },

package/src/eslint/mui/rules/docgen-ignore-before-comment.test.mjs

@@ -21,7 +21,6 @@ ruleTester.run('ignore-before-comment', rule, {
       errors: [
         {
           message: '@ignore should be at the beginning of a block comment.',
-          type: 'Block',
         },
       ],
     },
@@ -30,7 +29,6 @@ ruleTester.run('ignore-before-comment', rule, {
       errors: [
         {
           message: '@ignore should be at the beginning of a block comment.',
-          type: 'Block',
         },
       ],
     },
@@ -39,7 +37,6 @@ ruleTester.run('ignore-before-comment', rule, {
       errors: [
         {
           message: '@ignore should be at the beginning of a block comment.',
-          type: 'Block',
         },
       ],
     },
@@ -48,7 +45,6 @@ ruleTester.run('ignore-before-comment', rule, {
       errors: [
         {
           message: '@ignore should be at the beginning of a block comment.',
-          type: 'Block',
         },
       ],
     },

package/src/eslint/mui/rules/mui-name-matches-component-name.test.mjs

@@ -138,7 +138,6 @@ ruleTester.run('mui-name-matches-component-name', rule, {
         {
           message:
             "Expected `name` to be 'MuiStaticDateRangePicker' but instead got 'MuiPickersDateRangePicker'.",
-          type: 'Literal',
         },
       ],
     },
@@ -147,7 +146,6 @@ ruleTester.run('mui-name-matches-component-name', rule, {
       errors: [
         {
           message: 'Unable to find `name` property. Did you forget to pass `name`?',
-          type: 'ObjectExpression',
         },
       ],
     },
@@ -157,7 +155,6 @@ ruleTester.run('mui-name-matches-component-name', rule, {
         {
           message:
             'Unable to resolve `name`. Please hardcode the `name` i.e. use a string literal.',
-          type: 'Identifier',
         },
       ],
     },
@@ -166,7 +163,6 @@ ruleTester.run('mui-name-matches-component-name', rule, {
       errors: [
         {
           message: 'Unable to find component for this call.',
-          type: 'CallExpression',
         },
       ],
     },
@@ -184,7 +180,6 @@ ruleTester.run('mui-name-matches-component-name', rule, {
         {
           message:
             "Expected `name` to be 'MuiStaticDateRangePicker' but instead got 'MuiPickersDateRangePicker'.",
-          type: 'Literal',
         },
       ],
     },
@@ -203,7 +198,6 @@ ruleTester.run('mui-name-matches-component-name', rule, {
         {
           message:
             "Expected `name` to be 'MuiStaticDateRangePicker' but instead got 'MuiPickersDateRangePicker'.",
-          type: 'Literal',
         },
       ],
     },
@@ -212,7 +206,6 @@ ruleTester.run('mui-name-matches-component-name', rule, {
       errors: [
         {
           message: 'Unable to find `name` property. Did you forget to pass `name`?',
-          type: 'ObjectExpression',
         },
       ],
     },
@@ -222,7 +215,6 @@ ruleTester.run('mui-name-matches-component-name', rule, {
         {
           message:
             'Unable to resolve `name`. Please hardcode the `name` i.e. use a string literal.',
-          type: 'Identifier',
         },
       ],
     },
@@ -231,7 +223,6 @@ ruleTester.run('mui-name-matches-component-name', rule, {
       errors: [
         {
           message: 'Unable to find component for this call.',
-          type: 'CallExpression',
         },
       ],
     },
@@ -249,7 +240,6 @@ ruleTester.run('mui-name-matches-component-name', rule, {
         {
           message:
             "Unable to find name argument. Expected `useDatePickerDefaultizedProps(firstParameter, 'MuiComponent')`.",
-          type: 'Identifier',
         },
       ],
     },

package/src/eslint/mui/rules/no-empty-box.test.mjs

@@ -19,7 +19,6 @@ ruleTester.run('no-empty-box', rule, {
       errors: [
         {
           messageId: 'emptyBox',
-          type: 'JSXOpeningElement',
           data: {
             component: 'div',
           },
@@ -31,7 +30,6 @@ ruleTester.run('no-empty-box', rule, {
       errors: [
         {
           messageId: 'emptyBox',
-          type: 'JSXOpeningElement',
           data: {
             component: 'span',
           },

package/src/eslint/mui/rules/no-styled-box.test.mjs

@@ -35,7 +35,6 @@ const foo = styled(Box)({
       errors: [
         {
           messageId: 'noBox',
-          type: 'CallExpression',
         },
       ],
       output: `
@@ -59,7 +58,6 @@ const foo = styled(Box, {})({
       errors: [
         {
           messageId: 'noBox',
-          type: 'CallExpression',
         },
       ],
       output: `

package/src/utils/extractErrorCodes.mjs

@@ -147,7 +147,9 @@ export default async function extractErrorCodes(args) {
     Object.entries(existingErrorCodes).map(([key, value]) => [value, Number(key)]),
   );
   const originalErrorCount = inverseLookupCode.size;
-  let newErrorCodeStart = Math.max(...Array.from(inverseLookupCode.values()));
+  let newErrorCodeStart =
+    inverseLookupCode.size > 0 ? Math.max(...Array.from(inverseLookupCode.values())) : 0;
+
   if (newErrorCodeStart !== originalErrorCount) {
     console.warn(
       `⚠️ Warning: Detected a gap in the error codes. Current max code is ${newErrorCodeStart}, but there are only ${originalErrorCount} existing codes.\nThis can happen when codes have been removed. New codes will continue from the highest existing code.`,