@exodus/ethereum-api 8.6.0 → 8.7.0

package/CHANGELOG.md

@@ -3,6 +3,15 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [8.7.0](https://github.com/ExodusMovement/assets/compare/@exodus/ethereum-api@8.6.0...@exodus/ethereum-api@8.7.0) (2024-06-21)
+
+
+### Features
+
+* EVM NFT's fee and send ([#2626](https://github.com/ExodusMovement/assets/issues/2626)) ([2231954](https://github.com/ExodusMovement/assets/commit/22319543463e88046339ad6658683e13f0e71a46))
+
+
+
 ## [8.6.0](https://github.com/ExodusMovement/assets/compare/@exodus/ethereum-api@8.5.0...@exodus/ethereum-api@8.6.0) (2024-06-18)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/ethereum-api",
-  "version": "8.6.0",
+  "version": "8.7.0",
   "description": "Ethereum Api",
   "main": "src/index.js",
   "files": [
@@ -66,5 +66,5 @@
     "type": "git",
     "url": "git+https://github.com/ExodusMovement/assets.git"
   },
-  "gitHead": "1c873218d5be9cebd41509a7d88c4b003c09d353"
+  "gitHead": "685abde3787a364cdd00a675e95fe4dfa9053d96"
 }

package/src/gas-estimation.js

@@ -1,6 +1,6 @@
 import BN from 'bn.js'
 import * as ethUtil from '@exodus/ethereumjs-util'
-import { currency2buffer, isToken } from '@exodus/ethereum-lib'
+import { currency2buffer, isEthereumLikeToken } from '@exodus/ethereum-lib'
 import { estimateGas, isContractAddressCached } from './eth-like-util'
 
 const EXTRA_PERCENTAGE = 20
