Merge pull request #14 from paoloricciuti/fix-object-expressions-stat…

…ements fix: add parenthesis to object expressions statements
Rich-Harris · Nov 30, 2024 · af59c14 · af59c14
2 parents 03c6151 + 8954ec2
commit af59c14
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 4 deletions.
diff --git a/src/handlers.js b/src/handlers.js
@@ -785,8 +785,9 @@ const handlers = {
 
 	ExpressionStatement(node, state) {
 		if (
-			node.expression.type === 'AssignmentExpression' &&
-			node.expression.left.type === 'ObjectPattern'
+			node.expression.type === 'ObjectExpression' ||
+			(node.expression.type === 'AssignmentExpression' &&
+				node.expression.left.type === 'ObjectPattern')
 		) {
 			// is an AssignmentExpression to an ObjectPattern
 			state.commands.push('(');

diff --git a/test/samples/object-expressions/expected.js b/test/samples/object-expressions/expected.js
@@ -10,6 +10,7 @@ let blah;
 obj = { blah };
 obj = { blah };
 obj = { a: b };
+({ a: 1, b: 1 });
 
 obj = {
 	method() {

diff --git a/test/samples/object-expressions/expected.js.map b/test/samples/object-expressions/expected.js.map
diff --git a/test/samples/object-expressions/input.js b/test/samples/object-expressions/input.js
@@ -8,6 +8,7 @@ let blah;
 obj = { blah };
 obj = { blah };
 obj = { a: b };
+({ a: 1, b: 1 });
 
 obj = {
 	method() {