Fix broken NewExpressions containing require calls

internal/bundler/bundler_default_test.go

@@ -79,6 +79,27 @@ func TestNestedCommonJS(t *testing.T) {
 	})
 }
 
+// This test makes sure that NewExpressions containing require() calls aren't
+// broken.
+func TestNewExpressionCommonJS(t *testing.T) {
+	default_suite.expectBundled(t, bundled{
+		files: map[string]string{
+			"/entry.js": `
+        new (require("./foo.js")).Foo();
+			`,
+			"/foo.js": `
+        class Foo {}
+				module.exports = {Foo};
+			`,
+		},
+		entryPaths: []string{"/entry.js"},
+		options: config.Options{
+			IsBundling:    true,
+			AbsOutputFile: "/out.js",
+		},
+	})
+}
+
 func TestCommonJSFromES6(t *testing.T) {
 	default_suite.expectBundled(t, bundled{
 		files: map[string]string{

internal/bundler/snapshots/snapshots_default.txt

@@ -1203,6 +1203,19 @@ TestNestedScopeBug
   a();
 })();
 
+================================================================================
+TestNewExpressionCommonJS
+---------- /out.js ----------
+// /foo.js
+var require_foo = __commonJS((exports, module) => {
+  class Foo {
+  }
+  module.exports = {Foo};
+});
+
+// /entry.js
+new (require_foo()).Foo();
+
 ================================================================================
 TestNodeModules
 ---------- /Users/user/project/out.js ----------

internal/printer/printer.go

@@ -1226,6 +1226,7 @@ const (
 	forbidCall = 1 << iota
 	forbidIn
 	hasNonOptionalChainParent
+	wrapRequireCall
 )
 
 func (p *printer) printUndefined(level ast.L) {
@@ -1290,7 +1291,7 @@ func (p *printer) printExpr(expr ast.Expr, level ast.L, flags int) {
 		p.printSpaceBeforeIdentifier()
 		p.print("new")
 		p.printSpace()
-		p.printExpr(e.Target, ast.LNew, forbidCall)
+		p.printExpr(e.Target, ast.LNew, forbidCall|wrapRequireCall)
 
 		// TODO: Omit this while minifying
 		p.print("(")
@@ -1363,7 +1364,7 @@ func (p *printer) printExpr(expr ast.Expr, level ast.L, flags int) {
 		}
 
 	case *ast.ERequire:
-		wrap := level >= ast.LNew
+		wrap := level >= ast.LNew || (flags&wrapRequireCall) != 0
 		if wrap {
 			p.print("(")
 		}