@@ -42,7 +42,7 @@ export async function fetchGasLimit({
   asset,
   fromAddress,
   toAddress,
-  txInput = '0x',
+  txInput: providedTxInput,
   amount,
   bip70,
   throwOnError = true,
@@ -53,21 +53,22 @@ export async function fetchGasLimit({
 
   if (!amount) amount = asset.currency.ZERO
 
+  const isToken = isEthereumLikeToken(asset)
+
+  const txInput =
+    providedTxInput ||
+    (isToken
+      ? ethUtil.bufferToHex(
+          asset.contract.transfer.build(toAddress.toLowerCase(), amount.toBaseString())
+        )
+      : '0x')
+
   const defaultGasLimit = () =>
     asset.gasLimit + GAS_PER_NON_ZERO_BYTE * ethUtil.toBuffer(txInput).length
 
-  const _isToken = isToken(asset)
-
   const isContract = await isContractAddressCached({ asset, address: toAddress })
 
-  if (_isToken) {
-    // only create tx-input only if not pass tx-input to a token asset
-    if (txInput === '0x') {
-      txInput = ethUtil.bufferToHex(
-        asset.contract.transfer.build(toAddress.toLowerCase(), amount.toBaseString())
-      )
-    }
-
+  if (isToken) {
     amount = asset.baseAsset.currency.ZERO
     toAddress = asset.contract.address
   } else if (

package/src/get-fee-async.js

@@ -2,6 +2,7 @@ import assert from 'minimalistic-assert'
 import { fetchGasLimit } from './gas-estimation'
 import { isForwarderContractCached } from './eth-like-util'
 import { getFeeFactory } from './get-fee'
+import { getNftArguments } from './nft-utils'
 
 const FIXED_TRANSFER_GAS_LIMIT_ASSETS = new Set([
   'amp',
@@ -33,38 +34,47 @@ const getFeeAsyncFactory = ({
   assert(createUnsignedTx, 'createUnsignedTx is required')
   const getFee = getFeeFactory({ gasLimit: defaultGasLimit })
   return async ({
+    nft,
     asset,
     fromAddress = '0xffffffffffffffffffffffffffffffffffffffff', // sending from a random address
     toAddress = '0xffffffffffffffffffffffffffffffffffffffff', // sending to a random address,
     amount,
     bip70,
-    txInput,
+    txInput: txInputPram,
     isExchange,
     customFee,
     calculateEffectiveFee,
     isSendAll,
     isRbfAllowed = true, // Destkop, isRbfAllowed=true when advanced panel is on
-    throwOnError = false,
   }) => {
-    const extraPercentage = await resolveExtraPercentage({ asset, toAddress })
+    const resolveGasLimit = async () => {
+      if (nft) {
+        return getNftArguments({ asset, nft, fromAddress, toAddress })
+      }
+
+      const extraPercentage = await resolveExtraPercentage({ asset, toAddress })
+      const txInput = txInputPram ?? '0x'
+      const gasLimit = await fetchGasLimit({
+        asset,
+        fromAddress,
+        toAddress,
+        txInput, // default value is '0x'
+        amount,
+        bip70,
+        extraPercentage,
+      })
+      return { gasLimit, txInput }
+    }
+
+    const { txInput, gasLimit, contractAddress } = await resolveGasLimit()
 
-    const gasLimit = await fetchGasLimit({
-      asset,
-      fromAddress,
-      toAddress,
-      txInput: txInput ?? '0x', // default value is '0x'
-      amount,
-      bip70,
-      throwOnError,
-      extraPercentage,
-    })
     const feeData = await assetClientInterface.getFeeConfig({ assetName: asset.baseAsset.name })
 
     const optimismL1DataFee = asset.baseAsset.estimateL1DataFee
       ? await asset.baseAsset.estimateL1DataFee({
           unsignedTx: createUnsignedTx({
             asset,
-            address: toAddress,
+            address: contractAddress || toAddress,
             fromAddress,
             amount,
             nonce: 0,
@@ -90,7 +100,13 @@ const getFeeAsyncFactory = ({
     const l1DataFee = optimismL1DataFee
       ? asset.baseAsset.currency.baseUnit(optimismL1DataFee)
       : asset.baseAsset.currency.ZERO
-    return { fee: fee.add(l1DataFee), optimismL1DataFee, gasLimit, ...rest }
+    return {
+      fee: fee.add(l1DataFee),
+      optimismL1DataFee,
+      gasLimit,
+      txInput,
+      ...rest,
+    }
   }
 }
 

package/src/nft-utils.js

@@ -0,0 +1,53 @@
+import SolidityContract from '@exodus/solidity-contract'
+import { fetchGasLimit } from './gas-estimation'
+import assert from 'minimalistic-assert'
+
+export const getNftArguments = async ({ asset, nft, fromAddress, toAddress }) => {
+  assert(asset, 'asset is required')
+  assert(nft, 'nft is required')
+  assert(fromAddress, 'fromAddress is required')
+  assert(toAddress, 'toAddress is required')
+  const { contractAddress, tokenId, contractType } = nft
+
+  const txInputs = []
+
+  if (!contractType || contractType === 'erc1155') {
+    const txInput = `0x${SolidityContract.simpleErc1155(contractAddress)
+      .safeTransferFrom.build(fromAddress, toAddress, tokenId, 1, [])
+      .toString('hex')}`
+    txInputs.push({ txInput, contractType: 'erc1155' })
+  }
+
+  if (!contractType || contractType === 'erc721') {
+    const txInput = `0x${SolidityContract.simpleErc721(contractAddress)
+      .safeTransferFrom.build(fromAddress, toAddress, tokenId)
+      .toString('hex')}`
+    txInputs.push({ txInput, contractType: 'erc721' })
+  }
+
+  const errors = [`Cannot fetch gas for ${asset.name}`]
+  const { txInput, gasLimit } = await Promise.any(
+    txInputs.map(async ({ txInput, contractType }) => {
+      try {
+        const gasLimit = await fetchGasLimit({
+          asset,
+          fromAddress,
+          toAddress: contractAddress,
+          txInput,
+          amount: asset.baseAsset.currency.ZERO,
+        })
+        return { txInput, gasLimit }
+      } catch (e) {
+        errors.push(`ContractType ${contractType} nft input ${txInput}. ${e.message}`)
+        throw e
+      }
+    })
+  ).catch((e) => {
+    throw new Error(errors.join('\n'))
+  })
+  return {
+    contractAddress,
+    gasLimit,
+    txInput,
+  }
+}

package/src/tx-send/get-fee-info.js

@@ -28,7 +28,6 @@ const getFeeInfo = async function getFeeInfo({
       amount,
       txInput,
       throwOnError: false,
-      isContract: feeOpts.isContract ?? undefined,
     })
   }
 

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

@@ -1,18 +1,16 @@
 /* eslint-disable  @exodus/export-default/last */
 
 import { getNonce, transactionExists } from '../eth-like-util'
-import {
-  calculateBumpedGasPrice,
-  isToken as checkIsToken,
-  normalizeTxId,
-} from '@exodus/ethereum-lib'
+import { calculateBumpedGasPrice, isEthereumLikeToken, normalizeTxId } from '@exodus/ethereum-lib'
 import assert from 'minimalistic-assert'
 import getFeeInfo from './get-fee-info'
+import { getNftArguments } from '../nft-utils'
 
 const txSendFactory = ({ assetClientInterface, createUnsignedTx }) => {
   assert(assetClientInterface, 'assetClientInterface is required')
   assert(createUnsignedTx, 'createUnsignedTx is required')
   return async ({
+    nft,
     asset,
     walletAccount,
     amount,
@@ -43,13 +41,21 @@ const txSendFactory = ({ assetClientInterface, createUnsignedTx }) => {
       assetName: baseAsset.name,
       walletAccount,
     })
-
-    let nonceParam = _nonce
-
     const feeData = await assetClientInterface.getFeeData({
       assetName: baseAsset.name,
     })
 
+    let contractAddress
+    if (nft) {
+      const nftArguments = await getNftArguments({ asset, nft, fromAddress, toAddress: address })
+      contractAddress = nftArguments.contractAddress
+      feeOpts.gasLimit = nftArguments.gasLimit
+      txInput = nftArguments.txInput
+      amount = asset.baseAsset.currency.ZERO
+    }
+
+    let nonceParam = _nonce
+
     let eip1559Enabled = feeData.eip1559Enabled
 
     // `replacedTx` is always an ETH/ETC transaction (not a token)
@@ -102,7 +108,7 @@ const txSendFactory = ({ assetClientInterface, createUnsignedTx }) => {
       assetClientInterface,
       asset,
       walletAccount,
-      toAddress: address,
+      toAddress: contractAddress || address,
       amount,
       nonce: nonceParam,
       fromAddress,
@@ -171,7 +177,7 @@ const txSendFactory = ({ assetClientInterface, createUnsignedTx }) => {
       ],
     })
 
-    const isToken = checkIsToken(asset)
+    const isToken = isEthereumLikeToken(asset)
     if (isToken) {
       await assetClientInterface.updateTxLogAndNotify({
         assetName: baseAsset.name,
@@ -224,7 +230,7 @@ const createTx = async ({
   feeOpts,
   createUnsignedTx,
 }) => {
-  const isToken = checkIsToken(asset)
+  const isToken = isEthereumLikeToken(asset)
 
   if (txInput && isToken && !keepTxInput)
     throw new Error(`Additional data for Ethereum Token (${asset.name}) is not allowed`)