refactor(extension): Optimized the destruction of some events and ref…

…actor some code (#3780)
vuejs · Dec 21, 2023 · f1ce889 · f1ce889
1 parent 18c4329
commit f1ce889
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 16 deletions.
diff --git a/extensions/vscode/src/common.ts b/extensions/vscode/src/common.ts
@@ -161,14 +161,14 @@ async function doActivate(context: vscode.ExtensionContext, createLc: CreateLang
 	}
 
 	function activateServerMaxOldSpaceSizeChange() {
-		vscode.workspace.onDidChangeConfiguration((e) => {
+		context.subscriptions.push(vscode.workspace.onDidChangeConfiguration((e) => {
 			if (e.affectsConfiguration('vue.server.runtime') || e.affectsConfiguration('vue.server.path')) {
 				requestReloadVscode();
 			}
 			if (e.affectsConfiguration('vue')) {
 				vscode.commands.executeCommand('volar.action.restartServer');
 			}
-		});
+		}));
 	}
 
 	async function activateRestartRequest() {

diff --git a/extensions/vscode/src/features/nameCasing.ts b/extensions/vscode/src/features/nameCasing.ts
@@ -11,24 +11,25 @@ export async function activate(_context: vscode.ExtensionContext, client: BaseLa
 
 	await client.start();
 
+	const disposes: vscode.Disposable[] = [];
 	const statusBar = vscode.window.createStatusBarItem(vscode.StatusBarAlignment.Right);
 	statusBar.command = 'volar.action.nameCasing';
 
 	update(vscode.window.activeTextEditor?.document);
 
-	const d_1 = vscode.window.onDidChangeActiveTextEditor(e => {
+	disposes.push(vscode.window.onDidChangeActiveTextEditor(e => {
 		update(e?.document);
-	});
-	const d_2 = vscode.workspace.onDidChangeConfiguration(() => {
+	}));
+	disposes.push(vscode.workspace.onDidChangeConfiguration(() => {
 		attrNameCasings.clear();
 		tagNameCasings.clear();
 		update(vscode.window.activeTextEditor?.document);
-	});
-	const d_3 = vscode.workspace.onDidCloseTextDocument((doc) => {
+	}));
+	disposes.push(vscode.workspace.onDidCloseTextDocument((doc) => {
 		attrNameCasings.delete(doc.uri.toString());
 		tagNameCasings.delete(doc.uri.toString());
-	});
-	const d_4 = vscode.commands.registerCommand('volar.action.nameCasing', async () => {
+	}));
+	disposes.push(vscode.commands.registerCommand('volar.action.nameCasing', async () => {
 
 		if (!vscode.window.activeTextEditor?.document) return;
 
@@ -80,14 +81,11 @@ export async function activate(_context: vscode.ExtensionContext, client: BaseLa
 			await convertAttr(vscode.window.activeTextEditor, AttrNameCasing.Camel);
 		}
 		updateStatusBarText();
-	});
+	}));
 
 	client.onDidChangeState(e => {
 		if (e.newState === State.Stopped) {
-			d_1.dispose();
-			d_2.dispose();
-			d_3.dispose();
-			d_4.dispose();
+			disposes.forEach(d => d.dispose());
 			statusBar.dispose();
 		}
 	});

diff --git a/extensions/vscode/src/nodeClientMain.ts b/extensions/vscode/src/nodeClientMain.ts
@@ -18,7 +18,7 @@ export async function activate(context: vscode.ExtensionContext) {
 	let cancellationPipeUpdateKey: string | undefined;
 	let serverPathStatusItem: vscode.StatusBarItem | undefined;
 
-	vscode.workspace.onDidChangeTextDocument((e) => {
+	context.subscriptions.push(vscode.workspace.onDidChangeTextDocument((e) => {
 		let key = e.document.uri.toString() + '|' + e.document.version;
 		if (cancellationPipeUpdateKey === undefined) {
 			cancellationPipeUpdateKey = key;
@@ -28,7 +28,7 @@ export async function activate(context: vscode.ExtensionContext) {
 			cancellationPipeUpdateKey = key;
 			fs.writeFileSync(cancellationPipeName, '');
 		}
-	});
+	}));
 
 	await commonActivate(context, (
 		id,