fixup! fix: avoid creating invalid syntax async get method

Use the same check for isGetterOrSetter in shouldIgnoreChain as in unwindPromiseChain for consistency so that getters that return a short Promise chain are not converted to a verbose IIFE

Author: yonran

src/util/shouldIgnoreChain.ts

@@ -5,6 +5,7 @@ import iterateChain from './iterateChain'
 import getThenHandler from './getThenHandler'
 import getCatchHandler from './getCatchHandler'
 import getFinallyHandler from './getFinallyHandler'
+import isGetterOrSetter from './isGetterOrSetter'
 
 function chainLength(path: NodePath<t.CallExpression>): number {
   let length = 0
@@ -36,9 +37,10 @@ export default function shouldIgnoreChain(
 ): boolean {
   const { parentPath } = path
   if (
-    !parentPath.isReturnStatement() &&
-    !parentPath.isAwaitExpression() &&
-    !parentPath.isFunction()
+    (!parentPath.isReturnStatement() &&
+      !parentPath.isAwaitExpression() &&
+      !parentPath.isFunction()) ||
+    isGetterOrSetter(path.getFunctionParent())
   ) {
     if (chainLength(path) <= 2 && !hasComplexHandlers(path)) return true
   }

test/fixtures/bugs_GetterAndSetterCannotBeAsync.ts

@@ -3,6 +3,7 @@ class A {
   method() {return p.then(x => f(x))}
   get prop() {return p.then(x => f(x))}
   set prop(val) {return p.then(x => f(x))}
+  get longchain() {return p.then(x => f(x)).then(y => g(y)).then(z => h(z))}
 }
 const obj = {
   method() {return p.then(x => f(x))},
@@ -17,15 +18,17 @@ class A {
     return f(x);
   }
   get prop() {
-    return (async () => {
-      const x = await p;
-      return await f(x);
-    })()
+    return p.then(x => f(x))
   }
   set prop(val) {
+    return p.then(x => f(x))
+  }
+  get longchain() {
     return (async () => {
-      const x = await p;
-      return await f(x);
+      const x = await p
+      const y = await f(x)
+      const z = await g(y)
+      return await h(z)
     })()
   }
 }
@@ -35,16 +38,10 @@ const obj = {
     return f(x);
   },
   get prop() {
-    return (async () => {
-      const x = await p;
-      return await f(x);
-    })()
+    return p.then(x => f(x))
   },
   set prop(val) {
-    return (async () => {
-      const x = await p;
-      return await f(x);
-    })()
+    return p.then(x => f(x))
   }
 };
 `