@exodus/ethereum-api 8.33.7 → 8.34.0

package/CHANGELOG.md

@@ -3,6 +3,16 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [8.34.0](https://github.com/ExodusMovement/assets/compare/@exodus/ethereum-api@8.33.7...@exodus/ethereum-api@8.34.0) (2025-04-08)
+
+
+### Features
+
+
+* feat: demote batch of 20 tokens (#5347)
+
+
+
 ## [8.33.7](https://github.com/ExodusMovement/assets/compare/@exodus/ethereum-api@8.33.6...@exodus/ethereum-api@8.33.7) (2025-04-03)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/ethereum-api",
-  "version": "8.33.7",
+  "version": "8.34.0",
   "description": "Transaction monitors, fee monitors, RPC with the blockchain node, and other networking code for Ethereum and EVM-based blockchains",
   "type": "module",
   "main": "src/index.js",
@@ -28,8 +28,8 @@
     "@exodus/bip44-constants": "^195.0.0",
     "@exodus/crypto": "^1.0.0-rc.13",
     "@exodus/currency": "^6.0.1",
-    "@exodus/ethereum-lib": "^5.9.0",
-    "@exodus/ethereum-meta": "^2.3.0",
+    "@exodus/ethereum-lib": "^5.10.0",
+    "@exodus/ethereum-meta": "^2.5.0",
     "@exodus/ethereumholesky-meta": "^2.0.2",
     "@exodus/ethereumjs": "^1.0.0",
     "@exodus/fetch": "^1.3.0",
@@ -64,5 +64,5 @@
     "type": "git",
     "url": "git+https://github.com/ExodusMovement/assets.git"
   },
-  "gitHead": "c181bb6d727d45fe17b31d1cf1b70eeb34fb7fd4"
+  "gitHead": "1684d29840942a8b5dce8dc54e3de02d86964b36"
 }

package/src/eth-like-util.js

@@ -100,8 +100,8 @@ export function sendRawTransaction(asset) {
 export async function transactionExists({ asset, txId }) {
   const server = getServer(asset)
   txId = normalizeTxId(txId)
-  const txResult = server.getTransactionByHash(txId)
-  return txResult && txResult.hash === txId
+  const txResult = await server.getTransactionByHash(txId)
+  return Boolean(txResult && txResult.hash === txId)
 }
 
 export async function getTransaction({ asset, txId }) {

package/src/tx-send/tx-send.js

@@ -9,6 +9,58 @@ import { getNftArguments } from '../nft-utils.js'
 import getFeeInfo from './get-fee-info.js'
 import { resolveNonce } from './nonce-utils.js'
 
+// Exodus enforces a strict invariant that `sendAll` transactions
+// must not leave any dust in the sender's account. Currently, the
+// assets library has the expectation that the client frontend
+// should calculate the precise amount to send, but due to the
+// sheer amount of variables involved when resolving a `gasPrice`,
+// this is a significant undertaking.
+//
+// Therefore, although clients try their very best to calculate
+// the correct amount, in cases this fails we can fall back to
+// the implementation defined here with a warning.
+// eslint-disable-next-line camelcase
+export const HACK_maybeRefineSendAllAmount = async ({
+  amount: providedAmount,
+  asset,
+  assetClientInterface,
+  walletAccount,
+  gasLimit,
+  gasPrice,
+}) => {
+  try {
+    const { name: assetName } = asset
+
+    const [txLog, accountState] = await Promise.all([
+      assetClientInterface.getTxLog({
+        assetName,
+        walletAccount,
+      }),
+      assetClientInterface.getAccountState({
+        assetName,
+        walletAccount,
+      }),
+    ])
+
+    const { spendable } = await asset.api.getBalances({ asset, txLog, accountState })
+    const maxGasCost = gasPrice.mul(gasLimit)
+
+    if (maxGasCost.gt(spendable)) throw new Error('transaction gas cost exceeds spendable balance')
+
+    const expectedSendAllAmount = spendable.sub(maxGasCost)
+
+    // If the client attempted to send the correct
+    // amount, good job! You get a cookie!
+    if (providedAmount.equals(expectedSendAllAmount)) return null
+
+    // The client attempted to `sendAll` using the incorrect amount.
+    return expectedSendAllAmount
+  } catch (e) {
+    console.error('failed to refine send all amount', e)
+    return null
+  }
+}
+
 const txSendFactory = ({ assetClientInterface, createUnsignedTx }) => {
   assert(assetClientInterface, 'assetClientInterface is required')
   assert(createUnsignedTx, 'createUnsignedTx is required')
@@ -298,6 +350,8 @@ const createTx = async ({
     feeOpts,
   })
 
+  const isSendAllBaseAsset = isSendAll && !isToken
+
   if (eip1559Enabled) {
     if (customGasPrice) {
       gasPrice = customGasPrice // aka maxFeePerGas
@@ -310,12 +364,31 @@ const createTx = async ({
       tipGasPrice = customGasPrice // aka maxPriorityFeePerGas
     }
 
-    if (isSendAll && !isToken) {
+    if (isSendAllBaseAsset) {
       // force consuming all gas
       tipGasPrice = gasPrice
     }
+  }
 
-    // gasLimit = customGasLimit
+  // HACK: If we are handling a send all transaction, we must ensure
+  //       the send all invariant is maintained before producing the
+  //       final transaction.
+  const maybeOverrideSendAllAmount =
+    isSendAllBaseAsset &&
+    (await HACK_maybeRefineSendAllAmount({
+      amount,
+      asset,
+      assetClientInterface,
+      walletAccount,
+      gasLimit,
+      gasPrice,
+    }))
+
+  if (maybeOverrideSendAllAmount) {
+    console.log(
+      `Attempted to execute a sendAll transaction with an amount of ${amount.toDefaultString({ unit: true })}, but this would fail to maintain the no dust invariant! Overriding with ${maybeOverrideSendAllAmount.toDefaultString({ unit: true })}.`
+    )
+    amount = maybeOverrideSendAllAmount
   }
 
   const unsignedTx = await createUnsignedTx({