wrap arrow function body in parentheses when it is an object destruct…

…uring assignment (#70)
Rich-Harris · Jan 6, 2022 · 93f852c · 93f852c
1 parent 590c981
commit 93f852c
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 1 deletion.
diff --git a/src/print/handlers.js b/src/print/handlers.js
@@ -882,7 +882,10 @@ const handlers = {
 
 		chunks.push(c(' => '));
 
-		if (node.body.type === 'ObjectExpression') {
+		if (
+			node.body.type === 'ObjectExpression' ||
+			(node.body.type === 'AssignmentExpression' && node.body.left.type === 'ObjectPattern')
+		) {
 			chunks.push(
 				c('('),
 				...handle(node.body, state),

diff --git a/test/samples/arrow-function-assignment-object-pattern/expected.js b/test/samples/arrow-function-assignment-object-pattern/expected.js
@@ -0,0 +1 @@
+let a = () => ({ x } = { x: 42 });
diff --git a/test/samples/arrow-function-assignment-object-pattern/expected.js.map b/test/samples/arrow-function-assignment-object-pattern/expected.js.map
diff --git a/test/samples/arrow-function-assignment-object-pattern/input.js b/test/samples/arrow-function-assignment-object-pattern/input.js
@@ -0,0 +1 @@
+export default ({ b }) => b`let a = () => ({ x } = { x: 42 });`;
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		export default ({ b }) => b`let a = () => ({ x } = { x: 42 });`;