Avoid registering the same scheme multiple times

usethesource · Dec 13, 2024 · 6e26f07 · 6e26f07
1 parent 6725358
commit 6e26f07
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/rascal-vscode-extension/src/fs/VSCodeURIResolver.ts b/rascal-vscode-extension/src/fs/VSCodeURIResolver.ts
@@ -220,6 +220,10 @@ export class VSCodeUriResolverServer implements Disposable {
         toIgnore.forEach(v => this.rascalNativeSchemes.add(v));
     }
 
+    getIgnoredSchemes(): string[] {
+        return Array.from(this.rascalNativeSchemes.values());
+    }
+
     dispose() {
         this.server.close();
         this.activeClients.forEach(c => c.dispose());

diff --git a/rascal-vscode-extension/src/lsp/RascalLSPConnection.ts b/rascal-vscode-extension/src/lsp/RascalLSPConnection.ts
@@ -62,8 +62,12 @@ export async function activateLanguageClient(
     const schemesReply = client.sendRequest<string[]>("rascal/filesystem/schemes");
 
     schemesReply.then( schemes => {
-        vfsServer.ignoreSchemes(schemes);
-        new RascalFileSystemProvider(client).registerSchemes(schemes);
+        // avoid registering the same schemes multiple times
+        const knownRascalNativeSchemes = vfsServer.getIgnoredSchemes();
+        const remainingSchemes = schemes.filter(s => !knownRascalNativeSchemes.includes(s));
+        if (remainingSchemes.length) {
+            new RascalFileSystemProvider(client).registerSchemes(schemes);
+        }
     });