package-build-stats 8.2.4 → 8.2.5

package/build/utils/build.utils.js

@@ -14,6 +14,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');
@@ -58,18 +70,24 @@ const BuildUtils = {
             minify,
         });
         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);
                 }
             });
         });
@@ -198,7 +216,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.5",
   "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/utils/build.utils.ts

@@ -35,6 +35,8 @@ type CompilePackageReturn = {
   error: Error | null
 }
 
+type Compiler = NonNullable<ReturnType<typeof rspack>>
+
 type BuildPackageArgs = {
   name: string
   installPath: string
@@ -70,6 +72,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,
@@ -132,18 +146,25 @@ const BuildUtils = {
 
     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)
         }
       })
     })
@@ -313,7 +334,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())