fix: preserve existing package analysis APIs

Author: pastelsky

.changeset/calm-dragons-share.md

@@ -1,5 +1,5 @@
 ---
-'package-build-stats': major
+'package-build-stats': minor
 ---
 
-Add reusable installed-package handles so multiple analyses can share one workspace, and rename `getAllPackageExports` to `getPackageExports`.
+Add reusable installed-package handles so multiple analyses can share one workspace, and add `getPackageExports` as the canonical name while preserving `getAllPackageExports` as a deprecated alias.

src/getPackageExportSizes.ts

@@ -66,6 +66,11 @@ export async function getPackageExports(
   return getExportsForInstalledPackage(packageInput, options, startTime)
 }
 
+/**
+ * @deprecated Use `getPackageExports`; this alias preserves the existing API.
+ */
+export const getAllPackageExports = getPackageExports
+
 async function analyzePackageExportSizes(
   installation: InstalledPackage,
   options: GetPackageStatsOptions = {},

src/getPackageStats.ts

@@ -143,8 +143,14 @@ export default async function getPackageStats(
 ) {
   const startTime = performance.now()
   if (typeof packageInput === 'string') {
-    return withInstalledPackage(packageInput, options, installedPackage =>
-      getStatsForInstalledPackage(installedPackage, options, startTime),
+    return withInstalledPackage(
+      packageInput,
+      options,
+      installedPackage =>
+        getStatsForInstalledPackage(installedPackage, options, startTime),
+      // Preserve the existing debug contract: retain the workspace so callers
+      // can inspect emitted source maps and intermediate build artifacts.
+      !options.debug,
     )
   }
 

src/installedPackage.ts

@@ -64,12 +64,15 @@ export async function withInstalledPackage<TResult>(
   packageString: string,
   options: InstallPackageOptions,
   analyze: (installedPackage: InstalledPackage) => Promise<TResult>,
+  disposeAfterAnalysis = true,
 ) {
   const installedPackage = await installPackage(packageString, options)
 
   try {
     return await analyze(installedPackage)
   } finally {
-    await disposePackage(installedPackage)
+    if (disposeAfterAnalysis) {
+      await disposePackage(installedPackage)
+    }
   }
 }

tests/slow/export-sizes.test.ts

@@ -7,6 +7,7 @@
 
 import path from 'path'
 import {
+  getAllPackageExports,
   getPackageExports,
   getPackageExportSizes,
 } from '../../src/getPackageExportSizes'
@@ -31,6 +32,10 @@ vi.mock('../../src/utils/common.utils.js', async importOriginal => {
 })
 
 describe('getPackageExports', () => {
+  test('keeps the previous getAllPackageExports API as an alias', () => {
+    expect(getAllPackageExports).toBe(getPackageExports)
+  })
+
   test('should get exports from package with multiple exports', async () => {
     const fixturePath = path.resolve(
       __dirname,