@juspay/neurolink 7.14.4 → 7.14.5

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## [7.14.5](https://github.com/juspay/neurolink/compare/v7.14.4...v7.14.5) (2025-08-18)
+
+### Bug Fixes
+
+- **(mcp):** prevent memory leak from uncleared interval timer in MCPCircuitBreaker ([1f2ae47](https://github.com/juspay/neurolink/commit/1f2ae4743dc8657baac9ba28a053c4e9d199cdbc))
+
 ## [7.14.4](https://github.com/juspay/neurolink/compare/v7.14.3...v7.14.4) (2025-08-18)
 
 ### Bug Fixes

package/dist/lib/mcp/mcpCircuitBreaker.d.ts

@@ -91,6 +91,7 @@ export declare class MCPCircuitBreaker extends EventEmitter {
     private lastFailureTime;
     private halfOpenCalls;
     private lastStateChange;
+    private cleanupTimer?;
     constructor(name: string, config?: Partial<CircuitBreakerConfig>);
     /**
      * Execute an operation with circuit breaker protection
@@ -144,6 +145,12 @@ export declare class MCPCircuitBreaker extends EventEmitter {
      * Check if circuit is half-open
      */
     isHalfOpen(): boolean;
+    /**
+     * Destroy the circuit breaker and clean up resources
+     * This method should be called when the circuit breaker is no longer needed
+     * to prevent memory leaks from the cleanup timer
+     */
+    destroy(): void;
 }
 /**
  * Circuit breaker manager for multiple circuit breakers
@@ -155,7 +162,7 @@ export declare class CircuitBreakerManager {
      */
     getBreaker(name: string, config?: Partial<CircuitBreakerConfig>): MCPCircuitBreaker;
     /**
-     * Remove a circuit breaker
+     * Remove a circuit breaker and clean up its resources
      */
     removeBreaker(name: string): boolean;
     /**
@@ -180,5 +187,10 @@ export declare class CircuitBreakerManager {
         halfOpenBreakers: number;
         unhealthyBreakers: string[];
     };
+    /**
+     * Destroy all circuit breakers and clean up their resources
+     * This should be called during application shutdown to prevent memory leaks
+     */
+    destroyAll(): void;
 }
 export declare const globalCircuitBreakerManager: CircuitBreakerManager;

package/dist/lib/mcp/mcpCircuitBreaker.js

@@ -17,6 +17,8 @@ export class MCPCircuitBreaker extends EventEmitter {
     lastFailureTime = 0;
     halfOpenCalls = 0;
     lastStateChange = new Date();
+    // Store the cleanup timer reference for proper cleanup
+    cleanupTimer;
     constructor(name, config = {}) {
         super();
         this.name = name;
@@ -29,8 +31,8 @@ export class MCPCircuitBreaker extends EventEmitter {
             minimumCallsBeforeCalculation: config.minimumCallsBeforeCalculation ?? 10,
             statisticsWindowSize: config.statisticsWindowSize ?? 300000, // 5 minutes
         };
-        // Clean up old call records periodically
-        setInterval(() => this.cleanupCallHistory(), 60000);
+        // Clean up old call records periodically - now storing the timer reference
+        this.cleanupTimer = setInterval(() => this.cleanupCallHistory(), 60000);
     }
     /**
      * Execute an operation with circuit breaker protection
@@ -250,6 +252,24 @@ export class MCPCircuitBreaker extends EventEmitter {
     isHalfOpen() {
         return this.state === "half-open";
     }
+    /**
+     * Destroy the circuit breaker and clean up resources
+     * This method should be called when the circuit breaker is no longer needed
+     * to prevent memory leaks from the cleanup timer
+     */
+    destroy() {
+        // Clear the interval timer to prevent memory leaks
+        if (this.cleanupTimer) {
+            clearInterval(this.cleanupTimer);
+            this.cleanupTimer = undefined;
+            mcpLogger.debug(`[CircuitBreaker:${this.name}] Cleanup timer cleared`);
+        }
+        // Clear any remaining event listeners
+        this.removeAllListeners();
+        // Clear call history to free memory
+        this.callHistory = [];
+        mcpLogger.debug(`[CircuitBreaker:${this.name}] Destroyed and cleaned up`);
+    }
 }
 /**
  * Circuit breaker manager for multiple circuit breakers
@@ -268,14 +288,18 @@ export class CircuitBreakerManager {
         return this.breakers.get(name);
     }
     /**
-     * Remove a circuit breaker
+     * Remove a circuit breaker and clean up its resources
      */
     removeBreaker(name) {
-        const removed = this.breakers.delete(name);
-        if (removed) {
-            mcpLogger.debug(`[CircuitBreakerManager] Removed circuit breaker: ${name}`);
+        const breaker = this.breakers.get(name);
+        if (breaker) {
+            // Destroy the breaker to clean up its timer and resources
+            breaker.destroy();
+            this.breakers.delete(name);
+            mcpLogger.debug(`[CircuitBreakerManager] Removed and cleaned up circuit breaker: ${name}`);
+            return true;
         }
-        return removed;
+        return false;
     }
     /**
      * Get all circuit breaker names
@@ -333,6 +357,17 @@ export class CircuitBreakerManager {
             unhealthyBreakers,
         };
     }
+    /**
+     * Destroy all circuit breakers and clean up their resources
+     * This should be called during application shutdown to prevent memory leaks
+     */
+    destroyAll() {
+        for (const breaker of this.breakers.values()) {
+            breaker.destroy();
+        }
+        this.breakers.clear();
+        mcpLogger.info("[CircuitBreakerManager] Destroyed all circuit breakers");
+    }
 }
 // Global circuit breaker manager instance
 export const globalCircuitBreakerManager = new CircuitBreakerManager();

package/dist/mcp/mcpCircuitBreaker.d.ts

@@ -91,6 +91,7 @@ export declare class MCPCircuitBreaker extends EventEmitter {
     private lastFailureTime;
     private halfOpenCalls;
     private lastStateChange;
+    private cleanupTimer?;
     constructor(name: string, config?: Partial<CircuitBreakerConfig>);
     /**
      * Execute an operation with circuit breaker protection
@@ -144,6 +145,12 @@ export declare class MCPCircuitBreaker extends EventEmitter {
      * Check if circuit is half-open
      */
     isHalfOpen(): boolean;
+    /**
+     * Destroy the circuit breaker and clean up resources
+     * This method should be called when the circuit breaker is no longer needed
+     * to prevent memory leaks from the cleanup timer
+     */
+    destroy(): void;
 }
 /**
  * Circuit breaker manager for multiple circuit breakers
@@ -155,7 +162,7 @@ export declare class CircuitBreakerManager {
      */
     getBreaker(name: string, config?: Partial<CircuitBreakerConfig>): MCPCircuitBreaker;
     /**
-     * Remove a circuit breaker
+     * Remove a circuit breaker and clean up its resources
      */
     removeBreaker(name: string): boolean;
     /**
@@ -180,5 +187,10 @@ export declare class CircuitBreakerManager {
         halfOpenBreakers: number;
         unhealthyBreakers: string[];
     };
+    /**
+     * Destroy all circuit breakers and clean up their resources
+     * This should be called during application shutdown to prevent memory leaks
+     */
+    destroyAll(): void;
 }
 export declare const globalCircuitBreakerManager: CircuitBreakerManager;

