@mui/internal-code-infra 0.0.4-canary.3 → 0.0.4-canary.31

package/src/eslint/mui/config.mjs

@@ -1,4 +1,5 @@
-import { defineConfig } from 'eslint/config';
+// TODO: change back to 'eslint/config' once https://github.com/eslint/rewrite/issues/425 is fixed
+import { defineConfig } from '@eslint/config-helpers';
 
 const restrictedMethods = ['setTimeout', 'setInterval', 'clearTimeout', 'clearInterval'];
 
@@ -499,6 +500,15 @@ export function createCoreConfig(options = {}) {
             message: 'Do not call `Error(...)` without `new`. Use `new Error(...)` instead.',
             selector: "CallExpression[callee.name='Error']",
           },
+          {
+            // xmlns="http://www.w3.org/2000/svg" is only needed on standalone .svg files so the
+            // browser treats them as SVG instead of generic XML. Inside HTML the <svg> element is
+            // already recognised by the browser, so the attribute is dead weight.
+            // https://github.com/mui/mui-public/pull/1321
+            message:
+              'Remove xmlns from inline <svg>. The attribute is redundant in HTML and adds unnecessary bytes.',
+            selector: 'JSXOpeningElement[name.name="svg"] > JSXAttribute[name.name="xmlns"]',
+          },
           ...restrictedSyntaxRules,
         ],
 

package/src/eslint/testConfig.mjs

@@ -1,7 +1,8 @@
 import mochaPlugin from 'eslint-plugin-mocha';
 import vitestPlugin from '@vitest/eslint-plugin';
 import testingLibrary from 'eslint-plugin-testing-library';
-import { defineConfig } from 'eslint/config';
+// TODO: change back to 'eslint/config' once https://github.com/eslint/rewrite/issues/425 is fixed
+import { defineConfig } from '@eslint/config-helpers';
 import globals from 'globals';
 import * as tseslint from 'typescript-eslint';
 import { EXTENSION_TS } from './extensions.mjs';

package/src/estree-typescript.d.ts

@@ -1,4 +1,4 @@
-import { BaseExpression, Expression } from 'estree';
+import { type BaseExpression, type Expression } from 'estree';
 
 export * from 'estree';
 

package/src/untyped-plugins.d.ts

@@ -13,77 +13,77 @@ declare module 'eslint-config-airbnb' {
 }
 
 declare module 'eslint-config-airbnb-base/rules/best-practices' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   declare const config: Omit<Linter.LegacyConfig, 'extends' | 'plugins'>;
   export default config;
 }
 
 declare module 'eslint-config-airbnb-base/rules/errors' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   declare const config: Omit<Linter.LegacyConfig, 'extends' | 'plugins'>;
   export default config;
 }
 
 declare module 'eslint-config-airbnb-base/rules/es6' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   declare const config: Omit<Linter.LegacyConfig, 'extends' | 'plugins'>;
   export default config;
 }
 
 declare module 'eslint-config-airbnb-base/rules/imports' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   declare const config: Omit<Linter.LegacyConfig, 'extends' | 'plugins'>;
   export default config;
 }
 
 declare module 'eslint-config-airbnb-base/rules/node' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   declare const config: Omit<Linter.LegacyConfig, 'extends' | 'plugins'>;
   export default config;
 }
 
 declare module 'eslint-config-airbnb-base/rules/strict' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   declare const config: Omit<Linter.LegacyConfig, 'extends' | 'plugins'>;
   export default config;
 }
 
 declare module 'eslint-config-airbnb-base/rules/style' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   declare const config: Omit<Linter.LegacyConfig, 'extends' | 'plugins'>;
   export default config;
 }
 
 declare module 'eslint-config-airbnb-base/rules/variables' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   declare const config: Omit<Linter.LegacyConfig, 'extends' | 'plugins'>;
   export default config;
 }
 
 declare module 'eslint-config-airbnb/rules/react' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   declare const config: Omit<Linter.LegacyConfig, 'extends' | 'plugins'>;
   export default config;
 }
 
 declare module 'eslint-config-airbnb/rules/react-a11y' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   declare const config: Omit<Linter.LegacyConfig, 'extends' | 'plugins'>;
   export default config;
 }
 
 declare module '@next/eslint-plugin-next' {
-  import { Linter } from 'eslint';
+  import { type Linter } from 'eslint';
 
   interface NextEslintPluginConfig extends Linter.LegacyConfig {
     flatConfig: {