package-build-stats 8.2.3 → 8.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "package-build-stats",
-  "version": "8.2.3",
+  "version": "8.2.5",
   "type": "module",
   "author": "Shubham Kanodia <shubham.kanodia10@gmail.com>",
   "repository": "https://github.com/pastelsky/package-build-stats",
@@ -40,7 +40,7 @@
     "compare:top": "tsx scripts/index.ts top",
     "compare:advanced": "tsx scripts/index.ts compare",
     "perf:test": "tsx scripts/performance-test.ts",
-    "prepublish": "rm -rf build && yarn check && tsc && yarn format"
+    "prepack": "rm -rf build && yarn check && yarn build && yarn format"
   },
   "prettier": {
     "semi": false,
@@ -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)
         }
       })
     })
@@ -193,7 +214,7 @@ const BuildUtils = {
     externals,
     options,
   }: BuildPackageArgs) {
-    let entry: Entry = {}
+    let entry: any = {}
 
     if (options.splitCustomImports) {
       if (!options.customImports || !options.customImports.length) {
@@ -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())