Unify Go to Definition(F12) and Ctrl+Click through CCS API resolution (#20)

* revert: remove cross-workspace definition lookup (45b7b82)

* feat: add API-based Go to Definition command and integrate with keybinding

- Introduced DefinitionResolverClient for REST API resolution
- Implemented definitionLookup feature (extractQuery + lookup) for robust query handling
- Added `PrioritizedDefinitionProvider` to prefer CCS resolver before fallback
- Implemented new command vscode-objectscript.ccs.goToDefinition with API-first fallback
- Updated package.json to bind F12 and menus to the new command for ObjectScript files
- Registered new command in extension.ts and integrated telemetry
- New goToDefinitionLocalFirst command integrates CCS API before native definition

* feat: support cross-namespace definition lookup and request logging

* feat: implement full Ctrl+Click support with `CCS API` resolution

* feat: remove visual underline from `DocumentLinks` in `DefinitionDocumentLinkProvider`

Author: LeoAnders

README.md

@@ -76,23 +76,6 @@ intersystems-community.vscode-objectscript version X.Y.Z-beta.1 activating with
 
 After a subsequent update of the extension from Marketplace you will only have to download and install the new `vscode-objectscript-X.Y.Z-beta.1` VSIX. None of the other steps above are needed again.
 
-## Cross-workspace Go to Definition
-
-> **Implementation developed and maintained by Consistem Sistemas**
-
-When working in a multi-root workspace, the extension normally searches the current workspace folder (and any sibling folders connected to the same namespace) for local copies of ObjectScript code before requesting the server version. If you keep shared source code in other workspace folders with different connection settings, set the `objectscript.export.searchOtherWorkspaceFolders` array in the consuming folder's settings so those folders are considered first. Use workspace-folder names, or specify `"*"` to search every non-`isfs` folder.
-
-```json
-{
-  "objectscript.export": {
-    "folder": "src",
-    "searchOtherWorkspaceFolders": ["shared"]
-  }
-}
-```
-
-With this setting enabled, features such as Go to Definition resolve to the first matching local file across the configured workspace folders before falling back to the server copy.
-
 ## Notes
 
 - Connection-related output appears in the 'Output' view while switched to the 'ObjectScript' channel using the drop-down menu on the view titlebar.

package.json

@@ -482,6 +482,15 @@
         }
       ],
       "editor/context": [
+        {
+          "command": "vscode-objectscript.ccs.goToDefinition",
+          "group": "navigation@0",
+          "when": "editorTextFocus && editorLangId =~ /^objectscript/"
+        },
+        {
+          "command": "-editor.action.revealDefinition",
+          "when": "editorLangId =~ /^objectscript/"
+        },
         {
           "command": "vscode-objectscript.viewOthers",
           "when": "vscode-objectscript.connectActive",
@@ -529,6 +538,16 @@
         }
       ],
       "editor/title": [
+        {
+          "command": "vscode-objectscript.ccs.goToDefinition",
+          "group": "navigation@0",
+          "when": "editorTextFocus && editorLangId =~ /^objectscript/"
+        },
+        {
+          "command": "-editor.action.revealDefinition",
+          "group": "navigation@0",
+          "when": "editorLangId =~ /^objectscript/"
+        },
         {
           "command": "vscode-objectscript.serverCommands.sourceControl",
           "group": "navigation@1",
@@ -854,6 +873,16 @@
         "title": "Show Global Documentation",
         "enablement": "vscode-objectscript.connectActive"
       },
+      {
+        "category": "ObjectScript",
+        "command": "vscode-objectscript.ccs.goToDefinition",
+        "title": "Go to Definition"
+      },
+      {
+        "category": "ObjectScript",
+        "command": "vscode-objectscript.ccs.followDefinitionLink",
+        "title": "Follow Definition Link"
+      },
       {
         "category": "ObjectScript",
         "command": "vscode-objectscript.compile",
@@ -1228,6 +1257,11 @@
       }
     ],
     "keybindings": [
+      {
+        "command": "vscode-objectscript.ccs.goToDefinition",
+        "key": "F12",
+        "when": "editorTextFocus && editorLangId =~ /^objectscript/"
+      },
       {
         "command": "vscode-objectscript.compile",
         "key": "Ctrl+F7",
@@ -1439,14 +1473,6 @@
               },
               "additionalProperties": false
             },
-            "searchOtherWorkspaceFolders": {
-              "markdownDescription": "Additional workspace folders to search for client-side sources when resolving ObjectScript documents. Specify `\"*\"` to search all non-isfs workspace folders in the current multi-root workspace before falling back to the server.",
-              "type": "array",
-              "items": {
-                "type": "string"
-              },
-              "default": []
-            },
             "atelier": {
               "description": "Export source code as Atelier did it, with packages as subfolders. This setting only affects classes, routines, include files and DFI files.",
               "type": "boolean"

src/ccs/commands/goToDefinitionLocalFirst.ts

@@ -0,0 +1,26 @@
+import * as vscode from "vscode";
+
+import { lookupCcsDefinition } from "../features/definitionLookup/lookup";
+
+export async function goToDefinitionLocalFirst(): Promise<void> {
+  const editor = vscode.window.activeTextEditor;
+  if (!editor) {
+    return;
+  }
+
+  const { document, selection } = editor;
+  const position = selection.active;
+  const tokenSource = new vscode.CancellationTokenSource();
+
+  try {
+    const location = await lookupCcsDefinition(document, position, tokenSource.token);
+    if (location) {
+      await vscode.window.showTextDocument(location.uri, { selection: location.range });
+      return;
+    }
+  } finally {
+    tokenSource.dispose();
+  }
+
+  await vscode.commands.executeCommand("editor.action.revealDefinition");
+}

src/ccs/core/types.ts

@@ -1,3 +1,10 @@
+export interface LocationJSON {
+  uri?: string;
+  line?: number;
+}
+
+export type ResolveDefinitionResponse = LocationJSON;
+
 export interface ResolveContextExpressionResponse {
   status?: string;
   textExpression?: string;