package-build-stats 8.2.6 → 8.2.7

package/build/utils/build.utils.js

@@ -1,4 +1,5 @@
 import path from 'path';
+import { randomUUID } from 'crypto';
 import config from '../config/config.js';
 import { rspack } from '@rspack/core';
 import isValidNPMName from 'is-valid-npm-name';
@@ -121,10 +122,13 @@ const BuildUtils = {
     },
     async buildPackage({ name, installPath, externals, options, }) {
         var _a;
-        const outputPath = config.tmp;
+        const id = randomUUID().slice(0, 8);
+        const outputPath = path.join(config.tmp, 'dist', `build-${name}-${id}`);
+        fs.mkdirSync(outputPath, { recursive: true });
         let entry = {};
         if (options.splitCustomImports) {
             if (!options.customImports || !options.customImports.length) {
+                fs.rmSync(outputPath, { recursive: true, force: true });
                 return { assets: [] };
             }
             options.customImports.forEach(importt => {
@@ -149,92 +153,97 @@ const BuildUtils = {
             minify: options.minify,
             outputPath,
         });
-        const jsonStatsStartTime = performance.now();
-        let jsonStats = stats.toJson({
-            assets: true,
-            source: true,
-            chunks: false,
-            chunkGroups: false,
-            chunkModules: true,
-            modules: true,
-            nestedModules: true,
-            reasons: true,
-            depth: true,
-            errors: true,
-            entrypoints: false,
-            warnings: false,
-        });
-        if (!jsonStats) {
-            Telemetry.parseWebpackStats(name, false, jsonStatsStartTime);
-            throw new UnexpectedBuildError('Expected webpack json stats to be non-null, but was null');
-        }
-        else {
-            Telemetry.parseWebpackStats(name, true, jsonStatsStartTime);
-        }
-        const compilationErrors = getCompilationErrors(stats);
-        if (error && !stats) {
-            throw new BuildError(error);
-        }
-        else if (compilationErrors.length) {
-            const missingModules = BuildUtils._parseMissingModules(compilationErrors);
-            if (missingModules.length) {
-                if (missingModules.length === 1 && missingModules[0] === name) {
-                    throw new EntryPointError(compilationErrors.map(err => err.message));
-                }
-                else {
-                    throw new MissingDependencyError(compilationErrors.map(err => err.toString()), { missingModules });
-                }
+        try {
+            const jsonStatsStartTime = performance.now();
+            let jsonStats = stats.toJson({
+                assets: true,
+                source: true,
+                chunks: false,
+                chunkGroups: false,
+                chunkModules: true,
+                modules: true,
+                nestedModules: true,
+                reasons: true,
+                depth: true,
+                errors: true,
+                entrypoints: false,
+                warnings: false,
+            });
+            if (!jsonStats) {
+                Telemetry.parseWebpackStats(name, false, jsonStatsStartTime);
+                throw new UnexpectedBuildError('Expected webpack json stats to be non-null, but was null');
             }
-            else if (jsonStats.errors && jsonStats.errors.length > 0) {
-                if (jsonStats.errors.some(error => error.message.includes("Unexpected character '#'"))) {
-                    throw new CLIBuildError(jsonStats.errors);
+            else {
+                Telemetry.parseWebpackStats(name, true, jsonStatsStartTime);
+            }
+            const compilationErrors = getCompilationErrors(stats);
+            if (error && !stats) {
+                throw new BuildError(error);
+            }
+            else if (compilationErrors.length) {
+                const missingModules = BuildUtils._parseMissingModules(compilationErrors);
+                if (missingModules.length) {
+                    if (missingModules.length === 1 && missingModules[0] === name) {
+                        throw new EntryPointError(compilationErrors.map(err => err.message));
+                    }
+                    else {
+                        throw new MissingDependencyError(compilationErrors.map(err => err.toString()), { missingModules });
+                    }
+                }
+                else if (jsonStats.errors && jsonStats.errors.length > 0) {
+                    if (jsonStats.errors.some(error => error.message.includes("Unexpected character '#'"))) {
+                        throw new CLIBuildError(jsonStats.errors);
+                    }
+                    else {
+                        throw new BuildError(jsonStats.errors);
+                    }
                 }
                 else {
-                    throw new BuildError(jsonStats.errors);
+                    throw new UnexpectedBuildError('The webpack stats object was unexpectedly empty');
                 }
             }
             else {
-                throw new UnexpectedBuildError('The webpack stats object was unexpectedly empty');
-            }
-        }
-        else {
-            const getAssetStats = async (asset) => {
-                const bundle = path.join(outputPath, asset.name);
-                const bundleContents = await fs.promises.readFile(bundle);
-                const gzip = gzipSync(bundleContents, {}).length;
-                const matches = asset.name.match(/(.+?)\.bundle\.(.+)$/);
-                if (!matches) {
-                    throw new UnexpectedBuildError('Found an asset without the `.bundle` suffix. ' +
-                        'A loader customization might be needed to recognize this asset type' +
-                        asset.name);
+                const getAssetStats = async (asset) => {
+                    const bundle = path.join(outputPath, asset.name);
+                    const bundleContents = await fs.promises.readFile(bundle);
+                    const gzip = gzipSync(bundleContents, {}).length;
+                    const matches = asset.name.match(/(.+?)\.bundle\.(.+)$/);
+                    if (!matches) {
+                        throw new UnexpectedBuildError('Found an asset without the `.bundle` suffix. ' +
+                            'A loader customization might be needed to recognize this asset type' +
+                            asset.name);
+                    }
+                    const [, entryName, extension] = matches;
+                    return {
+                        name: entryName,
+                        type: extension,
+                        size: asset.size,
+                        gzip,
+                    };
+                };
+                const assetStatsPromises = ((_a = jsonStats === null || jsonStats === void 0 ? void 0 : jsonStats.assets) === null || _a === void 0 ? void 0 : _a.filter(asset => {
+                    var _a;
+                    return !((_a = asset.chunkNames) === null || _a === void 0 ? void 0 : _a.some(name => name === 'runtime' ||
+                        (typeof name === 'string' && name.startsWith('runtime~'))));
+                }).filter(asset => typeof asset.name === 'string' &&
+                    !asset.name.endsWith('LICENSE.txt')).map(getAssetStats)) || [];
+                const assetStats = await Promise.all(assetStatsPromises);
+                Telemetry.assetsGZIPParseTime(name, performance.now());
+                let dependencySizeResults = {};
+                if (options.includeDependencySizes) {
+                    const dependencySizes = await getDependencySizes(name, jsonStats);
+                    dependencySizeResults = {
+                        dependencySizes,
+                    };
                 }
-                const [, entryName, extension] = matches;
                 return {
-                    name: entryName,
-                    type: extension,
-                    size: asset.size,
-                    gzip,
-                };
-            };
-            const assetStatsPromises = ((_a = jsonStats === null || jsonStats === void 0 ? void 0 : jsonStats.assets) === null || _a === void 0 ? void 0 : _a.filter(asset => {
-                var _a;
-                return !((_a = asset.chunkNames) === null || _a === void 0 ? void 0 : _a.some(name => name === 'runtime' ||
-                    (typeof name === 'string' && name.startsWith('runtime~'))));
-            }).filter(asset => typeof asset.name === 'string' &&
-                !asset.name.endsWith('LICENSE.txt')).map(getAssetStats)) || [];
-            const assetStats = await Promise.all(assetStatsPromises);
-            Telemetry.assetsGZIPParseTime(name, performance.now());
-            let dependencySizeResults = {};
-            if (options.includeDependencySizes) {
-                const dependencySizes = await getDependencySizes(name, jsonStats);
-                dependencySizeResults = {
-                    dependencySizes,
+                    assets: assetStats || [],
+                    ...dependencySizeResults,
                 };
             }
-            return {
-                assets: assetStats || [],
-                ...dependencySizeResults,
-            };
+        }
+        finally {
+            fs.rmSync(outputPath, { recursive: true, force: true });
         }
     },
     async buildPackageIgnoringMissingDeps({ name, externals, installPath, options, }) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "package-build-stats",
-  "version": "8.2.6",
+  "version": "8.2.7",
   "type": "module",
   "author": "Shubham Kanodia <shubham.kanodia10@gmail.com>",
   "repository": "https://github.com/pastelsky/package-build-stats",

package/src/utils/build.utils.ts

@@ -1,4 +1,5 @@
 import path from 'path'
+import { randomUUID } from 'crypto'
 import config from '../config/config.js'
 import { Entry, rspack } from '@rspack/core'
 import isValidNPMName from 'is-valid-npm-name'
@@ -218,11 +219,14 @@ const BuildUtils = {
     externals,
     options,
   }: BuildPackageArgs) {
-    const outputPath = config.tmp
+    const id = randomUUID().slice(0, 8)
+    const outputPath = path.join(config.tmp, 'dist', `build-${name}-${id}`)
+    fs.mkdirSync(outputPath, { recursive: true })
     let entry: any = {}
 
     if (options.splitCustomImports) {
       if (!options.customImports || !options.customImports.length) {
+        fs.rmSync(outputPath, { recursive: true, force: true })
         return { assets: [] }
       }
       options.customImports.forEach(importt => {
@@ -248,119 +252,124 @@ const BuildUtils = {
       outputPath,
     })
 
-    const jsonStatsStartTime = performance.now()
-    let jsonStats = stats.toJson({
-      assets: true,
-      source: true,
-      chunks: false,
-      chunkGroups: false,
-      chunkModules: true,
-      modules: true,
-      nestedModules: true,
-      reasons: true,
-      depth: true,
-      errors: true,
-      entrypoints: false,
-      warnings: false,
-    })
-
-    if (!jsonStats) {
-      Telemetry.parseWebpackStats(name, false, jsonStatsStartTime)
-      throw new UnexpectedBuildError(
-        'Expected webpack json stats to be non-null, but was null',
-      )
-    } else {
-      Telemetry.parseWebpackStats(name, true, jsonStatsStartTime)
-    }
-
-    const compilationErrors = getCompilationErrors(stats)
-
-    if (error && !stats) {
-      throw new BuildError(error)
-    } else if (compilationErrors.length) {
-      const missingModules = BuildUtils._parseMissingModules(compilationErrors)
+    try {
+      const jsonStatsStartTime = performance.now()
+      let jsonStats = stats.toJson({
+        assets: true,
+        source: true,
+        chunks: false,
+        chunkGroups: false,
+        chunkModules: true,
+        modules: true,
+        nestedModules: true,
+        reasons: true,
+        depth: true,
+        errors: true,
+        entrypoints: false,
+        warnings: false,
+      })
 
-      if (missingModules.length) {
-        if (missingModules.length === 1 && missingModules[0] === name) {
-          throw new EntryPointError(compilationErrors.map(err => err.message))
-        } else {
-          throw new MissingDependencyError(
-            compilationErrors.map(err => err.toString()),
-            { missingModules },
-          )
-        }
-      } else if (jsonStats.errors && jsonStats.errors.length > 0) {
-        if (
-          jsonStats.errors.some(error =>
-            error.message.includes("Unexpected character '#'"),
-          )
-        ) {
-          throw new CLIBuildError(jsonStats.errors)
-        } else {
-          throw new BuildError(jsonStats.errors)
-        }
-      } else {
+      if (!jsonStats) {
+        Telemetry.parseWebpackStats(name, false, jsonStatsStartTime)
         throw new UnexpectedBuildError(
-          'The webpack stats object was unexpectedly empty',
+          'Expected webpack json stats to be non-null, but was null',
         )
+      } else {
+        Telemetry.parseWebpackStats(name, true, jsonStatsStartTime)
       }
-    } else {
-      const getAssetStats = async (asset: RspackStatsAsset) => {
-        const bundle = path.join(outputPath, asset.name)
-        const bundleContents = await fs.promises.readFile(bundle)
 
-        const gzip = gzipSync(bundleContents, {}).length
-        const matches = asset.name.match(/(.+?)\.bundle\.(.+)$/)
+      const compilationErrors = getCompilationErrors(stats)
 
-        if (!matches) {
+      if (error && !stats) {
+        throw new BuildError(error)
+      } else if (compilationErrors.length) {
+        const missingModules =
+          BuildUtils._parseMissingModules(compilationErrors)
+
+        if (missingModules.length) {
+          if (missingModules.length === 1 && missingModules[0] === name) {
+            throw new EntryPointError(compilationErrors.map(err => err.message))
+          } else {
+            throw new MissingDependencyError(
+              compilationErrors.map(err => err.toString()),
+              { missingModules },
+            )
+          }
+        } else if (jsonStats.errors && jsonStats.errors.length > 0) {
+          if (
+            jsonStats.errors.some(error =>
+              error.message.includes("Unexpected character '#'"),
+            )
+          ) {
+            throw new CLIBuildError(jsonStats.errors)
+          } else {
+            throw new BuildError(jsonStats.errors)
+          }
+        } else {
           throw new UnexpectedBuildError(
-            'Found an asset without the `.bundle` suffix. ' +
-              'A loader customization might be needed to recognize this asset type' +
-              asset.name,
+            'The webpack stats object was unexpectedly empty',
           )
         }
+      } else {
+        const getAssetStats = async (asset: RspackStatsAsset) => {
+          const bundle = path.join(outputPath, asset.name)
+          const bundleContents = await fs.promises.readFile(bundle)
+
+          const gzip = gzipSync(bundleContents, {}).length
+          const matches = asset.name.match(/(.+?)\.bundle\.(.+)$/)
+
+          if (!matches) {
+            throw new UnexpectedBuildError(
+              'Found an asset without the `.bundle` suffix. ' +
+                'A loader customization might be needed to recognize this asset type' +
+                asset.name,
+            )
+          }
 
-        const [, entryName, extension] = matches
+          const [, entryName, extension] = matches
 
-        return {
-          name: entryName,
-          type: extension,
-          size: asset.size,
-          gzip,
+          return {
+            name: entryName,
+            type: extension,
+            size: asset.size,
+            gzip,
+          }
         }
-      }
 
-      const assetStatsPromises =
-        jsonStats?.assets
-          ?.filter(
-            asset =>
-              !asset.chunkNames?.some(
-                name =>
-                  name === 'runtime' ||
-                  (typeof name === 'string' && name.startsWith('runtime~')),
-              ),
-          )
-          .filter(
-            asset =>
-              typeof asset.name === 'string' &&
-              !asset.name.endsWith('LICENSE.txt'),
-          )
-          .map(getAssetStats) || []
-      const assetStats = await Promise.all(assetStatsPromises)
-      Telemetry.assetsGZIPParseTime(name, performance.now())
-
-      let dependencySizeResults = {}
-      if (options.includeDependencySizes) {
-        const dependencySizes = await getDependencySizes(name, jsonStats)
-        dependencySizeResults = {
-          dependencySizes,
+        const assetStatsPromises =
+          jsonStats?.assets
+            ?.filter(
+              asset =>
+                !asset.chunkNames?.some(
+                  name =>
+                    name === 'runtime' ||
+                    (typeof name === 'string' && name.startsWith('runtime~')),
+                ),
+            )
+            .filter(
+              asset =>
+                typeof asset.name === 'string' &&
+                !asset.name.endsWith('LICENSE.txt'),
+            )
+            .map(getAssetStats) || []
+        const assetStats = await Promise.all(assetStatsPromises)
+        Telemetry.assetsGZIPParseTime(name, performance.now())
+
+        let dependencySizeResults = {}
+        if (options.includeDependencySizes) {
+          const dependencySizes = await getDependencySizes(name, jsonStats)
+          dependencySizeResults = {
+            dependencySizes,
+          }
         }
-      }
 
-      return {
-        assets: assetStats || [],
-        ...dependencySizeResults,
+        return {
+          assets: assetStats || [],
+          ...dependencySizeResults,
+        }
       }
+    } finally {
+      fs.rmSync(outputPath, { recursive: true, force: true })
     }
   },
   async buildPackageIgnoringMissingDeps({