fix: handle call expression param case

Author: Ilya Golovin

typescript/src/codeActions/custom/addDestructure/addSplittedDestructure.ts

@@ -22,8 +22,8 @@ export default (node: ts.Node, sourceFile: ts.SourceFile, formatOptions: ts.Form
 
         if (
             ts.isElementAccessExpression(highlightedNode.parent) ||
-            ts.isCallExpression(highlightedNode.parent.parent) ||
-            ts.isTypeQueryNode(highlightedNode.parent)
+            ts.isTypeQueryNode(highlightedNode.parent) ||
+            (ts.isCallExpression(highlightedNode.parent.parent) && highlightedNode.parent.parent.expression === highlightedNode.parent)
         )
             return
 

typescript/test/codeActions/addDestruct.spec.ts

@@ -253,9 +253,34 @@ describe('Add destructure', () => {
             })
         })
     })
+    describe('Should destruct param call expression param', () => {
+        test('Should skip if trying to destruct call expression', () => {
+            const initial = /* ts */ `
+              const /*t*/newVariable/*t*/ = { test: 1}
+
+              const obj = {
+                  tag: foo.map(newVariable.test),
+              }
+            `
+            const expected = /* ts */ `
+              const { test } = { test: 1}
+
+              const obj = {
+                  tag: foo.map(test),
+              }
+            `
+
+            const { codeAction } = fourslashLikeTester(initial, undefined, { dedent: true })
+
+            codeAction(0, {
+                refactorName: 'Add Destruct',
+                newContent: expected,
+            })
+        })
+    })
 
     describe('Skip cases', () => {
-        test('Should skip if trying to destruct call expression', () => {
+        test('Should skip if trying to destruct expression of call expression', () => {
             const initial = /* ts */ `
               const /*t*/newVariable/*t*/ = foo