@exodus/assets-feature 4.0.0 → 4.0.1

package/CHANGELOG.md

@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [4.0.1](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@4.0.0...@exodus/assets-feature@4.0.1) (2023-12-13)
+
+- refactor: cache CTR request failures (https://github.com/ExodusMovement/exodus-hydra/pull/4975)
+
 ## [4.0.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@3.8.0...@exodus/assets-feature@4.0.0) (2023-11-23)
 
 ### ⚠ BREAKING CHANGES

package/index.js

@@ -33,12 +33,10 @@ const assets = ({ config = {} } = {}) => {
       {
         definition: assetModuleDefinition,
         // storage: { namespace: 'customTokens' },
-        writesAtoms: ['assetsAtom'],
         aliases: [{ implementationId: 'customTokensStorage', interfaceId: 'storage' }],
       },
       {
         definition: assetPreferencesDefinition,
-        writesAtoms: ['multiAddressModeAtom', 'disabledPurposesAtom'],
       },
       { definition: customTokensMonitorDefinition },
     ],

package/module/assets-module.js

@@ -185,7 +185,7 @@ export class AssetsModule {
 
       // tokens may be:
       // 1. pure custom tokens from the CTR
-      // 2. built-in tokens previously from the CTR
+      // 2. built-in tokens previously from the CTR which may or may not be available
       // 3. both of the above from a network that has feature flag customTokens === false
       // It is OK to add all of these to the assets registry
       const { added, updated } = this.#handleFetchedTokens(Object.values(tokens))
@@ -208,12 +208,27 @@ export class AssetsModule {
 
     const key = getFetchCacheKey(baseAssetName, assetId)
     const cache = this.#getCache(key)
+    if (cache?.cachedError) throw cache.cachedError
     if (cache) return cache
 
-    const _token = await this.#fetch(`networks/${baseAssetName}`, { assetId }, 'token')
+    const fetchTokenAndCacheError = async () => {
+      try {
+        return await this.#fetch(`networks/${baseAssetName}`, { assetId }, 'token')
+      } catch (err) {
+        if (err.message !== 'Too Many Requests') {
+          this.#setCache(key, { cachedError: err })
+        }
+
+        throw err
+      }
+    }
+
+    const _token = await fetchTokenAndCacheError()
     if (!this.#validateCustomToken(_token)) {
       this.#logger.warn('Invalid Custom Token schema')
-      throw new Error('Token did not pass validation')
+      const err = new Error('Token did not pass validation')
+      this.#setCache(key, { cachedError: err })
+      throw err
     }
 
     const token = normalizeToken(_token)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/assets-feature",
-  "version": "4.0.0",
+  "version": "4.0.1",
   "license": "UNLICENSED",
   "description": "Assets module, clients and apis",
   "main": "index.js",
@@ -41,7 +41,7 @@
   },
   "devDependencies": {
     "@exodus/assets": "^8.0.95",
-    "@exodus/atoms": "^6.0.1",
+    "@exodus/atoms": "^7.0.0",
     "@exodus/available-assets": "^8.0.0",
     "@exodus/bitcoin-meta": "^1.0.1",
     "@exodus/bitcoin-plugin": "^1.0.3",
@@ -55,7 +55,7 @@
     "@exodus/osmosis-plugin": "^1.0.0",
     "@exodus/redux-dependency-injection": "^3.0.0",
     "@exodus/storage-memory": "^2.1.1",
-    "@exodus/wallet-accounts": "^14.2.0",
+    "@exodus/wallet-accounts": "^15.0.0",
     "@exodus/wild-emitter": "^1.0.0",
     "eslint": "^8.44.0",
     "events": "^3.3.0",
@@ -63,5 +63,5 @@
     "msw": "^2.0.0",
     "redux": "^4.0.0"
   },
-  "gitHead": "6fe52cd3aa2ffc3fa76d2334228a0dcd02dc9e2e"
+  "gitHead": "8a27fc57a2783ecc728030da8b3d2a10fa8f915c"
 }