[utils] Use consistent build approach (#40837)

mui · Feb 1, 2024 · 5e811b8 · 5e811b8
1 parent 56302c9
commit 5e811b8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 11 deletions.
diff --git a/packages/mui-utils/package.json b/packages/mui-utils/package.json
@@ -29,8 +29,8 @@
     "build": "pnpm build:legacy && pnpm build:modern && pnpm build:node && pnpm build:stable && pnpm build:types && pnpm build:copy-files",
     "build:legacy": "node ../../scripts/build.mjs legacy",
     "build:modern": "node ../../scripts/build.mjs modern",
-    "build:node": "node ../../scripts/build.mjs node --ignoreTopLevel",
-    "build:stable": "node ../../scripts/build.mjs stable --ignoreTopLevel",
+    "build:node": "node ../../scripts/build.mjs node",
+    "build:stable": "node ../../scripts/build.mjs stable",
     "build:copy-files": "node ../../scripts/copyFiles.mjs",
     "build:types": "node ../../scripts/buildTypes.mjs",
     "prebuild": "rimraf build tsconfig.build.tsbuildinfo",

diff --git a/scripts/build.mjs b/scripts/build.mjs
@@ -19,7 +19,7 @@ const validBundles = [
 ];
 
 async function run(argv) {
-  const { bundle, largeFiles, outDir: relativeOutDir, verbose, ignoreTopLevel } = argv;
+  const { bundle, largeFiles, outDir: relativeOutDir, verbose } = argv;
 
   if (validBundles.indexOf(bundle) === -1) {
     throw new TypeError(
@@ -62,9 +62,9 @@ async function run(argv) {
     //
     // TODO v6: Switch to `exports` field.
     {
-      node: !ignoreTopLevel && topLevelPathImportsCanBePackages ? './node' : './',
+      node: topLevelPathImportsCanBePackages ? './node' : './',
       modern: './modern',
-      stable: !ignoreTopLevel && topLevelPathImportsCanBePackages ? './' : './esm',
+      stable: topLevelPathImportsCanBePackages ? './' : './esm',
       legacy: './legacy',
     }[bundle],
   );
@@ -119,11 +119,6 @@ yargs(process.argv.slice(2))
           describe: 'Set to `true` if you know you are transpiling large files.',
         })
         .option('out-dir', { default: './build', type: 'string' })
-        .option('ignoreTopLevel', {
-          type: 'boolean',
-          default: false,
-          describe: 'Set to `true` to ignore switching out-dir based on top level import.',
-        })
         .option('verbose', { type: 'boolean' });
     },
     handler: run,

diff --git a/scripts/sizeSnapshot/webpack.config.js b/scripts/sizeSnapshot/webpack.config.js
@@ -139,7 +139,7 @@ async function getWebpackEntries() {
     ...coreComponents,
     {
       id: '@material-ui/utils',
-      path: 'packages/mui-utils/build/esm/index.js',
+      path: 'packages/mui-utils/build/index.js',
     },
     // TODO: Requires webpack v5
     // Resolution of webpack/acorn to 7.x is blocked by nextjs (https://github.com/vercel/next.js/issues/11947)