@icgio/icg-exchanges-wrapper 1.17.25 → 1.17.27

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,9 +305,12 @@ const get_exchanges_rates = function (Exchanges, exchange_pair_dict, cb, ws_enab
 
             check_status()
           })
-          // Keep the cached promise as long as the underlying socket is alive.
-          // The next tick's `is_opening_or_opened` check reads `Exchange.ws.market.status`;
-          // when that flips to 'closed'/'stopped' the else-branch below overwrites this slot.
+          // 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
@@ -435,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]) {
@@ -579,9 +576,12 @@ const get_exchanges_market_history = function (Exchanges, exchange_pair_dict, ti
             }
             check_status()
           })
-          // Keep the cached promise as long as the underlying socket is alive.
-          // The next tick's `is_opening_or_opened` check reads `Exchange.ws.market.status`;
-          // when that flips to 'closed'/'stopped' the else-branch below overwrites this slot.
+          // Do NOT cache this promise. `Exchange.market_history_ws(pair, tf, cb)` returns a
+          // filtered/sorted SNAPSHOT of `market_history_dict[pair]` at call time (see icg-exchanges
+          // mexc.js:214), so the result is frozen once the promise resolves. Rerunning the else
+          // branch each poll lets a new `market_history_ws` call pick up trades the WebSocket has
+          // appended since last tick; `first_time[Exchange_name]` still prevents the underlying
+          // WS from being reopened, so there's no reconnect cost.
         }
 
         // 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.25",
+  "version": "1.17.27",
   "description": "",
   "main": "index.js",
   "scripts": {