@exodus/assets-feature 4.1.1 → 5.0.0

package/CHANGELOG.md

@@ -3,6 +3,23 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [5.0.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@4.1.2...@exodus/assets-feature@5.0.0) (2024-03-11)
+
+### ⚠ BREAKING CHANGES
+
+- get public key from `publicKeyProvider` (#5924)
+
+### Code Refactoring
+
+- get public key from `publicKeyProvider` ([#5924](https://github.com/ExodusMovement/exodus-hydra/issues/5924)) ([5f656cf](https://github.com/ExodusMovement/exodus-hydra/commit/5f656cf41503ff59e98dc0259b0e76b30c94a5e2))
+
+## [4.1.2](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@4.1.1...@exodus/assets-feature@4.1.2) (2024-02-20)
+
+### Bug Fixes
+
+- make sure the check does not throw ([#5070](https://github.com/ExodusMovement/exodus-hydra/issues/5070)) ([b145ecc](https://github.com/ExodusMovement/exodus-hydra/commit/b145ecce8d5e2720edc9f422b913268e5f5b8364))
+- native tokens auto-disabling when already in registry ([#5736](https://github.com/ExodusMovement/exodus-hydra/issues/5736)) ([feb9e8c](https://github.com/ExodusMovement/exodus-hydra/commit/feb9e8cf79f43f5be3987108bb2bc307c3648365))
+
 ## [4.1.1](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@4.1.0...@exodus/assets-feature@4.1.1) (2024-01-19)
 
 ### Bug Fixes

package/README.md

@@ -7,3 +7,39 @@ Assets module, clients and apis
 ```sh
 yarn add @exodus/assets-feature
 ```
+
+## Usage
+
+`@exodus/assets-feature` is bundled with `@exodus/headless`. It injects the `assetsAtom` into the IOC, which you can then reference in your feature's dependencies to get all/any particular assets.
+
+```js
+import createExodus from '@exodus/headless'
+
+const container = createExodus({ adapters, config, port })
+const terribleTickersFeatureYouShouldNotUse = () => ({
+  id: 'tickers',
+  definitions: [
+    {
+      definition: {
+        id: 'tickers',
+        type: 'api',
+        factory: ({ assetsAtom }) => ({
+          tickers: {
+            get: async (name) => {
+              const { value: assets } = await assetsAtom.get()
+              return assets[name].ticker
+            },
+          },
+        }),
+
+        // provided by `@exodus/assets-feature`
+        dependencies: ['assetsAtom'],
+      },
+    },
+  ],
+})
+
+container.use(terribleTickersFeatureYouShouldNotUse())
+const exodus = container.resolve()
+exodus.tickers.get('bitcoin').then(console.log) // 'BTC'
+```

package/client/asset-client-interface.js

@@ -8,10 +8,10 @@ class AssetClientInterface {
   #walletAccountsAtom
   #enabledWalletAccountsAtom
   #transactionSigner
+  #publicKeyProvider
 
   constructor({
     blockchainMetadata,
-    wallet,
     walletAccountsAtom,
     enabledWalletAccountsAtom,
     assetsModule,
@@ -19,9 +19,9 @@ class AssetClientInterface {
     addressProvider,
     feeMonitors,
     transactionSigner,
+    publicKeyProvider,
   }) {
     this.blockchainMetadata = blockchainMetadata
-    this.wallet = wallet
     this.#walletAccountsAtom = walletAccountsAtom
     this.#enabledWalletAccountsAtom = enabledWalletAccountsAtom
     this.assetsModule = assetsModule
@@ -29,6 +29,7 @@ class AssetClientInterface {
     this.feeMonitors = feeMonitors
     this.#availableAssetNamesAtom = availableAssetNamesAtom
     this.#transactionSigner = transactionSigner
+    this.#publicKeyProvider = publicKeyProvider
 
     // TODO: remove conditional when clients updated
     if (assetsModule.initialize) {
@@ -172,13 +173,26 @@ class AssetClientInterface {
 
   // addresses interface
 
-  getPublicKey = async ({ assetName, walletAccount, addressIndex = 0, chainIndex = 0 }) => {
+  getPublicKey = async ({
+    assetName,
+    walletAccount,
+    purpose = 44,
+    addressIndex = 0,
+    chainIndex = 0,
+  }) => {
     const walletAccountInstance = await this.#getWalletAccount(walletAccount)
-    return this.wallet.getPublicKey({
+    const asset = this.assetsModule.getAsset(assetName)
+
+    const keyIdentifier = asset.baseAsset.api.getKeyIdentifier({
+      purpose,
       accountIndex: walletAccountInstance.index,
-      baseAssetName: assetName,
-      addressIndex,
       chainIndex,
+      addressIndex,
+    })
+
+    return this.#publicKeyProvider.getPublicKey({
+      walletAccount: walletAccountInstance,
+      keyIdentifier,
     })
   }
 

package/client/index.js

@@ -14,6 +14,7 @@ const assetsClientInterfaceDefinition = {
     'addressProvider',
     'feeMonitors',
     'transactionSigner',
+    'publicKeyProvider',
   ],
 }
 

package/module/assets-module.js

@@ -318,8 +318,8 @@ export class AssetsModule {
 
     if (isEmpty(tokens)) return
 
-    const validTokens = tokensToAdd.filter(this.#validateCustomToken).map(normalizeToken)
-    if (validTokens.length !== tokensToAdd.length) this.#logger.warn('Invalid Custom Token schema')
+    const validTokens = tokens.filter(this.#validateCustomToken).map(normalizeToken)
+    if (validTokens.length !== tokens.length) this.#logger.warn('Invalid Custom Token schema')
 
     const { fetchedAndHandledTokens, added, updated } = this.#handleFetchedTokens(validTokens)
 
@@ -481,8 +481,8 @@ export class AssetsModule {
 
   #assertSupportsCustomTokens = (baseAssetName) => {
     const baseAsset = this.getAsset(baseAssetName)
-    if (!baseAsset.api.hasFeature('customTokens')) {
-      const reason = `BUG: network ${baseAsset.name} does not support custom tokens.`
+    if (!baseAsset?.api?.hasFeature('customTokens')) {
+      const reason = `BUG: network ${baseAsset?.name} does not support custom tokens.`
       this.#logger.warn(reason)
       throw new Error(reason)
     }

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@exodus/assets-feature",
-  "version": "4.1.1",
+  "version": "5.0.0",
   "license": "UNLICENSED",
   "description": "Assets module, clients and apis",
   "main": "index.js",
   "author": "Exodus Movement Inc.",
   "scripts": {
-    "test": "jest",
-    "lint": "eslint . --ignore-path ../../.gitignore",
+    "test": "run -T jest",
+    "lint": "run -T eslint . --ignore-path ../../.gitignore",
     "lint:fix": "yarn lint --fix"
   },
   "files": [
@@ -44,6 +44,7 @@
   },
   "devDependencies": {
     "@exodus/available-assets": "^8.0.0",
+    "@exodus/bip44-constants": "^195.0.0",
     "@exodus/bitcoin-meta": "^1.0.1",
     "@exodus/bitcoin-plugin": "^1.0.3",
     "@exodus/bitcoinregtest-plugin": "^1.0.3",
@@ -51,18 +52,20 @@
     "@exodus/blockchain-metadata": "^15.1.0",
     "@exodus/combined-assets-meta": "^1.2.5",
     "@exodus/cosmos-plugin": "^1.0.0",
+    "@exodus/ethereum-lib": "^4.2.7",
     "@exodus/ethereum-meta": "^1.1.0",
+    "@exodus/keychain": "^6.1.1",
     "@exodus/models": "^9.1.1",
     "@exodus/osmosis-plugin": "^1.0.0",
+    "@exodus/public-key-provider": "^2.0.0",
     "@exodus/redux-dependency-injection": "^3.0.0",
     "@exodus/storage-memory": "^2.1.1",
-    "@exodus/wallet-accounts": "^15.1.0",
+    "@exodus/wallet-accounts": "^16.2.0",
     "@exodus/wild-emitter": "^1.0.0",
-    "eslint": "^8.44.0",
+    "bip39": "^3.1.0",
     "events": "^3.3.0",
-    "jest": "^29.1.2",
     "msw": "^2.0.0",
     "redux": "^4.0.0"
   },
-  "gitHead": "021b200f9c9b2ba5c9e5fb2c7002c1f88b50d3c6"
+  "gitHead": "d3a937080fe613b640536d3378eb002b4a1838a6"
 }