fix: auto patch for api_provider_clients

Author: kiwina

src/api/index.ts

@@ -56,6 +56,11 @@ export interface ApiHandler {
 	 * @returns A promise resolving to the token count
 	 */
 	countTokens(content: Array<Anthropic.Messages.ContentBlockParam>): Promise<number>
+
+	/**
+	 * Optional method to dispose of any resources held by the API handler.
+	 */
+	dispose?: () => void
 }
 
 export function buildApiHandler(configuration: ProviderSettings): ApiHandler {

src/api/providers/base-provider.ts

@@ -10,6 +10,8 @@ import { countTokens } from "../../utils/countTokens"
  * Base class for API providers that implements common functionality.
  */
 export abstract class BaseProvider implements ApiHandler {
+	protected client: any // Added protected client field
+
 	abstract createMessage(
 		systemPrompt: string,
 		messages: Anthropic.Messages.MessageParam[],
@@ -32,4 +34,23 @@ export abstract class BaseProvider implements ApiHandler {
 
 		return countTokens(content, { useWorker: true })
 	}
+
+	/**
+	 * Disposes of any resources held by the provider.
+	 * Attempts common disposal methods on the client if it exists.
+	 */
+	public dispose(): void {
+		if (this.client) {
+			// Try common disposal methods that SDKs might have
+			if (typeof (this.client as any).close === "function") {
+				;(this.client as any).close()
+			} else if (typeof (this.client as any).destroy === "function") {
+				;(this.client as any).destroy()
+			} else if (typeof (this.client as any).dispose === "function") {
+				;(this.client as any).dispose()
+			}
+			// Clear the reference
+			this.client = undefined
+		}
+	}
 }

src/core/task/Task.ts

@@ -1001,6 +1001,7 @@ export class Task extends EventEmitter<ClineEvents> {
 		this.browserSession.closeBrowser()
 		this.rooIgnoreController?.dispose()
 		this.fileContextTracker.dispose()
+		this.api?.dispose?.() // Added this line
 
 		// If we're not streaming then `abortStream` (which reverts the diff
 		// view changes) won't be called, so we need to revert the changes here.
@@ -1012,6 +1013,25 @@ export class Task extends EventEmitter<ClineEvents> {
 		await this.saveClineMessages()
 	}
 
+	// Added new dispose method as per leak report suggestion
+	public dispose(): void {
+		console.log(`[subtasks] disposing task ${this.taskId}.${this.instanceId}`)
+		this.abortTask(true) // Call abortTask to ensure all resources are released
+
+		// Explicitly call dispose on the api handler again, in case abortTask didn't catch it
+		// or if dispose is called directly without abortTask.
+		this.api?.dispose?.()
+
+		// Clear any other task-specific resources if they weren't handled by abortTask
+		if (this.pauseInterval) {
+			clearInterval(this.pauseInterval)
+			this.pauseInterval = undefined
+		}
+		// Ensure other disposables are handled if not already by abortTask
+		// this.rooIgnoreController?.dispose(); // Already called in abortTask
+		// this.fileContextTracker.dispose(); // Already called in abortTask
+	}
+
 	// Used when a sub-task is launched and the parent task is waiting for it to
 	// finish.
 	// TBD: The 1s should be added to the settings, also should add a timeout to