clear debounce on unsubscribe

Author: chipgpt

src/client/index.ts

@@ -245,7 +245,7 @@ export class Client<
     private _cachedKnownTaskTools: Set<string> = new Set();
     private _cachedRequiredTaskTools: Set<string> = new Set();
     private _experimental?: { tasks: ExperimentalClientTasks<RequestT, NotificationT, ResultT> };
-    private _toolListChangedOptions: ToolListChangedOptions | null = null;
+    private _toolListChangedOptions?: ToolListChangedOptions;
     private _toolListChangedDebounceTimer?: ReturnType<typeof setTimeout>;
 
     /**
@@ -822,7 +822,7 @@ export class Client<
             const refreshToolList = async () => {
                 // If autoRefresh is false, call the callback for the notification, but without tools data
                 if (!toolListChangedOptions.autoRefresh) {
-                    toolListChangedOptions.onToolListChanged?.(null, null);
+                    toolListChangedOptions.onToolListChanged(null, null);
                     return;
                 }
 
@@ -834,7 +834,7 @@ export class Client<
                 } catch (e) {
                     error = e instanceof Error ? e : new Error(String(e));
                 }
-                toolListChangedOptions.onToolListChanged?.(error, tools);
+                toolListChangedOptions.onToolListChanged(error, tools);
             };
 
             this.setNotificationHandler(ToolListChangedNotificationSchema, () => {
@@ -854,15 +854,19 @@ export class Client<
         }
         // Reset tool list changed options and remove notification handler
         else {
-            this._toolListChangedOptions = null;
+            this._toolListChangedOptions = undefined;
             this.removeNotificationHandler(ToolListChangedNotificationSchema.shape.method.value);
+            if (this._toolListChangedDebounceTimer) {
+                clearTimeout(this._toolListChangedDebounceTimer);
+                this._toolListChangedDebounceTimer = undefined;
+            }
         }
     }
 
     /**
      * Gets the current tool list changed options
      */
-    public getToolListChangedOptions(): ToolListChangedOptions | null {
+    public getToolListChangedOptions(): ToolListChangedOptions | undefined {
         return this._toolListChangedOptions;
     }