@atlaspack/transformer-js 19.0.3 → 19.0.4

package/lib/JSTransformer.js

@@ -1018,7 +1018,20 @@ var _default = exports.default = new (_plugin().Transformer)({
         let dep = deps.get(source);
         if (!dep) continue;
         if (local === '*' && imported === '*') {
-          dep.symbols.set('*', '*', convertLoc(loc), true);
+          // When a module has both `import * as ns from 'x'` and
+          // `export * from 'x'`, they share a single dep (merged by the
+          // core). The namespace import binding ('*' → local, isWeak:false)
+          // set by imported_symbols is strictly more inclusive than the
+          // wildcard re-export ('*' → '*', isWeak:true) for symbol
+          // propagation, so don't overwrite it. Instead, flag the dep so
+          // that symbol propagation still treats it as a wildcard
+          // re-exporter.
+          let existingStar = dep.symbols.get('*');
+          if ((0, _featureFlags().getFeatureFlag)('fixExportStarNamespaceOverwrite') && existingStar && existingStar.local !== '*') {
+            dep.meta.hasExportStar = true;
+          } else {
+            dep.symbols.set('*', '*', convertLoc(loc), true);
+          }
         } else {
           var _dep$symbols$get;
           let reExportName = ((_dep$symbols$get = dep.symbols.get(imported)) === null || _dep$symbols$get === void 0 ? void 0 : _dep$symbols$get.local) ?? `$${asset.id}$re_export$${local}`;
@@ -1111,7 +1124,16 @@ var _default = exports.default = new (_plugin().Transformer)({
           let dep = deps.get(source);
           if (!dep) continue;
           dep.symbols.ensure();
-          dep.symbols.set('*', '*', convertLoc(loc), true);
+          // When a module has both `import * as ns from 'x'` and
+          // `export * from 'x'`, they share a single dep. Don't overwrite
+          // the namespace import binding — flag the dep instead so symbol
+          // propagation still treats it as a wildcard re-exporter.
+          let existingStar = dep.symbols.get('*');
+          if ((0, _featureFlags().getFeatureFlag)('fixExportStarNamespaceOverwrite') && existingStar && existingStar.local !== '*') {
+            dep.meta.hasExportStar = true;
+          } else {
+            dep.symbols.set('*', '*', convertLoc(loc), true);
+          }
         }
 
         // Add * symbol if there are CJS exports, no imports/exports at all, or the asset is wrapped.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaspack/transformer-js",
-  "version": "19.0.3",
+  "version": "19.0.4",
   "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
     "access": "public"
@@ -24,14 +24,14 @@
     "src"
   ],
   "dependencies": {
-    "@atlaspack/build-cache": "2.13.15",
+    "@atlaspack/build-cache": "2.13.16",
     "@atlaspack/diagnostic": "2.14.4",
-    "@atlaspack/feature-flags": "2.31.1",
-    "@atlaspack/plugin": "2.14.61",
-    "@atlaspack/rust": "3.29.0",
-    "@atlaspack/source-map": "3.3.5",
-    "@atlaspack/utils": "3.4.3",
-    "@atlaspack/workers": "2.14.61",
+    "@atlaspack/feature-flags": "2.31.2",
+    "@atlaspack/plugin": "2.14.62",
+    "@atlaspack/rust": "3.29.1",
+    "@atlaspack/source-map": "3.3.6",
+    "@atlaspack/utils": "3.4.4",
+    "@atlaspack/workers": "2.14.62",
     "@swc/helpers": "^0.5.15",
     "browserslist": "^4.6.6",
     "nullthrows": "^1.1.1",
@@ -39,7 +39,7 @@
     "semver": "^7.5.2"
   },
   "peerDependencies": {
-    "@atlaspack/core": "2.38.3"
+    "@atlaspack/core": "2.38.4"
   },
   "type": "commonjs"
 }

package/src/JSTransformer.ts

@@ -1300,7 +1300,24 @@ export default new Transformer({
         let dep = deps.get(source);
         if (!dep) continue;
         if (local === '*' && imported === '*') {
-          dep.symbols.set('*', '*', convertLoc(loc), true);
+          // When a module has both `import * as ns from 'x'` and
+          // `export * from 'x'`, they share a single dep (merged by the
+          // core). The namespace import binding ('*' → local, isWeak:false)
+          // set by imported_symbols is strictly more inclusive than the
+          // wildcard re-export ('*' → '*', isWeak:true) for symbol
+          // propagation, so don't overwrite it. Instead, flag the dep so
+          // that symbol propagation still treats it as a wildcard
+          // re-exporter.
+          let existingStar = dep.symbols.get('*');
+          if (
+            getFeatureFlag('fixExportStarNamespaceOverwrite') &&
+            existingStar &&
+            existingStar.local !== '*'
+          ) {
+            dep.meta.hasExportStar = true;
+          } else {
+            dep.symbols.set('*', '*', convertLoc(loc), true);
+          }
         } else {
           let reExportName =
             dep.symbols.get(imported)?.local ??
@@ -1412,7 +1429,20 @@ export default new Transformer({
           let dep = deps.get(source);
           if (!dep) continue;
           dep.symbols.ensure();
-          dep.symbols.set('*', '*', convertLoc(loc), true);
+          // When a module has both `import * as ns from 'x'` and
+          // `export * from 'x'`, they share a single dep. Don't overwrite
+          // the namespace import binding — flag the dep instead so symbol
+          // propagation still treats it as a wildcard re-exporter.
+          let existingStar = dep.symbols.get('*');
+          if (
+            getFeatureFlag('fixExportStarNamespaceOverwrite') &&
+            existingStar &&
+            existingStar.local !== '*'
+          ) {
+            dep.meta.hasExportStar = true;
+          } else {
+            dep.symbols.set('*', '*', convertLoc(loc), true);
+          }
         }
 
         // Add * symbol if there are CJS exports, no imports/exports at all, or the asset is wrapped.