@exodus/sui-api 1.1.3 → 1.2.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.
 
+## [1.2.0](https://github.com/ExodusMovement/assets/compare/@exodus/sui-api@1.1.3...@exodus/sui-api@1.2.0) (2026-01-06)
+
+
+### Features
+
+
+* feat: add createTx for SUI (#7143)
+
+
+
 ## [1.1.3](https://github.com/ExodusMovement/assets/compare/@exodus/sui-api@1.1.2...@exodus/sui-api@1.1.3) (2025-11-03)
 
 **Note:** Version bump only for package @exodus/sui-api

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@exodus/sui-api",
   "packageManager": "yarn@4.3.1",
-  "version": "1.1.3",
+  "version": "1.2.0",
   "description": "Transaction monitors, RPC with the blockchain node, and other networking code for Sui",
   "type": "module",
   "main": "src/index.js",
@@ -50,5 +50,5 @@
     "type": "git",
     "url": "git+https://github.com/ExodusMovement/assets.git"
   },
-  "gitHead": "1792bda2b5f11b36e67d2e36669341b51002197f"
+  "gitHead": "f32e29d35f604bd6857bb1d19e2896b34352f122"
 }

package/src/get-fee-async.js

@@ -21,7 +21,6 @@ export const getFeeAsyncFactory = ({ assetClientInterface, serverApi, createUnsi
       walletAccount,
       recipientAddress: recipientAddress || senderAddress,
       amount: amount.toBaseString(),
-      serverApi,
       gasBudget: MIN_GAS_BUDGET,
       buildOptions: {
         onlyTransactionKind: true,

package/src/index.js

@@ -7,3 +7,4 @@ export { createSuiNoHistoryMonitor } from './sui-no-history-monitor.js'
 export { getFeeAsyncFactory } from './get-fee-async.js'
 export { txSendFactory } from './tx-send.js'
 export { createUnsignedTxFactory } from './create-unsigned-tx.js'
+export { createTxFactory } from './tx-create.js'

package/src/sui-no-history-monitor.js

@@ -53,7 +53,7 @@ export class SuiNoHistoryMonitor extends BaseMonitor {
 
     let batch = this.aci.createOperationsBatch()
 
-    batch = await this.aci.updateAccountStateBatch({
+    batch = this.aci.updateAccountStateBatch({
       assetName,
       walletAccount,
       newData: {
@@ -71,11 +71,11 @@ export class SuiNoHistoryMonitor extends BaseMonitor {
     })
 
     for (const [assetName, txsToRemove] of Object.entries(txsToRemoveByAsset)) {
-      batch = await this.aci.removeTxLogBatch({ assetName, walletAccount, txs: txsToRemove, batch })
+      batch = this.aci.removeTxLogBatch({ assetName, walletAccount, txs: txsToRemove, batch })
     }
 
     for (const [assetName, txLogItems] of Object.entries(txLogItemsByAsset)) {
-      batch = await this.aci.updateTxLogAndNotifyBatch({
+      batch = this.aci.updateTxLogAndNotifyBatch({
         assetName,
         walletAccount,
         txs: txLogItems,

package/src/tx-create.js

@@ -0,0 +1,40 @@
+import assert from 'minimalistic-assert'
+
+export const createTxFactory = ({
+  assetClientInterface: aci,
+  serverApi,
+  getFeeAsync,
+  createUnsignedTx,
+}) => {
+  assert(aci, 'assetClientInterface is required')
+  assert(serverApi, 'serverApi is required')
+  assert(getFeeAsync, 'getFeeAsync is required')
+  assert(createUnsignedTx, 'createUnsignedTx is required')
+
+  return async ({ asset, walletAccount, fromAddress, address: toAddress, amount, gasBudget }) => {
+    const { name: assetName } = asset
+
+    if (!fromAddress) {
+      fromAddress = await aci.getReceiveAddress({ assetName, walletAccount })
+    }
+
+    const unsignedTx = await createUnsignedTx({
+      asset,
+      senderAddress: fromAddress,
+      recipientAddress: toAddress,
+      amount: amount.toBaseString(),
+      gasBudget,
+      walletAccount,
+    })
+
+    const { fee } = await getFeeAsync({
+      asset,
+      walletAccount,
+      toAddress,
+      fromAddress,
+      amount,
+    })
+
+    return { unsignedTx, fee }
+  }
+}

package/src/tx-send.js

@@ -125,40 +125,26 @@ const getObjectAndBalanceChangesByType = ({
   }
 }
 
-const updateAccountStateAfterSend = async ({
-  aci,
-  asset,
-  walletAccount,
-  ownerAddress,
-  result,
-  amount,
-  recipientAddress,
-}) => {
+const getRecipientAddress = ({ balanceChanges, ownerAddress }) => {
+  const receiverItem = balanceChanges.find((item) => item.owner.AddressOwner !== ownerAddress)
+  return receiverItem?.owner?.AddressOwner
+}
+
+// TODO: extract into separate file
+const getStateChanges = async ({ aci, asset, walletAccount, result }) => {
   const { feeAsset, currency: assetCurrency, name: assetName } = asset
   const { objectChanges, balanceChanges, digest, effects } = result
   const { gasUsed } = effects
   const wasBaseAssetSent = assetName === asset.baseAssetName
 
   const fee = getFeeAmount({ gasUsed, feeAsset })
+  const ownerAddress = await aci.getReceiveAddress({ assetName, walletAccount })
 
   const accountState = await aci.getAccountState({
     assetName: asset.baseAssetName,
     walletAccount,
   })
 
-  const tx = {
-    txId: digest,
-    confirmations: 0,
-    coinName: assetName,
-    coinAmount: amount.abs().negate(),
-    feeAmount: fee,
-    feeCoinName: feeAsset.name,
-    selfSend: false,
-    to: recipientAddress,
-    data: Object.create(null),
-    currencies: { [assetName]: assetCurrency, [feeAsset.name]: feeAsset.currency },
-  }
-
   const { ownerObjectChangesByType, ownerBalanceChangesByType } = getObjectAndBalanceChangesByType({
     objectChanges,
     balanceChanges,
@@ -178,7 +164,10 @@ const updateAccountStateAfterSend = async ({
 
   newAccountState.utxos = newBaseUtxos.utxos
 
-  if (!wasBaseAssetSent) {
+  let amount
+  if (wasBaseAssetSent) {
+    amount = asset.currency.baseUnit(ownerBalanceChangesByType[coinTypes.Base]).add(fee)
+  } else {
     const coinType = toCoinType(asset.assetId)
     const newTokenUtxos = deriveNewUtxos({
       mutatedObject: ownerObjectChangesByType[coinType].mutated[0],
@@ -189,14 +178,26 @@ const updateAccountStateAfterSend = async ({
     })
 
     newAccountState.tokenUtxos = { [asset.name]: newTokenUtxos.utxos }
+    amount = asset.currency.baseUnit(ownerBalanceChangesByType[asset.assetId])
   }
 
-  let batch = aci.createOperationsBatch()
+  const to = getRecipientAddress({ balanceChanges, ownerAddress })
 
-  batch = await aci.updateAccountStateBatch({
-    assetName: asset.baseAssetName,
-    walletAccount,
-    newData: {
+  const tx = {
+    txId: digest,
+    confirmations: 0,
+    coinName: assetName,
+    coinAmount: amount.abs().negate(),
+    feeAmount: fee,
+    feeCoinName: feeAsset.name,
+    selfSend: false,
+    to,
+    data: Object.create(null),
+    currencies: { [assetName]: assetCurrency, [feeAsset.name]: feeAsset.currency },
+  }
+
+  return {
+    accountStateData: {
       utxos: newAccountState.utxos,
       tokenUtxos: {
         ...accountState.tokenUtxos,
@@ -205,52 +206,53 @@ const updateAccountStateAfterSend = async ({
           : Object.create(null)),
       },
     },
-    batch,
-  })
-
-  batch = await aci.updateTxLogAndNotifyBatch({ assetName, walletAccount, txs: [tx], batch })
-
-  await aci.executeOperationsBatch(batch)
+    txLogsByAssetName: {
+      [assetName]: [tx],
+      // TODO: add fee-to-send entry to the feeAsset txLog
+    },
+  }
 }
 
-export const txSendFactory = ({ assetClientInterface: aci, serverApi, createUnsignedTx }) => {
+export const txSendFactory = ({ assetClientInterface: aci, serverApi, createTx }) => {
   assert(aci, 'assetClientInterface is required')
+  assert(serverApi, 'serverApi is required')
+  assert(createTx, 'createTx is required')
 
-  return async ({
-    asset,
-    walletAccount,
-    address: recipientAddress,
-    amount,
-    gasBudget,
-    feeOpts,
-  }) => {
-    const { name: assetName } = asset
-
-    const senderAddress = await aci.getReceiveAddress({ assetName, walletAccount })
+  return async ({ asset, walletAccount, unsignedTx: providedUnsignedTx, ...legacyParams }) => {
+    const { unsignedTx } = providedUnsignedTx
+      ? { unsignedTx: providedUnsignedTx }
+      : await createTx({ asset, walletAccount, ...legacyParams })
 
-    const unsignedTx = await createUnsignedTx({
-      asset,
-      senderAddress,
-      recipientAddress,
-      amount: amount.toBaseString(),
-      serverApi,
-      gasBudget: gasBudget || feeOpts.gasBudget,
+    const { plainTx } = await aci.signTransaction({
+      assetName: asset.name,
+      unsignedTx,
       walletAccount,
     })
 
-    const { plainTx } = await aci.signTransaction({ assetName, unsignedTx, walletAccount })
+    const result = await asset.baseAsset.api.broadcastTx(plainTx)
 
-    const result = await serverApi.broadcastTx(plainTx)
-
-    await updateAccountStateAfterSend({
-      aci,
+    const { accountStateData: newData, txLogsByAssetName } = await getStateChanges({
       asset,
       walletAccount,
-      ownerAddress: senderAddress,
       result,
-      amount,
-      recipientAddress,
+      aci,
     })
+
+    let batch = aci.createOperationsBatch()
+
+    batch = await aci.updateAccountStateBatch({
+      assetName: asset.baseAsset.name,
+      walletAccount,
+      newData,
+      batch,
+    })
+
+    Object.entries(txLogsByAssetName).forEach(([assetName, txs]) => {
+      batch = aci.updateTxLogAndNotifyBatch({ assetName, walletAccount, txs, batch })
+    })
+
+    await aci.executeOperationsBatch(batch)
+
     return { txId: result.digest }
   }
 }