@exodus/ethereum-api 8.62.2 → 8.62.3

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.62.3](https://github.com/ExodusMovement/assets/compare/@exodus/ethereum-api@8.62.2...@exodus/ethereum-api@8.62.3) (2026-01-13)
+
+
+### Bug Fixes
+
+
+* fix: enfore sanity checking about clarity string numerics for canonical tx props (#7262)
+
+
+
 ## [8.62.2](https://github.com/ExodusMovement/assets/compare/@exodus/ethereum-api@8.62.1...@exodus/ethereum-api@8.62.2) (2026-01-07)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/ethereum-api",
-  "version": "8.62.2",
+  "version": "8.62.3",
   "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",
@@ -67,5 +67,5 @@
     "type": "git",
     "url": "git+https://github.com/ExodusMovement/assets.git"
   },
-  "gitHead": "be4898d14d842c79dfcc327501c390dfaf56157b"
+  "gitHead": "d1618b4c41ccde490c3e4aaa8dd239c0c5393a7b"
 }

package/src/staking/ethereum/service.js

@@ -23,6 +23,14 @@ const MINIMUM_DELEGATION_GAS_LIMIT = 180_000
 
 const EXTRA_GAS_LIMIT = 20_000 // extra gas Limit to prevent tx failing if something change on pool state (till tx is in mempool)
 
