fix: do not remove parens in var decl init when paren expr is assignment (#465)

Author: dsherret

Cargo.lock

@@ -26,7 +26,7 @@ tracing = ["dprint-core/tracing"]
 
 [dependencies]
 anyhow = "1.0.64"
-deno_ast = { version = "0.23.1", features = ["view"] }
+deno_ast = { version = "0.23.2", features = ["view"] }
 dprint-core = { version = "0.60.0", features = ["formatting"] }
 rustc-hash = "1.1.0"
 serde = { version = "1.0.144", features = ["derive"] }

Cargo.toml

@@ -2735,9 +2735,11 @@ fn should_skip_paren_expr(node: &ParenExpr, context: &Context) -> bool {
   }
 
   if let Node::VarDeclarator(var_decl) = parent {
-    if let Some(init) = var_decl.init {
-      if init.range().contains(&node.range()) {
-        return true;
+    if node.expr.kind() != NodeKind::AssignExpr {
+      if let Some(init) = var_decl.init {
+        if init.range().contains(&node.range()) {
+          return true;
+        }
       }
     }
   }

src/generation/generate.rs

@@ -115,10 +115,10 @@ mod tests {
       "./test.ts",
       "+value.",
       concat!(
+        // comment to keep this multi-line
         "Unexpected eof at ./test.ts:1:8\n\n",
         "  +value.\n",
-        // this excess whitespace is a bug, but not a big deal
-        "         "
+        "         ~"
       ),
     );
   }

src/swc.rs

@@ -33,6 +33,8 @@ const c = (1 + 1) + 1;
 const d = 1 + (1 + 1);
 // except here it should because of the type assertion
 const e = /** @type {number} */ (((1)));
+// keep this too as some people like it for clarity
+const a = (b = 5);
 
 [expect]
 const a = 1;
@@ -41,6 +43,8 @@ const c = (1 + 1) + 1;
 const d = 1 + (1 + 1);
 // except here it should because of the type assertion
 const e = /** @type {number} */ (1);
+// keep this too as some people like it for clarity
+const a = (b = 5);
 
 == should ignore paren exprs within paren exprs ==
 (((test)));