@exodus/assets-feature 5.11.1 → 5.11.2

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.
 
+## [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

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.1",
+  "version": "5.11.2",
   "license": "UNLICENSED",
   "description": "Assets module, clients and apis",
   "type": "module",
@@ -62,14 +62,14 @@
     "@exodus/osmosis-plugin": "^1.0.0",
     "@exodus/public-key-provider": "^2.5.0",
     "@exodus/public-key-store": "^1.2.2",
-    "@exodus/redux-dependency-injection": "^4.0.1",
+    "@exodus/redux-dependency-injection": "^4.0.2",
     "@exodus/storage-memory": "^2.2.0",
-    "@exodus/wallet-accounts": "^16.10.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": "08a439da808ec039338b0ca1b448b39267d131fb"
+  "gitHead": "a758fd2522b6a4b638f10a340cf87a6d64c814de"
 }