fix: improve adding parens when necessary for object literal exprs

Also, better handling of these nodes in binary exprs even though they would never appear in these contexts

Author: dsherret

src/parsing/parser.rs

@@ -2214,7 +2214,13 @@ fn should_add_parens_around_expr(node: Node, context: &Context) -> bool {
       Node::CondExpr(cond_expr) => {
         return cond_expr.test.span().contains(original_node.span());
       }
-      Node::OptChainExpr(_) | Node::BinExpr(_) => {
+      Node::BinExpr(bin_expr) => {
+        // we only care to add parens when it's the left most expr
+        if bin_expr.right.span().contains(original_node.span()) {
+          return false;
+        }
+      }
+      Node::OptChainExpr(_) => {
         // continue searching
       }
       _ => {
@@ -2307,7 +2313,7 @@ fn parse_non_null_expr<'a>(node: &'a TsNonNullExpr, context: &mut Context<'a>) -
 }
 
 fn parse_object_lit<'a>(node: &'a ObjectLit, context: &mut Context<'a>) -> PrintItems {
-  parse_object_like_node(
+  let items = parse_object_like_node(
     ParseObjectLikeNodeOptions {
       node: node.into(),
       members: node.props.iter().map(|x| x.into()).collect(),
@@ -2319,7 +2325,13 @@ fn parse_object_lit<'a>(node: &'a ObjectLit, context: &mut Context<'a>) -> Print
       node_sorter: None,
     },
     context,
-  )
+  );
+
+  return if should_add_parens_around_expr(node.into(), context) {
+    surround_with_parens(items)
+  } else {
+    items
+  };
 }
 
 fn parse_paren_expr<'a>(node: &'a ParenExpr, context: &mut Context<'a>) -> PrintItems {

tests/specs/expressions/ParenExpr/ParenExpr_All.txt

@@ -111,6 +111,7 @@ test;
 (function test() {}?.prop);
 (function test() {}[56]);
 (function test() {} + 2);
+(2 + function test() {});
 1 + (function test() {} + 2);
 call(function test() {}());
 obj[(function test() {})];
@@ -130,6 +131,7 @@ test = (function() {});
 (function test() {})?.prop;
 (function test() {})[56];
 (function test() {}) + 2;
+2 + function test() {};
 1 + (function test() {} + 2);
 call(function test() {}());
 obj[function test() {}];
@@ -177,3 +179,11 @@ new test(() => {});
 new (() => {})();
 test = () => {};
 test = () => {};
+
+== should handle object literal expr in paren expr in expr stmt ==
+({ prop: 5 }.prop);
+({ prop: 5 }).prop;
+
+[expect]
+({ prop: 5 }).prop;
+({ prop: 5 }).prop;