Merge pull request webpack#7561 from webpack/bugfix/stack-overflow

improve error message for circular reexports

Author: sokra

lib/optimize/ConcatenatedModule.js

@@ -108,7 +108,8 @@ const getFinalName = (
 	moduleToInfoMap,
 	requestShortener,
 	asCall,
-	strictHarmonyModule
+	strictHarmonyModule,
+	alreadyVisited = new Set()
 ) => {
 	switch (info.type) {
 		case "concatenated": {
@@ -134,6 +135,22 @@ const getFinalName = (
 			}
 			const reexport = info.reexportMap.get(exportName);
 			if (reexport) {
+				if (alreadyVisited.has(reexport)) {
+					throw new Error(
+						`Circular reexports ${Array.from(
+							alreadyVisited,
+							e =>
+								`"${e.module.readableIdentifier(requestShortener)}".${
+									e.exportName
+								}`
+						).join(
+							" --> "
+						)} -(circular)-> "${reexport.module.readableIdentifier(
+							requestShortener
+						)}".${reexport.exportName}`
+					);
+				}
+				alreadyVisited.add(reexport);
 				const refInfo = moduleToInfoMap.get(reexport.module);
 				if (refInfo) {
 					// module is in the concatenation
@@ -143,7 +160,8 @@ const getFinalName = (
 						moduleToInfoMap,
 						requestShortener,
 						asCall,
-						strictHarmonyModule
+						strictHarmonyModule,
+						alreadyVisited
 					);
 				}
 			}

test/configCases/errors/self-reexport/a.js

@@ -0,0 +1 @@
+export { something } from "./a";

test/configCases/errors/self-reexport/aa.js

@@ -0,0 +1,3 @@
+import { something } from "./a";
+
+something();

test/configCases/errors/self-reexport/b.js

@@ -0,0 +1,7 @@
+
+import { something, other } from "./b";
+
+export {
+	something as other,
+	other as something
+}

test/configCases/errors/self-reexport/bb.js

@@ -0,0 +1,3 @@
+import {something} from "./b";
+
+something();

test/configCases/errors/self-reexport/c1.js

@@ -0,0 +1 @@
+export { something } from "./c2";

test/configCases/errors/self-reexport/c2.js

@@ -0,0 +1 @@
+export { something } from "./c1";

test/configCases/errors/self-reexport/cc.js

@@ -0,0 +1,3 @@
+import {something} from "./c1";
+
+something();

test/configCases/errors/self-reexport/errors.js

@@ -0,0 +1,5 @@
+module.exports = [
+	[/Circular reexports "\.\/a.js"\.something -\(circular\)-> "\.\/a.js"\.something/],
+	[/Circular reexports "\.\/b.js"\.other --> "\.\/b.js"\.something -\(circular\)-> "\.\/b.js"\.other/],
+	[/Circular reexports "\.\/c2.js"\.something --> "\.\/c1.js"\.something -\(circular\)-> "\.\/c2.js"\.something/]
+];

test/configCases/errors/self-reexport/index.js

@@ -0,0 +1,7 @@
+it("should not crash on incorrect exports", function() {
+	if(Math.random() < -1) {
+		import(/* webpackChunkName: "a" */ "./aa");
+		import(/* webpackChunkName: "b" */ "./bb");
+		import(/* webpackChunkName: "c" */ "./cc");
+	}
+});