@atlaspack/packager-js 2.23.5-dev-9bc89ddee.0 → 2.23.5-dev-9dac9ed3a.0

package/dist/ScopeHoistingPackager.js

@@ -235,10 +235,14 @@ class ScopeHoistingPackager {
                 sourceMap.addSourceMap(map, lineCount);
             }
         }
-        return {
-            contents: res,
-            map: sourceMap,
-        };
+        const result = { contents: res, map: sourceMap };
+        if (utils_1.debugTools['scope-hoisting-stats']) {
+            result.scopeHoistingStats = {
+                totalAssets: this.assetOutputs.size,
+                wrappedAssets: this.wrappedAssets.size,
+            };
+        }
+        return result;
     }
     shouldBundleQueue(bundle) {
         let referencingBundles = this.bundleGraph.getReferencingBundles(bundle);

package/dist/index.js

@@ -55,6 +55,7 @@ exports.default = new plugin_1.Packager({
         // If this is a non-module script, and there is only one asset with no dependencies,
         // then we don't need to package at all and can pass through the original code un-wrapped.
         let contents, map;
+        let scopeHoistingStats;
         if (bundle.env.sourceType === 'script') {
             let entries = bundle.getEntryAssets();
             if (entries.length === 1 &&
@@ -64,10 +65,17 @@ exports.default = new plugin_1.Packager({
             }
         }
         if (contents == null) {
-            let packager = bundle.env.shouldScopeHoist
-                ? new ScopeHoistingPackager_1.ScopeHoistingPackager(options, bundleGraph, bundle, (0, nullthrows_1.default)(config).parcelRequireName, (0, nullthrows_1.default)(config).unstable_asyncBundleRuntime, (0, nullthrows_1.default)(config).unstable_manualStaticBindingExports, logger)
-                : new DevPackager_1.DevPackager(options, bundleGraph, bundle, (0, nullthrows_1.default)(config).parcelRequireName);
-            ({ contents, map } = await packager.package());
+            if (bundle.env.shouldScopeHoist) {
+                let packager = new ScopeHoistingPackager_1.ScopeHoistingPackager(options, bundleGraph, bundle, (0, nullthrows_1.default)(config).parcelRequireName, (0, nullthrows_1.default)(config).unstable_asyncBundleRuntime, (0, nullthrows_1.default)(config).unstable_manualStaticBindingExports, logger);
+                let packageResult = await packager.package();
+                ({ contents, map } = packageResult);
+                scopeHoistingStats = packageResult.scopeHoistingStats;
+            }
+            else {
+                let packager = new DevPackager_1.DevPackager(options, bundleGraph, bundle, (0, nullthrows_1.default)(config).parcelRequireName);
+                let packageResult = await packager.package();
+                ({ contents, map } = packageResult);
+            }
         }
         contents += '\n' + (await getSourceMapSuffix(getSourceMapReference, map));
         // For library builds, we need to replace URL references with their final resolved paths.
@@ -81,7 +89,7 @@ exports.default = new plugin_1.Packager({
                 getReplacement: (s) => JSON.stringify(s).slice(1, -1),
             }));
         }
-        return (0, utils_1.replaceInlineReferences)({
+        let result = await (0, utils_1.replaceInlineReferences)({
             bundle,
             bundleGraph,
             contents,
@@ -92,6 +100,10 @@ exports.default = new plugin_1.Packager({
             getInlineBundleContents,
             map,
         });
+        if (utils_1.debugTools['scope-hoisting-stats']) {
+            return { ...result, scopeHoistingStats };
+        }
+        return result;
     },
 });
 async function getSourceMapSuffix(getSourceMapReference, map) {

package/lib/ScopeHoistingPackager.js

@@ -253,10 +253,17 @@ class ScopeHoistingPackager {
         sourceMap.addSourceMap(map, lineCount);
       }
     }
-    return {
+    const result = {
       contents: res,
       map: sourceMap
     };
+    if (_utils().debugTools['scope-hoisting-stats']) {
+      result.scopeHoistingStats = {
+        totalAssets: this.assetOutputs.size,
+        wrappedAssets: this.wrappedAssets.size
+      };
+    }
+    return result;
   }
   shouldBundleQueue(bundle) {
     let referencingBundles = this.bundleGraph.getReferencingBundles(bundle);

package/lib/index.js

@@ -100,6 +100,7 @@ var _default = exports.default = new (_plugin().Packager)({
     // If this is a non-module script, and there is only one asset with no dependencies,
     // then we don't need to package at all and can pass through the original code un-wrapped.
     let contents, map;
+    let scopeHoistingStats;
     if (bundle.env.sourceType === 'script') {
       let entries = bundle.getEntryAssets();
       if (entries.length === 1 && bundleGraph.getDependencies(entries[0]).length === 0) {
@@ -108,11 +109,22 @@ var _default = exports.default = new (_plugin().Packager)({
       }
     }
     if (contents == null) {
-      let packager = bundle.env.shouldScopeHoist ? new _ScopeHoistingPackager.ScopeHoistingPackager(options, bundleGraph, bundle, (0, _nullthrows().default)(config).parcelRequireName, (0, _nullthrows().default)(config).unstable_asyncBundleRuntime, (0, _nullthrows().default)(config).unstable_manualStaticBindingExports, logger) : new _DevPackager.DevPackager(options, bundleGraph, bundle, (0, _nullthrows().default)(config).parcelRequireName);
-      ({
-        contents,
-        map
-      } = await packager.package());
+      if (bundle.env.shouldScopeHoist) {
+        let packager = new _ScopeHoistingPackager.ScopeHoistingPackager(options, bundleGraph, bundle, (0, _nullthrows().default)(config).parcelRequireName, (0, _nullthrows().default)(config).unstable_asyncBundleRuntime, (0, _nullthrows().default)(config).unstable_manualStaticBindingExports, logger);
+        let packageResult = await packager.package();
+        ({
+          contents,
+          map
+        } = packageResult);
+        scopeHoistingStats = packageResult.scopeHoistingStats;
+      } else {
+        let packager = new _DevPackager.DevPackager(options, bundleGraph, bundle, (0, _nullthrows().default)(config).parcelRequireName);
+        let packageResult = await packager.package();
+        ({
+          contents,
+          map
+        } = packageResult);
+      }
     }
     contents += '\n' + (await getSourceMapSuffix(getSourceMapReference, map));
 
@@ -130,7 +142,7 @@ var _default = exports.default = new (_plugin().Packager)({
         getReplacement: s => JSON.stringify(s).slice(1, -1)
       }));
     }
-    return (0, _utils().replaceInlineReferences)({
+    let result = await (0, _utils().replaceInlineReferences)({
       bundle,
       bundleGraph,
       contents,
@@ -141,6 +153,13 @@ var _default = exports.default = new (_plugin().Packager)({
       getInlineBundleContents,
       map
     });
+    if (_utils().debugTools['scope-hoisting-stats']) {
+      return {
+        ...result,
+        scopeHoistingStats
+      };
+    }
+    return result;
   }
 });
 async function getSourceMapSuffix(getSourceMapReference, map) {

package/lib/types/ScopeHoistingPackager.d.ts

@@ -4,6 +4,14 @@ export interface OutputFormat {
     buildBundlePrelude(): [string, number];
     buildBundlePostlude(): [string, number];
 }
+export type PackageResult = {
+    contents: string;
+    map: SourceMap | null | undefined;
+    scopeHoistingStats?: {
+        totalAssets: number;
+        wrappedAssets: number;
+    };
+};
 export declare class ScopeHoistingPackager {
     options: PluginOptions;
     bundleGraph: BundleGraph<NamedBundle>;
@@ -37,10 +45,7 @@ export declare class ScopeHoistingPackager {
     logger: PluginLogger;
     useBothScopeHoistingImprovements: boolean;
     constructor(options: PluginOptions, bundleGraph: BundleGraph<NamedBundle>, bundle: NamedBundle, parcelRequireName: string, useAsyncBundleRuntime: boolean, manualStaticBindingExports: string[] | null, logger: PluginLogger);
-    package(): Promise<{
-        contents: string;
-        map: SourceMap | null | undefined;
-    }>;
+    package(): Promise<PackageResult>;
     shouldBundleQueue(bundle: NamedBundle): boolean;
     runWhenReady(bundle: NamedBundle, codeToRun: string): string;
     loadAssets(): Promise<void>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaspack/packager-js",
-  "version": "2.23.5-dev-9bc89ddee.0",
+  "version": "2.23.5-dev-9dac9ed3a.0",
   "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
     "access": "public"
@@ -16,13 +16,13 @@
     "node": ">= 16.0.0"
   },
   "dependencies": {
-    "@atlaspack/diagnostic": "2.14.5-dev-9bc89ddee.0",
-    "@atlaspack/feature-flags": "2.25.5-dev-9bc89ddee.0",
-    "@atlaspack/plugin": "2.14.35-dev-9bc89ddee.0",
-    "@atlaspack/rust": "3.8.3-dev-9bc89ddee.0",
-    "@atlaspack/source-map": "3.0.1-dev-9bc89ddee.0",
-    "@atlaspack/types": "2.15.25-dev-9bc89ddee.0",
-    "@atlaspack/utils": "3.0.3-dev-9bc89ddee.0",
+    "@atlaspack/diagnostic": "2.14.5-dev-9dac9ed3a.0",
+    "@atlaspack/feature-flags": "2.25.5-dev-9dac9ed3a.0",
+    "@atlaspack/plugin": "2.14.35-dev-9dac9ed3a.0",
+    "@atlaspack/rust": "3.8.3-dev-9dac9ed3a.0",
+    "@atlaspack/source-map": "3.0.1-dev-9dac9ed3a.0",
+    "@atlaspack/types": "2.15.25-dev-9dac9ed3a.0",
+    "@atlaspack/utils": "3.0.3-dev-9dac9ed3a.0",
     "globals": "^13.2.0",
     "nullthrows": "^1.1.1",
     "outdent": "^0.8.0"
@@ -31,5 +31,5 @@
   "scripts": {
     "build:lib": "gulp build --gulpfile ../../../gulpfile.js --cwd ."
   },
-  "gitHead": "9bc89ddeed68b29ffa09a0e3a0525923572853e6"
+  "gitHead": "9dac9ed3ab18934ba96e184638b45e9f171ecfc7"
 }

package/src/ScopeHoistingPackager.ts

@@ -82,6 +82,15 @@ export interface OutputFormat {
   buildBundlePostlude(): [string, number];
 }
 
+export type PackageResult = {
+  contents: string;
+  map: SourceMap | null | undefined;
+  scopeHoistingStats?: {
+    totalAssets: number;
+    wrappedAssets: number;
+  };
+};
+
 export class ScopeHoistingPackager {
   options: PluginOptions;
   bundleGraph: BundleGraph<NamedBundle>;
@@ -153,10 +162,7 @@ export class ScopeHoistingPackager {
     this.globalNames = GLOBALS_BY_CONTEXT[bundle.env.context];
   }
 
-  async package(): Promise<{
-    contents: string;
-    map: SourceMap | null | undefined;
-  }> {
+  async package(): Promise<PackageResult> {
     await this.loadAssets();
     this.buildExportedSymbols();
 
@@ -319,10 +325,16 @@ export class ScopeHoistingPackager {
       }
     }
 
-    return {
-      contents: res,
-      map: sourceMap,
-    };
+    const result: PackageResult = {contents: res, map: sourceMap};
+
+    if (debugTools['scope-hoisting-stats']) {
+      result.scopeHoistingStats = {
+        totalAssets: this.assetOutputs.size,
+        wrappedAssets: this.wrappedAssets.size,
+      };
+    }
+
+    return result;
   }
 
   shouldBundleQueue(bundle: NamedBundle): boolean {

package/src/index.ts

@@ -1,4 +1,4 @@
-import type {Async} from '@atlaspack/types';
+import type {Async, BundleResult} from '@atlaspack/types';
 import type SourceMap from '@atlaspack/source-map';
 import {Packager} from '@atlaspack/plugin';
 import {
@@ -6,12 +6,16 @@ import {
   replaceURLReferences,
   validateSchema,
   SchemaEntity,
+  debugTools,
 } from '@atlaspack/utils';
 import {encodeJSONKeyComponent} from '@atlaspack/diagnostic';
 import {hashString} from '@atlaspack/rust';
 import nullthrows from 'nullthrows';
 import {DevPackager} from './DevPackager';
-import {ScopeHoistingPackager} from './ScopeHoistingPackager';
+import {
+  type PackageResult as ScopeHoistingPackageResult,
+  ScopeHoistingPackager,
+} from './ScopeHoistingPackager';
 
 type JSPackagerConfig = {
   parcelRequireName: string;
@@ -91,6 +95,8 @@ export default new Packager({
     // If this is a non-module script, and there is only one asset with no dependencies,
     // then we don't need to package at all and can pass through the original code un-wrapped.
     let contents, map;
+    let scopeHoistingStats: ScopeHoistingPackageResult['scopeHoistingStats'];
+
     if (bundle.env.sourceType === 'script') {
       let entries = bundle.getEntryAssets();
       if (
@@ -103,24 +109,31 @@ export default new Packager({
     }
 
     if (contents == null) {
-      let packager = bundle.env.shouldScopeHoist
-        ? new ScopeHoistingPackager(
-            options,
-            bundleGraph,
-            bundle,
-            nullthrows(config).parcelRequireName,
-            nullthrows(config).unstable_asyncBundleRuntime,
-            nullthrows(config).unstable_manualStaticBindingExports,
-            logger,
-          )
-        : new DevPackager(
-            options,
-            bundleGraph,
-            bundle,
-            nullthrows(config).parcelRequireName,
-          );
-
-      ({contents, map} = await packager.package());
+      if (bundle.env.shouldScopeHoist) {
+        let packager = new ScopeHoistingPackager(
+          options,
+          bundleGraph,
+          bundle,
+          nullthrows(config).parcelRequireName,
+          nullthrows(config).unstable_asyncBundleRuntime,
+          nullthrows(config).unstable_manualStaticBindingExports,
+          logger,
+        );
+
+        let packageResult = await packager.package();
+        ({contents, map} = packageResult);
+        scopeHoistingStats = packageResult.scopeHoistingStats;
+      } else {
+        let packager = new DevPackager(
+          options,
+          bundleGraph,
+          bundle,
+          nullthrows(config).parcelRequireName,
+        );
+
+        let packageResult = await packager.package();
+        ({contents, map} = packageResult);
+      }
     }
 
     contents += '\n' + (await getSourceMapSuffix(getSourceMapReference, map));
@@ -137,7 +150,7 @@ export default new Packager({
       }));
     }
 
-    return replaceInlineReferences({
+    let result = await replaceInlineReferences({
       bundle,
       bundleGraph,
       contents,
@@ -148,6 +161,12 @@ export default new Packager({
       getInlineBundleContents,
       map,
     });
+
+    if (debugTools['scope-hoisting-stats']) {
+      return {...result, scopeHoistingStats};
+    }
+
+    return result;
   },
 }) as Packager<unknown, unknown>;