put an error on the swap & bridge page indicating that the user doesn't have enough native even before going for quotes

Author: Oxbobby

src/controllers/swapAndBridge/swapAndBridge.ts

@@ -1206,7 +1206,7 @@ export class SwapAndBridgeController extends EventEmitter implements ISwapAndBri
     )
     if (!native) return 0n
 
-    if (this.fromSelectedToken?.address !== ZeroAddress) return native.amount
+    if (this.fromSelectedToken?.address !== ZeroAddress || amount === 0n) return native.amount
 
     // subtract the from amount from the portfolio available balance
     if (amount > native.amount) return 0n
@@ -2465,6 +2465,33 @@ export class SwapAndBridgeController extends EventEmitter implements ISwapAndBri
       })
     }
 
+    // if the account cannot use other fee payment options than native
+    // and the account doesn't have any native,
+    // show an error
+    const accountNativeBalance = this.#accountNativeBalance(0n)
+    if (this.#selectedAccount.account && this.fromChainId && accountNativeBalance === 0n) {
+      const network = this.#networks.networks.find(
+        (net) => net.chainId === BigInt(this.fromChainId!)
+      )
+      if (network) {
+        const accountState =
+          this.#accounts.accountStates[this.#selectedAccount.account.addr][this.fromChainId]
+        if (accountState) {
+          const baseAcc = getBaseAccount(
+            this.#selectedAccount.account,
+            accountState,
+            this.#keystore.getAccountKeys(this.#selectedAccount.account),
+            network
+          )
+          if (!baseAcc.canPayWithTokens() && !baseAcc.canPayWithEOA()) {
+            errors.push({
+              title: `Insufficient ${network.nativeAssetSymbol} to cover the gas fees`
+            })
+          }
+        }
+      }
+    }
+
     return errors
   }
 

src/interfaces/account.ts

@@ -54,6 +54,7 @@ export interface AccountOnchainState {
   isSmarterEoa: boolean
   delegatedContract: Hex | null
   delegatedContractName: 'AMBIRE' | 'METAMASK' | 'UNKNOWN' | null
+  hasEoaWithNative: boolean
 }
 
 export type AccountStates = {

src/libs/account/BaseAccount.ts

@@ -86,6 +86,16 @@ export abstract class BaseAccount {
    */
   abstract getNonceId(): string
 
+  /**
+   * Can the account pay with tokens the transaction fee
+   */
+  abstract canPayWithTokens(): boolean
+
+  /**
+   * Can the account pay with tokens the transaction fee
+   */
+  abstract canPayWithEOA(): boolean
+
   // this is specific for v2 accounts, hardcoding a false for all else
   shouldIncludeActivatorCall(broadcastOption: string) {
     return false

src/libs/account/EOA.ts

@@ -117,4 +117,12 @@ export class EOA extends BaseAccount {
     // EOAs have only an execution layer nonce
     return this.accountState.eoaNonce!.toString()
   }
+
+  canPayWithTokens(): boolean {
+    return false
+  }
+
+  canPayWithEOA(): boolean {
+    return false
+  }
 }

src/libs/account/EOA7702.ts

@@ -3,6 +3,7 @@ import { Interface } from 'ethers'
 import AmbireAccount from '../../../contracts/compiled/AmbireAccount.json'
 import AmbireAccount7702 from '../../../contracts/compiled/AmbireAccount7702.json'
 import { Hex } from '../../interfaces/hex'
+import { has7702 } from '../7702/7702'
 import { AccountOp, getSignableCalls } from '../accountOp/accountOp'
 import { BROADCAST_OPTIONS } from '../broadcast/broadcast'
 import {
@@ -197,4 +198,12 @@ export class EOA7702 extends BaseAccount {
     // 7702 accounts have an execution layer nonce and an entry point nonce
     return `${this.accountState.eoaNonce!.toString()}-${this.accountState.erc4337Nonce.toString()}`
   }
+
+  canPayWithTokens(): boolean {
+    return has7702(this.network) && this.network.erc4337.hasPaymaster
+  }
+
+  canPayWithEOA(): boolean {
+    return false
+  }
 }

src/libs/account/V1.ts

@@ -98,4 +98,12 @@ export class V1 extends BaseAccount {
     // v1 accounts can only have an ambire smart contract nonce
     return this.accountState.nonce.toString()
   }
+
+  canPayWithTokens(): boolean {
+    return this.network.hasRelayer
+  }
+
+  canPayWithEOA(): boolean {
+    return this.accountState.hasEoaWithNative
+  }
 }

src/libs/account/V2.ts

@@ -173,4 +173,16 @@ export class V2 extends BaseAccount {
     // v2 accounts have two nonces: ambire smart account & entry point nonce
     return `${this.accountState.nonce.toString()}-${this.accountState.erc4337Nonce.toString()}`
   }
+
+  canPayWithTokens(): boolean {
+    if (this.network.erc4337.enabled) {
+      return this.network.erc4337.hasPaymaster
+    }
+
+    return this.network.hasRelayer
+  }
+
+  canPayWithEOA(): boolean {
+    return this.accountState.hasEoaWithNative
+  }
 }

src/libs/accountState/accountState.ts

@@ -73,6 +73,7 @@ export async function getAccountState(
     getEOAsCode(eoas)
   ])
 
+  const hasEoaWithNative = !!accountStateResult.find((res: any) => res.isEOA && res.balance > 0)
   const result: AccountOnchainState[] = accountStateResult.map((accResult: any, index: number) => {
     const associatedKeys = accResult.associatedKeyPrivileges.map(
       (privilege: string, keyIndex: number) => {
@@ -130,7 +131,8 @@ export async function getAccountState(
         account.associatedKeys.length > 0 && accResult.associatedKeyPrivileges.length === 0,
       isSmarterEoa,
       delegatedContract,
-      delegatedContractName
+      delegatedContractName,
+      hasEoaWithNative
     }
   })