@exodus/bitcoin-api 2.27.0 → 2.28.0

package/CHANGELOG.md

@@ -3,6 +3,20 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [2.28.0](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@2.27.0...@exodus/bitcoin-api@2.28.0) (2024-10-31)
+
+
+### Features
+
+* update bip322-js to 2.0.0 ([#4414](https://github.com/ExodusMovement/assets/issues/4414)) ([da2a7dd](https://github.com/ExodusMovement/assets/commit/da2a7dd217a6664f02fa722968bf6ff16f53ea39))
+
+
+### Bug Fixes
+
+* stringify sent amount NU's ([#4433](https://github.com/ExodusMovement/assets/issues/4433)) ([f2886a8](https://github.com/ExodusMovement/assets/commit/f2886a85060f5e347df9bb2bf892bb52257ece6b)), closes [#4437](https://github.com/ExodusMovement/assets/issues/4437)
+
+
+
 ## [2.27.0](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@2.26.1...@exodus/bitcoin-api@2.27.0) (2024-10-17)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/bitcoin-api",
-  "version": "2.27.0",
+  "version": "2.28.0",
   "description": "Exodus bitcoin-api",
   "type": "module",
   "main": "src/index.js",
@@ -23,7 +23,7 @@
     "@exodus/asset-lib": "^5.0.0",
     "@exodus/basic-utils": "^3.0.1",
     "@exodus/bip32": "^3.3.0",
-    "@exodus/bip322-js": "^1.1.0",
+    "@exodus/bip322-js": "^2.0.0",
     "@exodus/bip44-constants": "^195.0.0",
     "@exodus/bitcoin-lib": "^2.4.2",
     "@exodus/bitcoinjs": "^1.1.0",
@@ -56,5 +56,5 @@
     "type": "git",
     "url": "git+https://github.com/ExodusMovement/assets.git"
   },
-  "gitHead": "3218754fdfad087b6d789eb81f7fe9f21b636fef"
+  "gitHead": "65c334463a6fd913d73d2873ac455c5ec675705a"
 }

package/src/fee/fee-utils.js

@@ -58,3 +58,11 @@ export function parseCurrency(val, currency) {
   if (typeof val === 'string') return currency.parse(val)
   return currency.parse(val.value + ' ' + val.unit)
 }
+
+export function serializeCurrency(val, currency) {
+  if (val === undefined) {
+    return val
+  }
+
+  return parseCurrency(val, currency).toDefaultString({ unit: true })
+}

package/src/tx-log/bitcoin-monitor-scanner.js

@@ -505,7 +505,10 @@ export class BitcoinMonitorScanner {
           if (isSent && !txLogItem.to) {
             const val = currency.defaultUnit(vout.value)
             const sentDisplayAddress = asset.address.displayAddress?.(sentAddress) || sentAddress
-            txLogItem.data.sent.push({ address: sentDisplayAddress, amount: val })
+            txLogItem.data.sent.push({
+              address: sentDisplayAddress,
+              amount: val.toDefaultString({ unit: true }),
+            })
           }
 
           return

package/src/tx-send/index.js

@@ -5,7 +5,7 @@ import { retry } from '@exodus/simple-retry'
 import lodash from 'lodash'
 import assert from 'minimalistic-assert'
 
-import { parseCurrency } from '../fee/fee-utils.js'
+import { parseCurrency, serializeCurrency } from '../fee/fee-utils.js'
 import { selectUtxos } from '../fee/utxo-selector.js'
 import { findUnconfirmedSentRbfTxs } from '../tx-utils.js'
 import { getUnconfirmedTxAncestorMap } from '../unconfirmed-ancestor-data.js'
@@ -376,7 +376,7 @@ export const getPrepareSendTransaction =
       replaceTx = replaceTx.clone()
       replaceTx = replaceTx.update({ data: { ...replaceTx.data } })
       replaceTx.data.sent = replaceTx.data.sent.map((to) => {
-        return { ...to, amount: parseCurrency(to.amount, asset.currency) }
+        return { ...to, amount: serializeCurrency(to.amount, asset.currency) }
       })
       selectedUtxos = selectedUtxos.union(
         // how to avoid replace tx inputs when inputs are ordinals? !!!!
@@ -389,7 +389,9 @@ export const getPrepareSendTransaction =
     // Inputs and Outputs
     const inputs = shuffle(createInputs(assetName, selectedUtxos.toArray(), rbfEnabled))
     let outputs = replaceTx
-      ? replaceTx.data.sent.map(({ address, amount }) => createOutput(assetName, address, amount))
+      ? replaceTx.data.sent.map(({ address, amount }) =>
+          createOutput(assetName, address, parseCurrency(amount, currency))
+        )
       : []
 
     // Send output
@@ -408,7 +410,10 @@ export const getPrepareSendTransaction =
     }
 
     const totalAmount = replaceTx
-      ? replaceTx.data.sent.reduce((total, { amount }) => total.add(amount), sendAmount)
+      ? replaceTx.data.sent.reduce(
+          (total, { amount }) => total.add(parseCurrency(amount, asset.currency)),
+          sendAmount
+        )
       : sendAmount
 
     const change = selectedUtxos.value
@@ -655,8 +660,11 @@ export const createAndBroadcastTXFactory =
         ? replaceTx.data.sent
         : []
       : replaceTx
-        ? [...replaceTx.data.sent, { address: displayReceiveAddress, amount }]
-        : [{ address: displayReceiveAddress, amount }]
+        ? [
+            ...replaceTx.data.sent,
+            { address: displayReceiveAddress, amount: serializeCurrency(amount, asset.currency) },
+          ]
+        : [{ address: displayReceiveAddress, amount: serializeCurrency(amount, asset.currency) }]
 
     const calculateCoinAmount = () => {
       if (selfSend) {