@mui/internal-code-infra 0.0.4-canary.64 → 0.0.4-canary.66

package/build/eslint/mui/rules/no-floating-cleanup.d.mts

@@ -0,0 +1,5 @@
+import { ESLintUtils } from '@typescript-eslint/utils';
+declare const _default: ESLintUtils.RuleModule<"floatingCleanup", [], unknown, ESLintUtils.RuleListener> & {
+    name: string;
+};
+export default _default;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/internal-code-infra",
-  "version": "0.0.4-canary.64",
+  "version": "0.0.4-canary.66",
   "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"
@@ -192,7 +188,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitSha": "42ab800dc8926bfe1c6ea3020a013565143f1bd6",
+  "gitSha": "db0c201b6c697cff1aab50b9b33270720c93f424",
   "scripts": {
     "build": "tsgo -p tsconfig.build.json",
     "typescript": "tsgo -noEmit",

package/src/eslint/mui/index.mjs

@@ -13,6 +13,7 @@ import straightQuotes from './rules/straight-quotes.mjs';
 import addUndefToOptional from './rules/add-undef-to-optional.mjs';
 import flattenParentheses from './rules/flatten-parentheses.mjs';
 import noPresentationRole from './rules/no-presentation-role.mjs';
+import noFloatingCleanup from './rules/no-floating-cleanup.mjs';
 
 /** @type {import('eslint').ESLint.Plugin} */
 const muiPlugin = {
@@ -37,6 +38,7 @@ const muiPlugin = {
     'add-undef-to-optional': /** @type {any} */ (addUndefToOptional),
     'flatten-parentheses': /** @type {any} */ (flattenParentheses),
     'no-presentation-role': noPresentationRole,
+    'no-floating-cleanup': /** @type {any} */ (noFloatingCleanup),
   },
 };
 

package/src/eslint/mui/rules/no-floating-cleanup.mjs

@@ -0,0 +1,187 @@
+import { ESLintUtils } from '@typescript-eslint/utils';
+import ts from 'typescript';
+
+/**
+ * no-floating-cleanup
+ *
+ * Like `@typescript-eslint/no-floating-promises`, but for functions: reports
+ * call expressions used as statements whose return value is a function
+ * (e.g. an `unsubscribe` / cleanup callback). Ignoring such a value usually
+ * means the subscription can never be torn down — a leak.
+ *
+ * Requires type information (parserOptions.project / projectService).
+ *
+ * Calls whose resolved signature declares a `this` return type are never
+ * reported: fluent / builder APIs (e.g. d3 scales) chain by returning `this`,
+ * so the discarded result is callable but not a leaked cleanup function.
+ *
+ * Opt-out at a call site, same convention as no-floating-promises:
+ *   void subscribe(cb);
+ */
+
+const RULE_NAME = 'no-floating-cleanup';
+
+const createRule = ESLintUtils.RuleCreator(
+  (name) =>
+    `https://github.com/mui/mui-public/blob/master/packages/code-infra/src/eslint/mui/rules/${name}.mjs`,
+);
+
+export default createRule({
+  meta: {
+    type: 'problem',
+    docs: {
+      description:
+        'Disallow ignoring returned cleanup/unsubscribe functions, which leaks subscriptions.',
+    },
+    messages: {
+      floatingCleanup:
+        "Return value of type '{{typeName}}' is ignored. This likely leaks a subscription — " +
+        'store the cleanup function, or discard it explicitly with the `void` operator.',
+    },
+    schema: [],
+  },
+  name: RULE_NAME,
+  defaultOptions: [],
+  create(context) {
+    const services = ESLintUtils.getParserServices(context);
+    const checker = services.program.getTypeChecker();
+
+    /**
+     * Does this type (or any union member) look like a cleanup function?
+     * @param {import('typescript').Type} type
+     * @returns {boolean}
+     */
+    function isCleanupType(type) {
+      if (type.isUnion()) {
+        return type.types.some(isCleanupType);
+      }
+      // Never fire on any/unknown/never — too noisy and not provable.
+      // eslint-disable-next-line no-bitwise -- TypeScript type flags are a bitmask
+      if (type.flags & (ts.TypeFlags.Any | ts.TypeFlags.Unknown | ts.TypeFlags.Never)) {
+        return false;
+      }
+
+      return checker.getSignaturesOfType(type, ts.SignatureKind.Call).length > 0;
+    }
+
+    /**
+     * @param {import('typescript').Type} type
+     * @returns {string}
+     */
+    function describeType(type) {
+      return (type.aliasSymbol && type.aliasSymbol.getName()) || checker.typeToString(type);
+    }
+
+    /**
+     * Does this call resolve to a signature that returns `this`? Fluent /
+     * builder APIs (e.g. d3 scales) chain by returning `this`, so the
+     * discarded result is callable but never a leaked cleanup function —
+     * those calls should not be reported. Covers both an explicit `: this`
+     * annotation and an inferred `this` return (no annotation).
+     * @param {import('@typescript-eslint/utils').TSESTree.Node} estreeCall
+     * @returns {boolean}
+     */
+    function returnsThis(estreeCall) {
+      /** @type {import('typescript').Node | undefined} */
+      let tsNode = services.esTreeNodeToTSNodeMap.get(estreeCall);
+      // Unwrap `await (...)` / parens down to the underlying call.
+      while (tsNode && (ts.isAwaitExpression(tsNode) || ts.isParenthesizedExpression(tsNode))) {
+        tsNode = tsNode.expression;
+      }
+      if (!tsNode || !(ts.isCallExpression(tsNode) || ts.isNewExpression(tsNode))) {
+        return false;
+      }
+      const declaration = checker.getResolvedSignature(tsNode)?.getDeclaration();
+      if (!declaration) {
+        return false;
+      }
+      // Explicit `: this` annotation.
+      if (declaration.type?.kind === ts.SyntaxKind.ThisType) {
+        return true;
+      }
+      // Inferred `this` return: the *declared* (uninstantiated) signature's
+      // return type is the polymorphic `this` type, which prints as `this`.
+      // (The resolved signature would have it bound to the receiver type.)
+      const declaredSignature = checker.getSignatureFromDeclaration(declaration);
+      return (
+        !!declaredSignature &&
+        checker.typeToString(checker.getReturnTypeOfSignature(declaredSignature)) === 'this'
+      );
+    }
+
+    /**
+     * Unwrap an expression-statement expression down to the calls whose
+     * results are being discarded. Mirrors the cases no-floating-promises
+     * handles: plain calls, optional chains, `await`, comma sequences,
+     * and the operand(s) of unary / logical / conditional expressions.
+     * A top-level `void` operator is the explicit opt-out, so we stop there.
+     * @param {import('@typescript-eslint/utils').TSESTree.Node} expr
+     * @param {import('@typescript-eslint/utils').TSESTree.Node[]} [out]
+     * @returns {import('@typescript-eslint/utils').TSESTree.Node[]}
+     */
+    function findDiscardedCalls(expr, out = []) {
+      switch (expr.type) {
+        case 'CallExpression':
+        case 'NewExpression':
+          out.push(expr);
+          break;
+        case 'ChainExpression':
+          findDiscardedCalls(expr.expression, out);
+          break;
+        case 'AwaitExpression':
+          // type of the AwaitExpression node is already the awaited type
+          if (expr.argument.type === 'CallExpression' || expr.argument.type === 'ChainExpression') {
+            out.push(expr);
+          }
+          break;
+        case 'UnaryExpression':
+          // `void expr` is the explicit opt-out; any other unary operator
+          // (e.g. `!subscribe()`) still discards the operand's result.
+          if (expr.operator !== 'void') {
+            findDiscardedCalls(expr.argument, out);
+          }
+          break;
+        case 'LogicalExpression':
+          // `a && subscribe()`, `a || subscribe()`, `a ?? subscribe()`:
+          // either side may be the discarded value.
+          findDiscardedCalls(expr.left, out);
+          findDiscardedCalls(expr.right, out);
+          break;
+        case 'ConditionalExpression':
+          // `cond ? subscribe() : other()`: the branches are discarded,
+          // not the test.
+          findDiscardedCalls(expr.consequent, out);
+          findDiscardedCalls(expr.alternate, out);
+          break;
+        case 'SequenceExpression':
+          // every value except the last is discarded; the last one is the
+          // statement's value, which is also discarded here
+          expr.expressions.forEach((subExpr) => findDiscardedCalls(subExpr, out));
+          break;
+        default:
+          break;
+      }
+      return out;
+    }
+
+    return {
+      /** @param {import('@typescript-eslint/utils').TSESTree.ExpressionStatement} node */
+      ExpressionStatement(node) {
+        findDiscardedCalls(node.expression).forEach((call) => {
+          // Fluent/builder calls return `this` for chaining — not a leak.
+          if (returnsThis(call)) {
+            return;
+          }
+          const type = services.getTypeAtLocation(call);
+          if (isCleanupType(type)) {
+            context.report({
+              node: call,
+              messageId: 'floatingCleanup',
+              data: { typeName: describeType(type) },
+            });
+          }
+        });
+      },
+    };
+  },
+});

package/src/eslint/mui/rules/no-floating-cleanup.test.mjs

@@ -0,0 +1,141 @@
+import { RuleTester } from '@typescript-eslint/rule-tester';
+import TSESlintParser from '@typescript-eslint/parser';
+import rule from './no-floating-cleanup.mjs';
+
+const ruleTester = new RuleTester({
+  languageOptions: {
+    parser: TSESlintParser,
+    parserOptions: {
+      tsconfigRootDir: import.meta.dirname,
+      projectService: {
+        allowDefaultProject: ['*.ts'],
+      },
+    },
+  },
+});
+
+ruleTester.run('no-floating-cleanup', rule, {
+  valid: [
+    {
+      name: 'stored cleanup function',
+      code: `
+type Unsubscribe = () => void;
+declare function subscribe(cb: () => void): Unsubscribe;
+function f() {
+  const un = subscribe(() => {});
+  un();
+}
+      `,
+    },
+    {
+      name: 'explicit void opt-out',
+      code: `
+type Unsubscribe = () => void;
+declare function subscribe(cb: () => void): Unsubscribe;
+function f() {
+  void subscribe(() => {});
+}
+      `,
+    },
+    {
+      name: 'call returning void',
+      code: `
+declare function noop(): void;
+function f() {
+  noop();
+}
+      `,
+    },
+    {
+      name: 'fluent builder methods returning `this`',
+      code: `
+interface Scale {
+  (value: number): number;
+  domain(d: number[]): this;
+  range(r: number[]): this;
+}
+declare const scale: Scale;
+function f() {
+  scale.domain([0, 1]);
+  scale.range([0, 100]);
+}
+      `,
+    },
+    {
+      name: 'fluent builder method with an inferred `this` return',
+      code: `
+class Builder {
+  configure() {
+    return this;
+  }
+}
+declare const builder: Builder;
+function f() {
+  builder.configure();
+}
+      `,
+    },
+  ],
+  invalid: [
+    {
+      name: 'discarded unsubscribe function',
+      code: `
+type Unsubscribe = () => void;
+declare function subscribe(cb: () => void): Unsubscribe;
+function f() {
+  subscribe(() => {});
+}
+      `,
+      errors: [{ messageId: 'floatingCleanup', line: 5 }],
+    },
+    {
+      name: 'call returning a fresh callable (not `this`)',
+      code: `
+interface Scale {
+  (value: number): number;
+  copy(): Scale;
+}
+declare const scale: Scale;
+function f() {
+  scale.copy();
+}
+      `,
+      errors: [{ messageId: 'floatingCleanup', line: 8 }],
+    },
+    {
+      name: 'discarded cleanup behind a unary operator',
+      code: `
+type Unsubscribe = () => void;
+declare function subscribe(cb: () => void): Unsubscribe;
+function f() {
+  !subscribe(() => {});
+}
+      `,
+      errors: [{ messageId: 'floatingCleanup', line: 5 }],
+    },
+    {
+      name: 'discarded cleanup in a logical expression',
+      code: `
+type Unsubscribe = () => void;
+declare function subscribe(cb: () => void): Unsubscribe;
+declare const enabled: boolean;
+function f() {
+  enabled && subscribe(() => {});
+}
+      `,
+      errors: [{ messageId: 'floatingCleanup', line: 6 }],
+    },
+    {
+      name: 'discarded cleanup in a conditional expression',
+      code: `
+type Unsubscribe = () => void;
+declare function subscribe(cb: () => void): Unsubscribe;
+declare const enabled: boolean;
+function f() {
+  enabled ? subscribe(() => {}) : undefined;
+}
+      `,
+      errors: [{ messageId: 'floatingCleanup', line: 6 }],
+    },
+  ],
+});

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".`,
-        });
-      }
-    });
-  },
-};