@mui/internal-code-infra 0.0.2-canary.52 → 0.0.2-canary.54

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/internal-code-infra",
-  "version": "0.0.2-canary.52",
+  "version": "0.0.2-canary.54",
   "description": "Infra scripts and configs to be used across MUI repos.",
   "type": "module",
   "license": "MIT",
@@ -21,13 +21,15 @@
   },
   "dependencies": {
     "@argos-ci/core": "^4.0.3",
+    "@babel/cli": "^7.28.3",
     "@babel/core": "^7.28.3",
-    "@babel/plugin-transform-runtime": "^7.28.3",
     "@babel/plugin-syntax-typescript": "^7.27.1",
+    "@babel/plugin-transform-runtime": "^7.28.3",
     "@babel/preset-env": "^7.28.3",
     "@babel/preset-react": "^7.27.1",
     "@babel/preset-typescript": "^7.27.1",
     "@eslint/compat": "^1.3.2",
+    "@eslint/js": "^9.33.0",
     "@next/eslint-plugin-next": "^15.4.6",
     "@octokit/rest": "^22.0.0",
     "@pnpm/find-workspace-dir": "^1000.1.2",
@@ -36,8 +38,6 @@
     "babel-plugin-transform-react-remove-prop-types": "^0.4.24",
     "babel-plugin-transform-remove-imports": "^1.8.0",
     "chalk": "^5.5.0",
-    "eslint-config-airbnb": "^19.0.4",
-    "eslint-config-airbnb-base": "^15.0.0",
     "eslint-config-prettier": "^10.1.8",
     "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-module-utils": "^2.12.1",
@@ -57,13 +57,14 @@
     "semver": "^7.7.2",
     "typescript-eslint": "^8.39.1",
     "yargs": "^18.0.0",
-    "@mui/internal-babel-plugin-display-name": "1.0.4-canary.4",
+    "@mui/internal-babel-plugin-display-name": "1.0.4-canary.5",
     "@mui/internal-babel-plugin-minify-errors": "2.0.8-canary.6",
-    "@mui/internal-babel-plugin-resolve-imports": "2.0.7-canary.16"
+    "@mui/internal-babel-plugin-resolve-imports": "2.0.7-canary.17"
   },
   "peerDependencies": {
     "eslint": "^9.0.0",
-    "prettier": "^3.5.3"
+    "prettier": "^3.5.3",
+    "typescript": "^5.0.0"
   },
   "devDependencies": {
     "@types/babel__core": "^7.20.5",
@@ -89,7 +90,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitSha": "eca4b1b3b0d63fff48e756a7d6264bae3877fa50",
+  "gitSha": "79ad58df711429581fc497ec6344e9c16c9f7de5",
   "scripts": {
     "typescript": "tsc -p tsconfig.json",
     "test": "pnpm -w test --project @mui/internal-code-infra",

package/src/cli/babel.mjs

@@ -134,11 +134,13 @@ export async function babelBuild({
   };
   const res = await $({
     stdio: 'inherit',
+    preferLocal: true,
+    localDir: import.meta.dirname,
     env: {
       ...process.env,
       ...env,
     },
-  })`pnpm babel --config-file ${configFile} --extensions ${TO_TRANSFORM_EXTENSIONS.join(',')} ${sourceDir} --out-dir ${outDir} --ignore ${BASE_IGNORES.concat(ignores).join(',')} --out-file-extension ${outExtension !== '.js' ? outExtension : '.js'} --compact ${hasLargeFiles ? 'false' : 'auto'}`;
+  })`babel --config-file ${configFile} --extensions ${TO_TRANSFORM_EXTENSIONS.join(',')} ${sourceDir} --out-dir ${outDir} --ignore ${BASE_IGNORES.concat(ignores).join(',')} --out-file-extension ${outExtension !== '.js' ? outExtension : '.js'} --compact ${hasLargeFiles ? 'false' : 'auto'}`;
 
   if (res.stderr) {
     throw new Error(`Command: '${res.escapedCommand}' failed with \n${res.stderr}`);

package/src/cli/cmdBuild.mjs

@@ -1,6 +1,5 @@
 /* eslint-disable no-console */
 import { $ } from 'execa';
-import { globby } from 'globby';
 import set from 'lodash-es/set.js';
 import * as fs from 'node:fs/promises';
 import * as path from 'node:path';
@@ -307,6 +306,8 @@ export default /** @type {import('yargs').CommandModule<{}, Args>} */ ({
 
     console.log(`Selected output directory: "${buildDirBase}"`);
 
+    await fs.rm(buildDir, { recursive: true, force: true });
+
     let babelRuntimeVersion = packageJson.dependencies['@babel/runtime'];
     if (babelRuntimeVersion === 'catalog:') {
       // resolve the version from the given package
@@ -423,16 +424,5 @@ export default /** @type {import('yargs').CommandModule<{}, Args>} */ ({
       outputDir: buildDir,
       addTypes: buildTypes,
     });
-
-    // cleanup
-    const tsbuildinfo = await globby('**/*.tsbuildinfo', {
-      absolute: true,
-      cwd: buildDir,
-    });
-    await Promise.all(
-      tsbuildinfo.map(async (file) => {
-        await fs.rm(file);
-      }),
-    );
   },
 });

package/src/cli/typescript.mjs

@@ -17,8 +17,17 @@ const $$ = $({ stdio: 'inherit' });
  * @param {string} outDir - The output directory for the declaration files.
  */
 export async function emitDeclarations(tsconfig, outDir) {
-  console.log(`Building types for ${tsconfig} in ${outDir}`);
-  await $$`tsc -p ${tsconfig} --outDir ${outDir} --declaration --emitDeclarationOnly`;
+  const tsconfigDir = path.dirname(tsconfig);
+  const rootDir = path.resolve(tsconfigDir, './src');
+  await $$`tsc
+    -p ${tsconfig}
+    --rootDir ${rootDir}
+    --outDir ${outDir}
+    --declaration
+    --emitDeclarationOnly
+    --noEmit false
+    --composite false
+    --incremental false`;
 }
 
 /**
@@ -137,6 +146,7 @@ export async function createTypes({ bundles, srcDir, buildDir, cwd, skipTsc, isM
             `The package root is '${cwd}'`,
         );
       }
+      console.log(`Building types for ${tsconfigPath} in ${tmpDir}`);
       await emitDeclarations(tsconfigPath, tmpDir);
     }
 

package/src/eslint/baseConfig.mjs

@@ -1,14 +1,16 @@
 import { includeIgnoreFile } from '@eslint/compat';
+import eslintJs from '@eslint/js';
 import prettier from 'eslint-config-prettier/flat';
-import reactCompilerPlugin from 'eslint-plugin-react-compiler';
+import importPlugin from 'eslint-plugin-import';
+import jsxA11yPlugin from 'eslint-plugin-jsx-a11y';
+import reactPlugin from 'eslint-plugin-react';
+import { configs as reactCompilerPluginConfigs } from 'eslint-plugin-react-compiler';
 import { configs as reactHookConfigs } from 'eslint-plugin-react-hooks';
 import globals from 'globals';
-import * as tseslint from 'typescript-eslint';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
+import * as tseslint from 'typescript-eslint';
 
-import { airbnbBaseConfig, airbnbReactConfig } from './airbnb/base.mjs';
-import airbnbTypescript from './airbnb/typescript.mjs';
 import { createCoreConfig } from './material-ui/config.mjs';
 import muiPlugin from './material-ui/index.mjs';
 /**
@@ -36,16 +38,19 @@ export function createBaseConfig(
   return /** @type {import('eslint').Linter.Config[]} */ (
     tseslint.config(
       ...ignoreRules,
-      airbnbBaseConfig,
-      airbnbReactConfig,
-      airbnbTypescript,
+      eslintJs.configs.recommended,
+      importPlugin.flatConfigs.recommended,
+      importPlugin.flatConfigs.react,
+      jsxA11yPlugin.flatConfigs.recommended,
+      reactPlugin.configs.flat.recommended,
       reactHookConfigs.recommended,
-      enableReactCompiler ? reactCompilerPlugin.configs.recommended : {},
+      tseslint.configs.recommended,
+      importPlugin.flatConfigs.typescript,
+      enableReactCompiler ? reactCompilerPluginConfigs.recommended : {},
       prettier,
       {
         name: 'typescript-eslint-parser',
         languageOptions: {
-          parser: tseslint.parser,
           ecmaVersion: 7,
           globals: {
             ...globals.es2020,
@@ -54,19 +59,8 @@ export function createBaseConfig(
           },
         },
         plugins: {
-          '@typescript-eslint': tseslint.plugin,
           'material-ui': muiPlugin,
         },
-        settings: {
-          'import/parsers': {
-            '@typescript-eslint/parser': ['.ts', '.tsx'],
-          },
-          'import/resolver': {
-            typescript: {
-              project: ['tsconfig.node.json', 'apps/*/tsconfig.json', 'packages/*/tsconfig.json'],
-            },
-          },
-        },
         extends: createCoreConfig({ reactCompilerEnabled: enableReactCompiler }),
       },
       {
@@ -82,7 +76,6 @@ export function createBaseConfig(
           ],
         },
       },
-
       // Lint rule to disallow usage of typescript namespaces.We've seen at least two problems with them:
       //   * Creates non-portable types in base ui. [1]
       //   * This pattern [2] leads to broken bundling in codesandbox [3].
@@ -96,6 +89,14 @@ export function createBaseConfig(
           '@typescript-eslint/no-namespace': 'error',
         },
       },
+      // Part of the migration away from airbnb config. Turned of initially.
+      {
+        rules: {
+          '@typescript-eslint/no-explicit-any': 'off',
+          '@typescript-eslint/no-unsafe-function-type': 'off',
+          '@typescript-eslint/no-empty-object-type': 'off',
+        },
+      },
     )
   );
 }

