fix: avoid adding parens to static property in new expression

Author: fs0414

src/printer.mjs

@@ -2102,11 +2102,28 @@ function printNode(path, options, print) {
         );
       } else {
         const isExpression = ["call", "offsetlookup"].includes(node.what.kind);
+
+        const isInMemberChain =
+          path.parent &&
+          (isLookupNode(path.parent) || path.parent.kind === "call");
+        const whatEndOffset = locEnd(node.what);
+        const nextCharIndex = getNextNonSpaceNonCommentCharacterIndex(
+          options.originalText,
+          whatEndOffset
+        );
+        const hasOriginalParens =
+          options.originalText.charAt(nextCharIndex) === "(";
+        const shouldSkipParens =
+          isInMemberChain &&
+          node.arguments.length === 0 &&
+          !hasOriginalParens &&
+          node.what.kind === "staticlookup";
+
         const printed = [
           isExpression ? "(" : "",
           print("what"),
           isExpression ? ")" : "",
-          printArgumentsList(path, options, print),
+          shouldSkipParens ? "" : printArgumentsList(path, options, print),
         ];
 
         parts.push(hasLeadingComment(node.what) ? indent(printed) : printed);

tests/parens/__snapshots__/jsfmt.spec.mjs.snap

@@ -5021,6 +5021,10 @@ $var = ((new $a)->b());
 
 $var = (new class {})->foo;
 
+// Issue #2441: static property access should not add parens
+$identifier = new Yii::$app->class([]);
+$identifier = new Yii::$app()->class([]);
+
 =====================================output=====================================
 <?php
 new Translator(
@@ -5075,10 +5079,14 @@ $var = new $a->b();
 $var = (new $a())->b();
 $var = (new $a())->b();
 (new class {})->foo;
+
 (new class {})->foo();
 (new class {})();
 (new class {})["foo"];
 $var = (new class {})->foo;
+// Issue #2441: static property access should not add parens
+$identifier = (new Yii::$app)->class([]);
+$identifier = (new Yii::$app)->class([]);
 
 
 ================================================================================
@@ -5163,6 +5171,10 @@ $var = ((new $a)->b());
 
 $var = (new class {})->foo;
 
+// Issue #2441: static property access should not add parens
+$identifier = new Yii::$app->class([]);
+$identifier = new Yii::$app()->class([]);
+
 =====================================output=====================================
 <?php
 new Translator(
@@ -5217,10 +5229,14 @@ $var = new $a->b();
 $var = new $a()->b();
 $var = new $a()->b();
 new class {}->foo;
+
 new class {}->foo();
 new class {}();
 new class {}["foo"];
 $var = new class {}->foo;
+// Issue #2441: static property access should not add parens
+$identifier = new Yii::$app->class([]);
+$identifier = new Yii::$app->class([]);
 
 
 ================================================================================

tests/parens/new.php

@@ -70,3 +70,7 @@ public function log($msg)
 (new class {})['foo'];
 
 $var = (new class {})->foo;
+
+// Issue #2441: static property access should not add parens
+$identifier = new Yii::$app->class([]);
+$identifier = new Yii::$app()->class([]);