Debugger improvements

README.md

@@ -1,4 +1,4 @@
-<!-- DO NOT REMOVE - contributor_list:data:start:["ivojawer", "fdodino", "PalumboN", "npasserini", "Miranda-03", "dependabot[bot]", "FerRomMu"]:end -->
+<!-- DO NOT REMOVE - contributor_list:data:start:["ivojawer", "fdodino", "PalumboN", "npasserini", "dependabot[bot]", "Miranda-03", "FerRomMu"]:end -->
 
 
 # Wollok IDE
@@ -40,6 +40,6 @@ Do you want to contribute? Great, you are always welcome!
 <!-- DO NOT REMOVE - contributor_list:start -->
 ## 👥 Contributors
 
-<img src="https://avatars.githubusercontent.com/u/18614957?v=4" height="40" width="40" alt="ivojawer" title="ivojawer" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/u/4549002?v=4" height="40" width="40" alt="fdodino" title="fdodino" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/u/4098184?v=4" height="40" width="40" alt="PalumboN" title="PalumboN" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/u/4633913?v=4" height="40" width="40" alt="npasserini" title="npasserini" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/u/72475370?v=4" height="40" width="40" alt="Miranda-03" title="Miranda-03" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/in/29110?v=4" height="40" width="40" alt="dependabot[bot]" title="dependabot[bot]" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/u/70177008?v=4" height="40" width="40" alt="FerRomMu" title="FerRomMu" class="avatar circle"/>&nbsp;
+<img src="https://avatars.githubusercontent.com/u/18614957?v=4" height="40" width="40" alt="ivojawer" title="ivojawer" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/u/4549002?v=4" height="40" width="40" alt="fdodino" title="fdodino" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/u/4098184?v=4" height="40" width="40" alt="PalumboN" title="PalumboN" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/u/4633913?v=4" height="40" width="40" alt="npasserini" title="npasserini" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/in/29110?v=4" height="40" width="40" alt="dependabot[bot]" title="dependabot[bot]" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/u/72475370?v=4" height="40" width="40" alt="Miranda-03" title="Miranda-03" class="avatar circle"/>&nbsp;<img src="https://avatars.githubusercontent.com/u/70177008?v=4" height="40" width="40" alt="FerRomMu" title="FerRomMu" class="avatar circle"/>&nbsp;
 <!-- DO NOT REMOVE - contributor_list:end -->
 <!-- prettier-ignore-end -->

package.json

@@ -101,38 +101,53 @@
         "languages": [
           "wollok"
         ],
