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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/internal-code-infra",
-  "version": "0.0.4-canary.70",
+  "version": "0.0.4-canary.72",
   "author": "MUI Team",
   "description": "Infra scripts and configs to be used across MUI repos.",
   "license": "MIT",
@@ -135,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.37",
-    "@mui/internal-babel-plugin-minify-errors": "2.0.8-canary.28",
-    "@mui/internal-babel-plugin-display-name": "1.0.4-canary.21"
+    "@mui/internal-babel-plugin-display-name": "1.0.4-canary.21",
+    "@mui/internal-babel-plugin-resolve-imports": "2.0.7-canary.38",
+    "@mui/internal-babel-plugin-minify-errors": "2.0.8-canary.28"
   },
   "peerDependencies": {
     "@next/eslint-plugin-next": "*",
@@ -186,7 +186,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitSha": "92bf20d1aa981a2b1b44d8a9773613d5a76f170c",
+  "gitSha": "d5f96b89b5b960ad6272652eb2734ddb305a85ae",
   "scripts": {
     "build": "tsgo -p tsconfig.build.json",
     "typescript": "tsgo -noEmit",

package/src/brokenLinksChecker/index.test.ts

@@ -4,10 +4,13 @@ import { describe, expect, it } from 'vitest';
 
 import {
   crawl,
-  type BrokenLinkIssue,
-  type HtmlValidateIssue,
-  type Issue,
-  type Link,
+  // eslint-disable-next-line import/extensions
+} from './index.mjs';
+import type {
+  BrokenLinkIssue,
+  HtmlValidateIssue,
+  Issue,
+  Link,
   // eslint-disable-next-line import/extensions
 } from './index.mjs';
 

package/src/eslint/mui/config.mjs

@@ -527,9 +527,10 @@ export function createCoreConfig(options = {}) {
               '@typescript-eslint/consistent-type-imports': [
                 'error',
                 {
-                  fixStyle: 'inline-type-imports',
+                  fixStyle: 'separate-type-imports',
                 },
               ],
+              'import/consistent-type-specifier-style': ['error', 'prefer-top-level'],
             }
           : {}),
         // Prevent the use of `e` as a shorthand for `event`, `error`, etc.

package/src/estree-typescript.d.ts

@@ -1,4 +1,4 @@
-import { type BaseExpression, type Expression } from 'estree';
+import type { BaseExpression, 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 { type 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 { type 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 { type 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 { type 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 { type 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 { type 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 { type 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 { type 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 { type 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 { type 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 { type Linter } from 'eslint';
+  import type { Linter } from 'eslint';
 
   interface NextEslintPluginConfig extends Linter.LegacyConfig {
     flatConfig: {

package/src/utils/pnpm.mjs

@@ -78,18 +78,33 @@ import { parseDocument, isMap } from 'yaml';
 export async function getWorkspacePackages(options = {}) {
   const { sinceRef = null, publicOnly = false, nonPublishedOnly = false, filter = [] } = options;
 
-  // Build command with conditional filter
-  const filterArg = sinceRef ? ['--filter', `...[${sinceRef}]`] : [];
-  if (filter.length > 0) {
-    filter.forEach((f) => {
-      filterArg.push('--filter', f);
-    });
+  /**
+   * Run `pnpm ls` with the given --filter args and return the parsed list.
+   * @param {string[]} filterArg
+   * @returns {Promise<PnpmListResultItem[]>}
+   */
+  const listPackages = async (filterArg) => {
+    const result = await $({ cwd: options.cwd })`pnpm ls -r --json --depth -1 ${filterArg}`;
+    return JSON.parse(result.stdout);
+  };
+
+  // pnpm ORs --filter args, so intersect "matches filter" with "changed since ref"
+  // in JS. The `[ref]` selector (no `...`) excludes dependents of changed packages.
+  const patternFilterArg = filter.flatMap((f) => ['--filter', f]);
+  const [candidatePackages, changedPackages] = await Promise.all([
+    listPackages(patternFilterArg),
+    // null when no sinceRef (skip the constraint); [] when nothing changed.
+    sinceRef ? listPackages(['--filter', `[${sinceRef}]`]) : Promise.resolve(null),
+  ]);
+  let packageData = candidatePackages;
+  if (changedPackages) {
+    // sinceRef given but nothing changed → no packages, regardless of filter.
+    if (changedPackages.length === 0) {
+      return [];
+    }
+    const changedPaths = new Set(changedPackages.map((pkg) => pkg.path));
+    packageData = packageData.filter((pkg) => changedPaths.has(pkg.path));
   }
-  const result = options.cwd
-    ? await $({ cwd: options.cwd })`pnpm ls -r --json --depth -1 ${filterArg}`
-    : await $`pnpm ls -r --json --depth -1 ${filterArg}`;
-  /** @type {PnpmListResultItem[]} */
-  const packageData = JSON.parse(result.stdout);
 
   // Filter packages based on options
   const filteredPackages = packageData.flatMap((pkg) => {