Print necessary parentheses for functions in postfix expressions (#14532

) fix: isFirstInContext doesn't handle postfix unary operators Co-authored-by: xiawenqi <xwq167663@alibaba-inc.com>
babel · May 6, 2022 · 0984a2a · 0984a2a
1 parent fbd0652
commit 0984a2a
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/packages/babel-generator/src/node/parentheses.ts b/packages/babel-generator/src/node/parentheses.ts
@@ -5,6 +5,7 @@ import {
   isAwaitExpression,
   isBinary,
   isBinaryExpression,
+  isUpdateExpression,
   isCallExpression,
   isClassDeclaration,
   isClassExpression,
@@ -414,6 +415,7 @@ function isFirstInContext(
     if (
       (hasPostfixPart(node, parent) && !isNewExpression(parent)) ||
       (isSequenceExpression(parent) && parent.expressions[0] === node) ||
+      (isUpdateExpression(parent) && !parent.prefix) ||
       isConditional(parent, { test: node }) ||
       isBinary(parent, { left: node }) ||
       isAssignmentExpression(parent, { left: node })

diff --git a/packages/babel-generator/test/fixtures/parentheses/unary-postfix/input.js b/packages/babel-generator/test/fixtures/parentheses/unary-postfix/input.js
@@ -0,0 +1 @@
+(function () { return {x: 24} }()).x++;
diff --git a/packages/babel-generator/test/fixtures/parentheses/unary-postfix/output.js b/packages/babel-generator/test/fixtures/parentheses/unary-postfix/output.js
@@ -0,0 +1,5 @@
+(function () {
+  return {
+    x: 24
+  };
+})().x++;