Merge pull request #14663 from webpack/hotfix/14662

sokra · web-flow · commit 707a6d360f39 · 2021-11-05T17:25:33.000+01:00
fix #14662
diff --git a/lib/dependencies/CommonJsFullRequireDependency.js b/lib/dependencies/CommonJsFullRequireDependency.js
@@ -121,7 +121,7 @@ CommonJsFullRequireDependency.Template = class CommonJsFullRequireDependencyTemp
 				requireExpr =
 					dep.asiSafe === true
 						? `(${requireExpr}${access})`
-						: `${requireExpr}${access})`;
+						: `${requireExpr}${access}`;
 			}
 		}
 		source.replace(dep.range[0], dep.range[1] - 1, requireExpr);
diff --git a/test/cases/parsing/asi/index.js b/test/cases/parsing/asi/index.js
@@ -41,5 +41,8 @@ it("should respect asi flag", () => {
 		return callme()
 	})()
 
+	;(donotcallme)
+	require("./b.json").prop
+
 	expect(getCount()).toBe(29)
 });

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ CommonJsFullRequireDependency.Template = class CommonJsFullRequireDependencyTemp`
`121`	`121`	`requireExpr =`
`122`	`122`	`dep.asiSafe === true`
`123`	`123`	? `(${requireExpr}${access})`
`124`		- : `${requireExpr}${access})`;
	`124`	+ : `${requireExpr}${access}`;
`125`	`125`	`}`
`126`	`126`	`}`
`127`	`127`	`source.replace(dep.range[0], dep.range[1] - 1, requireExpr);`