package/dist/mcp/mcpCircuitBreaker.js

@@ -17,6 +17,8 @@ export class MCPCircuitBreaker extends EventEmitter {
     lastFailureTime = 0;
     halfOpenCalls = 0;
     lastStateChange = new Date();
+    // Store the cleanup timer reference for proper cleanup
+    cleanupTimer;
     constructor(name, config = {}) {
         super();
         this.name = name;
@@ -29,8 +31,8 @@ export class MCPCircuitBreaker extends EventEmitter {
             minimumCallsBeforeCalculation: config.minimumCallsBeforeCalculation ?? 10,
             statisticsWindowSize: config.statisticsWindowSize ?? 300000, // 5 minutes
         };
-        // Clean up old call records periodically
-        setInterval(() => this.cleanupCallHistory(), 60000);
+        // Clean up old call records periodically - now storing the timer reference
+        this.cleanupTimer = setInterval(() => this.cleanupCallHistory(), 60000);
     }
     /**
      * Execute an operation with circuit breaker protection
@@ -250,6 +252,24 @@ export class MCPCircuitBreaker extends EventEmitter {
     isHalfOpen() {
         return this.state === "half-open";
     }
+    /**
+     * Destroy the circuit breaker and clean up resources
+     * This method should be called when the circuit breaker is no longer needed
+     * to prevent memory leaks from the cleanup timer
+     */
+    destroy() {
+        // Clear the interval timer to prevent memory leaks
+        if (this.cleanupTimer) {
+            clearInterval(this.cleanupTimer);
+            this.cleanupTimer = undefined;
+            mcpLogger.debug(`[CircuitBreaker:${this.name}] Cleanup timer cleared`);
+        }
+        // Clear any remaining event listeners
+        this.removeAllListeners();
+        // Clear call history to free memory
+        this.callHistory = [];
+        mcpLogger.debug(`[CircuitBreaker:${this.name}] Destroyed and cleaned up`);
+    }
 }
 /**
  * Circuit breaker manager for multiple circuit breakers
@@ -268,14 +288,18 @@ export class CircuitBreakerManager {
         return this.breakers.get(name);
     }
     /**
-     * Remove a circuit breaker
+     * Remove a circuit breaker and clean up its resources
      */
     removeBreaker(name) {
-        const removed = this.breakers.delete(name);
-        if (removed) {
-            mcpLogger.debug(`[CircuitBreakerManager] Removed circuit breaker: ${name}`);
+        const breaker = this.breakers.get(name);
+        if (breaker) {
+            // Destroy the breaker to clean up its timer and resources
+            breaker.destroy();
+            this.breakers.delete(name);
+            mcpLogger.debug(`[CircuitBreakerManager] Removed and cleaned up circuit breaker: ${name}`);
+            return true;
         }
-        return removed;
+        return false;
     }
     /**
      * Get all circuit breaker names
@@ -333,6 +357,17 @@ export class CircuitBreakerManager {
             unhealthyBreakers,
         };
     }
+    /**
+     * Destroy all circuit breakers and clean up their resources
+     * This should be called during application shutdown to prevent memory leaks
+     */
+    destroyAll() {
+        for (const breaker of this.breakers.values()) {
+            breaker.destroy();
+        }
+        this.breakers.clear();
+        mcpLogger.info("[CircuitBreakerManager] Destroyed all circuit breakers");
+    }
 }
 // Global circuit breaker manager instance
 export const globalCircuitBreakerManager = new CircuitBreakerManager();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@juspay/neurolink",
-  "version": "7.14.4",
+  "version": "7.14.5",
   "description": "Universal AI Development Platform with working MCP integration, multi-provider support, and professional CLI. Built-in tools operational, 58+ external MCP servers discoverable. Connect to filesystem, GitHub, database operations, and more. Build, test, and deploy AI applications with 9 major providers: OpenAI, Anthropic, Google AI, AWS Bedrock, Azure, Hugging Face, Ollama, and Mistral AI.",
   "author": {
     "name": "Juspay Technologies",