fix: don't remove parens around sequence expressions

Author: dsherret

src/parsing/parser.rs

@@ -2398,6 +2398,11 @@ fn should_skip_paren_expr(node: &ParenExpr, context: &Context) -> bool {
     }
   }
 
+  if matches!(node.expr.kind(), NodeKind::SeqExpr) {
+    // don't care about extra logic for sequence expressions
+    return false;
+  }
+
   if matches!(node.expr.kind(), NodeKind::ArrayLit) {
     return true;
   }

tests/specs/expressions/SequenceExpression/SequenceExpression_All.txt

@@ -7,11 +7,11 @@ const x = (2, 3, 4);
 
 == should format inside a member expression ==
 member[1,2];
-member[(1,2)];
+member[(1,2)]; // could remove, but who writes this?
 
 [expect]
 member[1, 2];
-member[1, 2];
+member[(1, 2)]; // could remove, but who writes this?
 
 == should format multi-line with a hanging indent ==
 testttttttttting, thhhhisssouttttaaaabbitt, testing
@@ -20,3 +20,9 @@ testttttttttting, thhhhisssouttttaaaabbitt, testing
 testttttttttting,
     thhhhisssouttttaaaabbitt,
     testing;
+
+== should not remove parens around the sequence expression in this bin expr ==
+c ??= (a++, b());
+
+[expect]
+c ??= (a++, b());