@mui/internal-code-infra 0.0.4-canary.72 → 0.0.4-canary.74

package/src/eslint/mui/rules/no-styled-box.test.mjs

@@ -1,73 +0,0 @@
-import eslint from 'eslint';
-import parser from '@typescript-eslint/parser';
-import rule from './no-styled-box.mjs';
-
-const ruleTester = new eslint.RuleTester({
-  languageOptions: {
-    parser,
-    parserOptions: {
-      ecmaFeatures: { jsx: true },
-    },
-  },
-});
-
-ruleTester.run('no-styled-box', rule, {
-  valid: [
-    `
-import { styled } from '@mui/system';
-styled('div');
-`,
-    `
-import { styled } from '@mui/system';
-styled('div', {});
-`,
-  ],
-  invalid: [
-    {
-      code: `
-import { styled } from '@mui/system';
-import Box from '@mui/material/Box';
-
-const foo = styled(Box)({
-  color: 'red',
-});
-`,
-      errors: [
-        {
-          messageId: 'noBox',
-        },
-      ],
-      output: `
-import { styled } from '@mui/system';
-import Box from '@mui/material/Box';
-
-const foo = styled('div')({
-  color: 'red',
-});
-`,
-    },
-    {
-      code: `
-import { styled } from '@mui/system';
-import Box from '@mui/material/Box';
-
-const foo = styled(Box, {})({
-  color: 'red',
-});
-`,
-      errors: [
-        {
-          messageId: 'noBox',
-        },
-      ],
-      output: `
-import { styled } from '@mui/system';
-import Box from '@mui/material/Box';
-
-const foo = styled('div', {})({
-  color: 'red',
-});
-`,
-    },
-  ],
-});

package/src/eslint/mui/rules/require-dev-wrapper.test.mjs

