@exodus/ethereum-api 8.46.0 → 8.46.1

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.46.1](https://github.com/ExodusMovement/assets/compare/@exodus/ethereum-api@8.46.0...@exodus/ethereum-api@8.46.1) (2025-09-03)
+
+
+### Bug Fixes
+
+
+* fix: tx-create simplification (#6310)
+
+
+
 ## [8.46.0](https://github.com/ExodusMovement/assets/compare/@exodus/ethereum-api@8.45.6...@exodus/ethereum-api@8.46.0) (2025-08-21)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/ethereum-api",
-  "version": "8.46.0",
+  "version": "8.46.1",
   "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",
@@ -64,5 +64,5 @@
     "type": "git",
     "url": "git+https://github.com/ExodusMovement/assets.git"
   },
-  "gitHead": "9d7bd1606fa42f6e05c68e0396d5aed7f7ea8852"
+  "gitHead": "e84949a7310f03feb4893668abd44fe537bf250c"
 }

package/src/create-asset.js

@@ -33,7 +33,6 @@ import { createFeeData } from './fee-data-factory.js'
 import { createGetBalanceForAddress } from './get-balance-for-address.js'
 import { getBalancesFactory } from './get-balances.js'
 import { getFeeFactory } from './get-fee.js'
-import getFeeAsyncFactory from './get-fee-async.js'
 import { estimateL1DataFeeFactory, getL1GetFeeFactory } from './optimism-gas/index.js'
 import { serverBasedFeeMonitorFactoryFactory } from './server-based-fee-monitor.js'
 import { createStakingApi } from './staking-api.js'
@@ -258,7 +257,7 @@ export const createAssetFactory = ({
       getBalanceForAddress: createGetBalanceForAddress({ asset, server }),
       getConfirmationsNumber: () => confirmationsNumber,
       getDefaultAddressPath: () => defaultAddressPath,
-      getFeeAsync: getFeeAsyncFactory({ assetClientInterface, createTx }),
+      getFeeAsync: createTx, // createTx alias, remove me when possible
       getFee,
       getFeeData: () => feeData,
       getKeyIdentifier: createGetKeyIdentifier({

package/src/tx-create.js

@@ -6,7 +6,7 @@ import * as ErrorWrapper from './error-wrapper.js'
 import { isContractAddressCached } from './eth-like-util.js'
 import { ensureSaneEip1559GasPriceForTipGasPrice } from './fee-utils.js'
 import { ARBITRARY_ADDRESS, fetchGasLimit, resolveDefaultTxInput } from './gas-estimation.js'
-import { getFeeFactoryGasPrices } from './get-fee.js'
+import { getExtraFeeData, getFeeFactoryGasPrices } from './get-fee.js'
 import { getNftArguments } from './nft-utils.js'
 
 async function createUnsignedTxWithFees({
@@ -66,7 +66,7 @@ async function createUnsignedTxWithFees({
     : asset.baseAsset.currency.ZERO
 
   const fee = baseFee.add(l1DataFee)
-
+  const extraFeeData = getExtraFeeData({ asset, amount: coinAmount })
   const unsignedTx = {
     txData: { transactionBuffer, chainId },
     txMeta: {
@@ -81,6 +81,8 @@ async function createUnsignedTxWithFees({
   }
   return {
     unsignedTx,
+    fee,
+    extraFeeData,
     // exhcange compatibility until the use usignedTx, remove me!
     gasPrice,
     tipGasPrice,
@@ -195,7 +197,6 @@ export const createTxFactory = ({ chainId, assetClientInterface, useAbsoluteNonc
   return async ({
     asset,
     walletAccount,
-    feeData,
     nft, // when sending nfts
     fromAddress: providedFromAddress, // wallet from address
     toAddress: providedToAddress, // user's to address, not the token or the dex contract
@@ -213,8 +214,16 @@ export const createTxFactory = ({ chainId, assetClientInterface, useAbsoluteNonc
     keepTxInput, // @deprecated this flag is used by swaps when swapping a token via DEX. The asset is token but the tx TO address is not the token address. Update swap to use `contractAddress`
   }) => {
     assert(asset, 'asset is required')
-    assert(feeData, 'feeData is required')
-    const fromAddress = providedFromAddress || ARBITRARY_ADDRESS
+    assert(walletAccount, 'walletAccount is required')
+
+    const feeData = await assetClientInterface.getFeeConfig({ assetName: asset.baseAsset.name })
+
+    const fromAddress =
+      providedFromAddress ??
+      (await assetClientInterface.getReceiveAddress({
+        assetName: asset.baseAsset.name,
+        walletAccount,
+      }))
 
     const baseAssetTxLog = await assetClientInterface.getTxLog({
       assetName: asset.baseAsset.name,
@@ -320,7 +329,7 @@ export const createTxFactory = ({ chainId, assetClientInterface, useAbsoluteNonc
       (await fetchGasLimit({
         asset,
         feeData,
-        fromAddress: providedFromAddress,
+        fromAddress,
         toAddress: providedToAddress,
         txInput: providedTxInput,
         contractAddress: txToAddress,

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

@@ -33,24 +33,9 @@ const txSendFactory = ({ assetClientInterface, createTx }) => {
         return { unsignedTx: providedUnsignedTx }
       }
 
-      const feeData =
-        legacyParams.feeData ??
-        (await assetClientInterface.getFeeData({
-          assetName: baseAsset.name,
-        }))
-
-      const fromAddress =
-        legacyParams.fromAddress ??
-        (await assetClientInterface.getReceiveAddress({
-          assetName: baseAsset.name,
-          walletAccount,
-        }))
-
       return createTx({
         asset,
         walletAccount,
-        feeData,
-        fromAddress,
         toAddress: legacyParams.address,
         ...legacyParams,
         ...legacyParams.options,

package/src/get-fee-async.js

@@ -1,29 +0,0 @@
-import assert from 'minimalistic-assert'
-
-import { getExtraFeeData } from './get-fee.js'
-
-const getFeeAsyncFactory = ({ assetClientInterface, createTx }) => {
-  assert(assetClientInterface, 'assetClientInterface is required')
-  assert(createTx, 'createTx is required')
-
-  return async (params) => {
-    const { asset } = params
-    const { unsignedTx, gasPrice, tipGasPrice, gasLimit } = params.unsignedTx
-      ? params
-      : await createTx(params)
-    const fee = asset.feeAsset.currency.parse(unsignedTx.txMeta.fee)
-    const coinAmount = asset.currency.parse(unsignedTx.txMeta.amount)
-    const extraFeeData = getExtraFeeData({ asset, amount: coinAmount })
-    return {
-      fee,
-      extraFeeData,
-      unsignedTx,
-      // deprecated, exchanges uses these params to recreate the tx during send. It should just use unsignedTx...
-      gasPrice,
-      tipGasPrice,
-      gasLimit,
-    }
-  }
-}
-
-export default getFeeAsyncFactory