package/src/eslint/material-ui/config.mjs

@@ -5,6 +5,301 @@ const restrictedSyntaxRules = restrictedMethods.map((method) => ({
   selector: `MemberExpression[object.name='window'][property.name='${method}']`,
 }));
 
+/**
+ * Critical Airbnb rules missing from individual plugin recommended configs
+ * @type {import('eslint').Linter.Config['rules']}
+ */
+const criticalAirbnbRules = {
+  // Security & Best Practices
+  'no-eval': 'error',
+  'no-script-url': 'error',
+  'array-callback-return': ['error', { allowImplicit: true }],
+  'consistent-return': 'error',
+  eqeqeq: ['error', 'always', { null: 'ignore' }],
+  // 'no-param-reassign': ['error', { props: true }],
+  'guard-for-in': 'error',
+  radix: 'error',
+  // disallow usage of __proto__ property
+  // https://eslint.org/docs/rules/no-proto
+  'no-proto': 'error',
+  'vars-on-top': 'error',
+  // disallow use of labels for anything other than loops and switches
+  // https://eslint.org/docs/rules/no-labels
+  'no-labels': ['error', { allowLoop: false, allowSwitch: false }],
+  // disallow unnecessary nested blocks
+  // https://eslint.org/docs/rules/no-lone-blocks
+  'no-lone-blocks': 'error',
+  // disallow use of new operator when not part of the assignment or comparison
+  // https://eslint.org/docs/rules/no-new
+  'no-new': 'error',
+  // disallow use of new operator for Function object
+  // https://eslint.org/docs/rules/no-new-func
+  'no-new-func': 'error',
+
+  // disallow comparisons where both sides are exactly the same
+  // https://eslint.org/docs/rules/no-self-compare
+  'no-self-compare': 'error',
+  'no-restricted-globals': ['error', 'isFinite', 'isNaN'],
+
+  // Styles
+  // require or disallow an empty line between class members
+  // https://eslint.org/docs/rules/lines-between-class-members
+  'lines-between-class-members': ['error', 'always', { exceptAfterSingleLine: false }],
+  // require a capital letter for constructors
+  'new-cap': [
+    'error',
+    {
+      newIsCap: true,
+      newIsCapExceptions: [],
+      capIsNew: false,
+      capIsNewExceptions: ['Immutable.Map', 'Immutable.Set', 'Immutable.List'],
+    },
+  ],
+  // disallow use of unary operators, ++ and --
+  // https://eslint.org/docs/rules/no-plusplus
+  'no-plusplus': 'error',
+  // disallow use of bitwise operators
+  // https://eslint.org/docs/rules/no-bitwise
+  'no-bitwise': 'error',
+  // disallow if as the only statement in an else block
+  // https://eslint.org/docs/rules/no-lonely-if
+  'no-lonely-if': 'error',
+
+  // ES6+ Modern JavaScript
+  'no-var': 'error',
+  'prefer-const': ['error', { destructuring: 'any', ignoreReadBeforeAssign: true }],
+  'prefer-template': 'error',
+  'object-shorthand': ['error', 'always'],
+
+  // Error
+  // Disallow template literal placeholder syntax in regular strings
+  // https://eslint.org/docs/rules/no-template-curly-in-string
+  'no-template-curly-in-string': 'error',
+  // Disallow returning values from Promise executor functions
+  // https://eslint.org/docs/rules/no-promise-executor-return
+  'no-promise-executor-return': 'error',
+
+  // Import rules (critical ones not in recommended)
+  'import/order': ['error', { groups: [['builtin', 'external', 'internal']] }],
+  'import/first': 'error',
+  'import/no-mutable-exports': 'error',
+  'import/newline-after-import': 'error',
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/namespace.md
+  'import/namespace': 'off',
+  // Forbid require() calls with expressions
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-dynamic-require.md
+  'import/no-dynamic-require': 'error',
+
+  // Additional best practices
+  'default-case': ['error', { commentPattern: '^no default$' }],
+  'default-case-last': 'error',
+  'no-else-return': ['error', { allowElseIf: false }],
+  'no-multi-assign': 'error',
+  'no-nested-ternary': 'error',
+  'no-unneeded-ternary': ['error', { defaultAssignment: false }],
+  'spaced-comment': [
+    'error',
+    'always',
+    {
+      line: { markers: ['/'], exceptions: ['-', '+'] },
+      block: { markers: ['*'], exceptions: ['*'], balanced: true },
+    },
+  ],
+  // require all requires be top-level
+  // https://eslint.org/docs/rules/global-require
+  'global-require': 'error',
+  // disallow use of assignment in return statement
+  // https://eslint.org/docs/rules/no-return-assign
+  'no-return-assign': ['error', 'always'],
+  // disallow useless string concatenation
+  // https://eslint.org/docs/rules/no-useless-concat
+  'no-useless-concat': 'error',
+
+  // Disallow await inside of loops
+  // https://eslint.org/docs/rules/no-await-in-loop
+  'no-await-in-loop': 'error',
+  // disallow assignment in conditional expressions
+  'no-cond-assign': ['error', 'always'],
+
+  // React
+  // Prevent usage of button elements without an explicit type attribute
+  // https://github.com/jsx-eslint/eslint-plugin-react/blob/843d71a432baf0f01f598d7cf1eea75ad6896e4b/docs/rules/button-has-type.md
+  'react/button-has-type': [
+    'error',
+    {
+      button: true,
+      submit: true,
+      reset: false,
+    },
+  ],
+  // Prevent missing displayName in a React component definition
+  // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/display-name.md
+  'react/display-name': ['off', { ignoreTranspilerName: false }],
+  // Enforce a specific function type for function components
+  // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/function-component-definition.md
+  'react/function-component-definition': [
+    'error',
+    {
+      namedComponents: ['function-declaration', 'function-expression'],
+      unnamedComponents: 'function-expression',
+    },
+  ],
+  // Prevent react contexts from taking non-stable values
+  // https://github.com/jsx-eslint/eslint-plugin-react/blob/e2eaadae316f9506d163812a09424eb42698470a/docs/rules/jsx-no-constructed-context-values.md
+  'react/jsx-no-constructed-context-values': 'error',
+  // Require stateless functions when not using lifecycle methods, setState or ref
+  // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/prefer-stateless-function.md
+  'react/prefer-stateless-function': ['error', { ignorePureComponents: true }],
+  // Forbids using non-exported propTypes
+  // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/forbid-foreign-prop-types.md
+  // this is intentionally set to "warn". it would be "error",
+  // but it's only critical if you're stripping propTypes in production.
+  'react/forbid-foreign-prop-types': ['warn', { allowInPropTypes: true }],
+  // Validate JSX has key prop when in array or iterator
+  // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-key.md
+  // Turned off because it has too many false positives
+  'react/jsx-key': 'off',
+  // Prevent unused propType definitions
+  // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-unused-prop-types.md
+  'react/no-unused-prop-types': [
+    'error',
+    {
+      customValidators: [],
+      skipShapeProps: true,
+    },
+  ],
+};
+
+/**
+ * TypeScript-specific rules (replaces Airbnb TypeScript config)
+ * @type {import('eslint').Linter.Config['rules']}
+ */
+const typescriptOverrides = {
+  // The following rules are enabled in Airbnb config, but are recommended to be disabled within TypeScript projects
+  // See: https://github.com/typescript-eslint/typescript-eslint/blob/13583e65f5973da2a7ae8384493c5e00014db51b/docs/linting/TROUBLESHOOTING.md#eslint-plugin-import
+  'import/named': 'off',
+  'import/no-named-as-default-member': 'off',
+  'import/no-unresolved': 'off',
+
+  // TypeScript equivalents of ESLint rules
+  'default-param-last': 'off',
+  '@typescript-eslint/default-param-last': 'error',
+  'no-array-constructor': 'off',
+  '@typescript-eslint/no-array-constructor': 'error',
+  '@typescript-eslint/triple-slash-reference': 'off',
+  'no-empty-function': 'off',
+  // disallow empty functions, except for standalone funcs/arrows
+  // https://eslint.org/docs/rules/no-empty-function
+  '@typescript-eslint/no-empty-function': [
+    'error',
+    {
+      allow: ['arrowFunctions', 'functions', 'methods'],
+    },
+  ],
+  'no-loss-of-precision': 'error',
+  'no-loop-func': 'off',
+  '@typescript-eslint/no-loop-func': 'error',
+  'no-shadow': 'off',
+  '@typescript-eslint/no-shadow': 'error',
+  'no-unused-expressions': 'off',
+  '@typescript-eslint/no-unused-expressions': [
+    'error',
+    { allowShortCircuit: false, allowTernary: false },
+  ],
+  'no-useless-constructor': 'off',
+  '@typescript-eslint/no-useless-constructor': 'error',
+  'require-await': 'off',
+
+  // TypeScript naming convention (replaces camelcase)
+  camelcase: 'off',
+
+  // The `@typescript-eslint/naming-convention` rule allows `leadingUnderscore` and `trailingUnderscore` settings. However, the existing `no-underscore-dangle` rule already takes care of this.
+  '@typescript-eslint/naming-convention': [
+    'error',
+    // Allow camelCase variables (23.2), PascalCase variables (23.8), and UPPER_CASE variables (23.10)
+    {
+      selector: 'variable',
+      format: ['camelCase', 'PascalCase', 'UPPER_CASE'],
+    },
+    // Allow camelCase functions (23.2), and PascalCase functions (23.8)
+    {
+      selector: 'function',
+      format: ['camelCase', 'PascalCase'],
+    },
+    // Airbnb recommends PascalCase for classes (23.3), and although Airbnb does not make TypeScript recommendations, we are assuming this rule would similarly apply to anything "type like", including interfaces, type aliases, and enums
+    {
+      selector: 'typeLike',
+      format: ['PascalCase'],
+    },
+  ],
+
+  // Namespace rule
+  '@typescript-eslint/no-namespace': 'off',
+  '@typescript-eslint/no-this-alias': 'off',
+
+  // TypeScript extensions handling
+  'import/extensions': [
+    'error',
+    'ignorePackages',
+    {
+      js: 'never',
+      mjs: 'never',
+      jsx: 'never',
+      ts: 'never',
+      tsx: 'never',
+    },
+  ],
+};
+
+/**
+ * @type {import('eslint').Linter.Config['rules']}
+ */
+const airbnbJsxA11y = {
+  // Enforce that all elements that require alternative text have meaningful information
+  // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/alt-text.md
+  'jsx-a11y/alt-text': [
+    'error',
+    {
+      elements: ['img'],
+    },
+  ],
+
+  // ensure <a> tags are valid
+  // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/0745af376cdc8686d85a361ce36952b1fb1ccf6e/docs/rules/anchor-is-valid.md
+  'jsx-a11y/anchor-is-valid': [
+    'error',
+    {
+      components: ['Link'],
+      specialLink: ['to'],
+      aspects: ['noHref', 'invalidHref', 'preferButton'],
+    },
+  ],
+
+  // Enforce that a control (an interactive element) has a text label.
+  // https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/master/docs/rules/control-has-associated-label.md
+  'jsx-a11y/control-has-associated-label': [
+    'error',
+    {
+      labelAttributes: ['label'],
+      controlComponents: [],
+      ignoreElements: ['audio', 'canvas', 'embed', 'input', 'textarea', 'tr', 'video'],
+      ignoreRoles: [
+        'grid',
+        'listbox',
+        'menu',
+        'menubar',
+        'radiogroup',
+        'row',
+        'tablist',
+        'toolbar',
+        'tree',
+        'treegrid',
+      ],
+      depth: 5,
+    },
+  ],
+};
+
 /**
  * @param {Object} [options]
  * @param {boolean} [options.reactCompilerEnabled] - Whether the config is for spec files.
@@ -14,7 +309,29 @@ export function createCoreConfig(options = {}) {
   return [
     {
       name: 'material-ui-base',
+      settings: {
+        'import/resolver': {
+          node: {
+            extensions: ['.mjs', '.js', '.json'],
+          },
+          typescript: {
+            project: ['tsconfig.node.json', 'apps/*/tsconfig.json', 'packages/*/tsconfig.json'],
+          },
+        },
+        'import/extensions': ['.js', '.mjs', '.jsx', '.ts', '.tsx', '.d.ts'],
+        'import/core-modules': [],
+        'import/ignore': ['node_modules', '\\.(css|svg|json)$'],
+        // Override with TypeScript-specific settings
+        'import/parsers': {
+          '@typescript-eslint/parser': ['.ts', '.tsx'],
+        },
+        // Extend Airbnb extensions with TypeScript
+        'import/external-module-folders': ['node_modules', 'node_modules/@types'],
+      },
       rules: {
+        ...criticalAirbnbRules,
+        ...airbnbJsxA11y,
+        ...typescriptOverrides,
         'no-redeclare': 'off',
         '@typescript-eslint/no-redeclare': 'off',
         'consistent-this': ['error', 'self'],
@@ -39,9 +356,7 @@ export function createCoreConfig(options = {}) {
         'no-throw-literal': 'error',
         // Use the proptype inheritance chain
         'no-prototype-builtins': 'off',
-        'no-return-await': 'error',
         'no-underscore-dangle': 'error',
-        'nonblock-statement-body-position': 'error',
         'prefer-arrow-callback': ['error', { allowNamedFunctions: true }],
         // Destructuring harm grep potential.
         'prefer-destructuring': 'off',

package/src/eslint/testConfig.mjs

@@ -18,6 +18,7 @@ export const baseSpecRules = {
     '@typescript-eslint/no-unused-expressions': 'off',
     '@typescript-eslint/no-unused-vars': 'off',
     '@typescript-eslint/no-use-before-define': 'off',
+    '@typescript-eslint/ban-ts-comment': 'off',
 
     'import/prefer-default-export': 'off',
 
@@ -84,6 +85,12 @@ export function createTestConfig(options = {}) {
           // that they don't need type-checking
           'react/prop-types': 'off',
           'react/no-unused-prop-types': 'off',
+          // Part of the migration away from airbnb config. Turned of initially.
+          '@typescript-eslint/no-empty-function': 'off',
+          '@typescript-eslint/ban-ts-comment': 'off',
+          'testing-library/no-node-access': 'off',
+          '@typescript-eslint/no-non-null-asserted-optional-chain': 'off',
+          // end migration
           ...(useMocha
             ? {
                 'mocha/consistent-spacing-between-blocks': 'off',

package/src/eslint/airbnb/base.mjs

@@ -1,113 +0,0 @@
-/**
- * Reassembles the ESLint Airbnb base configuration for usage with
- * flat Eslint configuration.
- */
-import baseBestPractices from 'eslint-config-airbnb-base/rules/best-practices';
-import baseErrors from 'eslint-config-airbnb-base/rules/errors';
-import baseEs6 from 'eslint-config-airbnb-base/rules/es6';
-import baseImports from 'eslint-config-airbnb-base/rules/imports';
-import baseNode from 'eslint-config-airbnb-base/rules/node';
-import baseStrict from 'eslint-config-airbnb-base/rules/strict';
-import baseStyle from 'eslint-config-airbnb-base/rules/style';
-import baseVariables from 'eslint-config-airbnb-base/rules/variables';
-import airbnbReact from 'eslint-config-airbnb/rules/react';
-import airbnbReactA11y from 'eslint-config-airbnb/rules/react-a11y';
-import eslintPluginImport from 'eslint-plugin-import';
-import eslintPluginJsxA11y from 'eslint-plugin-jsx-a11y';
-import eslintPluginReact from 'eslint-plugin-react';
-
-import globals from 'globals';
-import * as tseslint from 'typescript-eslint';
-
-const baseES6Plugin = {
-  languageOptions: {
-    globals: {
-      ...globals.es2016,
-    },
-    parserOptions: baseEs6.parserOptions,
-  },
-  rules: baseEs6.rules,
-};
-
-const baseImportPlugin = {
-  languageOptions: {
-    globals: {
-      ...globals.es2016,
-    },
-    parserOptions: baseImports.parserOptions,
-  },
-  settings: baseImports.settings,
-  rules: baseImports.rules,
-  plugins: {
-    import: eslintPluginImport,
-  },
-};
-
-const baseNodePlugin = {
-  languageOptions: {
-    globals: {
-      ...globals.node,
-    },
-  },
-  rules: baseNode.rules,
-};
-
-export const airbnbBaseConfig = /** @type {import('eslint').Linter.Config[]} */ (
-  tseslint.config(
-    {
-      name: 'base-best-practices',
-      ...baseBestPractices,
-    },
-    {
-      name: 'base-errors',
-      ...baseErrors,
-    },
-    {
-      name: 'base-node',
-      ...baseNodePlugin,
-    },
-    { name: 'base-style', ...baseStyle },
-    { name: 'base-variables', ...baseVariables },
-    { name: 'base-es6-plugin', ...baseES6Plugin },
-    { name: 'base-import-plugin', ...baseImportPlugin },
-    { name: 'base-strict', ...baseStrict },
-  )
-);
-
-/**
- * @type {import('typescript-eslint').InfiniteDepthConfigWithExtends}
- */
-const airbnbReactPlugin = {
-  languageOptions: {
-    parserOptions: {
-      ecmaFeatures: {
-        jsx: true,
-      },
-    },
-  },
-  plugins: {
-    react: eslintPluginReact,
-  },
-  rules: airbnbReact.rules,
-};
-
-const airbnbReactA11yPlugin = {
-  plugins: {
-    'jsx-a11y': eslintPluginJsxA11y,
-  },
-  languageOptions: {
-    parserOptions: {
-      ecmaFeatures: {
-        jsx: true,
-      },
-    },
-  },
-  rules: airbnbReactA11y.rules,
-};
-
-export const airbnbReactConfig = /** @type {import('eslint').Linter.Config[]} */ (
-  tseslint.config(
-    { name: 'airbnb-react', ...airbnbReactPlugin },
-    { name: 'airbnb-react-a11y', ...airbnbReactA11yPlugin },
-  )
-);

