@exodus/market-history 10.2.3 → 10.2.4

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.2.4](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/market-history@10.2.3...@exodus/market-history@10.2.4) (2025-06-17)
+
+### Performance
+
+- perf(market-history): Optimize price transformation logic (#12932)
+
 ## [10.2.3](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/market-history@10.2.2...@exodus/market-history@10.2.3) (2025-06-16)
 
 ### Performance

package/module/index.js

@@ -1,4 +1,3 @@
-import { mapValues } from '@exodus/basic-utils'
 import dayjs from '@exodus/dayjs'
 import { fetchHistoricalPrices } from '@exodus/price-api'
 import delay from 'delay'
@@ -12,14 +11,6 @@ const CLEAR_MARKET_HISTORY_CACHE_FROM_REMOTE_CONFIG_KEY =
   'clear-market-history-cache-from-remote-config'
 const MARKET_HISTORY_REFRESH_KEY = 'market-history-cache-refresh'
 
-const transformPriceEntries = (entries = []) => {
-  const closePrices = entries.map((entry) => [entry[0], entry[1].close])
-  return Object.fromEntries(closePrices)
-}
-
-const transformPricesByAssetName = (pricesByAssetName) =>
-  mapValues(pricesByAssetName, (entries) => transformPriceEntries(entries))
-
 // when provided cache version from local or remote config is different from stored on device we clear the cache and fetch new prices
 const _invalidateStorageCache = async ({
   storage,
@@ -62,6 +53,36 @@ const filterMapAssetDataFromServer = (data) =>
     .filter((piece) => piece.close !== 0)
     .map((piece) => [piece.time * 1000, { close: piece.close }])
 
+const mapPricesForCache = (pricesMap, assetNames, assets) => {
+  const result = Object.create(null)
+  for (const assetName of assetNames) {
+    const assetPricesMap = pricesMap.get(assets[assetName].ticker)
+    result[assetName] = assetPricesMap ? [...assetPricesMap] : []
+  }
+
+  return result
+}
+
+const transformHistoricalPrices = (pricesMap, assetNames, assets) => {
+  const result = Object.create(null)
+  for (const assetName of assetNames) {
+    const assetPrices = pricesMap.get(assets[assetName].ticker)
+    if (assetPrices) {
+      const closePrices = Object.create(null)
+      // assetPrices is a Map, iterate it directly to avoid creating an intermediate array
+      for (const [time, priceData] of assetPrices.entries()) {
+        closePrices[time] = priceData.close
+      }
+
+      result[assetName] = closePrices
+    } else {
+      result[assetName] = Object.create(null)
+    }
+  }
+
+  return result
+}
+
 const delayWithJitter = (ms, jitter = 0, signal) =>
   delay(Math.floor(ms + Math.random() * jitter), { signal })
 
@@ -229,29 +250,23 @@ class MarketHistoryMonitor {
       requestLimit: requestLimit || this.#granularityRequestLimits[granularity],
     })
 
-    const mapPrices = (pricesMap) => {
-      return Object.fromEntries(
-        assetNames.map((assetName) => {
-          const assetPricesMap = pricesMap.get(assets[assetName].ticker)
-          const value = assetPricesMap ? [...assetPricesMap] : []
-          return [assetName, value]
-        })
-      )
-    }
-
-    const pricesByAssetName = mapPrices(fetchedPricesMap)
-
-    const fullHistoryByAssetName =
-      fetchedPricesMap.size === historicalPricesMap.size
-        ? pricesByAssetName
-        : mapPrices(historicalPricesMap)
-
     if (fetchedPricesMap.size > 0) {
-      await this.#setCache({ currency, granularity, pricesByAssetName })
+      const pricesByAssetNameForCache = mapPricesForCache(fetchedPricesMap, assetNames, assets)
+      await this.#setCache({
+        currency,
+        granularity,
+        pricesByAssetName: pricesByAssetNameForCache,
+      })
     }
 
+    const fullHistoryByAssetName = transformHistoricalPrices(
+      historicalPricesMap,
+      assetNames,
+      assets
+    )
+
     return {
-      prices: transformPricesByAssetName(fullHistoryByAssetName),
+      prices: fullHistoryByAssetName,
       hasNewPrices: fetchedPricesMap.size > 0,
     }
   }
@@ -631,7 +646,9 @@ class MarketHistoryMonitor {
       })
 
       await this.#marketHistoryAtom.set(async (current) => {
-        const transformedPrices = transformPriceEntries([...updatedAssetHistoryMap])
+        const transformedPrices = Object.fromEntries(
+          [...updatedAssetHistoryMap].map(([time, priceData]) => [time, priceData.close])
+        )
         const parsedGranularity = parseGranularity(granularity)
 
         return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/market-history",
-  "version": "10.2.3",
+  "version": "10.2.4",
   "description": "Fetches historical prices for assets",
   "author": "Exodus Movement, Inc.",
   "license": "MIT",
@@ -59,5 +59,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "a75b31501250fd4a5001ee29eaf929c4dead9c44"
+  "gitHead": "724247d832b6972c09ecf018bbeac77d80829457"
 }