reapply fix for method snippet with code actions for windows in really verbose way

Author: zardoy

README.MD

@@ -315,6 +315,12 @@ function Foo() {
 }
 ```
 
+`tsEssentialPlugins.methodSnippetsInsertText`:
+
+Optionally resolve insertText of all completion at suggest trigger:
+
+![method-snippets-insert-text](media/method-snippets-insert-text.png)
+
 ### Ambiguous Suggestions
 
 Some variables like `Object` or `lodash` are common to access properties as well as call directly:

media/method-snippets-insert-text.png

@@ -43,9 +43,22 @@ export default (tsApi: { onCompletionAccepted }) => {
             const dataMarker = '<!--tep '
             if (!documentation?.startsWith(dataMarker)) return
             const parsed = JSON.parse(documentation.slice(dataMarker.length, documentation.indexOf('e-->')))
-            const { methodSnippet: params, isAmbiguous } = parsed
-            // nextChar check also duplicated in completionEntryDetails for perf, but we need to run this check again with correct position
+            const { methodSnippet: params, isAmbiguous, wordStartOffset } = parsed
             const startPos = editor.selection.start
+            const acceptedWordStartOffset = wordStartOffset !== undefined && editor.document.getWordRangeAtPosition(startPos, /[\w\d]+/i)?.start
+            if (!oneOf(acceptedWordStartOffset, false, undefined) && wordStartOffset === editor.document.offsetAt(acceptedWordStartOffset)) {
+                await new Promise<void>(resolve => {
+                    vscode.workspace.onDidChangeTextDocument(({ document, contentChanges }) => {
+                        if (document !== editor.document || contentChanges.length === 0) return
+                        resolve()
+                    })
+                })
+                await new Promise(resolve => {
+                    setTimeout(resolve, 0)
+                })
+            }
+
+            // nextChar check also duplicated in completionEntryDetails for perf, but we need to run this check again with correct position
             const nextChar = editor.document.getText(new vscode.Range(startPos, startPos.translate(0, 1)))
             if (!params || ['(', '.', '`'].includes(nextChar)) return
 

src/onCompletionAccepted.ts

@@ -3,6 +3,7 @@ import constructMethodSnippet from './constructMethodSnippet'
 import { RequestResponseTypes } from './ipcTypes'
 import namespaceAutoImports from './namespaceAutoImports'
 import { GetConfig } from './types'
+import { wordStartAtPos } from './utils'
 
 export const lastResolvedCompletion = {
     value: undefined as undefined | RequestResponseTypes['getLastResolvedCompletion'],
@@ -60,7 +61,12 @@ export default function completionEntryDetails(
             }
             const methodSnippet = constructMethodSnippet(languageService, sourceFile, position, symbol, c, resolveData)
             if (methodSnippet) {
-                const data = JSON.stringify({ methodSnippet, isAmbiguous: resolveData.isAmbiguous })
+                const wordStartOffset = source ? wordStartAtPos(sourceFile.getFullText(), position) : undefined
+                const data = JSON.stringify({
+                    methodSnippet,
+                    isAmbiguous: resolveData.isAmbiguous,
+                    wordStartOffset,
+                })
                 prior.documentation = [{ kind: 'text', text: `<!--tep ${data} e-->` }, ...(prior.documentation ?? [])]
             }
         }

typescript/src/completionEntryDetails.ts

@@ -258,6 +258,17 @@ const wordRangeAtPos = (text: string, position: number) => {
     return text.slice(startPos + 1, endPos)
 }
 
+export const wordStartAtPos = (text: string, position: number) => {
+    const isGood = (pos: number) => {
+        return /[\w\d]/i.test(text.at(pos - 1) ?? '')
+    }
+    let startPos = position
+    while (isGood(startPos)) {
+        startPos--
+    }
+    return startPos
+}
+
 type GetIs<T> = T extends (elem: any) => elem is infer T ? T : never
 
 export const createDummySourceFile = (code: string) => {