fix: prevent duplicate liquidation events by cleaning up old listeners on bot restart

Author: birdbathd

src/bot/index.ts

@@ -461,8 +461,14 @@ logErrorWithTimestamp('⚠️  Position Manager failed to start:', error.message
         logWithTimestamp('ℹ️  Tranche Management disabled for all symbols');
       }
 
-      // Initialize Hunter
-      this.hunter = new Hunter(this.config, this.isHedgeMode);
+      // Initialize Hunter (or reuse existing instance to prevent duplicate listeners)
+      if (!this.hunter) {
+        this.hunter = new Hunter(this.config, this.isHedgeMode);
+      } else {
+        // Remove all old listeners before re-attaching to prevent duplicates
+        this.hunter.removeAllListeners();
+        console.log('[Bot] Removed all old hunter event listeners to prevent duplicates');
+      }
 
       // Inject status broadcaster for order events
       this.hunter.setStatusBroadcaster(this.statusBroadcaster);
@@ -492,6 +498,9 @@ logErrorWithTimestamp('⚠️  Position Manager failed to start:', error.message
         this.statusBroadcaster.logActivity(`Blocked: ${data.symbol} ${data.side} - ${data.blockType}`);
       });
 
+      // Remove old threshold monitor listeners to prevent duplicates
+      thresholdMonitor.removeAllListeners('thresholdUpdate');
+      
       // Listen for threshold updates and broadcast to UI
       thresholdMonitor.on('thresholdUpdate', (thresholdUpdate: any) => {
         this.statusBroadcaster.broadcastThresholdUpdate(thresholdUpdate);