-        "program": "./out/debug-adapter/src/index.js",
         "runtime": "node",
         "configurationAttributes": {
           "launch": {
             "required": [
-              "file",
               "target"
             ],
             "properties": {
               "target": {
                 "type": "object",
                 "description": "Program or test to run.",
                 "properties": {
+                  "type": {
+                    "type": "string",
+                    "description": "The type of targetting.",
+                    "enum": [
+                      "program",
+                      "test",
+                      "fqn"
+                    ],
+                    "enumDescriptions": [
+                      "Target a program, should specify program and file",
+                      "Target a test, should specify test, describe and file",
+                      "Target a fully qualified name, should specify fqn"
+                    ]
+                  },
                   "program": {
                     "type": "string",
                     "description": "The program's name."
                   },
                   "describe": {
                     "type": "string",
-                    "description": "(optional) The describe's name.",
-                    "optional": true
+                    "description": "(optional) The describe's name."
                   },
                   "test": {
                     "type": "string",
                     "description": "The test's name."
+                  },
+                  "fqn": {
+                    "type": "string",
+                    "description": "The fully qualified name of the describe/test/program you want to run."
+                  },
+                  "file": {
+                    "type": "string",
+                    "description": "Absolute path to a Wollok file."
                   }
                 }
               },
-              "file": {
-                "type": "string",
-                "description": "Absolute path to a Wollok file."
-              },
               "stopOnEntry": {
                 "type": "boolean",
                 "description": "Stop on the first line of the program/test.",

packages/client/src/commands.ts

@@ -14,7 +14,7 @@
   asShellString,
   fsToShell,
 } from './platform-string-utils'
-import { COMMAND_RUN_ALL_TESTS, COMMAND_RUN_GAME, COMMAND_RUN_PROGRAM, COMMAND_RUN_TEST, COMMAND_START_REPL, wollokLSPExtensionCode, COMMAND_INIT_PROJECT } from './shared-definitions'
+import { COMMAND_RUN_ALL_TESTS, COMMAND_RUN_GAME, COMMAND_RUN_PROGRAM, COMMAND_RUN_TEST, COMMAND_START_REPL, wollokLSPExtensionCode, COMMAND_INIT_PROJECT, COMMAND_DEBUG } from './shared-definitions'
 import { getLSPMessage } from './messages'
 
 export const subscribeWollokCommands = (context: ExtensionContext): void => {
@@ -32,6 +32,7 @@
   context.subscriptions.push(
     registerCLICommand(COMMAND_INIT_PROJECT, initProject),
   )
+  context.subscriptions.push(registerDebuggerCommand())
 }
 
 /**
@@ -141,3 +142,20 @@
     new ShellExecution(fsToShell(wollokCliPath), shellCommandArgs),
   )
 }
+function registerDebuggerCommand(): { dispose(): any } {
+  return commands.registerCommand(COMMAND_DEBUG, async (fqn: string) => {
+    vscode.debug.startDebugging(
+      vscode.workspace.workspaceFolders[0],
+      {
+        type: "wollok",
+        request: "launch",
+        name: "Launch Debug Session",
+        stopOnEntry: false,
+        target: {
+          type: 'fqn',
+          fqn,
+        },
+      }
+    )
+  })
+}

packages/client/src/shared-definitions.ts

@@ -10,4 +10,6 @@ export const COMMAND_RUN_GAME = 'wollok.run.game'
 export const COMMAND_RUN_PROGRAM = 'wollok.run.program'
 export const COMMAND_RUN_ALL_TESTS = 'wollok.run.allTests'
 export const COMMAND_RUN_TEST = 'wollok.run.test'
-export const COMMAND_INIT_PROJECT = 'wollok.init.project'
+export const COMMAND_INIT_PROJECT = 'wollok.init.project'
+//ToDo: Create shared package
+export const COMMAND_DEBUG = 'wollok.debug'

packages/client/src/test/code-lens.test.ts

@@ -23,6 +23,14 @@ suite('Should do code lenses', () => {
           arguments: ['pepitaGame.juego'],
         },
       ),
+      new CodeLens(
+        new Range(new Position(0, 0), new Position(2, 1)),
+        {
+          title: 'Debug program',
+          command: 'wollok.debug',
+          arguments: ['pepitaGame.juego'],
+        }
+      ),
     ])
   })
 
@@ -52,6 +60,14 @@ suite('Should do code lenses', () => {
           arguments: [null, 'test.wtest', 'pepita test', 'pepita is happy'],
         },
       ),
+      new CodeLens(
+        new Range(new Position(1, 4), new Position(3, 5)),
+        {
+          title: 'Debug test',
+          command: 'wollok.debug',
+          arguments: ['test."pepita test"."pepita is happy"'],
+        }
+      ),
     ])
   })
 

packages/debug-adapter/package.json

@@ -17,13 +17,14 @@
     "@vscode/debugprotocol": "^1.66.0"
   },
   "devDependencies": {
+    "@types/chai": "^4",
     "@types/mocha": "^10",
     "@types/node": "^18.14.1",
     "@types/vscode": "^1.92.0",
     "@vscode/debugadapter-testsupport": "^1.67.0",
+    "chai": "4.3.10",
     "mocha": "^10.7.3",
     "nyc": "^17.0.0",
-    "ts-mocha": "^10.0.0",
-    "wollok-ts": "4.2.0"
+    "ts-mocha": "^10.0.0"
   }
 }

packages/debug-adapter/src/debug-configuration-provider.ts

@@ -1,25 +1,26 @@
 import { CancellationToken, DebugConfiguration, DebugConfigurationProvider, ProviderResult, WorkspaceFolder } from 'vscode'
 import * as vscode from 'vscode'
 
+const targetTypeRequiredKeys = {
+  'fqn': ['fqn'],
+  'program': ['file', 'program'],
+  'test': ['file', 'test'],
+}
+
 export class WollokDebugConfigurationProvider implements DebugConfigurationProvider {
 
   resolveDebugConfiguration(_folder: WorkspaceFolder | undefined, config: DebugConfiguration, _token?: CancellationToken): ProviderResult<DebugConfiguration> {
-    if (!config.file) {
-      return vscode.window.showErrorMessage("Cannot find a file to debug").then(_ => {
-        return undefined	// abort launch
-      })
+    if(!config.target.type) {
+      return vscode.window.showErrorMessage('No target type provided').then(() => undefined)
     }
 
-    if(config.target.program && config.target.test){
-      return vscode.window.showErrorMessage("Cannot specify both program and test properties at the same time").then(_ => {
-        return undefined	// abort launch
-      })
+    if(!targetTypeRequiredKeys[config.target.type]) {
+      return vscode.window.showErrorMessage(`Unknown target type: ${config.target.type}`).then(() => undefined)
     }
 
-    if(!config.target.program && !config.target.test) {
-      return vscode.window.showErrorMessage("Must specify either program or test property").then(_ => {
-        return undefined	// abort launch
-      })
+    const missingKeys = targetTypeRequiredKeys[config.target.type].filter(key => !config.target[key])
+    if(missingKeys.length) {
+      return vscode.window.showErrorMessage(`Missing required target parameters: ${missingKeys.join(', ')}`).then(() => undefined)
     }
 
     return config