@exodus/balances 7.0.0 → 7.2.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.
 
+## [7.2.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/balances@7.1.0...@exodus/balances@7.2.0) (2023-08-01)
+
+### Features
+
+- add balances report ([#3059](https://github.com/ExodusMovement/exodus-hydra/issues/3059)) ([a06ae1c](https://github.com/ExodusMovement/exodus-hydra/commit/a06ae1c988fb9abc8899caa0093528495496df1f))
+
+## [7.1.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/balances@6.1.0...@exodus/balances@7.1.0) (2023-07-21)
+
+### Features
+
+- **balances:** has balance atom ([#2761](https://github.com/ExodusMovement/exodus-hydra/issues/2761)) ([1b9e51e](https://github.com/ExodusMovement/exodus-hydra/commit/1b9e51e3074e368840832fedc7da45298d442347))
+- move balances to feature dir ([#2757](https://github.com/ExodusMovement/exodus-hydra/issues/2757)) ([8d0729a](https://github.com/ExodusMovement/exodus-hydra/commit/8d0729a470e9f1329814cca7e1ceb01bbc5db4b6))
+
+### Bug Fixes
+
+- don't crash when a single asset fails to get balance ([#2828](https://github.com/ExodusMovement/exodus-hydra/issues/2828)) ([547ddac](https://github.com/ExodusMovement/exodus-hydra/commit/547ddac4b1a7e1e81db1f3a504dacada50e9ae62))
+
 ## [7.0.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/balances@6.1.0...@exodus/balances@7.0.0) (2023-07-21)
 
 ### ⚠ BREAKING CHANGES

package/index.js

@@ -1,5 +1,6 @@
 import balancesDefinition from './module'
 import { balancesAtomDefinition, hasBalanceAtomDefinition } from './atoms'
+import balancesReportDefinition from './report'
 
 const balances = () => {
   return {
@@ -11,6 +12,7 @@ const balances = () => {
       },
       { definition: balancesAtomDefinition },
       { definition: hasBalanceAtomDefinition },
+      { definition: balancesReportDefinition },
     ],
   }
 }

package/module/index.js

@@ -53,13 +53,15 @@ class Balances extends ExodusModule {
         this.#pushIntoSharedQueue(() => fn.apply(this, args))
       }
 
+    // sanitize args in case someone passes something sensitive, e.g. getBalance({ walletAccount, mnemonic })
+    // when only `{ walletAccount }` is expected
     const captureErrors =
-      (fn, name) =>
+      ({ fn, name, sanitizeArgs = () => {} }) =>
       async (...args) => {
         try {
           return await fn(...args)
         } catch (error) {
-          this.emit('error', { fn: name, error })
+          this.emit('error', { fn: name, error, args: sanitizeArgs(...args) })
           throw error
         }
       }
@@ -68,12 +70,17 @@ class Balances extends ExodusModule {
     this.#updateAssetSource = shareQueue(this.#updateAssetSource)
     this.#handleTxLogUpdate = shareQueue(this.#handleTxLogUpdate)
     this.#recompute = shareQueue(this.#recompute)
-    this.#getBalancesForAssetSource = captureErrors(
-      this.#getBalancesForAssetSource,
-      'getBalancesForAssetSource'
-    )
+    this.#getBalancesForAssetSource = captureErrors({
+      fn: this.#getBalancesForAssetSource,
+      name: 'getBalancesForAssetSource',
+      sanitizeArgs: ({ walletAccount, assetName }) => ({ walletAccount, assetName }),
+    })
 
-    this.#setWalletAccounts = captureErrors(this.#setWalletAccounts, 'setWalletAccounts')
+    this.#setWalletAccounts = captureErrors({
+      fn: this.#setWalletAccounts,
+      name: 'setWalletAccounts',
+      sanitizeArgs: (walletAccounts) => walletAccounts,
+    })
 
     blockchainMetadata.once('load-wallet-accounts', () => {
       // eslint-disable-next-line hydra/module-side-effects
@@ -106,6 +113,7 @@ class Balances extends ExodusModule {
       })
       return
     }
+
     return this.#updateAssetSource(opts)
   }
 
@@ -177,10 +185,11 @@ class Balances extends ExodusModule {
 
   #getBalancesForWalletAccount = async ({ walletAccount }) => {
     const assetNames = this.#getSupportedAssetNames()
-    const balances = await Promise.all(
+    const results = await Promise.allSettled(
       assetNames.map((assetName) => this.#getBalancesForAssetSource({ assetName, walletAccount }))
     )
 
+    const balances = results.map(({ value }) => value).filter(Boolean)
     return merge(Object.create(null), ...balances)
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/balances",
-  "version": "7.0.0",
+  "version": "7.2.0",
   "description": "Tracks crypto balances across enabled wallet accounts.",
   "author": "Exodus Movement Inc.",
   "license": "UNLICENSED",
@@ -11,7 +11,6 @@
   },
   "main": "index.js",
   "files": [
-    "index.js",
     "module",
     "atoms",
     "README.md",
@@ -27,7 +26,7 @@
     "url": "https://github.com/ExodusMovement/exodus-hydra/issues?q=is%3Aissue+is%3Aopen+label%3Abalances"
   },
   "dependencies": {
-    "@exodus/atoms": "^5.2.2",
+    "@exodus/atoms": "^5.4.0",
     "@exodus/basic-utils": "^2.0.0",
     "@exodus/currency": "^2.1.2",
     "@exodus/module": "^1.1.0",
@@ -38,6 +37,8 @@
     "@exodus/blockchain-metadata": "^6.0.0"
   },
   "devDependencies": {
+    "@exodus/assets": "^8.0.94",
+    "@exodus/assets-base": "^8.1.10",
     "@exodus/blockchain-metadata": "^8.0.0",
     "@exodus/models": "^8.9.0",
     "@exodus/storage-memory": "*",
@@ -46,5 +47,5 @@
     "events": "^3.3.0",
     "jest": "^29.1.2"
   },
-  "gitHead": "c85a6f44669c8d173606c3eb89fceaf650365df7"
+  "gitHead": "91e322589102bc3fa4886b7bd254e08d1dd766da"
 }