@icgio/icg-exchanges-wrapper 1.17.24 → 1.17.26

package/functions/get-exchanges-rates-market-history-ohlcv.js

@@ -162,9 +162,6 @@ const get_exchanges_rates = function (Exchanges, exchange_pair_dict, cb, ws_enab
               return
             }
 
-            // Store promise before starting connection
-            ws_connection_promises[ws_key] = ws_promise
-
             // Create WebSocket with ALL pairs collected in first pass
             // This ensures only ONE WebSocket per exchange, even with multiple clients/pairs
             Exchange.ws_market(all_pairs)
@@ -308,19 +305,12 @@ const get_exchanges_rates = function (Exchanges, exchange_pair_dict, cb, ws_enab
 
             check_status()
           })
-          // Clean up promise reference when done (but keep connection open for reuse)
-          ws_promise.finally(() => {
-            // Don't delete immediately - keep for a short time to allow reuse
-            setTimeout(() => {
-              if (ws_connection_promises[ws_key] === ws_promise) {
-                delete ws_connection_promises[ws_key]
-                // Also clean up wrapped promise if it exists
-                if (ws_connection_promises_with_fallback[ws_key]) {
-                  delete ws_connection_promises_with_fallback[ws_key]
-                }
-              }
-            }, 1000)
-          })
+          // Store the cached promise AFTER `new Promise(...)` returns — assigning inside
+          // the synchronous executor reads the outer binding while it's still undefined.
+          // Keep the entry alive while the socket is alive; the next tick's
+          // `is_opening_or_opened` check reads `Exchange.ws.market.status` and the
+          // else-branch above overwrites this slot when the socket is closed/stopped.
+          ws_connection_promises[ws_key] = ws_promise
         }
 
         // Wrap WebSocket promise to handle rejection and fall back to REST
@@ -445,9 +435,6 @@ const get_exchanges_market_history = function (Exchanges, exchange_pair_dict, ti
               return
             }
 
-            // Store promise before starting connection
-            ws_market_history_promises[ws_key] = ws_promise
-
             // Only start if not already started (reuse connection from rates if available)
             // If starting new, use ALL pairs collected in first pass
             if (!first_time[Exchange_name]) {
@@ -589,18 +576,11 @@ const get_exchanges_market_history = function (Exchanges, exchange_pair_dict, ti
             }
             check_status()
           })
-          // Clean up promise reference when done (but keep connection open for reuse)
-          ws_promise.finally(() => {
-            setTimeout(() => {
-              if (ws_market_history_promises[ws_key] === ws_promise) {
-                delete ws_market_history_promises[ws_key]
-                // Also clean up wrapped promise if it exists
-                if (ws_market_history_promises_with_fallback[ws_key]) {
-                  delete ws_market_history_promises_with_fallback[ws_key]
-                }
-              }
-            }, 1000)
-          })
+          // Store the cached promise AFTER `new Promise(...)` returns — assigning inside
+          // the synchronous executor reads the outer binding while it's still undefined.
+          // Keep the entry alive while the socket is alive; the else-branch above
+          // overwrites this slot when the socket is closed/stopped.
+          ws_market_history_promises[ws_key] = ws_promise
         }
 
         // If WebSocket promise rejects (timeout/error), fall back to REST for all pairs

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@icgio/icg-exchanges-wrapper",
-  "version": "1.17.24",
+  "version": "1.17.26",
   "description": "",
   "main": "index.js",
   "scripts": {