package-build-stats 8.2.5 → 8.2.7

package/build/config/makeRspackConfig.d.ts

@@ -6,6 +6,7 @@ type MakeRspackConfigOptions = {
     debug?: boolean;
     minify?: boolean;
     entry: Entry;
+    outputPath: string;
 };
-export default function makeRspackConfig({ packageName: _packageName, entry, externals, debug: _debug, minify, }: MakeRspackConfigOptions): Configuration;
+export default function makeRspackConfig({ packageName: _packageName, entry, externals, debug: _debug, minify, outputPath, }: MakeRspackConfigOptions): Configuration;
 export {};

package/build/config/makeRspackConfig.js

@@ -3,7 +3,7 @@ import escapeRegex from 'escape-string-regexp';
 import rspack from '@rspack/core';
 import { createRequire } from 'module';
 const require = createRequire(import.meta.url);
-export default function makeRspackConfig({ packageName: _packageName, entry, externals, debug: _debug, minify = true, }) {
+export default function makeRspackConfig({ packageName: _packageName, entry, externals, debug: _debug, minify = true, outputPath, }) {
     const externalsRegex = makeExternalsRegex(externals.externalPackages);
     const isExternalRequest = (request) => {
         const isPeerDep = externals.externalPackages.length
@@ -154,6 +154,7 @@ export default function makeRspackConfig({ packageName: _packageName, entry, ext
         },
         output: {
             filename: '[name].bundle.js',
+            path: outputPath,
         },
         externals: ({ request }, callback) => isExternalRequest(request || '')
             ? callback(undefined, 'commonjs ' + request)

package/build/utils/build.utils.d.ts

@@ -7,6 +7,7 @@ type CompilePackageArgs = {
     entry: Entry;
     debug?: boolean;
     minify?: boolean;
+    outputPath: string;
 };
 type CompilePackageReturn = {
     stats: Stats;
@@ -36,7 +37,7 @@ type BuildPackageResultWithIgnored = BuildPackageResult & {
 declare function getCompilationErrors(stats: Stats): import("@rspack/core").RspackError[];
 declare const BuildUtils: {
     createEntryPoint(packageName: string, installPath: string, options: CreateEntryPointOptions): string;
-    compilePackage({ name, entry, externals, debug, minify, }: CompilePackageArgs): Promise<CompilePackageReturn>;
+    compilePackage({ name, entry, externals, debug, minify, outputPath, }: CompilePackageArgs): Promise<CompilePackageReturn>;
     _parseMissingModules(errors: ReturnType<typeof getCompilationErrors>): string[];
     buildPackage({ name, installPath, externals, options, }: BuildPackageArgs): Promise<{
         assets: {

package/build/utils/build.utils.js

@@ -1,4 +1,6 @@
 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';
 import { gzipSync } from 'zlib';
@@ -60,7 +62,7 @@ const BuildUtils = {
             throw new EntryPointError(err);
         }
     },
-    compilePackage({ name, entry, externals, debug, minify, }) {
+    compilePackage({ name, entry, externals, debug, minify, outputPath, }) {
         const startTime = performance.now();
         const options = makeRspackConfig({
             packageName: name,
@@ -68,6 +70,7 @@ const BuildUtils = {
             externals,
             debug,
             minify,
+            outputPath,
         });
         const compiler = rspack(options);
         return new Promise((resolve, reject) => {
@@ -119,9 +122,13 @@ const BuildUtils = {
     },
     async buildPackage({ name, installPath, externals, options, }) {
         var _a;
+        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 => {
@@ -144,93 +151,99 @@ const BuildUtils = {
             externals,
             debug: options.debug,
             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(process.cwd(), 'dist', 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.5",
+  "version": "8.2.7",
   "type": "module",
   "author": "Shubham Kanodia <shubham.kanodia10@gmail.com>",
   "repository": "https://github.com/pastelsky/package-build-stats",

package/src/config/makeRspackConfig.ts

@@ -15,6 +15,7 @@ type MakeRspackConfigOptions = {
   debug?: boolean
   minify?: boolean
   entry: Entry
+  outputPath: string
 }
 
 export default function makeRspackConfig({
@@ -23,6 +24,7 @@ export default function makeRspackConfig({
   externals,
   debug: _debug,
   minify = true,
+  outputPath,
 }: MakeRspackConfigOptions): Configuration {
   const externalsRegex = makeExternalsRegex(externals.externalPackages)
   const isExternalRequest = (request: string) => {
@@ -176,6 +178,7 @@ export default function makeRspackConfig({
     },
     output: {
       filename: '[name].bundle.js',
+      path: outputPath,
     },
     externals: ({ request }, callback) =>
       isExternalRequest(request || '')

package/src/utils/build.utils.ts

@@ -1,4 +1,6 @@
 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'
 import { gzipSync } from 'zlib'
@@ -28,6 +30,7 @@ type CompilePackageArgs = {
   entry: Entry
   debug?: boolean
   minify?: boolean
+  outputPath: string
 }
 
 type CompilePackageReturn = {
@@ -133,6 +136,7 @@ const BuildUtils = {
     externals,
     debug,
     minify,
+    outputPath,
   }: CompilePackageArgs) {
     const startTime = performance.now()
 
@@ -142,6 +146,7 @@ const BuildUtils = {
       externals,
       debug,
       minify,
+      outputPath,
     })
 
     const compiler = rspack(options)
@@ -214,10 +219,14 @@ const BuildUtils = {
     externals,
     options,
   }: BuildPackageArgs) {
+    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 => {
@@ -240,121 +249,127 @@ const BuildUtils = {
       externals,
       debug: options.debug,
       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)
+    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(process.cwd(), 'dist', 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({