fix: correct "silence" typo to "silent" in needsParens (#2464)

The PHP parser emits node kind "silent" (not "silence"), so the
case "silence" branch was dead code. Silent expressions with parens
(e.g. (@$obj)->method()) would always have their parens dropped.

Author: jorgsowa

src/needs-parens.mjs

@@ -215,7 +215,7 @@ function needsParens(path, options) {
         default:
           return false;
       }
-    case "silence":
+    case "silent":
     case "cast":
       // TODO: bug https://github.com/glayzzle/php-parser/issues/172
       return node.parenthesizedExpression;

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

@@ -1461,7 +1461,7 @@ $var = $page > 0 || ($page == 0 && $this->forcePageParam);
 $var = $page > 0 || ($page == 0 && $this->forcePageParam);
 
 @foo() || @foo();
-@foo() || @foo();
+(@foo()) || (@foo());
 $var = @foo() || @foo();
 $var = @foo() || @foo();
 
@@ -2228,7 +2228,7 @@ $var = $page > 0 || ($page == 0 && $this->forcePageParam);
 $var = $page > 0 || ($page == 0 && $this->forcePageParam);
 
 @foo() || @foo();
-@foo() || @foo();
+(@foo()) || (@foo());
 $var = @foo() || @foo();
 $var = @foo() || @foo();
 
@@ -6192,7 +6192,7 @@ echo @(1 / 0);
 @foo();
 
 $var = @foo();
-$var = @foo();
+$var = (@foo());
 
 $var = @call() || @other();
 $var = @call() || @other_class();
@@ -6204,7 +6204,7 @@ $var = @$cache[$key];
 $var = @new MyClass();
 $var = @new MyClass();
 
-($var = @include "file.php") or die("Could not find file.php!");
+($var = (@include "file.php")) or die("Could not find file.php!");
 
 $var = @$_GET["data"];
 
@@ -6305,7 +6305,7 @@ echo @(1 / 0);
 @foo();
 
 $var = @foo();
-$var = @foo();
+$var = (@foo());
 
 $var = @call() || @other();
 $var = @call() || @other_class();
@@ -6317,7 +6317,7 @@ $var = @$cache[$key];
 $var = @new MyClass();
 $var = @new MyClass();
 
-($var = @include "file.php") or die("Could not find file.php!");
+($var = (@include "file.php")) or die("Could not find file.php!");
 
 $var = @$_GET["data"];
 

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

@@ -19,6 +19,8 @@ $variable = @foo()->veryVeryVeryVeryVeryLongMethod()->veryVeryVeryVeryVeryLongMe
 $variable = @foo('VeryVeryVeryVeryVeryVeryLongArgument', 'VeryVeryVeryVeryVeryVeryLongArgument', 'VeryVeryVeryVeryVeryVeryLongArgument');
 @trigger_error($error, E_USER_DEPRECATED);
 $value = @     $cache[$key];
+(@$obj)->method();
+(@$arr)[0];
 
 =====================================output=====================================
 <?php
@@ -45,6 +47,8 @@ $variable = @foo(
 );
 @trigger_error($error, E_USER_DEPRECATED);
 $value = @$cache[$key];
+(@$obj)->method();
+(@$arr)[0];
 
 ================================================================================
 `;

tests/silent/silent.php

@@ -11,3 +11,5 @@
 $variable = @foo('VeryVeryVeryVeryVeryVeryLongArgument', 'VeryVeryVeryVeryVeryVeryLongArgument', 'VeryVeryVeryVeryVeryVeryLongArgument');
 @trigger_error($error, E_USER_DEPRECATED);
 $value = @     $cache[$key];
+(@$obj)->method();
+(@$arr)[0];