fix: dispose

Author: bajrangCoder

src/cm/lsp/clientManager.ts

@@ -1074,6 +1074,22 @@ export class LspClientManager {
       logLspInfo(`[LSP:${server.id}] attached to ${uri}${suffix}`);
     };
 
+    const dispose = async (): Promise<void> => {
+      if (disposed) return;
+      disposed = true;
+      this.#clients.delete(key);
+      try {
+        client.disconnect();
+      } catch (error) {
+        console.warn(`Error disconnecting LSP client ${server.id}`, error);
+      }
+      try {
+        await transportHandle.dispose?.();
+      } catch (error) {
+        console.warn(`Error disposing LSP transport ${server.id}`, error);
+      }
+    };
+
     const detach = (uri: string, view?: EditorView): void => {
       const actualUri = uriAliases.get(uri) ?? uri;
       const existing = fileRefs.get(actualUri);
@@ -1100,26 +1116,11 @@ export class LspClientManager {
           server,
           client,
           rootUri: effectiveRoot,
+          dispose,
         });
       }
     };
 
-    const dispose = async (): Promise<void> => {
-      if (disposed) return;
-      disposed = true;
-      this.#clients.delete(key);
-      try {
-        client.disconnect();
-      } catch (error) {
-        console.warn(`Error disconnecting LSP client ${server.id}`, error);
-      }
-      try {
-        await transportHandle.dispose?.();
-      } catch (error) {
-        console.warn(`Error disposing LSP transport ${server.id}`, error);
-      }
-    };
-
     return {
       server,
       client,

src/cm/lsp/types.ts

@@ -413,6 +413,8 @@ export interface ClientIdleInfo {
 	server: LspServerDefinition;
 	client: LSPClient;
 	rootUri: string | null;
+	/** Disposes only this idle client instance (not every client for the server id). */
+	dispose: () => Promise<void>;
 }
 
 export interface ClientState {

src/lib/editorManager.js

@@ -3290,10 +3290,22 @@ async function EditorManager($header, $body) {
 
 	lspClientManager.setOptions({
 		resolveRoot: resolveRootUriForContext,
-		onClientIdle: ({ server }) => {
-			if (!server?.id) return;
-			stopManagedServer(server.id);
-			void lspClientManager.disposeServer(server.id);
+		onClientIdle: ({ server, dispose }) => {
+			if (!server?.id || typeof dispose !== "function") return;
+			// Dispose only this idle client. disposeServer(server.id) would tear
+			// down every workspace sharing the server id (e.g. web-worker LSPs).
+			void (async () => {
+				await dispose();
+				const stillActive = lspClientManager
+					.getActiveClients()
+					.some(
+						(state) =>
+							state.server?.id?.toLowerCase() === server.id.toLowerCase(),
+					);
+				if (!stillActive) {
+					stopManagedServer(server.id);
+				}
+			})();
 		},
 		displayFile: async (targetUri) => {
 			if (!targetUri) return null;