@mui/internal-code-infra 0.0.4-canary.5 → 0.0.4-canary.51

package/src/remark/straightQuotes.test.mjs

@@ -0,0 +1,25 @@
+import { describe, it, expect } from 'vitest';
+import { createLintTester } from './createLintTester.mjs';
+import plugin from './straightQuotes.mjs';
+
+const lint = createLintTester(plugin);
+
+describe('remark-lint-mui-straight-quotes', () => {
+  it('accepts straight quotes', () => {
+    expect(lint(`# Title\n\nA paragraph with "straight" and 'simple' quotes.\n`)).toEqual([]);
+  });
+
+  it('flags curly double quotes with correct location', () => {
+    const messages = lint(`# Title\n\nA paragraph with “curly” quotes.\n`);
+    expect(messages).toHaveLength(2);
+    expect(messages[0]).toMatchObject({ line: 3, column: 18 });
+    expect(messages[1]).toMatchObject({ line: 3, column: 24 });
+  });
+
+  it('flags curly single quotes', () => {
+    const messages = lint(`Use ‘single’ curly quotes too.\n`);
+    expect(messages).toHaveLength(2);
+    expect(messages[0]).toMatchObject({ line: 1, column: 5 });
+    expect(messages[1]).toMatchObject({ line: 1, column: 12 });
+  });
+});

package/src/remark/tableAlignment.mjs

@@ -0,0 +1,23 @@
+import { lintRule } from 'unified-lint-rule';
+import { visit } from 'unist-util-visit';
+
+const remarkLintMuiTableAlignment = lintRule(
+  {
+    origin: 'remark-lint:mui-table-alignment',
+    url: 'https://github.com/mui/mui-public',
+  },
+  /** @param {import('mdast').Root} tree */
+  (tree, file) => {
+    visit(tree, 'table', (node) => {
+      const align = node.align ?? [];
+      if (align.some((value) => value == null)) {
+        file.message(
+          'Table columns should declare an explicit alignment (`:---`, `---:`, or `:---:`).',
+          node,
+        );
+      }
+    });
+  },
+);
+
+export default remarkLintMuiTableAlignment;

package/src/remark/tableAlignment.test.mjs

@@ -0,0 +1,28 @@
+import { describe, it, expect } from 'vitest';
+import { createLintTester } from './createLintTester.mjs';
+import plugin from './tableAlignment.mjs';
+
+const lint = createLintTester(plugin);
+
+describe('remark-lint-mui-table-alignment', () => {
+  it('accepts tables with explicit alignment on every column', () => {
+    const input = `| A | B |\n| :- | -: |\n| 1 | 2 |\n`;
+    expect(lint(input)).toEqual([]);
+  });
+
+  it('flags tables with at least one column missing alignment', () => {
+    const input = `| A | B |\n| :- | -- |\n| 1 | 2 |\n`;
+    const messages = lint(input);
+    expect(messages).toHaveLength(1);
+  });
+
+  it('flags tables with no alignment at all', () => {
+    const input = `| A | B |\n| - | - |\n| 1 | 2 |\n`;
+    const messages = lint(input);
+    expect(messages).toHaveLength(1);
+  });
+
+  it('does not flag non-table content', () => {
+    expect(lint(`# Heading\n\nSome paragraph.\n`)).toEqual([]);
+  });
+});

package/src/remark/terminalLanguage.mjs

@@ -0,0 +1,19 @@
+import { lintRule } from 'unified-lint-rule';
+import { visit } from 'unist-util-visit';
+
+const remarkLintMuiTerminalLanguage = lintRule(
+  {
+    origin: 'remark-lint:mui-terminal-language',
+    url: 'https://github.com/mui/mui-public',
+  },
+  /** @param {import('mdast').Root} tree */
+  (tree, file) => {
+    visit(tree, 'code', (node) => {
+      if (node.lang === 'sh' && node.position) {
+        file.message('Use `bash` instead of `sh` as the code block language.', node);
+      }
+    });
+  },
+);
+
+export default remarkLintMuiTerminalLanguage;

package/src/remark/terminalLanguage.test.mjs

@@ -0,0 +1,17 @@
+import { describe, it, expect } from 'vitest';
+import { createLintTester } from './createLintTester.mjs';
+import plugin from './terminalLanguage.mjs';
+
+const lint = createLintTester(plugin);
+
+describe('remark-lint-mui-terminal-language', () => {
+  it('accepts bash and other languages', () => {
+    expect(lint(`\`\`\`bash\necho hi\n\`\`\`\n`)).toEqual([]);
+    expect(lint(`\`\`\`js\nconst x = 1;\n\`\`\`\n`)).toEqual([]);
+  });
+
+  it('flags `sh` code fences', () => {
+    const messages = lint(`\`\`\`sh\necho hi\n\`\`\`\n`);
+    expect(messages).toHaveLength(1);
+  });
+});

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: {

package/src/utils/build.mjs

@@ -34,6 +34,20 @@ export function getOutExtension(bundle, options = {}) {
   return bundle === 'cjs' ? '.js' : '.mjs';
 }
 
+/**
+ * Returns a new object with `import` first, `require` second, `default` last,
+ * and any other condition keys preserved in their original relative order in between.
+ * @param {Record<string, any>} conditions
+ * @returns {Record<string, any>}
+ */
+function sortExportConditions(conditions) {
+  /** @type {Record<string, number | undefined>} */
+  const order = { import: 0, require: 1, default: 3 };
+  return Object.fromEntries(
+    Object.entries(conditions).sort(([a], [b]) => (order[a] ?? 2) - (order[b] ?? 2)),
+  );
+}
+
 /**
  * @param {Object} param0
  * @param {NonNullable<import('../cli/packageJson').PackageJson.Exports>} param0.importPath
@@ -367,7 +381,8 @@ export async function createPackageExports({
     }
   });
 
-  // Transform import/require to default/require pattern
+  // Rebuild condition objects with stable key order; bundles run in parallel so
+  // import/require insertion order would otherwise depend on Promise timing.
   Object.keys(newExports).forEach((key) => {
     const exportVal = newExports[key];
     if (Array.isArray(exportVal)) {
@@ -387,6 +402,8 @@ export async function createPackageExports({
             ? defaultExport.default
             : defaultExport;
       }
+
+      newExports[key] = sortExportConditions(exportVal);
     }
   });