package/src/eslint/airbnb/typescript.mjs

@@ -1,126 +0,0 @@
-/**
- * Reassembles the ESLint Airbnb typescript configuration for usage with
- * flat Eslint configuration.
- */
-import baseBestPractices from 'eslint-config-airbnb-base/rules/best-practices';
-import baseEs6 from 'eslint-config-airbnb-base/rules/es6';
-import baseImports from 'eslint-config-airbnb-base/rules/imports';
-import baseStyle from 'eslint-config-airbnb-base/rules/style';
-import baseVariables from 'eslint-config-airbnb-base/rules/variables';
-import * as tseslint from 'typescript-eslint';
-
-const baseImportsRules = baseImports.rules;
-
-if (!Array.isArray(baseImportsRules?.['import/extensions'])) {
-  throw new Error(
-    'Expected `import/extensions` rule to be an array in `eslint-config-airbnb-base/rules/imports`',
-  );
-}
-
-export default /** @type {import('typescript-eslint').ConfigArray} */ (
-  tseslint.config(
-    {
-      settings: {
-        'import/parsers': {
-          '@typescript-eslint/parser': ['.ts', '.tsx', '.d.ts'],
-        },
-        'import/resolver': {
-          node: {
-            extensions: ['.mjs', '.js', '.jsx', '.json', '.ts', '.tsx', '.d.ts'],
-          },
-        },
-        // Append 'ts' extensions to Airbnb 'import/extensions' setting
-        // Original: ['.js', '.mjs', '.jsx']
-        'import/extensions': ['.js', '.mjs', '.jsx', '.ts', '.tsx', '.d.ts'],
-        // Resolve type definition packages
-        'import/external-module-folders': ['node_modules', 'node_modules/@types'],
-      },
-      rules: {
-        camelcase: 'off',
-        // The `@typescript-eslint/naming-convention` rule allows `leadingUnderscore` and `trailingUnderscore` settings. However, the existing `no-underscore-dangle` rule already takes care of this.
-        '@typescript-eslint/naming-convention': [
-          'error',
-          // Allow camelCase variables (23.2), PascalCase variables (23.8), and UPPER_CASE variables (23.10)
-          {
-            selector: 'variable',
-            format: ['camelCase', 'PascalCase', 'UPPER_CASE'],
-          },
-          // Allow camelCase functions (23.2), and PascalCase functions (23.8)
-          {
-            selector: 'function',
-            format: ['camelCase', 'PascalCase'],
-          },
-          // Airbnb recommends PascalCase for classes (23.3), and although Airbnb does not make TypeScript recommendations, we are assuming this rule would similarly apply to anything "type like", including interfaces, type aliases, and enums
-          {
-            selector: 'typeLike',
-            format: ['PascalCase'],
-          },
-        ],
-        'default-param-last': 'off',
-        '@typescript-eslint/default-param-last': baseBestPractices.rules?.['default-param-last'],
-        'no-array-constructor': 'off',
-        '@typescript-eslint/no-array-constructor': baseStyle.rules?.['no-array-constructor'],
-        'no-empty-function': 'off',
-        '@typescript-eslint/no-empty-function': baseBestPractices.rules?.['no-empty-function'],
-        'no-loss-of-precision': 'error',
-        'no-loop-func': 'off',
-        '@typescript-eslint/no-loop-func': baseBestPractices.rules?.['no-loop-func'],
-        'no-magic-numbers': 'off',
-        '@typescript-eslint/no-magic-numbers': baseBestPractices.rules?.['no-magic-numbers'],
-        'no-shadow': 'off',
-        '@typescript-eslint/no-shadow': baseVariables.rules?.['no-shadow'],
-        'no-unused-expressions': 'off',
-        '@typescript-eslint/no-unused-expressions':
-          baseBestPractices.rules?.['no-unused-expressions'],
-        'no-useless-constructor': 'off',
-        '@typescript-eslint/no-useless-constructor': baseEs6.rules?.['no-useless-constructor'],
-        'require-await': 'off',
-        '@typescript-eslint/require-await': baseBestPractices.rules?.['require-await'],
-
-        // Append 'ts' and 'tsx' to Airbnb 'import/extensions' rule
-        // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/extensions.md
-        'import/extensions': [
-          baseImportsRules['import/extensions'][0],
-          baseImportsRules['import/extensions'][1],
-          typeof baseImportsRules['import/extensions'][2] === 'object'
-            ? {
-                ...baseImportsRules['import/extensions'][2],
-                ts: 'never',
-                tsx: 'never',
-              }
-            : { ts: 'never', tsx: 'never' },
-        ],
-      },
-    },
-    {
-      files: ['**/*.ts', '**/*.tsx'],
-      rules: {
-        // The following rules are enabled in Airbnb config, but are already checked (more thoroughly) by the TypeScript compiler
-        // Some of the rules also fail in TypeScript files, for example: https://github.com/typescript-eslint/typescript-eslint/issues/662#issuecomment-507081586
-        // Rules are inspired by: https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/eslint-recommended.ts
-        'constructor-super': 'off',
-        'getter-return': 'off',
-        'no-const-assign': 'off',
-        'no-dupe-args': 'off',
-        'no-dupe-class-members': 'off',
-        'no-dupe-keys': 'off',
-        'no-func-assign': 'off',
-        'no-import-assign': 'off',
-        'no-new-symbol': 'off',
-        'no-obj-calls': 'off',
-        'no-redeclare': 'off',
-        'no-setter-return': 'off',
-        'no-this-before-super': 'off',
-        'no-undef': 'off',
-        'no-unreachable': 'off',
-        'no-unsafe-negation': 'off',
-        'valid-typeof': 'off',
-        // The following rules are enabled in Airbnb config, but are recommended to be disabled within TypeScript projects
-        // See: https://github.com/typescript-eslint/typescript-eslint/blob/13583e65f5973da2a7ae8384493c5e00014db51b/docs/linting/TROUBLESHOOTING.md#eslint-plugin-import
-        'import/named': 'off',
-        'import/no-named-as-default-member': 'off',
-        'import/no-unresolved': 'off',
-      },
-    },
-  )
-);