fix(formatter): should not wrap parenthesis for ComputedMemberExpression when it is not an option or it doesn't contain a call expression

Author: Dunqing

crates/oxc_formatter/src/parentheses/expression.rs

@@ -262,7 +262,7 @@ impl NeedsParentheses<'_> for AstNode<'_, MemberExpression<'_>> {
 impl NeedsParentheses<'_> for AstNode<'_, ComputedMemberExpression<'_>> {
     fn needs_parentheses(&self, f: &Formatter<'_, '_>) -> bool {
         matches!(self.parent, AstNodes::NewExpression(_))
-            && (!self.optional || member_chain_callee_needs_parens(&self.expression))
+            && (self.optional || member_chain_callee_needs_parens(&self.object))
     }
 }
 

crates/oxc_formatter/tests/fixtures/js/computed-members/parenthesis.js

@@ -1,2 +1,7 @@
 new (get(win))[ty](xxx);
 new (get(win))[ty][ty](xxx);
+
+new (A?.[ty])(xxx);
+new (A?.B[ty])(xxx);
+
+new window['TouchEvent'](xxx);

crates/oxc_formatter/tests/fixtures/js/computed-members/parenthesis.js.snap

@@ -5,8 +5,18 @@ source: crates/oxc_formatter/tests/fixtures/mod.rs
 new (get(win))[ty](xxx);
 new (get(win))[ty][ty](xxx);
 
+new (A?.[ty])(xxx);
+new (A?.B[ty])(xxx);
+
+new window['TouchEvent'](xxx);
+
 ==================== Output ====================
 new (get(win)[ty])(xxx);
 new (get(win)[ty][ty])(xxx);
 
+new (A?.[ty])(xxx);
+new (A?.B[ty])(xxx);
+
+new window["TouchEvent"](xxx);
+
 ===================== End =====================