Better error handling and only fetch if workspace/playground is experimental capability

Author: award999

src/playgrounds/LSPPlaygroundsDiscovery.ts

@@ -14,6 +14,7 @@
 import { checkExperimentalCapability } from "../sourcekit-lsp/LanguageClientManager";
 import { LanguageClientManager } from "../sourcekit-lsp/LanguageClientManager";
 import { Playground, WorkspacePlaygroundsRequest } from "../sourcekit-lsp/extensions";
+import { Version } from "../utilities/version";
 
 export { Playground };
 
@@ -25,6 +26,11 @@ export { Playground };
  */
 export class LSPPlaygroundsDiscovery {
     constructor(private languageClient: LanguageClientManager) {}
+
+    private get toolchainVersion(): Version {
+        return this.languageClient.folderContext.toolchain.swiftVersion;
+    }
+
     /**
      * Return list of workspace playgrounds
      */
@@ -39,4 +45,13 @@ export class LSPPlaygroundsDiscovery {
             }
         });
     }
+
+    async supportsPlaygrounds(): Promise<boolean> {
+        if (this.toolchainVersion.isLessThan(new Version(6, 3, 0))) {
+            return false;
+        }
+        return await this.languageClient.useLanguageClient(async client => {
+            return checkExperimentalCapability(client, WorkspacePlaygroundsRequest.method, 1);
+        });
+    }
 }

src/playgrounds/PlaygroundProvider.ts

@@ -15,6 +15,7 @@ import * as vscode from "vscode";
 
 import { FolderContext } from "../FolderContext";
 import { FolderOperation, WorkspaceContext } from "../WorkspaceContext";
+import { SwiftLogger } from "../logging/SwiftLogger";
 import { LSPPlaygroundsDiscovery, Playground } from "./LSPPlaygroundsDiscovery";
 
 export { Playground };
@@ -42,6 +43,10 @@ export class PlaygroundProvider implements vscode.Disposable {
         return new LSPPlaygroundsDiscovery(this.folderContext.languageClientManager);
     }
 
+    private get logger(): SwiftLogger {
+        return this.folderContext.workspaceContext.logger;
+    }
+
     /**
      * Create folder observer that creates a PlaygroundProvider when a folder is added and
      * discovers available playgrounds when the folder is in focus
@@ -107,14 +112,28 @@ export class PlaygroundProvider implements vscode.Disposable {
         this.didChangePlaygroundsEmitter.event;
 
     async fetch() {
+        if (!(await this.lspPlaygroundDiscovery.supportsPlaygrounds())) {
+            this.logger.debug(
+                `Fetching playgrounds not supported by the language server`,
+                this.folderContext.name
+            );
+            return;
+        }
         this.fetchPromise = this.lspPlaygroundDiscovery.getWorkspacePlaygrounds();
-        const playgrounds = await this.fetchPromise;
-        this.documentPlaygrounds.clear();
-        for (const playground of playgrounds) {
-            const uri = playground.location.uri;
-            this.documentPlaygrounds.set(
-                uri,
-                (this.documentPlaygrounds.get(uri) ?? []).concat(playground)
+        try {
+            const playgrounds = await this.fetchPromise;
+            this.documentPlaygrounds.clear();
+            for (const playground of playgrounds) {
+                const uri = playground.location.uri;
+                this.documentPlaygrounds.set(
+                    uri,
+                    (this.documentPlaygrounds.get(uri) ?? []).concat(playground)
+                );
+            }
+        } catch (error) {
+            this.logger.error(
+                `Failed to fetch workspace playgrounds: ${error}`,
+                this.folderContext.name
             );
         }
         this.fetchPromise = undefined;