@parcel/packager-js 2.0.0-dev.1528 → 2.0.0-dev.1548

package/lib/ScopeHoistingPackager.js

@@ -271,6 +271,7 @@ class ScopeHoistingPackager {
     // TODO: handle ESM exports of wrapped entry assets...
     let entry = this.bundle.getMainEntry();
     if (entry && !this.wrappedAssets.has(entry.id)) {
+      let hasNamespace = entry.symbols.hasExportSymbol('*');
       for (let {
         asset,
         exportAs,
@@ -279,6 +280,12 @@ class ScopeHoistingPackager {
       } of this.bundleGraph.getExportedSymbols(entry)) {
         if (typeof symbol === 'string') {
           var _this$exportedSymbols, _entry$symbols$get2;
+          // If the module has a namespace (e.g. commonjs), and this is not an entry, only export the namespace
+          // as default, without individual exports. This mirrors the importing logic in addExternal, avoiding
+          // extra unused exports and potential for non-identifier export names.
+          if (hasNamespace && this.isAsyncBundle && exportAs !== '*') {
+            continue;
+          }
           let symbols = (_this$exportedSymbols = this.exportedSymbols.get(symbol === '*' ? (0, _nullthrows().default)((_entry$symbols$get2 = entry.symbols.get('*')) === null || _entry$symbols$get2 === void 0 ? void 0 : _entry$symbols$get2.local) : symbol)) === null || _this$exportedSymbols === void 0 ? void 0 : _this$exportedSymbols.exportAs;
           if (!symbols) {
             symbols = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@parcel/packager-js",
-  "version": "2.0.0-dev.1528+70889ca07",
+  "version": "2.0.0-dev.1548+79b158883",
   "license": "MIT",
   "publishConfig": {
     "access": "public"
@@ -17,17 +17,17 @@
   "source": "src/index.js",
   "engines": {
     "node": ">= 12.0.0",
-    "parcel": "^2.0.0-dev.1526+70889ca07"
+    "parcel": "^2.0.0-dev.1546+79b158883"
   },
   "dependencies": {
-    "@parcel/diagnostic": "2.0.0-dev.1528+70889ca07",
-    "@parcel/plugin": "2.0.0-dev.1528+70889ca07",
-    "@parcel/rust": "2.12.1-dev.3151+70889ca07",
+    "@parcel/diagnostic": "2.0.0-dev.1548+79b158883",
+    "@parcel/plugin": "2.0.0-dev.1548+79b158883",
+    "@parcel/rust": "2.12.1-dev.3171+79b158883",
     "@parcel/source-map": "^2.1.1",
-    "@parcel/types": "2.0.0-dev.1528+70889ca07",
-    "@parcel/utils": "2.0.0-dev.1528+70889ca07",
+    "@parcel/types": "2.0.0-dev.1548+79b158883",
+    "@parcel/utils": "2.0.0-dev.1548+79b158883",
     "globals": "^13.2.0",
     "nullthrows": "^1.1.1"
   },
-  "gitHead": "70889ca07dccb95e21e4c6a2844d632e3723faf4"
+  "gitHead": "79b158883170daac9cd17bdecfebff163bc99e52"
 }

package/src/ScopeHoistingPackager.js

@@ -354,6 +354,8 @@ export class ScopeHoistingPackager {
     // TODO: handle ESM exports of wrapped entry assets...
     let entry = this.bundle.getMainEntry();
     if (entry && !this.wrappedAssets.has(entry.id)) {
+      let hasNamespace = entry.symbols.hasExportSymbol('*');
+
       for (let {
         asset,
         exportAs,
@@ -361,6 +363,13 @@ export class ScopeHoistingPackager {
         exportSymbol,
       } of this.bundleGraph.getExportedSymbols(entry)) {
         if (typeof symbol === 'string') {
+          // If the module has a namespace (e.g. commonjs), and this is not an entry, only export the namespace
+          // as default, without individual exports. This mirrors the importing logic in addExternal, avoiding
+          // extra unused exports and potential for non-identifier export names.
+          if (hasNamespace && this.isAsyncBundle && exportAs !== '*') {
+            continue;
+          }
+
           let symbols = this.exportedSymbols.get(
             symbol === '*' ? nullthrows(entry.symbols.get('*')?.local) : symbol,
           )?.exportAs;