@@ -1,265 +0,0 @@
-import eslint from 'eslint';
-import parser from '@typescript-eslint/parser';
-import rule from './require-dev-wrapper.mjs';
-
-const ruleTester = new eslint.RuleTester({
-  languageOptions: {
-    parser,
-  },
-});
-
-ruleTester.run('require-dev-wrapper', rule, {
-  valid: [
-    // Should pass: Function wrapped with !== production check
-    {
-      code: `
-if (process.env.NODE_ENV !== 'production') {
-  checkSlot(key, overrides[k]);
-}
-      `,
-    },
-    // Should pass: Function wrapped with === 'development' check (tree-shakes in production)
-    {
-      code: `
-if (process.env.NODE_ENV === 'development') {
-  checkSlot(key, overrides[k]);
-}
-      `,
-    },
-    // Should pass: Function wrapped in a for loop inside production check
-    {
-      code: `
-if (process.env.NODE_ENV !== 'production') {
-  for (key in slots) {
-    checkSlot(key, overrides[k]);
-  }
-}
-      `,
-    },
-    // Should pass: Other functions not in the list
-    {
-      code: `
-otherFunction('hello');
-      `,
-    },
-    // Should pass: warnOnce wrapped correctly
-    {
-      code: `
-if (process.env.NODE_ENV !== 'production') {
-  warnOnce('Some warning message');
-}
-      `,
-    },
-    // Should pass: Multiple statements in production check
-    {
-      code: `
-if (process.env.NODE_ENV !== 'production') {
-  const message = 'Warning';
-  warn(message);
-  checkSlot(key, value);
-}
-      `,
-    },
-    // Should pass: Nested blocks
-    {
-      code: `
-if (process.env.NODE_ENV !== 'production') {
-  if (someCondition) {
-    warnOnce('nested warning');
-  }
-}
-      `,
-    },
-    // Should pass: Reversed comparison (literal on left)
-    {
-      code: `
-if ('production' !== process.env.NODE_ENV) {
-  checkSlot(key, value);
-}
-      `,
-    },
-    // Should pass: Function in else block when if checks for production
-    {
-      code: `
-if (process.env.NODE_ENV === 'production') {
-  // production code
-} else {
-  checkSlot(key, value);
-}
-      `,
-    },
-    // Should pass: Nested if statement in else block
-    {
-      code: `
-if (process.env.NODE_ENV === 'production') {
-  // production code
-} else {
-  if (someCondition) {
-    warnOnce('nested warning in else');
-  }
-}
-      `,
-    },
-    // Should pass: else-if chain where inner if is a valid dev guard
-    {
-      code: `
-if (a) {
-} else if (process.env.NODE_ENV !== 'production') {
-  warn('dev only');
-}
-      `,
-    },
-    // Should pass: else-if after production check (call is in the non-production branch)
-    {
-      code: `
-if (process.env.NODE_ENV === 'production') {
-} else if (someCondition) {
-  warn('not production');
-}
-      `,
-    },
-  ],
-  invalid: [
-    // Should fail: checkSlot without production check
-    {
-      code: `
-checkSlot(key, overrides[k]);
-      `,
-      errors: [
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'checkSlot' },
-        },
-      ],
-    },
-    // Should fail: warnOnce without production check
-    {
-      code: `
-warnOnce('Some warning');
-      `,
-      errors: [
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'warnOnce' },
-        },
-      ],
-    },
-    // Should fail: warn without production check
-    {
-      code: `
-warn('Some warning');
-      `,
-      errors: [
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'warn' },
-        },
-      ],
-    },
-    // Should fail: Multiple unwrapped calls
-    {
-      code: `
-checkSlot(key, value);
-warn('Warning message');
-      `,
-      errors: [
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'checkSlot' },
-        },
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'warn' },
-        },
-      ],
-    },
-    // Should fail: Inside wrong conditional (no process.env.NODE_ENV)
-    {
-      code: `
-if (someOtherCondition) {
-  checkSlot(key, value);
-}
-      `,
-      errors: [
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'checkSlot' },
-        },
-      ],
-    },
-    // Should fail: Inside else block of production check (!== 'production')
-    {
-      code: `
-if (process.env.NODE_ENV !== 'production') {
-  // ok
-} else {
-  checkSlot(key, value);
-}
-      `,
-      errors: [
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'checkSlot' },
-        },
-      ],
-    },
-    // Should fail: In then block of === 'production' (would run in production!)
-    {
-      code: `
-if (process.env.NODE_ENV === 'production') {
-  checkSlot(key, value);
-}
-      `,
-      errors: [
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'checkSlot' },
-        },
-      ],
-    },
-    // Should fail: !== 'test' doesn't reliably tree-shake
-    {
-      code: `
-if (process.env.NODE_ENV !== 'test') {
-  checkSlot(key, value);
-}
-      `,
-      errors: [
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'checkSlot' },
-        },
-      ],
-    },
-    // Should fail: Non-static check doesn't tree-shake
-    {
-      code: `
-const env = 'production';
-if (process.env.NODE_ENV !== env) {
-  checkSlot(key, value);
-}
-      `,
-      errors: [
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'checkSlot' },
-        },
-      ],
-    },
-    // Should fail: else-if after !== 'production' means call runs IN production
-    {
-      code: `
-if (process.env.NODE_ENV !== 'production') {
-} else if (someCondition) {
-  warn('this runs in production');
-}
-      `,
-      errors: [
-        {
-          messageId: 'missingDevWrapper',
-          data: { functionName: 'warn' },
-        },
-      ],
-    },
-  ],
-});

package/src/eslint/mui/rules/rules-of-use-theme-variants.test.mjs

