package-build-stats 8.2.4 → 8.2.6

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,5 @@
 import path from 'path';
+import config from '../config/config.js';
 import { rspack } from '@rspack/core';
 import isValidNPMName from 'is-valid-npm-name';
 import { gzipSync } from 'zlib';
@@ -14,6 +15,18 @@ function notEmpty(value) {
 function getCompilationErrors(stats) {
     return [...stats.compilation.errors].filter(notEmpty).flat();
 }
+function closeCompiler(compiler) {
+    return new Promise((resolve, reject) => {
+        compiler.close(error => {
+            if (error) {
+                reject(error);
+            }
+            else {
+                resolve();
+            }
+        });
+    });
+}
 const BuildUtils = {
     createEntryPoint(packageName, installPath, options) {
         const entryPath = path.join(installPath, options.entryFilename || 'index.js');
@@ -48,7 +61,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,
@@ -56,20 +69,27 @@ const BuildUtils = {
             externals,
             debug,
             minify,
+            outputPath,
         });
         const compiler = rspack(options);
-        return new Promise(resolve => {
-            compiler.run((error, stats) => {
-                if (!stats) {
-                    throw new Error('stats is null');
-                }
-                resolve({ stats, error });
-                if (error) {
-                    console.error(error);
-                    Telemetry.compilePackage(name, false, startTime, {}, error);
+        return new Promise((resolve, reject) => {
+            compiler.run(async (error, stats) => {
+                try {
+                    if (!stats) {
+                        throw new Error('stats is null');
+                    }
+                    await closeCompiler(compiler);
+                    if (error) {
+                        console.error(error);
+                        Telemetry.compilePackage(name, false, startTime, {}, error);
+                    }
+                    else {
+                        Telemetry.compilePackage(name, true, startTime, {});
+                    }
+                    resolve({ stats, error });
                 }
-                else {
-                    Telemetry.compilePackage(name, true, startTime, {});
+                catch (closeError) {
+                    reject(closeError);
                 }
             });
         });
@@ -101,6 +121,7 @@ const BuildUtils = {
     },
     async buildPackage({ name, installPath, externals, options, }) {
         var _a;
+        const outputPath = config.tmp;
         let entry = {};
         if (options.splitCustomImports) {
             if (!options.customImports || !options.customImports.length) {
@@ -126,6 +147,7 @@ const BuildUtils = {
             externals,
             debug: options.debug,
             minify: options.minify,
+            outputPath,
         });
         const jsonStatsStartTime = performance.now();
         let jsonStats = stats.toJson({
@@ -177,7 +199,7 @@ const BuildUtils = {
         }
         else {
             const getAssetStats = async (asset) => {
-                const bundle = path.join(process.cwd(), 'dist', asset.name);
+                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\.(.+)$/);
@@ -198,7 +220,8 @@ const BuildUtils = {
                 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 => !asset.name.endsWith('LICENSE.txt')).map(getAssetStats)) || [];
+            }).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 = {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "package-build-stats",
-  "version": "8.2.4",
+  "version": "8.2.6",
   "type": "module",
   "author": "Shubham Kanodia <shubham.kanodia10@gmail.com>",
   "repository": "https://github.com/pastelsky/package-build-stats",
@@ -61,6 +61,7 @@
     "@types/rimraf": "^4.0.5",
     "@types/sanitize-filename": "^1.6.3",
     "@types/semver": "^7",
+    "@types/server": "^1",
     "@types/yargs": "^17.0.34",
     "@vitest/coverage-v8": "^4.0.6",
     "@vitest/ui": "4.0.6",
@@ -105,6 +106,7 @@
     "sanitize-filename": "^1.6.3",
     "sass": "^1.69.0",
     "sass-loader": "^14.0.0",
+    "server": "^1.0.42",
     "svelte": "^4.0.0",
     "svelte-loader": "^3.1.0",
     "terser": "^5.44.0",

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,5 @@
 import path from 'path'
+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 +29,7 @@ type CompilePackageArgs = {
   entry: Entry
   debug?: boolean
   minify?: boolean
+  outputPath: string
 }
 
 type CompilePackageReturn = {
@@ -35,6 +37,8 @@ type CompilePackageReturn = {
   error: Error | null
 }
 
+type Compiler = NonNullable<ReturnType<typeof rspack>>
+
 type BuildPackageArgs = {
   name: string
   installPath: string
@@ -70,6 +74,18 @@ function getCompilationErrors(stats: Stats) {
   return [...stats.compilation.errors].filter(notEmpty).flat()
 }
 
+function closeCompiler(compiler: Compiler) {
+  return new Promise<void>((resolve, reject) => {
+    compiler.close(error => {
+      if (error) {
+        reject(error)
+      } else {
+        resolve()
+      }
+    })
+  })
+}
+
 const BuildUtils = {
   createEntryPoint(
     packageName: string,
@@ -119,6 +135,7 @@ const BuildUtils = {
     externals,
     debug,
     minify,
+    outputPath,
   }: CompilePackageArgs) {
     const startTime = performance.now()
 
@@ -128,22 +145,30 @@ const BuildUtils = {
       externals,
       debug,
       minify,
+      outputPath,
     })
 
     const compiler = rspack(options)
 
-    return new Promise<CompilePackageReturn>(resolve => {
-      compiler.run((error, stats) => {
-        if (!stats) {
-          throw new Error('stats is null')
-        }
-        resolve({ stats, error })
-
-        if (error) {
-          console.error(error)
-          Telemetry.compilePackage(name, false, startTime, {}, error)
-        } else {
-          Telemetry.compilePackage(name, true, startTime, {})
+    return new Promise<CompilePackageReturn>((resolve, reject) => {
+      compiler.run(async (error, stats) => {
+        try {
+          if (!stats) {
+            throw new Error('stats is null')
+          }
+
+          await closeCompiler(compiler)
+
+          if (error) {
+            console.error(error)
+            Telemetry.compilePackage(name, false, startTime, {}, error)
+          } else {
+            Telemetry.compilePackage(name, true, startTime, {})
+          }
+
+          resolve({ stats, error })
+        } catch (closeError) {
+          reject(closeError)
         }
       })
     })
@@ -193,6 +218,7 @@ const BuildUtils = {
     externals,
     options,
   }: BuildPackageArgs) {
+    const outputPath = config.tmp
     let entry: any = {}
 
     if (options.splitCustomImports) {
@@ -219,6 +245,7 @@ const BuildUtils = {
       externals,
       debug: options.debug,
       minify: options.minify,
+      outputPath,
     })
 
     const jsonStatsStartTime = performance.now()
@@ -279,7 +306,7 @@ const BuildUtils = {
       }
     } else {
       const getAssetStats = async (asset: RspackStatsAsset) => {
-        const bundle = path.join(process.cwd(), 'dist', asset.name)
+        const bundle = path.join(outputPath, asset.name)
         const bundleContents = await fs.promises.readFile(bundle)
 
         const gzip = gzipSync(bundleContents, {}).length
@@ -313,7 +340,11 @@ const BuildUtils = {
                   (typeof name === 'string' && name.startsWith('runtime~')),
               ),
           )
-          .filter(asset => !asset.name.endsWith('LICENSE.txt'))
+          .filter(
+            asset =>
+              typeof asset.name === 'string' &&
+              !asset.name.endsWith('LICENSE.txt'),
+          )
           .map(getAssetStats) || []
       const assetStats = await Promise.all(assetStatsPromises)
       Telemetry.assetsGZIPParseTime(name, performance.now())