Fix go to definition for AHK v1, other misc work (#562)

demos/manualTests/definitionProvider/included.ahk1

@@ -0,0 +1,9 @@
+#NoEnv
+#SingleInstance, Force
+SendMode, Input
+SetBatchLines, -1
+SetWorkingDir, %A_ScriptDir%
+
+MyDefProviderFunc() {
+    MsgBox % "Hi from MyDefProviderFunc"
+}

demos/manualTests/definitionProvider/main.ahk1

@@ -0,0 +1,10 @@
+#NoEnv
+#SingleInstance, Force
+SendMode, Input
+SetBatchLines, -1
+SetWorkingDir, %A_ScriptDir%
+
+#Include ./included.ahk1
+
+;* Should be able to ctrl+click to jump to `included.ahk1` 
+MyDefProviderFunc()

package.json

@@ -1167,7 +1167,7 @@
         "@types/xml2js": "^0.4.11",
         "@vscode/test-cli": "^0.0.10",
         "@vscode/test-electron": "^2.3.10",
-        "@vscode/vsce": "^3.1.1",
+        "@vscode/vsce": "^3.2.1",
         "del-cli": "^6.0.0",
         "esbuild": "0.24.0",
         "eslint": "^9.9.0",

src/parser/parser.ts

@@ -54,8 +54,9 @@ export class Parser {
         options: BuildScriptOptions = {},
     ): Promise<Script> {
         const funcName = 'buildScript';
-        if (options.usingCache && documentCache.get(document.uri.path)) {
-            return documentCache.get(document.uri.path);
+        const cachedDocument = documentCache.get(document.uri.path);
+        if (options.usingCache && cachedDocument) {
+            return cachedDocument;
         }
 
         const maxParseLength =

src/providers/defProvider.e2e.ts

@@ -0,0 +1,15 @@
+import assert from 'assert';
+import { tryGetFileLink } from './defProvider';
+
+suite(tryGetFileLink.name, () => {
+    const tests: [
+        name: string,
+        args: Parameters<typeof tryGetFileLink>,
+        expected: ReturnType<typeof tryGetFileLink>,
+    ][] = [['non-match', ['/c:/path/to/file.ahk', 'foo'], undefined]];
+    tests.forEach(([name, args, expected]) =>
+        test(name, async () => {
+            assert.strictEqual(await tryGetFileLink(...args), expected);
+        }),
+    );
+});

src/providers/defProvider.ts

@@ -1,30 +1,42 @@
 import * as vscode from 'vscode';
 import { Parser } from '../parser/parser';
 import { resolveIncludedPath } from './defProvider.utils';
-import { Out } from 'src/common/out';
+import { Out } from '../common/out';
 import { stat } from 'fs/promises';
 
 export class DefProvider implements vscode.DefinitionProvider {
     public async provideDefinition(
         document: vscode.TextDocument,
         position: vscode.Position,
-    ): Promise<vscode.Location | vscode.Location[] | vscode.LocationLink[]> {
-        const fileLink = await tryGetFileLink(document, position);
+    ): Promise<
+        vscode.Location | vscode.Location[] | vscode.LocationLink[] | null
+    > {
+        const funcName = 'provideDefinition';
+        Out.debug(funcName);
+
+        const docPath = document.uri.path;
+        const { text } = document.lineAt(position.line);
+        const fileLink = await tryGetFileLink(docPath, text);
         if (fileLink) {
+            Out.debug(`${funcName} returning fileLink`);
             return fileLink;
         }
+        Out.debug(`${funcName} after tryGetFileLink`);
 
         const word = document.getText(
             document.getWordRangeAtPosition(position),
         );
+        Out.debug(`${funcName} word: ${word}`);
 
         // get method
         if (
             new RegExp(word + '\\s*\\(.*?\\)').test(
                 document.lineAt(position.line).text,
             )
         ) {
+            Out.debug(`${funcName} calling getMethodByName for word: ${word}`);
             const method = await Parser.getMethodByName(document, word);
+            Out.debug(`${funcName} method.name: ${method?.name}`);
             if (method) {
                 return new vscode.Location(
                     vscode.Uri.parse(method.uriString),
@@ -100,17 +112,22 @@ export class DefProvider implements vscode.DefinitionProvider {
  ** Currently assumes the working directory is the script path and
  * does not respect previous `#include dir` directives
  */
-async function tryGetFileLink(
-    document: vscode.TextDocument,
-    position: vscode.Position,
-): Promise<vscode.Location> | undefined {
+export async function tryGetFileLink(
     /** @example '/c:/path/to/file.ahk' */
-    const docPath = document.uri.path;
-    const { text } = document.lineAt(position.line);
+    docPath: string,
+    text: string,
+): Promise<vscode.Location | undefined> {
+    const funcName = 'tryGetFileLink';
+    Out.debug(`${funcName}('${docPath}', '${text}')`);
+
     /** @example 'c:/path/to/included.ahk' */
     const resolvedPath = resolveIncludedPath(docPath, text);
-    Out.debug(`resolvedPath: ${resolvedPath}`);
+    Out.debug(`${funcName} resolvedPath: ${resolvedPath}`);
+    if (!resolvedPath) return undefined;
+
     const fsStat = await stat(resolvedPath);
+    const isFile = fsStat.isFile();
+    Out.debug(`${funcName} isFile: ${isFile}`);
     return fsStat.isFile()
         ? new vscode.Location(
               vscode.Uri.file(resolvedPath),

src/providers/defProvider.utils.ts

@@ -55,6 +55,8 @@ export const resolveIncludedPath = (
     ahkLine: string,
 ): string | undefined => {
     const includedPath = getIncludedPath(ahkLine);
+    if (!includedPath) return undefined;
+
     /** @example 'c:/path/to' */
     const parentGoodPath = basePath.substring(1, basePath.lastIndexOf('/'));
     const normalizedPath = normalizeIncludedPath(