+// Estimates based on real life transaction averages.
+// Overestimated averages by 4x.
+const STAKING_OPERATION_FALLBACK_FEE_ESTIMATES = {
+  delegate: '0.00081512',
+  undelegate: '0.000554',
+  claimUndelegatedBalance: '0.0001800912',
+}
+
 const getStakingApi = memoize(
   (asset) => new EthereumStaking(asset, undefined, asset.server),
   (asset) => asset.name
@@ -346,61 +354,76 @@ export function createEthereumStakingService({
 
   async function estimateDelegateOperation({ walletAccount, operation, args, feeData }) {
     const asset = await getAsset(assetName)
-    const staking = getStakingApi(asset)
-
-    const requestedAmount = args.amount
-      ? amountToCurrency({ asset, amount: args.amount })
-      : asset.currency.ZERO
 
-    let delegatorAddress
-    ;({ delegatorAddress, feeData } = await getTransactionProps({ feeData, walletAccount }))
-
-    if (operation === 'undelegate') {
-      return estimateUndelegate({ walletAccount, amount: requestedAmount, feeData })
-    }
+    try {
+      const staking = getStakingApi(asset)
 
-    const NAMING_MAP = {
-      delegate: 'stake',
-      undelegate: 'unstake',
-      claimUndelegatedBalance: 'claimWithdrawRequest',
-    }
+      const requestedAmount = args.amount
+        ? amountToCurrency({ asset, amount: args.amount })
+        : asset.currency.ZERO
 
-    const delegateOperation = staking[NAMING_MAP[operation]].bind(staking)
+      let delegatorAddress
+      ;({ delegatorAddress, feeData } = await getTransactionProps({ feeData, walletAccount }))
 
-    if (!delegateOperation) throw new Error('Invalid staking operation')
+      if (operation === 'undelegate') {
+        return estimateUndelegate({ walletAccount, amount: requestedAmount, feeData })
+      }
 
-    const { amount, data } = await delegateOperation({ ...args, amount: requestedAmount })
+      const NAMING_MAP = {
+        delegate: 'stake',
+        undelegate: 'unstake',
+        claimUndelegatedBalance: 'claimWithdrawRequest',
+      }
 
-    const { fee } = await (operation === 'claimUndelegatedBalance'
-      ? estimateTxFee({
-          from: delegatorAddress,
-          to: staking.accountingAddress,
-          amount,
-          txInput: data,
-          feeData,
-        })
-      : // The `gasUsed` of a delegation transaction can vary
-        // significantly depending upon whether it will result
-        // in the activation of new slots (i.e. validator creation).
-        //
-        // This can result in transaction `revert` due to slippage
-        // from incompatible transaction ordering.
-        //
-        // To mitigate this, we:
-        // 1. Simulate the delegation at an amplified deposit amount
-        //    to show enhanced fees close to a proximity buffer.
-        // 2. Originate the transaction from the WETH contract,
-        //    which guarantees deep native ether liquidity which
-        //    exceeds any rational user deposit.
-        estimateTxFee({
-          from: WETH9_ADDRESS,
-          to: staking.poolAddress,
-          amount: amount.add(asset.currency.defaultUnit(SLOT_ACTIVATION_PROXIMITY_ETH)),
-          txInput: data,
-          feeData,
-        }))
+      const delegateOperation = staking[NAMING_MAP[operation]].bind(staking)
+
+      if (!delegateOperation) throw new Error('Invalid staking operation')
+
+      const { amount, data } = await delegateOperation({ ...args, amount: requestedAmount })
+
+      const { fee } = await (operation === 'claimUndelegatedBalance'
+        ? estimateTxFee({
+            from: delegatorAddress,
+            to: staking.accountingAddress,
+            amount,
+            txInput: data,
+            feeData,
+          })
+        : // The `gasUsed` of a delegation transaction can vary
+          // significantly depending upon whether it will result
+          // in the activation of new slots (i.e. validator creation).
+          //
+          // This can result in transaction `revert` due to slippage
+          // from incompatible transaction ordering.
+          //
+          // To mitigate this, we:
+          // 1. Simulate the delegation at an amplified deposit amount
+          //    to show enhanced fees close to a proximity buffer.
+          // 2. Originate the transaction from the WETH contract,
+          //    which guarantees deep native ether liquidity which
+          //    exceeds any rational user deposit.
+          estimateTxFee({
+            from: WETH9_ADDRESS,
+            to: staking.poolAddress,
+            amount: amount.add(asset.currency.defaultUnit(SLOT_ACTIVATION_PROXIMITY_ETH)),
+            txInput: data,
+            feeData,
+          }))
+
+      return fee
+    } catch (error) {
+      const fallbackFee = STAKING_OPERATION_FALLBACK_FEE_ESTIMATES[operation]
+      if (fallbackFee) {
+        console.warn(
+          `ETH staking fee estimation failed for '${operation}', using fallback:`,
+          error.message
+        )
+        return asset.currency.defaultUnit(fallbackFee)
+      }
 
-    return fee
+      error.message = `StakingFeeEstimationError: Failed to estimate fee for '${operation}' (and no fallback configured). Cause: ${error.message}`
+      throw error
+    }
   }
 
   async function estimateTxFee({ from, to, amount, txInput, feeData }) {

package/src/tx-log/clarity-utils/get-log-items-from-server-tx.js

@@ -9,9 +9,10 @@ import isConsideredSentTokenTx from '../monitor-utils/is-considered-sent-token-t
 import filterEffects from './filter-effects.js'
 import getNamesOfTokensTransferredByServerTx from './get-names-of-tokens-transferred-by-server-tx.js'
 
-// This function takes a server transaction object fetched from magnifier,
-// and transforms it into Tx models to update the exodus state.
+const isFiniteInteger = (n) => Number.isFinite(n) && Number.isInteger(n)
 
+// This function takes a server transaction object fetched from clarity,
+// and transforms it into Tx models to update the exodus state.
 export default function getLogItemsFromServerTx({
   serverTx,
   asset,
@@ -93,8 +94,8 @@ export default function getLogItemsFromServerTx({
             nonceChange: nonceUpdate,
             ...methodId,
             ...(sent?.length > 0 ? { sent } : undefined),
-            blockNumber,
-            transactionIndex,
+            ...(isFiniteInteger(blockNumber) ? { blockNumber } : null),
+            ...(isFiniteInteger(transactionIndex) ? { transactionIndex } : null),
           },
           ...(ourWalletWasSender
             ? { from: [], to: toAddress, feeAmount, feeCoinName: asset.feeAsset.name }
@@ -159,8 +160,8 @@ export default function getLogItemsFromServerTx({
           gasLimit,
           balanceChange,
           ...methodId,
-          blockNumber,
-          transactionIndex,
+          ...(isFiniteInteger(blockNumber) ? { blockNumber } : null),
+          ...(isFiniteInteger(transactionIndex) ? { transactionIndex } : null),
         },
         ...(isConsideredSent
           ? { from: [], to: tokenTransferToAddress, feeAmount, feeCoinName: token.feeAsset.name }

package/src/tx-log/monitor-utils/get-value-of-transfers.js

@@ -4,19 +4,17 @@
 // array is a transfer for that asset. If we sent more than we received, the value is negative.
 
 export default function getValueOfTransfers(ourWalletAddress, asset, transfers) {
-  return transfers
-    .reduce((balanceDifference, { from, to, value }) => {
-      const transferAmount = asset.currency.baseUnit(value)
+  return transfers.reduce((balanceDifference, { from, to, value }) => {
+    const transferAmount = asset.currency.baseUnit(value)
 
-      if (from === ourWalletAddress) {
-        balanceDifference = balanceDifference.sub(transferAmount)
-      }
+    if (from === ourWalletAddress) {
+      balanceDifference = balanceDifference.sub(transferAmount)
+    }
 
-      if (to === ourWalletAddress) {
-        balanceDifference = balanceDifference.add(transferAmount)
-      }
+    if (to === ourWalletAddress) {
+      balanceDifference = balanceDifference.add(transferAmount)
+    }
 
-      return balanceDifference
-    }, asset.currency.ZERO)
-    .to(asset.ticker)
+    return balanceDifference
+  }, asset.currency.ZERO)
 }