From ce1aff4981ad799158228dd7a65c05f69940e2f1 Mon Sep 17 00:00:00 2001
From: chemzqm <chemzqm@gmail.com>
Date: Sat, 24 Nov 2018 20:56:49 +0800
Subject: [PATCH] check position for function snippet

---
 src/server/features/completionItemProvider.ts | 34 ++++++++++++++++++-
 1 file changed, 33 insertions(+), 1 deletion(-)

diff --git a/src/server/features/completionItemProvider.ts b/src/server/features/completionItemProvider.ts
index 77cf98f..ef43bb7 100644
--- a/src/server/features/completionItemProvider.ts
+++ b/src/server/features/completionItemProvider.ts
@@ -223,7 +223,10 @@ export default class TypeScriptCompletionItemProvider implements CompletionItemP
     if (command) item.command = command
     item.additionalTextEdits = additionalTextEdits
     if (detail && item.insertTextFormat == InsertTextFormat.Snippet) {
-      this.createSnippetOfFunctionCall(item, detail)
+      const shouldCompleteFunction = await this.isValidFunctionCompletionContext(filepath, position, token)
+      if (shouldCompleteFunction) {
+        this.createSnippetOfFunctionCall(item, detail)
+      }
     }
 
     return item
@@ -395,6 +398,35 @@ export default class TypeScriptCompletionItemProvider implements CompletionItemP
     textEdit.newText = snippet
     item.textEdit = textEdit
   }
+
+  private async isValidFunctionCompletionContext(
+    filepath: string,
+    position: Position,
+    token: CancellationToken
+  ): Promise<boolean> {
+    // Workaround for https://github.com/Microsoft/TypeScript/issues/12677
+    // Don't complete function calls inside of destructive assigments or imports
+    try {
+      const args: Proto.FileLocationRequestArgs = typeConverters.Position.toFileLocationRequestArgs(filepath, position)
+      const response = await this.client.execute('quickinfo', args, token)
+      if (response.type !== 'response') {
+        return true
+      }
+
+      const { body } = response
+      switch (body && body.kind) {
+        case 'var':
+        case 'let':
+        case 'const':
+        case 'alias':
+          return false
+        default:
+          return true
+      }
+    } catch (e) {
+      return true
+    }
+  }
 }
 
 function shouldExcludeCompletionEntry(