fixup! Hack support for non-USD targetFiat

Author: peachbits

src/v3/router.ts

@@ -138,30 +138,37 @@ const getNonUsdRates = async (
   // Loop over the USD rates and store them in a map. At the same time,
   // save the date strings we'll need to query the original requested fiat for
   const usdCryptoRatesMap = new Map<string, number | undefined>()
-  const fiatUsdDateExchangeRateMap = new Map<string, number | undefined>()
+  const dateSet = new Set<string>()
   for (const crypto of usdResult.crypto) {
     usdCryptoRatesMap.set(toDatedCryptoKey(crypto), crypto.rate)
-    fiatUsdDateExchangeRateMap.set(crypto.isoDate.toISOString(), undefined)
+    dateSet.add(crypto.isoDate.toISOString())
   }
   const usdFiatRatesMap = new Map<string, number | undefined>()
   for (const fiat of usdResult.fiat) {
     usdFiatRatesMap.set(toDatedFiatKey(fiat), fiat.rate)
-    fiatUsdDateExchangeRateMap.set(fiat.isoDate.toISOString(), undefined)
+    dateSet.add(fiat.isoDate.toISOString())
   }
 
   // Get fiat/USD rates
-  const fiatParams = {
-    targetFiat: 'USD',
-    crypto: [],
-    fiat: [...fiatUsdDateExchangeRateMap.keys()].map(date => ({
-      isoDate: new Date(date),
-      fiatCode: initialParams.targetFiat,
-      rate: undefined
-    }))
-  }
-  const fiatResult = await getRates(fiatParams, rightNow)
-  for (const fiat of fiatResult.fiat) {
-    fiatUsdDateExchangeRateMap.set(fiat.isoDate.toISOString(), fiat.rate)
+  // The number of unique dates across crypto and fiat could exceed 256 so we need to chunk them
+  const dateArray = Array.from(dateSet)
+  const chunkSize = FIAT_LIMIT
+  const fiatUsdDateExchangeRateMap = new Map<string, number | undefined>()
+  for (let i = 0; i < dateArray.length; i += chunkSize) {
+    const chunk = dateArray.slice(i, i + chunkSize)
+    const fiatParams = {
+      targetFiat: 'USD',
+      crypto: [],
+      fiat: chunk.map(date => ({
+        isoDate: new Date(date),
+        fiatCode: initialParams.targetFiat,
+        rate: undefined
+      }))
+    }
+    const fiatResult = await getRates(fiatParams, rightNow)
+    for (const fiat of fiatResult.fiat) {
+      fiatUsdDateExchangeRateMap.set(fiat.isoDate.toISOString(), fiat.rate)
+    }
   }
 
   // Loop over the initial request and bridge the rates