perf: fix potential memory leak for old comment block configurations.

- Added new `commentBlocksDisposables` disposable array to keep track of the comment blocks.

- Changed the variable name for the comment block disposable to `commentBlocksDisposables`.

- Fixed potential memory leak in the `onDidOpenTextDocument` event handler. On each document opening, we reconfigure the comment blocks configs which pushes new disposables everytime. This could lead to memory leaks over time.

Fixed by looping through the `commentBlocksDisposables` array and disposing of each configuration, before new ones are created.

Author: yCodeTech

src/extension.ts

@@ -24,14 +24,16 @@ export function activate(context: vscode.ExtensionContext) {
 
 	// Store disposables for cleanup
 	const disposables: vscode.Disposable[] = [];
-	const configureCommentBlocksDisposable = configuration.configureCommentBlocks();
-	disposables.push(...configureCommentBlocksDisposable);
-
-	configuration.registerCommands(context);
+	let commentBlocksDisposables: vscode.Disposable[] = [];
 
+	// Initial configuration
+	commentBlocksDisposables = configuration.configureCommentBlocks();
+	disposables.push(...commentBlocksDisposables);
 
-	let disabledLangConfig: string[] = configuration.getConfigurationValue("disabledLanguages");
+	configuration.registerCommands(context);
 
+	// Show disabled languages message
+	const disabledLangConfig: string[] = configuration.getConfigurationValue("disabledLanguages");
 	if (disabledLangConfig.length > 0) {
 		vscode.window.showInformationMessage(`${disabledLangConfig.join(", ")} languages are disabled for ${extensionDisplayName}.`);
 	}
@@ -85,8 +87,14 @@ export function activate(context: vscode.ExtensionContext) {
 	 */
 	const documentOpenDisposable = vscode.workspace.onDidOpenTextDocument(() => {
 		logger.info("Active editor language changed, re-configuring comment blocks.");
-		const configureCommentBlocksDisposable = configuration.configureCommentBlocks();
-		disposables.push(...configureCommentBlocksDisposable);
+
+		// Dispose of old comment block configurations to prevent memory leaks
+		commentBlocksDisposables.forEach((disposable) => disposable.dispose());
+		commentBlocksDisposables = [];
+
+		// Create new comment block configurations
+		commentBlocksDisposables = configuration.configureCommentBlocks();
+		disposables.push(...commentBlocksDisposables);
 	});
 
 	disposables.push(documentOpenDisposable);