@mui/internal-code-infra 0.0.4-canary.65 → 0.0.4-canary.67

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/internal-code-infra",
-  "version": "0.0.4-canary.65",
+  "version": "0.0.4-canary.67",
   "author": "MUI Team",
   "description": "Infra scripts and configs to be used across MUI repos.",
   "license": "MIT",
@@ -29,10 +29,6 @@
       "types": "./build/utils/github.d.mts",
       "default": "./src/utils/github.mjs"
     },
-    "./markdownlint": {
-      "types": "./build/markdownlint/index.d.mts",
-      "default": "./src/markdownlint/index.mjs"
-    },
     "./remark": {
       "types": "./build/remark/config.d.mts",
       "default": "./src/remark/config.mjs"
@@ -67,7 +63,6 @@
     "@babel/preset-react": "^7.28.5",
     "@babel/preset-typescript": "^7.28.5",
     "@eslint/compat": "^2.1.0",
-    "@eslint/config-helpers": "^0.6.0",
     "@eslint/js": "^10.0.1",
     "@eslint/json": "^1.2.0",
     "@inquirer/confirm": "^6.0.13",
@@ -140,9 +135,9 @@
     "unist-util-visit": "^5.1.0",
     "yaml": "^2.9.0",
     "yargs": "^18.0.0",
+    "@mui/internal-babel-plugin-resolve-imports": "2.0.7-canary.36",
     "@mui/internal-babel-plugin-display-name": "1.0.4-canary.20",
-    "@mui/internal-babel-plugin-minify-errors": "2.0.8-canary.27",
-    "@mui/internal-babel-plugin-resolve-imports": "2.0.7-canary.36"
+    "@mui/internal-babel-plugin-minify-errors": "2.0.8-canary.27"
   },
   "peerDependencies": {
     "@next/eslint-plugin-next": "*",
@@ -192,7 +187,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitSha": "308d63cc49db1419769226ebad3e2ceec1dd49d2",
+  "gitSha": "b0b08ac265cb2f9411c8cc32b09a8d0bd5093fac",
   "scripts": {
     "build": "tsgo -p tsconfig.build.json",
     "typescript": "tsgo -noEmit",

package/src/eslint/baseConfig.mjs

@@ -1,7 +1,6 @@
 import { includeIgnoreFile, fixupConfigRules } from '@eslint/compat';
 import eslintJs from '@eslint/js';
-// TODO: change back to 'eslint/config' once https://github.com/eslint/rewrite/issues/425 is fixed
-import { defineConfig } from '@eslint/config-helpers';
+import { defineConfig } from 'eslint/config';
 import prettier from 'eslint-config-prettier/flat';
 import compatPlugin from 'eslint-plugin-compat';
 import importPlugin from 'eslint-plugin-import';

package/src/eslint/docsConfig.mjs

@@ -1,6 +1,5 @@
 import nextjs from '@next/eslint-plugin-next';
-// TODO: change back to 'eslint/config' once https://github.com/eslint/rewrite/issues/425 is fixed
-import { defineConfig } from '@eslint/config-helpers';
+import { defineConfig } from 'eslint/config';
 import { EXTENSION_TS } from './extensions.mjs';
 
 /**

package/src/eslint/jsonConfig.mjs

@@ -1,5 +1,4 @@
-// TODO: change back to 'eslint/config' once https://github.com/eslint/rewrite/issues/425 is fixed
-import { defineConfig } from '@eslint/config-helpers';
+import { defineConfig } from 'eslint/config';
 import json from '@eslint/json';
 
 /**

package/src/eslint/mui/config.mjs

@@ -1,5 +1,4 @@
-// TODO: change back to 'eslint/config' once https://github.com/eslint/rewrite/issues/425 is fixed
-import { defineConfig } from '@eslint/config-helpers';
+import { defineConfig } from 'eslint/config';
 import { EXTENSION_DTS } from '../extensions.mjs';
 
 const restrictedMethods = ['setTimeout', 'setInterval', 'clearTimeout', 'clearInterval'];

package/src/eslint/testConfig.mjs

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

package/build/markdownlint/duplicate-h1.d.mts

@@ -1,56 +0,0 @@
-/**
- * @typedef {[string, string]} Attr
- */
-export type Attr = [string, string];
-export type Token = {
-    type: string;
-    info: string;
-    tag: string;
-    content: string;
-    lineNumber: number;
-    attrs: Attr[];
-};
-export type OnErrorObj = {
-    lineNumber: number;
-    detail?: string;
-};
-export type OnError = (err: OnErrorObj) => void;
-export type MdParams = {
-    name: string;
-    lines: string[];
-    tokens: Token[];
-};
-/**
- * @typedef {Object} Token
- * @property {string} type
- * @property {string} info
- * @property {string} tag
- * @property {string} content
- * @property {number} lineNumber
- * @property {Attr[]} attrs
- */
-/**
- * @typedef {Object} OnErrorObj
- * @property {number} lineNumber
- * @property {string} [detail]
- */
-/**
- * @typedef {(err: OnErrorObj) => void} OnError
- */
-/**
- * @typedef {Object} MdParams
- * @property {string} name
- * @property {string[]} lines
- * @property {Token[]} tokens
- */
-declare const _default: {
-    names: string[];
-    description: string;
-    tags: string[];
-    /**
-     * @param {import('./duplicate-h1.mjs').MdParams} params
-     * @param {import('./duplicate-h1.mjs').OnError} onError
-     */
-    function: (params: import('./duplicate-h1.mjs').MdParams, onError: import('./duplicate-h1.mjs').OnError) => void;
-};
-export default _default;

package/build/markdownlint/git-diff.d.mts

@@ -1,11 +0,0 @@
-declare const _default: {
-    names: string[];
-    description: string;
-    tags: string[];
-    /**
-     * @param {import('./duplicate-h1.mjs').MdParams} params
-     * @param {import('./duplicate-h1.mjs').OnError} onError
-     */
-    function: (params: import('./duplicate-h1.mjs').MdParams, onError: import('./duplicate-h1.mjs').OnError) => void;
-};
-export default _default;

package/build/markdownlint/index.d.mts

@@ -1,52 +0,0 @@
-import straightQuotes from './straight-quotes.mjs';
-/**
- * Create a base configuration for markdownlint.
- * @param {Object} options
- * @param {(typeof straightQuotes)[]} [options.rules] - An array of custom rules to include.
- * @param {string[]} [options.ignores] - An array of glob patterns to ignore.
- * @returns
- */
-export declare function createBaseConfig(options?: {
-    rules?: (typeof straightQuotes)[];
-    ignores?: string[];
-}): {
-    config: {
-        default: boolean;
-        MD004: boolean;
-        MD009: {
-            br_spaces: number;
-            strict: boolean;
-            list_item_empty_lines: boolean;
-        };
-        MD013: boolean;
-        MD014: boolean;
-        MD024: {
-            siblings_only: boolean;
-        };
-        MD025: {
-            level: number;
-            front_matter_title: string;
-        };
-        MD033: boolean;
-        MD034: boolean;
-        MD028: boolean;
-        MD029: boolean;
-        MD031: boolean;
-        MD036: boolean;
-        MD051: boolean;
-        MD052: boolean;
-        MD059: boolean;
-        straightQuotes: boolean;
-        gitDiff: boolean;
-        tableAlignment: boolean;
-        terminalLanguage: boolean;
-        duplicateH1: boolean;
-    };
-    customRules: {
-        names: string[];
-        description: string;
-        tags: string[];
-        function: (params: import('./duplicate-h1.mjs').MdParams, onError: import('./duplicate-h1.mjs').OnError) => void;
-    }[];
-    ignores: string[];
-};

package/build/markdownlint/straight-quotes.d.mts

@@ -1,11 +0,0 @@
-declare const _default: {
-    names: string[];
-    description: string;
-    tags: string[];
-    /**
-     * @param {import('./duplicate-h1.mjs').MdParams} params
-     * @param {import('./duplicate-h1.mjs').OnError} onError
-     */
-    function: (params: import('./duplicate-h1.mjs').MdParams, onError: import('./duplicate-h1.mjs').OnError) => void;
-};
-export default _default;

package/build/markdownlint/table-alignment.d.mts

@@ -1,11 +0,0 @@
-declare const _default: {
-    names: string[];
-    description: string;
-    tags: string[];
-    /**
-     * @param {import('./duplicate-h1.mjs').MdParams} params
-     * @param {import('./duplicate-h1.mjs').OnError} onError
-     */
-    function: (params: import('./duplicate-h1.mjs').MdParams, onError: import('./duplicate-h1.mjs').OnError) => void;
-};
-export default _default;

package/build/markdownlint/terminal-language.d.mts

@@ -1,11 +0,0 @@
-declare const _default: {
-    names: string[];
-    description: string;
-    tags: string[];
-    /**
-     * @param {import('./duplicate-h1.mjs').MdParams} params
-     * @param {import('./duplicate-h1.mjs').OnError} onError
-     */
-    function: (params: import('./duplicate-h1.mjs').MdParams, onError: import('./duplicate-h1.mjs').OnError) => void;
-};
-export default _default;

package/src/markdownlint/duplicate-h1.mjs

@@ -1,69 +0,0 @@
-/**
- * @typedef {[string, string]} Attr
- */
-
-/**
- * @typedef {Object} Token
- * @property {string} type
- * @property {string} info
- * @property {string} tag
- * @property {string} content
- * @property {number} lineNumber
- * @property {Attr[]} attrs
- */
-
-/**
- * @typedef {Object} OnErrorObj
- * @property {number} lineNumber
- * @property {string} [detail]
- */
-
-/**
- * @typedef {(err: OnErrorObj) => void} OnError
- */
-
-/**
- * @typedef {Object} MdParams
- * @property {string} name
- * @property {string[]} lines
- * @property {Token[]} tokens
- */
-
-// This rule is an extension of MD025/no-multiple-top-level-headings.
-// The rule is buggy https://github.com/DavidAnson/markdownlint/pull/1109
-// but also blog headers don't tell you that h1 is already injected.
-export default {
-  names: ['duplicateH1'],
-  description: 'Multiple top-level headings in the same document.',
-  tags: ['headings'],
-  /**
-   * @param {import('./duplicate-h1.mjs').MdParams} params
-   * @param {import('./duplicate-h1.mjs').OnError} onError
-   */
-  function: (params, onError) => {
-    /**
-     * @type {number|boolean}
-     */
-    let hasTopLevelHeading = false;
-    params.tokens.forEach((token) => {
-      if (token.type === 'heading_open' && token.tag === 'h1') {
-        // Avoid duplicate errors with MD025.
-        if (hasTopLevelHeading !== false && hasTopLevelHeading !== 1) {
-          onError({
-            lineNumber: token.lineNumber,
-          });
-        } else if (params.name.includes('/docs/pages/blog/')) {
-          onError({
-            lineNumber: token.lineNumber,
-            detail: 'In the blog, the h1 is already added using the markdown header.title value.',
-          });
-        }
-
-        // Store the first h1 of the page.
-        if (hasTopLevelHeading === false) {
-          hasTopLevelHeading = token.lineNumber;
-        }
-      }
-    });
-  },
-};

package/src/markdownlint/git-diff.mjs

@@ -1,31 +0,0 @@
-export default {
-  names: ['gitDiff'],
-  description: 'Respect the format output of git diff.',
-  tags: ['spaces'],
-  /**
-   * @param {import('./duplicate-h1.mjs').MdParams} params
-   * @param {import('./duplicate-h1.mjs').OnError} onError
-   */
-  function: (params, onError) => {
-    params.tokens.forEach((token) => {
-      if (token.type === 'fence' && token.info === 'diff') {
-        token.content.split('\n').forEach((line, index) => {
-          if (
-            line[0] !== ' ' &&
-            line[0] !== '-' &&
-            line[0] !== '+' &&
-            line !== '' &&
-            line.indexOf('@@ ') !== 0 &&
-            line.indexOf('diff --git ') !== 0 &&
-            line.indexOf('index ') !== 0
-          ) {
-            onError({
-              lineNumber: token.lineNumber + index + 1,
-              detail: `The line start with "+" or "-" or " ": ${line}`,
-            });
-          }
-        });
-      }
-    });
-  },
-};

package/src/markdownlint/index.mjs

@@ -1,62 +0,0 @@
-import straightQuotes from './straight-quotes.mjs';
-import gitDiff from './git-diff.mjs';
-import tableAlignment from './table-alignment.mjs';
-import terminalLanguage from './terminal-language.mjs';
-import duplicateH1 from './duplicate-h1.mjs';
-
-/**
- * Create a base configuration for markdownlint.
- * @param {Object} options
- * @param {(typeof straightQuotes)[]} [options.rules] - An array of custom rules to include.
- * @param {string[]} [options.ignores] - An array of glob patterns to ignore.
- * @returns
- */
-export function createBaseConfig(options = {}) {
-  const { rules = [], ignores = [] } = options;
-  // https://github.com/DavidAnson/markdownlint#rules--aliases
-  return {
-    config: {
-      default: true,
-      MD004: false, // MD004/ul-style. Buggy
-      MD009: {
-        // MD009/no-trailing-spaces
-        br_spaces: 0,
-        strict: true,
-        list_item_empty_lines: false,
-      },
-      MD013: false, // MD013/line-length. Already handled by Prettier.
-      MD014: false, // MD014/commands-show-output. It's OK.
-      MD024: { siblings_only: true }, // MD024/no-duplicate-heading/no-duplicate-header
-      MD025: {
-        // Heading level
-        level: 1,
-        // RegExp for matching title in front matter
-        front_matter_title: '',
-      },
-      MD033: false, // MD033/no-inline-html. We use it from time to time, it's fine.
-      MD034: false, // MD034/no-bare-urls. Not a concern for us, our Markdown interpreter supports it.
-      MD028: false, // MD028/no-blanks-blockquote prevent double blockquote
-      MD029: false, // MD029/ol-prefix. Buggy
-      MD031: false, // MD031/blanks-around-fences Some code blocks inside li
-      MD036: false, // MD036/no-emphasis-as-heading/no-emphasis-as-header. It's OK.
-      MD051: false, // MD051/link-fragments. Many false positives in the changelog.
-      MD052: false, // MD052/reference-links-images. Many false positives in the changelog.
-      MD059: false, // MD059/descriptive-link-text. Does not allow links on text like "link", whereas we redirect to "Link" component.
-      straightQuotes: true,
-      gitDiff: true,
-      tableAlignment: true,
-      terminalLanguage: true,
-      duplicateH1: true,
-    },
-    customRules: [straightQuotes, gitDiff, tableAlignment, terminalLanguage, duplicateH1, ...rules],
-    ignores: [
-      'CHANGELOG.old.md',
-      '**/node_modules/**',
-      '**/build/**',
-      '.github/PULL_REQUEST_TEMPLATE.md',
-      'docs/public/**',
-      'docs/export/**',
-      ...ignores,
-    ],
-  };
-}

package/src/markdownlint/straight-quotes.mjs

@@ -1,26 +0,0 @@
-const nonStraightQuotes = /[‘’“”]/;
-
-export default {
-  names: ['straightQuotes'],
-  description: 'Only allow straight quotes.',
-  tags: ['spelling'],
-  /**
-   * @param {import('./duplicate-h1.mjs').MdParams} params
-   * @param {import('./duplicate-h1.mjs').OnError} onError
-   */
-  function: (params, onError) => {
-    params.lines.forEach((line, lineNumber) => {
-      // It will match
-      // opening single quote: \xE2\x80\x98
-      // closing single quote: \xE2\x80\x99
-      // opening double quote: \xE2\x80\x9C
-      // closing double quote: \xE2\x80\x9D
-      if (nonStraightQuotes.test(line)) {
-        onError({
-          lineNumber: lineNumber + 1,
-          detail: `For line: ${line}`,
-        });
-      }
-    });
-  },
-};

package/src/markdownlint/table-alignment.mjs

@@ -1,42 +0,0 @@
-/**
- * @param {import('./duplicate-h1.mjs').Attr[]} attrs
- * @returns {Record<string, string>}
- */
-function attr(attrs) {
-  return (attrs || []).reduce((acc, item) => ({ ...acc, [item[0]]: item[1] }), {});
-}
-
-export default {
-  names: ['tableAlignment'],
-  description: 'Set table alignment.',
-  tags: ['table'],
-  /**
-   * @param {import('./duplicate-h1.mjs').MdParams} params
-   * @param {import('./duplicate-h1.mjs').OnError} onError
-   */
-  function: (params, onError) => {
-    params.tokens.forEach((token) => {
-      // This is wrong:
-      // | Version | Supported          |
-      // | ------- | ------------------ |
-      //
-      // The second column should be left aligned because it contains text:
-      // | Version | Supported          |
-      // | ------- | :----------------- |
-      //
-      // However, columns that includes numbers should be right aligned:
-      // | Version | Supported          |
-      // | ------: | :----------------- |
-      //
-      // More details: https://ux.stackexchange.com/questions/24066/what-is-the-best-practice-for-data-table-cell-content-alignment
-      //
-      // In this check we expect the style to be 'text-align:right' or equivalent.
-      if (token.type === 'th_open' && attr(token.attrs).style == null) {
-        onError({
-          lineNumber: token.lineNumber,
-          detail: `${params.lines[token.lineNumber - 1]}`,
-        });
-      }
-    });
-  },
-};

package/src/markdownlint/terminal-language.mjs

@@ -1,19 +0,0 @@
-export default {
-  names: ['terminalLanguage'],
-  description: 'Set the right language for terminal code.',
-  tags: ['code'],
-  /**
-   * @param {import('./duplicate-h1.mjs').MdParams} params
-   * @param {import('./duplicate-h1.mjs').OnError} onError
-   */
-  function: (params, onError) => {
-    params.tokens.forEach((token) => {
-      if (token.type === 'fence' && token.info === 'sh') {
-        onError({
-          lineNumber: token.lineNumber,
-          detail: `Use "bash" instead of "sh".`,
-        });
-      }
-    });
-  },
-};