@exodus/assets-feature 4.0.0 → 4.0.2

package/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [4.0.2](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@4.0.1...@exodus/assets-feature@4.0.2) (2024-01-11)
+
+### Bug Fixes
+
+- don't use monitor for unsupported asset ([#5315](https://github.com/ExodusMovement/exodus-hydra/issues/5315)) ([6432c86](https://github.com/ExodusMovement/exodus-hydra/commit/6432c863e5547ba1a74d730082a9e2e450729d0d))
+- missing dependencies ([#5322](https://github.com/ExodusMovement/exodus-hydra/issues/5322)) ([01efedc](https://github.com/ExodusMovement/exodus-hydra/commit/01efedc7508fb14925277fdcd388afb721ac3dd1))
+
+## [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/client/asset-client-interface.js

@@ -1,6 +1,7 @@
 import { pickBy } from '@exodus/basic-utils'
 import { isEmpty } from 'lodash'
 import assert from 'minimalistic-assert'
+import { filterAsync } from '@exodus/basic-utils/lib/async'
 
 class AssetClientInterface {
   #availableAssetNamesAtom
@@ -119,7 +120,16 @@ class AssetClientInterface {
     // In the future, the list of wallets may be different based on the provided assets
     const asset = this.assetsModule.getAsset(assetName)
     assert(asset, `${assetName} is not supported`)
-    return Object.keys(await this.#enabledWalletAccountsAtom.get())
+    const enabledWalletAccounts = await this.#enabledWalletAccountsAtom.get()
+    return filterAsync(Object.keys(enabledWalletAccounts), async (walletAccount) => {
+      try {
+        // hardware wallet account may not support coin, to check this get address
+        await this.getReceiveAddress({ assetName, walletAccount, useCache: true })
+        return true
+      } catch {
+        return false
+      }
+    })
   }
 
   // assets interface

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.2",
   "license": "UNLICENSED",
   "description": "Assets module, clients and apis",
   "main": "index.js",
@@ -31,8 +31,11 @@
     "url": "git+https://github.com/ExodusMovement/exodus-hydra.git"
   },
   "dependencies": {
+    "@exodus/assets": "^8.0.95",
+    "@exodus/atoms": "^7.0.1",
     "@exodus/basic-utils": "^2.1.0",
     "@exodus/fetch": "^1.3.0",
+    "@exodus/module": "^1.2.2",
     "@exodus/timer": "^1.0.0",
     "lodash": "^4.17.21",
     "minimalistic-assert": "^1.0.1",
@@ -40,8 +43,6 @@
     "reselect": "^3.0.1"
   },
   "devDependencies": {
-    "@exodus/assets": "^8.0.95",
-    "@exodus/atoms": "^6.0.1",
     "@exodus/available-assets": "^8.0.0",
     "@exodus/bitcoin-meta": "^1.0.1",
     "@exodus/bitcoin-plugin": "^1.0.3",
@@ -51,11 +52,10 @@
     "@exodus/cosmos-plugin": "^1.0.0",
     "@exodus/ethereum-meta": "^1.1.0",
     "@exodus/models": "^9.1.1",
-    "@exodus/module": "^1.2.2",
     "@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.1.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": "20a9f63a04f28b3da5c32c4507ba023ed81394dc"
 }