Move to vscode-languageclient 8.0.1

Fixes redhat-developer#711

Signed-off-by: azerr <[email protected]>

package.json

@@ -93,7 +93,7 @@
     "fs-extra": "^8.1.0",
     "glob": "^7.1.4",
     "path-exists": "^4.0.0",
-    "vscode-languageclient": "^7.0.0",
+    "vscode-languageclient": "^8.0.1",
     "yauzl": "^2.10.0"
   },
   "contributes": {

src/client/clientErrorHandler.ts

@@ -1,6 +1,6 @@
 import * as fs from "fs-extra";
 import { commands, ExtensionContext, window, workspace } from "vscode";
-import { CloseAction, ErrorAction, ErrorHandler, Message } from "vscode-languageclient";
+import { CloseAction, CloseHandlerResult, ErrorAction, ErrorHandler, ErrorHandlerResult, Message } from "vscode-languageclient";
 import { ClientCommandConstants } from "../commands/commandConstants";
 import { HEAP_DUMP_LOCATION } from "../server/java/jvmArguments";
 import { Telemetry } from "../telemetry";
@@ -27,11 +27,19 @@ export class ClientErrorHandler implements ErrorHandler {
     this.heapDumpFolder = getHeapDumpFolderFromSettings() || context.globalStorageUri.fsPath;
   }
 
-  error(_error: Error, _message: Message, _count: number): ErrorAction {
-    return ErrorAction.Continue;
+  error(error: Error, message: Message, count: number): ErrorHandlerResult {
+    return {
+      action: ErrorAction.Continue
+    };
   }
 
-  closed(): CloseAction {
+  closed(): CloseHandlerResult {
+    return {
+      action: this.doClosed()
+    }
+  }
+
+  doClosed(): CloseAction {
     this.restarts.push(Date.now());
     const heapProfileGlob = new glob.GlobSync(`${this.heapDumpFolder}/java_*.hprof`);
     if (heapProfileGlob.found.length) {

src/client/xmlClient.ts

@@ -44,8 +44,7 @@ export async function startLanguageClient(context: ExtensionContext, executable:
     return Telemetry.sendTelemetry(e.name, e.properties);
   });
 
-  context.subscriptions.push(languageClient.start());
-  await languageClient.onReady();
+  await languageClient.start();
 
   // ---
 
@@ -155,8 +154,9 @@ function getLanguageClientOptions(
     middleware: {
       workspace: {
         didChangeConfiguration: () => {
-          languageClient.sendNotification(DidChangeConfigurationNotification.type, { settings: getXMLSettings(requirementsData.java_home, logfile, externalXmlSettings) });
+          const result = languageClient.sendNotification(DidChangeConfigurationNotification.type, { settings: getXMLSettings(requirementsData.java_home, logfile, externalXmlSettings) });
           onConfigurationChange();
+          return result;
         }
       }
     }

src/extension.ts

@@ -70,5 +70,6 @@ export async function activate(context: ExtensionContext): Promise<XMLExtensionA
 export async function deactivate(): Promise<void> {
   if (languageClient) {
     await languageClient.stop();
+    languageClient = undefined;
   }
 }