prevent duplicate event emissions on flaky connections with multiple sequential reconnect attempts

Author: kuan121

packages/xrpl/HISTORY.md

@@ -5,7 +5,7 @@ Subscribe to [the **xrpl-announce** mailing list](https://groups.google.com/g/xr
 ## Unreleased
 
 ### Fixed
-* Fix event listener accumulation bug where `'connected'` event handlers would fire multiple times after each reconnection. Changed `.on()` to `.once()` in the Client constructor's reconnect handler to prevent listener accumulation.
+* Fix event listener accumulation bug where `'connected'` event handlers would fire multiple times after each reconnection. The fix cleans up stale listeners from previous reconnect attempts to prevent duplicate event emissions on flaky connections with multiple sequential reconnect attempts.
 
 ## 4.6.0 (2026-02-12)
 

packages/xrpl/src/client/index.ts

@@ -262,8 +262,15 @@ class Client extends EventEmitter<EventTypes> {
       this.emit('error', errorCode, errorMessage, data)
     })
 
+    let connectedListener: (() => void) | undefined
     this.connection.on('reconnect', () => {
-      this.connection.once('connected', () => this.emit('connected'))
+      // Clean up any stale listener from previous reconnect attempt
+      if (connectedListener !== undefined) {
+        this.connection.off('connected', connectedListener)
+      }
+
+      connectedListener = (): boolean => this.emit('connected')
+      this.connection.once('connected', connectedListener)
     })
 
     this.connection.on('disconnected', (code: number) => {