@exodus/assets-feature 5.11.0 → 5.11.2

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [5.11.2](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@5.11.1...@exodus/assets-feature@5.11.2) (2024-09-05)
+
+### Bug Fixes
+
+- addRemoteTokens can add builtIn token as CT ([#8879](https://github.com/ExodusMovement/exodus-hydra/issues/8879)) ([2597011](https://github.com/ExodusMovement/exodus-hydra/commit/2597011ed1cd88755d174953e90afe467ecaa689))
+
+## [5.11.1](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@5.11.0...@exodus/assets-feature@5.11.1) (2024-08-26)
+
+### Bug Fixes
+
+- conditional customTokensMonitor creation ([#8269](https://github.com/ExodusMovement/exodus-hydra/issues/8269)) ([622bca7](https://github.com/ExodusMovement/exodus-hydra/commit/622bca7ae230ae302e9589ac00f3f7c8d5809a79))
+
 ## [5.11.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@5.10.1...@exodus/assets-feature@5.11.0) (2024-08-20)
 
 ### Features

package/index.js

@@ -34,7 +34,7 @@ const assets = ({ config = {} } = {}) => {
         aliases: [{ implementationId: 'customTokensStorage', interfaceId: 'storage' }],
       },
       { definition: assetPreferencesDefinition },
-      { definition: customTokensMonitorDefinition },
+      { if: { registered: ['customTokensStorage'] }, definition: customTokensMonitorDefinition },
     ],
   }
 }

package/module/assets-module.js

@@ -5,7 +5,7 @@ import {
   CT_STATUS as STATUS,
   CT_UPDATEABLE_PROPERTIES,
 } from '@exodus/assets'
-import { mapValues, pick, pickBy } from '@exodus/basic-utils'
+import { keyBy, mapValues, partition, pick, pickBy } from '@exodus/basic-utils'
 // eslint-disable-next-line @exodus/restricted-imports/prefer-basic-utils
 import lodash from 'lodash'
 import assert from 'minimalistic-assert'
@@ -19,7 +19,7 @@ import {
 import { getAssetFromAssetId, getFetchErrorMessage, isDisabledCustomToken } from './utils.js'
 import createFetchival from '@exodus/fetch/create-fetchival'
 
-const { get, isEmpty, keyBy, once } = lodash
+const { get, isEmpty, once } = lodash
 
 const FILTERED_FIELDS = [
   'assetId',
@@ -304,20 +304,27 @@ export class AssetsModule {
     this.#assertCustomTokensStorageSupported()
 
     const assets = this.getAssets()
-    const tokenNamesToFetch = tokenNames.filter((tokenName) => !assets[tokenName])
+    const [tokenNamesToUpdate, tokenNamesToFetch] = partition(
+      tokenNames,
+      (tokenName) => !!assets[tokenName]
+    )
 
-    const tokensToAdd =
+    const fetchedTokens =
       tokenNamesToFetch.length > 0
         ? await this.#fetch('tokens', { tokenNames: tokenNamesToFetch }, 'tokens')
         : []
-    const validTokensToAdd = tokensToAdd.filter(this.#validateCustomToken).map(normalizeToken)
-    if (validTokensToAdd.length !== tokensToAdd.length)
+
+    const validTokens = fetchedTokens.filter(this.#validateCustomToken).map(normalizeToken)
+    if (validTokens.length !== fetchedTokens.length)
       this.#logger.warn('Invalid Custom Token schema')
 
-    const tokensToUpdate = tokenNames
-      .map((tokenName) => assets[tokenName])
-      .filter((token) => !!token)
-    const tokens = [...validTokensToAdd, ...tokensToUpdate]
+    const tokens = tokenNamesToUpdate.map((tokenName) => assets[tokenName])
+
+    validTokens.forEach((token) => {
+      // replace fetched CT with existing built-in token if found by assetId
+      const asset = token.assetId && this.#getAssetFromAssetId(token.assetId, token.baseAssetName)
+      tokens.push(asset ?? token)
+    })
 
     if (isEmpty(tokens)) return
 
@@ -455,7 +462,7 @@ export class AssetsModule {
     const asset = this.getAsset(token.name)
     if (asset) return { asset, isAdded: false }
 
-    const { name } = this.#registry.addCustomToken(token) // add to `#/assets`
+    const { name } = this.#registry.addCustomToken(token) // add to registry
     this.#logger.log('Custom token added:', name)
     return { asset: this.getAsset(name), isAdded: true }
   }

package/module/utils.js

@@ -1,13 +1,16 @@
 import { CT_STATUS as STATUS } from '@exodus/assets'
 import lodash from 'lodash'
+import assert from 'minimalistic-assert'
 
 const { get } = lodash
 
 const BLACKLISTED_ERRORS = [/Cannot destructure property 'owner'/]
 
 export const getAssetFromAssetId = (assets, assetId, baseAssetName) => {
+  assert(assetId, 'getAssetFromAssetId(): assetId is required')
+  assert(baseAssetName, 'getAssetFromAssetId(): baseAssetName is required')
   // ignore letter-case for assets that support checksummed addresses
-  assetId = assetId?.toLowerCase()
+  assetId = assetId.toLowerCase()
   return Object.values(assets).find(
     (asset) => assetId === asset.assetId?.toLowerCase() && baseAssetName === asset.baseAsset.name
   )

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/assets-feature",
-  "version": "5.11.0",
+  "version": "5.11.2",
   "license": "UNLICENSED",
   "description": "Assets module, clients and apis",
   "type": "module",
@@ -45,13 +45,13 @@
     "reselect": "^3.0.1"
   },
   "devDependencies": {
-    "@exodus/available-assets": "^8.4.1",
+    "@exodus/available-assets": "^8.5.0",
     "@exodus/bip44-constants": "^195.0.0",
     "@exodus/bitcoin-meta": "^2.0.0",
     "@exodus/bitcoin-plugin": "^1.0.3",
     "@exodus/bitcoinregtest-plugin": "^1.0.3",
     "@exodus/bitcointestnet-plugin": "^1.0.3",
-    "@exodus/blockchain-metadata": "^15.3.5",
+    "@exodus/blockchain-metadata": "^15.4.0",
     "@exodus/cardano-lib": "^2.2.0",
     "@exodus/combined-assets-meta": "^3.0.0",
     "@exodus/cosmos-plugin": "^1.0.0",
@@ -60,16 +60,16 @@
     "@exodus/keychain": "^6.9.1",
     "@exodus/models": "^12.0.1",
     "@exodus/osmosis-plugin": "^1.0.0",
-    "@exodus/public-key-provider": "^2.4.1",
+    "@exodus/public-key-provider": "^2.5.0",
     "@exodus/public-key-store": "^1.2.2",
-    "@exodus/redux-dependency-injection": "^4.0.0",
-    "@exodus/storage-memory": "^2.1.1",
-    "@exodus/wallet-accounts": "^16.9.0",
+    "@exodus/redux-dependency-injection": "^4.0.2",
+    "@exodus/storage-memory": "^2.2.0",
+    "@exodus/wallet-accounts": "^16.10.1",
     "@exodus/wild-emitter": "^1.0.0",
     "bip39": "^3.1.0",
     "events": "^3.3.0",
     "msw": "^2.0.0",
     "redux": "^4.0.0"
   },
-  "gitHead": "6a72d364b98c73b79483e114ebaf8d3e757246c9"
+  "gitHead": "a758fd2522b6a4b638f10a340cf87a6d64c814de"
 }

package/plugin/index.js

@@ -60,11 +60,11 @@ const createAssetsPlugin = ({
   }
 
   const onUnlock = () => {
-    customTokensMonitor.start()
+    customTokensMonitor?.start()
   }
 
   const onLock = () => {
-    customTokensMonitor.stop()
+    customTokensMonitor?.stop()
   }
 
   const onStop = () => {
@@ -87,7 +87,7 @@ const assetsPluginDefinition = {
     'port',
     'assetsModule',
     'assetsAtom',
-    'customTokensMonitor',
+    'customTokensMonitor?',
     'disabledPurposesAtom',
     'multiAddressModeAtom',
   ],