@@ -1,149 +0,0 @@
-import eslint from 'eslint';
-import parser from '@typescript-eslint/parser';
-import rule from './rules-of-use-theme-variants.mjs';
-
-const ruleTester = new eslint.RuleTester({
-  languageOptions: {
-    parser,
-    parserOptions: {
-      ecmaFeatures: { jsx: true },
-    },
-  },
-});
-
-ruleTester.run('rules-of-use-theme-variants', rule, {
-  valid: [
-    // allowed but dangerous
-    {
-      name: 'custom useThemeVariants hook',
-      code: `
-{
-  const useCustomThemeVariants = props => useThemeVariants(props);
-}`,
-    },
-    {
-      name: 'basic useThemeVariants call',
-      code: `
-{
-  useThemeVariants(props);
-}
-`,
-    },
-    {
-      name: 'useThemeVariants with destructured props',
-      code: `
-{
-  const { className, value: valueProp, ...other } = props;
-  useThemeVariants(props);
-}
-`,
-    },
-    {
-      name: 'useThemeVariants with disabled prop override',
-      code: `
-{
-  const { className, disabled = false, value: valueProp, ...other } = props;
-  useThemeVariants({ ...props, disabled });
-}
-`,
-    },
-    {
-      name: 'useThemeVariants with state variables',
-      code: `
-{
-  const { className, value: valueProp, ...other } = props;
-  const [stateA, setStateA] = React.useState(0);
-  const [stateB, setStateB] = React.useState(0);
-  useThemeVariants({ stateA, ...props, stateB });
-}
-`,
-    },
-    // unnecessary spread but it's not the responsibility of this rule to catch "unnecessary" spread
-    {
-      name: 'useThemeVariants with unnecessary spread',
-      code: `
-{
-  const { className, value: valueProp, ...other } = props;
-  useThemeVariants({ ...props});
-}
-  `,
-    },
-  ],
-  invalid: [
-    {
-      name: 'disabled prop not passed to useThemeVariants',
-      code: `
-{
-  const { disabled = false, ...other } = props;
-  useThemeVariants({ ...props});
-}
-  `,
-      errors: [
-        {
-          message: 'Prop `disabled` is not passed to `useThemeVariants` props.',
-          line: 4,
-          column: 20,
-          endLine: 4,
-          endColumn: 31,
-        },
-      ],
-    },
-    {
-      name: 'variant prop not passed to useThemeVariants',
-      code: `
-{
-  const { disabled = false, variant = 'text', ...other } = props;
-  useThemeVariants({ ...props, disabled });
-}
-  `,
-      errors: [
-        {
-          message: 'Prop `variant` is not passed to `useThemeVariants` props.',
-          line: 4,
-          column: 20,
-          endLine: 4,
-          endColumn: 42,
-        },
-      ],
-    },
-    {
-      name: 'props spread must come first',
-      code: `
-{
-  const { disabled = false, ...other } = props;
-  useThemeVariants({ disabled, ...props });
-}
-  `,
-      errors: [
-        {
-          message:
-            'The props spread must come first in the `useThemeVariants` props. Otherwise destructured props with default values could be overridden.',
-          line: 4,
-          column: 32,
-          endLine: 4,
-          endColumn: 40,
-        },
-      ],
-    },
-    // this is valid code but not analyzable by this rule
-    {
-      name: 'cannot analyze identifier pattern',
-      code: `
-{
-  const { disabled = false, ...other } = props;
-  const themeVariantProps = { ...props, disabled };
-  useThemeVariants(themeVariantProps);
-}
-  `,
-      errors: [
-        {
-          message: "Can only analyze object patterns but found 'Identifier'. Prefer `{...props}`.",
-          line: 5,
-          column: 20,
-          endLine: 5,
-          endColumn: 37,
-        },
-      ],
-    },
-  ],
-});

package/src/eslint/mui/rules/straight-quotes.test.mjs

@@ -1,67 +0,0 @@
-import eslint from 'eslint';
-import parser from '@typescript-eslint/parser';
-import rule from './straight-quotes.mjs';
-
-const ruleTester = new eslint.RuleTester({
-  languageOptions: {
-    parser,
-  },
-});
-
-ruleTester.run('straight-quotes', rule, {
-  valid: [
-    `
-const values = [
-  {
-    title: 'Put community first 💙',
-  },
-];
-  `,
-  ],
-  invalid: [
-    {
-      code: `
-const values = [
-  {
-    title: 'Put community first 💙',
-    description: 'We never lose sight of who we’re serving and why.',
-  },
-];
-      `,
-      errors: [
-        {
-          messageId: 'wrongQuotes',
-          line: 5,
-        },
-      ],
-    },
-    {
-      code: `
-// reference ID (also known as “SHA” or “hash”) of the commit we're building.
-const values = 'foo';
-      `,
-      errors: [
-        {
-          line: 2,
-          column: 32,
-          messageId: 'wrongQuotes',
-        },
-        {
-          line: 2,
-          column: 36,
-          messageId: 'wrongQuotes',
-        },
-        {
-          line: 2,
-          column: 41,
-          messageId: 'wrongQuotes',
-        },
-        {
-          line: 2,
-          column: 46,
-          messageId: 'wrongQuotes',
-        },
-      ],
-    },
-  ],
-});

package/src/remark/firstBlockHeading.test.mjs

