Merge pull request #10146 from liximomo/fix-sideEffects

fix: bundle wrong modules

lib/optimize/SideEffectsFlagPlugin.js

@@ -115,7 +115,7 @@ class SideEffectsFlagPlugin {
 								(dep instanceof HarmonyImportSpecifierDependency &&
 									!dep.namespaceObjectAsContext)
 							) {
-								const mapping = map.get(dep.id);
+								const mapping = map.get(dep._id);
 								if (mapping) {
 									dep.redirectedModule = mapping.module;
 									dep.redirectedId = mapping.exportName;

test/cases/side-effects/wrong-module-10146/index.js

@@ -0,0 +1,5 @@
+import { A } from "./module";
+
+it("should return the correct module", () => {
+	expect(A()).toEqual("A/index.js");
+});

test/cases/side-effects/wrong-module-10146/module/A/A.js

@@ -0,0 +1,3 @@
+export default function hello() {
+  return 'A/A.js'
+}

test/cases/side-effects/wrong-module-10146/module/A/index.js

@@ -0,0 +1,6 @@
+
+export { default as A } from "./A";
+
+export default function hello() {
+  return 'A/index.js'
+}

test/cases/side-effects/wrong-module-10146/module/index.js

@@ -0,0 +1 @@
+export { default as A } from "./A";

test/cases/side-effects/wrong-module-10146/module/package.json

@@ -0,0 +1,3 @@
+{
+  "sideEffects": false
+}