@atlaspack/packager-js 2.14.5-canary.186 → 2.14.5-canary.189

package/lib/ScopeHoistingPackager.js

@@ -1225,18 +1225,24 @@ ${code}
         // for the symbol so that when the value changes the object property also changes. This is
         // required to simulate ESM live bindings. It's easier to do it this way rather than inserting
         // additional assignments after each mutation of the original binding.
-        prepend += `\n${usedExports.map(exp => {
+        for (let exp of usedExports) {
           var _asset$symbols$get2;
           let resolved = this.getSymbolResolution(asset, asset,
           // @ts-expect-error TS2345
           exp, undefined, replacements);
-          let get = this.buildFunctionExpression([], resolved);
-          let isEsmExport = !!((_asset$symbols$get2 = asset.symbols.get(exp)) !== null && _asset$symbols$get2 !== void 0 && (_asset$symbols$get2 = _asset$symbols$get2.meta) !== null && _asset$symbols$get2 !== void 0 && _asset$symbols$get2.isEsm);
-          let set = !isEsmExport && asset.meta.hasCJSExports ? ', ' + this.buildFunctionExpression(['v'], `${resolved} = v`) : '';
-          return `$parcel$export($${assetId}$exports, ${JSON.stringify(exp)}, ${get}${set});`;
-        }).join('\n')}\n`;
-        this.usedHelpers.add('$parcel$export');
-        prependLineCount += 1 + usedExports.length;
+          const meta = (_asset$symbols$get2 = asset.symbols.get(exp)) === null || _asset$symbols$get2 === void 0 ? void 0 : _asset$symbols$get2.meta;
+          if ((0, _featureFlags().getFeatureFlag)('exportsRebindingOptimisation') && meta !== null && meta !== void 0 && meta.isStaticBindingSafe) {
+            append += `$${assetId}$exports[${JSON.stringify(exp)}] = ${resolved};\n`;
+          } else {
+            var _asset$symbols$get3;
+            let get = this.buildFunctionExpression([], resolved);
+            let isEsmExport = !!((_asset$symbols$get3 = asset.symbols.get(exp)) !== null && _asset$symbols$get3 !== void 0 && (_asset$symbols$get3 = _asset$symbols$get3.meta) !== null && _asset$symbols$get3 !== void 0 && _asset$symbols$get3.isEsm);
+            let set = !isEsmExport && asset.meta.hasCJSExports ? ', ' + this.buildFunctionExpression(['v'], `${resolved} = v`) : '';
+            prepend += `$parcel$export($${assetId}$exports, ${JSON.stringify(exp)}, ${get}${set});\n`;
+            this.usedHelpers.add('$parcel$export');
+            prependLineCount += 1 + usedExports.length;
+          }
+        }
       }
     }
     return [prepend, prependLineCount, append];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaspack/packager-js",
-  "version": "2.14.5-canary.186+f6532d7a4",
+  "version": "2.14.5-canary.189+0684997c1",
   "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
     "access": "public"
@@ -16,12 +16,12 @@
     "node": ">= 16.0.0"
   },
   "dependencies": {
-    "@atlaspack/diagnostic": "2.14.1-canary.254+f6532d7a4",
-    "@atlaspack/feature-flags": "2.14.1-canary.254+f6532d7a4",
-    "@atlaspack/plugin": "2.14.5-canary.186+f6532d7a4",
-    "@atlaspack/rust": "3.2.1-canary.186+f6532d7a4",
-    "@atlaspack/types": "2.14.5-canary.186+f6532d7a4",
-    "@atlaspack/utils": "2.14.5-canary.186+f6532d7a4",
+    "@atlaspack/diagnostic": "2.14.1-canary.257+0684997c1",
+    "@atlaspack/feature-flags": "2.14.1-canary.257+0684997c1",
+    "@atlaspack/plugin": "2.14.5-canary.189+0684997c1",
+    "@atlaspack/rust": "3.2.1-canary.189+0684997c1",
+    "@atlaspack/types": "2.14.5-canary.189+0684997c1",
+    "@atlaspack/utils": "2.14.5-canary.189+0684997c1",
     "@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": "f6532d7a4f7f007bd4e5e36af04dd466f0b9f572"
+  "gitHead": "0684997c127a6204af1376b36e556fddd1ba5557"
 }

package/src/ScopeHoistingPackager.ts

@@ -1648,29 +1648,37 @@ ${code}
         // for the symbol so that when the value changes the object property also changes. This is
         // required to simulate ESM live bindings. It's easier to do it this way rather than inserting
         // additional assignments after each mutation of the original binding.
-        prepend += `\n${usedExports
-          .map((exp) => {
-            let resolved = this.getSymbolResolution(
-              asset,
-              asset,
-              // @ts-expect-error TS2345
+        for (let exp of usedExports) {
+          let resolved = this.getSymbolResolution(
+            asset,
+            asset,
+            // @ts-expect-error TS2345
+            exp,
+            undefined,
+            replacements,
+          );
+          const meta = asset.symbols.get(exp)?.meta;
+          if (
+            getFeatureFlag('exportsRebindingOptimisation') &&
+            meta?.isStaticBindingSafe
+          ) {
+            append += `$${assetId}$exports[${JSON.stringify(
               exp,
-              undefined,
-              replacements,
-            );
+            )}] = ${resolved};\n`;
+          } else {
             let get = this.buildFunctionExpression([], resolved);
             let isEsmExport = !!asset.symbols.get(exp)?.meta?.isEsm;
             let set =
               !isEsmExport && asset.meta.hasCJSExports
                 ? ', ' + this.buildFunctionExpression(['v'], `${resolved} = v`)
                 : '';
-            return `$parcel$export($${assetId}$exports, ${JSON.stringify(
+            prepend += `$parcel$export($${assetId}$exports, ${JSON.stringify(
               exp,
-            )}, ${get}${set});`;
-          })
-          .join('\n')}\n`;
-        this.usedHelpers.add('$parcel$export');
-        prependLineCount += 1 + usedExports.length;
+            )}, ${get}${set});\n`;
+            this.usedHelpers.add('$parcel$export');
+            prependLineCount += 1 + usedExports.length;
+          }
+        }
       }
     }