Merge pull request #1449 from tgodzik/show-error-on-lenses

bugfix: Show error properly if lenses cannot be run
scalameta · Jan 10, 2024 · cee20ea · cee20ea
2 parents eaa0897 + dcd6f94
commit cee20ea
Showing 1 changed file with 22 additions and 8 deletions.
diff --git a/packages/metals-vscode/src/extension.ts b/packages/metals-vscode/src/extension.ts
@@ -593,22 +593,36 @@ function launchMetals(
       registerCommand(
         ClientCommands.StartDebugSession,
         (param: ScalaCodeLensesParams) => {
-          scalaDebugger.start(false, param).then((wasStarted) => {
-            if (!wasStarted) {
-              window.showErrorMessage("Debug session not started");
+          scalaDebugger.start(false, param).then(
+            (wasStarted) => {
+              if (!wasStarted) {
+                window.showErrorMessage("Debug session not started");
+              }
+            },
+            (reason) => {
+              if (reason instanceof Error) {
+                window.showErrorMessage(reason.message);
+              }
             }
-          });
+          );
         }
       );
 
       registerCommand(
         ClientCommands.StartRunSession,
         (param: ScalaCodeLensesParams) => {
-          scalaDebugger.start(true, param).then((wasStarted) => {
-            if (!wasStarted) {
-              window.showErrorMessage("Run session not started");
+          scalaDebugger.start(true, param).then(
+            (wasStarted) => {
+              if (!wasStarted) {
+                window.showErrorMessage("Run session not started");
+              }
+            },
+            (reason) => {
+              if (reason instanceof Error) {
+                window.showErrorMessage(reason.message);
+              }
             }
-          });
+          );
         }
       );