@@ -1,107 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import remarkFrontmatter from 'remark-frontmatter';
-import remarkMdx from 'remark-mdx';
-import { remark } from 'remark';
-import remarkGfm from 'remark-gfm';
-import plugin from './firstBlockHeading.mjs';
-
-/**
- * @param {string} input
- * @param {Parameters<typeof plugin>[0]} [options]
- */
-function lint(input, options) {
-  const file = remark()
-    .use(remarkFrontmatter, ['yaml', 'toml'])
-    .use(remarkGfm)
-    .use(plugin, options)
-    .processSync(input);
-  return file.messages.map((message) => ({
-    reason: message.reason,
-    line: message.line ?? 0,
-  }));
-}
-
-/**
- * @param {string} input
- * @param {Parameters<typeof plugin>[0]} [options]
- */
-function lintMdx(input, options) {
-  const file = remark()
-    .use(remarkFrontmatter, ['yaml', 'toml'])
-    .use(remarkGfm)
-    .use(remarkMdx)
-    .use(plugin, options)
-    .processSync(input);
-  return file.messages.map((message) => ({
-    reason: message.reason,
-    line: message.line ?? 0,
-  }));
-}
-
-describe('remark-lint-mui-first-block-heading', () => {
-  it('accepts a document starting with an h1', () => {
-    expect(lint(`# Title\n\nSome content.\n`)).toEqual([]);
-  });
-
-  it('accepts an h1 after YAML frontmatter', () => {
-    expect(lint(`---\nfoo: bar\n---\n\n# Title\n`)).toEqual([]);
-  });
-
-  it('flags a paragraph before the h1', () => {
-    expect(lint(`Lead paragraph.\n\n# Title\n`)).toHaveLength(1);
-  });
-
-  it('flags an h2 as the first block', () => {
-    expect(lint(`## Subtitle\n\nContent.\n`)).toHaveLength(1);
-  });
-
-  it('flags an empty document', () => {
-    expect(lint(``)).toHaveLength(1);
-  });
-
-  it('flags a document starting with HTML', () => {
-    expect(lint(`<div>Hello</div>\n\n# Title\n`)).toHaveLength(1);
-  });
-
-  it('accepts an h1 after a <style> block', () => {
-    expect(lint(`<style>.x { color: red; }</style>\n\n# Title\n`)).toEqual([]);
-  });
-
-  it('accepts an h1 after a <script> block', () => {
-    expect(lint(`<script>var x = 1;</script>\n\n# Title\n`)).toEqual([]);
-  });
-
-  it('accepts an h1 after an HTML comment', () => {
-    expect(lint(`<!-- a comment -->\n\n# Title\n`)).toEqual([]);
-  });
-
-  it('accepts an h1 after frontmatter and a <style> block', () => {
-    expect(lint(`---\nfoo: bar\n---\n\n<style>.x{}</style>\n\n# Title\n`)).toEqual([]);
-  });
-
-  it('accepts a document with a title in YAML frontmatter', () => {
-    expect(lint(`---\ntitle: Hello\n---\n\nSome content.\n`)).toEqual([]);
-  });
-
-  it('still flags missing h1 when frontmatterTitle is disabled', () => {
-    expect(
-      lint(`---\ntitle: Hello\n---\n\nSome content.\n`, { frontMatterTitle: false }),
-    ).toHaveLength(1);
-  });
-
-  it('accepts an h1 after MDX imports', () => {
-    expect(lintMdx(`import Foo from './foo';\n\n# Title\n`)).toEqual([]);
-  });
-
-  it('accepts an h1 after an MDX block comment expression', () => {
-    expect(lintMdx(`{/* a comment */}\n\n# Title\n`)).toEqual([]);
-  });
-
-  it('accepts an h1 after MDX comment + imports', () => {
-    expect(lintMdx(`{/* lint disable */}\n\nimport Foo from './foo';\n\n# Title\n`)).toEqual([]);
-  });
-
-  it('flags an MDX expression that is not a comment', () => {
-    expect(lintMdx(`{value}\n\n# Title\n`)).toHaveLength(1);
-  });
-});

package/src/remark/gitDiff.test.mjs

@@ -1,45 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { createLintTester } from './createLintTester.mjs';
-import plugin from './gitDiff.mjs';
-
-const lint = createLintTester(plugin);
-
-describe('remark-lint-mui-git-diff', () => {
-  it('accepts a well-formed unified diff', () => {
-    const input = `# Title
-
-\`\`\`diff
-diff --git a/foo.txt b/foo.txt
-index 0000..1111 100644
---- a/foo.txt
-+++ b/foo.txt
-@@ -1,2 +1,2 @@
- unchanged line
--removed line
-+added line
-\`\`\`
-`;
-    expect(lint(input)).toEqual([]);
-  });
-
-  it('ignores non-diff code blocks', () => {
-    const input = `\`\`\`js
-const unrelated = 'line';
-\`\`\`
-`;
-    expect(lint(input)).toEqual([]);
-  });
-
-  it('flags lines that do not match the unified diff prefixes', () => {
-    const input = `\`\`\`diff
-this line has no prefix
- valid line
-bad line too
-\`\`\`
-`;
-    const messages = lint(input);
-    expect(messages).toHaveLength(2);
-    expect(messages[0].line).toBe(2);
-    expect(messages[1].line).toBe(4);
-  });
-});