@atlaspack/packager-js 2.14.5-canary.155 → 2.14.5-canary.157

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # @atlaspack/packager-js
 
+## 2.21.0
+
+### Minor Changes
+
+- [#732](https://github.com/atlassian-labs/atlaspack/pull/732) [`7f5841c`](https://github.com/atlassian-labs/atlaspack/commit/7f5841c39df049f9546cccbeea2a7337e0337b45) Thanks [@vykimnguyen](https://github.com/vykimnguyen)! - add tesseract context
+
+### Patch Changes
+
+- [#744](https://github.com/atlassian-labs/atlaspack/pull/744) [`3718bff`](https://github.com/atlassian-labs/atlaspack/commit/3718bff20484138e405e321472336ac058432e10) Thanks [@benjervis](https://github.com/benjervis)! - Fix the scope hoisting improvements feature by making all entry assets of a bundle
+  wrapped, which means there is no top level scope.
+- Updated dependencies [[`ad26146`](https://github.com/atlassian-labs/atlaspack/commit/ad26146f13b4c1cc65d4a0f9c67060b90ef14ff3), [`f1b48e7`](https://github.com/atlassian-labs/atlaspack/commit/f1b48e7a04e005cef0f36a3e692087a9ecdb6f7a), [`7f5841c`](https://github.com/atlassian-labs/atlaspack/commit/7f5841c39df049f9546cccbeea2a7337e0337b45), [`73dd7ba`](https://github.com/atlassian-labs/atlaspack/commit/73dd7baab69456ef2f6e4a0cc7dbb04f407eb148)]:
+  - @atlaspack/rust@3.6.0
+  - @atlaspack/feature-flags@2.22.0
+  - @atlaspack/utils@2.18.1
+  - @atlaspack/plugin@2.14.24
+  - @atlaspack/types@2.15.14
+
 ## 2.20.0
 
 ### Minor Changes

package/lib/ScopeHoistingPackager.js

@@ -316,6 +316,14 @@ class ScopeHoistingPackager {
       }
     });
     if ((0, _featureFlags().getFeatureFlag)('applyScopeHoistingImprovement')) {
+      // Make all entry assets wrapped, to avoid any top level hoisting
+      for (let entryAsset of this.bundle.getEntryAssets()) {
+        if (!this.wrappedAssets.has(entryAsset.id)) {
+          this.wrappedAssets.add(entryAsset.id);
+          wrapped.push(entryAsset);
+        }
+      }
+
       // Tracks which assets have been assigned to a wrap group
       let assignedAssets = new Set();
       for (let wrappedAsset of wrapped) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaspack/packager-js",
-  "version": "2.14.5-canary.155+7f5841c39",
+  "version": "2.14.5-canary.157+15a002dd2",
   "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
     "access": "public"
@@ -16,12 +16,12 @@
     "node": ">= 16.0.0"
   },
   "dependencies": {
-    "@atlaspack/diagnostic": "2.14.1-canary.223+7f5841c39",
-    "@atlaspack/feature-flags": "2.14.1-canary.223+7f5841c39",
-    "@atlaspack/plugin": "2.14.5-canary.155+7f5841c39",
-    "@atlaspack/rust": "3.2.1-canary.155+7f5841c39",
-    "@atlaspack/types": "2.14.5-canary.155+7f5841c39",
-    "@atlaspack/utils": "2.14.5-canary.155+7f5841c39",
+    "@atlaspack/diagnostic": "2.14.1-canary.225+15a002dd2",
+    "@atlaspack/feature-flags": "2.14.1-canary.225+15a002dd2",
+    "@atlaspack/plugin": "2.14.5-canary.157+15a002dd2",
+    "@atlaspack/rust": "3.2.1-canary.157+15a002dd2",
+    "@atlaspack/types": "2.14.5-canary.157+15a002dd2",
+    "@atlaspack/utils": "2.14.5-canary.157+15a002dd2",
     "@parcel/source-map": "^2.1.1",
     "globals": "^13.2.0",
     "nullthrows": "^1.1.1",
@@ -32,5 +32,5 @@
     "check-ts": "tsc --emitDeclarationOnly --rootDir src",
     "build:lib": "gulp build --gulpfile ../../../gulpfile.js --cwd ."
   },
-  "gitHead": "7f5841c39df049f9546cccbeea2a7337e0337b45"
+  "gitHead": "15a002dd20ce80711376e69044fb87b944084ac7"
 }

package/src/ScopeHoistingPackager.ts

@@ -388,6 +388,7 @@ export class ScopeHoistingPackager {
     let queue = new PromiseQueue({maxConcurrent: 32});
     let wrapped: Array<Asset> = [];
     let constant: Array<Asset> = [];
+
     this.bundle.traverseAssets((asset) => {
       queue.add(async () => {
         let [code, map] = await Promise.all([
@@ -426,6 +427,14 @@ export class ScopeHoistingPackager {
     });
 
     if (getFeatureFlag('applyScopeHoistingImprovement')) {
+      // Make all entry assets wrapped, to avoid any top level hoisting
+      for (let entryAsset of this.bundle.getEntryAssets()) {
+        if (!this.wrappedAssets.has(entryAsset.id)) {
+          this.wrappedAssets.add(entryAsset.id);
+          wrapped.push(entryAsset);
+        }
+      }
+
       // Tracks which assets have been assigned to a wrap group
       let assignedAssets = new Set<Asset>();