From e48f17c1d3b1314846c4e05fcf5f1b215a12a9bb Mon Sep 17 00:00:00 2001
From: Francisco Ramos <jscriptcoder@gmail.com>
Date: Fri, 2 Jun 2023 20:22:52 +0200
Subject: [PATCH] chore(bridge-ui): improve type checking (#13856)

Co-authored-by: dave | d1onys1us <13951458+d1onys1us@users.noreply.github.com>
---
 .../bridge-ui/src/bridge/ETHBridge.spec.ts    |  2 +
 .../components/BridgeForm/BridgeForm.svelte   |  4 +-
 packages/bridge-ui/src/constants/envVars.ts   | 35 ++++----
 packages/bridge-ui/src/domain/chain.ts        |  3 +-
 packages/bridge-ui/src/domain/relayerApi.ts   | 12 +--
 packages/bridge-ui/src/domain/token.ts        |  7 +-
 packages/bridge-ui/src/globals.d.ts           |  1 +
 .../bridge-ui/src/proof/ProofService.spec.ts  | 80 +++++++++----------
 .../src/relayer-api/RelayerAPIService.spec.ts |  2 +-
 .../src/relayer-api/RelayerAPIService.ts      |  8 +-
 .../bridge-ui/src/signer/subscriber.spec.ts   |  2 +-
 packages/bridge-ui/src/signer/subscriber.ts   |  9 ++-
 .../src/storage/StorageService.spec.ts        |  2 +-
 .../bridge-ui/src/storage/StorageService.ts   | 11 +--
 .../bridge-ui/src/utils/addressAvatar.spec.ts |  2 -
 .../src/utils/addressSubsection.spec.ts       |  2 -
 .../checkIfTokenIsDeployedCrossChain.spec.ts  |  4 +-
 .../bridge-ui/src/utils/getAddressForToken.ts |  3 +-
 .../src/utils/isOnCorrectChain.spec.ts        |  2 +-
 .../src/utils/recommendProcessingFee.spec.ts  |  3 +-
 .../src/utils/truncateString.spec.ts          |  2 -
 packages/bridge-ui/src/vault/tokenVaults.ts   |  4 +-
 22 files changed, 97 insertions(+), 103 deletions(-)

diff --git a/packages/bridge-ui/src/bridge/ETHBridge.spec.ts b/packages/bridge-ui/src/bridge/ETHBridge.spec.ts
index 466b0d58428..e99479e61aa 100644
--- a/packages/bridge-ui/src/bridge/ETHBridge.spec.ts
+++ b/packages/bridge-ui/src/bridge/ETHBridge.spec.ts
@@ -68,6 +68,8 @@ describe('bridge tests', () => {
       contractAddress: '0x1234',
       spenderAddress: '0x',
     });
+
+    expect(tx).toEqual({});
   });
 
   it('bridges with processing fee, owner !== to', async () => {
diff --git a/packages/bridge-ui/src/components/BridgeForm/BridgeForm.svelte b/packages/bridge-ui/src/components/BridgeForm/BridgeForm.svelte
index 59485954107..b43a49fe32c 100644
--- a/packages/bridge-ui/src/components/BridgeForm/BridgeForm.svelte
+++ b/packages/bridge-ui/src/components/BridgeForm/BridgeForm.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-  import { fetchFeeData } from '@wagmi/core';
+  import { type Address,fetchFeeData } from '@wagmi/core';
   import { BigNumber, Contract, ethers, type Signer } from 'ethers';
   import { _ } from 'svelte-i18n';
 
@@ -357,7 +357,7 @@
       // Set it manually.
       tx.chainId = $srcChain.id;
 
-      const userAddress = await $signer.getAddress();
+      const userAddress = (await $signer.getAddress()) as Address;
 
       let transactions: BridgeTransaction[] =
         await storageService.getAllByAddress(userAddress);
diff --git a/packages/bridge-ui/src/constants/envVars.ts b/packages/bridge-ui/src/constants/envVars.ts
index cc595ab70c7..a105b65fabb 100644
--- a/packages/bridge-ui/src/constants/envVars.ts
+++ b/packages/bridge-ui/src/constants/envVars.ts
@@ -1,22 +1,23 @@
 // TODO: think about a way to blow up the build if env var is missing
 //       dotenv-safe ?
 // TODO: explain each of these env vars
+import type { Address } from '@wagmi/core';
 
-import type { Address, ChainID } from '../domain/chain';
+import type { ChainID } from '../domain/chain';
 
 export const L1_RPC: string = import.meta.env?.VITE_L1_RPC_URL;
 
-export const L1_TOKEN_VAULT_ADDRESS: Address = import.meta.env
-  ?.VITE_L1_TOKEN_VAULT_ADDRESS;
+export const L1_TOKEN_VAULT_ADDRESS = import.meta.env
+  ?.VITE_L1_TOKEN_VAULT_ADDRESS as Address;
 
-export const L1_BRIDGE_ADDRESS: Address = import.meta.env
-  ?.VITE_L1_BRIDGE_ADDRESS;
+export const L1_BRIDGE_ADDRESS = import.meta.env
+  ?.VITE_L1_BRIDGE_ADDRESS as Address;
 
-export const L1_CROSS_CHAIN_SYNC_ADDRESS: Address = import.meta.env
-  ?.VITE_L1_CROSS_CHAIN_SYNC_ADDRESS;
+export const L1_CROSS_CHAIN_SYNC_ADDRESS = import.meta.env
+  ?.VITE_L1_CROSS_CHAIN_SYNC_ADDRESS as Address;
 
-export const L1_SIGNAL_SERVICE_ADDRESS: Address = import.meta.env
-  ?.VITE_L1_SIGNAL_SERVICE_ADDRESS;
+export const L1_SIGNAL_SERVICE_ADDRESS = import.meta.env
+  ?.VITE_L1_SIGNAL_SERVICE_ADDRESS as Address;
 
 export const L1_CHAIN_ID: ChainID = parseInt(import.meta.env?.VITE_L1_CHAIN_ID);
 
@@ -26,17 +27,17 @@ export const L1_EXPLORER_URL: string = import.meta.env?.VITE_L1_EXPLORER_URL;
 
 export const L2_RPC: string = import.meta.env?.VITE_L2_RPC_URL;
 
-export const L2_TOKEN_VAULT_ADDRESS: Address = import.meta.env
-  ?.VITE_L2_TOKEN_VAULT_ADDRESS;
+export const L2_TOKEN_VAULT_ADDRESS = import.meta.env
+  ?.VITE_L2_TOKEN_VAULT_ADDRESS as Address;
 
-export const L2_BRIDGE_ADDRESS: Address = import.meta.env
-  ?.VITE_L2_BRIDGE_ADDRESS;
+export const L2_BRIDGE_ADDRESS = import.meta.env
+  ?.VITE_L2_BRIDGE_ADDRESS as Address;
 
-export const L2_CROSS_CHAIN_SYNC_ADDRESS: Address = import.meta.env
-  ?.VITE_L2_CROSS_CHAIN_SYNC_ADDRESS;
+export const L2_CROSS_CHAIN_SYNC_ADDRESS = import.meta.env
+  ?.VITE_L2_CROSS_CHAIN_SYNC_ADDRESS as Address;
 
-export const L2_SIGNAL_SERVICE_ADDRESS: Address = import.meta.env
-  ?.VITE_L2_SIGNAL_SERVICE_ADDRESS;
+export const L2_SIGNAL_SERVICE_ADDRESS = import.meta.env
+  ?.VITE_L2_SIGNAL_SERVICE_ADDRESS as Address;
 
 export const L2_CHAIN_ID: ChainID = parseInt(import.meta.env?.VITE_L2_CHAIN_ID);
 
diff --git a/packages/bridge-ui/src/domain/chain.ts b/packages/bridge-ui/src/domain/chain.ts
index c99db063a3b..e9ca5702070 100644
--- a/packages/bridge-ui/src/domain/chain.ts
+++ b/packages/bridge-ui/src/domain/chain.ts
@@ -1,9 +1,8 @@
+import type { Address } from '@wagmi/core';
 import type { ComponentType } from 'svelte';
 
 export type ChainID = number;
 
-export type Address = string;
-
 export type Chain = {
   id: ChainID;
   name: string;
diff --git a/packages/bridge-ui/src/domain/relayerApi.ts b/packages/bridge-ui/src/domain/relayerApi.ts
index 9b399fbb689..831f3bf7f66 100644
--- a/packages/bridge-ui/src/domain/relayerApi.ts
+++ b/packages/bridge-ui/src/domain/relayerApi.ts
@@ -1,4 +1,6 @@
-import type { Address, ChainID } from './chain';
+import type { Address } from '@wagmi/core';
+
+import type { ChainID } from './chain';
 import type { BridgeTransaction } from './transaction';
 
 export type GetAllByAddressResponse = {
@@ -13,7 +15,7 @@ export type PaginationParams = {
 
 export interface RelayerAPI {
   getAllBridgeTransactionByAddress(
-    address: string,
+    address: Address,
     pagination: PaginationParams,
     chainID?: number,
   ): Promise<GetAllByAddressResponse>;
@@ -49,13 +51,13 @@ export type APIResponseTransaction = {
   status: number;
   eventType: number;
   chainID: number;
-  canonicalTokenAddress: string;
+  canonicalTokenAddress: Address;
   canonicalTokenSymbol: string;
   canonicalTokenName: string;
   canonicalTokenDecimals: number;
   amount: string;
   msgHash: string;
-  messageOwner: string;
+  messageOwner: Address;
   event: string;
 };
 
@@ -66,7 +68,7 @@ export type RelayerBlockInfo = {
 };
 
 export type APIRequestParams = {
-  address: string;
+  address: Address;
   chainID?: number;
   event?: string;
 };
diff --git a/packages/bridge-ui/src/domain/token.ts b/packages/bridge-ui/src/domain/token.ts
index 7c23698f4e5..506ba443027 100644
--- a/packages/bridge-ui/src/domain/token.ts
+++ b/packages/bridge-ui/src/domain/token.ts
@@ -1,13 +1,14 @@
+import type { Address } from '@wagmi/core';
 import type { ComponentType } from 'svelte';
 
-type Address = {
+type TokenAddress = {
   chainId: number;
-  address: string;
+  address: Address;
 };
 
 export type Token = {
   name: string;
-  addresses: Address[];
+  addresses: TokenAddress[];
   symbol: string;
   decimals: number;
   logoUrl?: string;
diff --git a/packages/bridge-ui/src/globals.d.ts b/packages/bridge-ui/src/globals.d.ts
index 1820e7ab737..1c230e6b869 100644
--- a/packages/bridge-ui/src/globals.d.ts
+++ b/packages/bridge-ui/src/globals.d.ts
@@ -1 +1,2 @@
 declare module '@lottiefiles/svelte-lottie-player';
+declare module 'identicon.js';
diff --git a/packages/bridge-ui/src/proof/ProofService.spec.ts b/packages/bridge-ui/src/proof/ProofService.spec.ts
index 93434c1b461..f2e1a8113c6 100644
--- a/packages/bridge-ui/src/proof/ProofService.spec.ts
+++ b/packages/bridge-ui/src/proof/ProofService.spec.ts
@@ -121,17 +121,15 @@ describe('prover tests', () => {
   });
 
   it('throws on invalid proof', async () => {
-    mockProvider.send.mockImplementation(
-      (method: string, params: unknown[]) => {
-        if (method === 'eth_getBlockByHash') {
-          return block;
-        }
-
-        if (method === 'eth_getProof') {
-          return invalidStorageProof;
-        }
-      },
-    );
+    mockProvider.send.mockImplementation((method: string) => {
+      if (method === 'eth_getBlockByHash') {
+        return block;
+      }
+
+      if (method === 'eth_getProof') {
+        return invalidStorageProof;
+      }
+    });
 
     const prover: ProofService = new ProofService(map);
 
@@ -149,17 +147,15 @@ describe('prover tests', () => {
   });
 
   it('generates proof', async () => {
-    mockProvider.send.mockImplementation(
-      (method: string, params: unknown[]) => {
-        if (method === 'eth_getBlockByHash') {
-          return block;
-        }
-
-        if (method === 'eth_getProof') {
-          return storageProof;
-        }
-      },
-    );
+    mockProvider.send.mockImplementation((method: string) => {
+      if (method === 'eth_getBlockByHash') {
+        return block;
+      }
+
+      if (method === 'eth_getProof') {
+        return storageProof;
+      }
+    });
 
     const prover: ProofService = new ProofService(map);
 
@@ -183,17 +179,15 @@ describe('generate release proof tests', () => {
   });
 
   it('throws on invalid proof', async () => {
-    mockProvider.send.mockImplementation(
-      (method: string, params: unknown[]) => {
-        if (method === 'eth_getBlockByHash') {
-          return block;
-        }
-
-        if (method === 'eth_getProof') {
-          return invalidStorageProof2;
-        }
-      },
-    );
+    mockProvider.send.mockImplementation((method: string) => {
+      if (method === 'eth_getBlockByHash') {
+        return block;
+      }
+
+      if (method === 'eth_getProof') {
+        return invalidStorageProof2;
+      }
+    });
 
     const prover: ProofService = new ProofService(map);
 
@@ -211,17 +205,15 @@ describe('generate release proof tests', () => {
   });
 
   it('generates proof', async () => {
-    mockProvider.send.mockImplementation(
-      (method: string, params: unknown[]) => {
-        if (method === 'eth_getBlockByHash') {
-          return block;
-        }
-
-        if (method === 'eth_getProof') {
-          return storageProof2;
-        }
-      },
-    );
+    mockProvider.send.mockImplementation((method: string) => {
+      if (method === 'eth_getBlockByHash') {
+        return block;
+      }
+
+      if (method === 'eth_getProof') {
+        return storageProof2;
+      }
+    });
 
     const prover: ProofService = new ProofService(map);
 
diff --git a/packages/bridge-ui/src/relayer-api/RelayerAPIService.spec.ts b/packages/bridge-ui/src/relayer-api/RelayerAPIService.spec.ts
index 6a42d31c170..b0a4f508244 100644
--- a/packages/bridge-ui/src/relayer-api/RelayerAPIService.spec.ts
+++ b/packages/bridge-ui/src/relayer-api/RelayerAPIService.spec.ts
@@ -146,7 +146,7 @@ describe('RelayerAPIService', () => {
       data: dataFromAPI,
     });
 
-    const data = await relayerApi.getAllBridgeTransactionByAddress('0x123', {
+    await relayerApi.getAllBridgeTransactionByAddress('0x123', {
       page: 0,
       size: 100,
     });
diff --git a/packages/bridge-ui/src/relayer-api/RelayerAPIService.ts b/packages/bridge-ui/src/relayer-api/RelayerAPIService.ts
index 59b6b9f0da9..594c593382c 100644
--- a/packages/bridge-ui/src/relayer-api/RelayerAPIService.ts
+++ b/packages/bridge-ui/src/relayer-api/RelayerAPIService.ts
@@ -1,9 +1,9 @@
+import type { Address } from '@wagmi/core';
 import axios from 'axios';
 import { BigNumber, Contract, ethers } from 'ethers';
 
 import { chains } from '../chain/chains';
 import { bridgeABI, erc20ABI, tokenVaultABI } from '../constants/abi';
-import type { Address } from '../domain/chain';
 import { MessageStatus } from '../domain/message';
 import type { RecordProviders } from '../domain/provider';
 import type {
@@ -126,14 +126,10 @@ export class RelayerAPIService implements RelayerAPI {
   }
 
   async getAllBridgeTransactionByAddress(
-    address: string,
+    address: Address,
     paginationParams: PaginationParams,
     chainID?: number,
   ): Promise<GetAllByAddressResponse> {
-    if (!address) {
-      throw new Error('address needed to fetch transactions');
-    }
-
     const params = {
       address,
       chainID,
diff --git a/packages/bridge-ui/src/signer/subscriber.spec.ts b/packages/bridge-ui/src/signer/subscriber.spec.ts
index 5abbeed7061..8a22991c6cd 100644
--- a/packages/bridge-ui/src/signer/subscriber.spec.ts
+++ b/packages/bridge-ui/src/signer/subscriber.spec.ts
@@ -1,4 +1,4 @@
-import type { Signer, Transaction } from 'ethers';
+import type { Signer } from 'ethers';
 
 import type { PaginationInfo, RelayerBlockInfo } from '../domain/relayerApi';
 import type { Token } from '../domain/token';
diff --git a/packages/bridge-ui/src/signer/subscriber.ts b/packages/bridge-ui/src/signer/subscriber.ts
index ed517bada55..51b6ee534fb 100644
--- a/packages/bridge-ui/src/signer/subscriber.ts
+++ b/packages/bridge-ui/src/signer/subscriber.ts
@@ -1,4 +1,5 @@
-import type { Signer } from 'ethers';
+import type { Address } from '@wagmi/core';
+import { constants, type Signer } from 'ethers';
 
 import type { PaginationInfo } from '../domain/relayerApi';
 import type { BridgeTransaction } from '../domain/transaction';
@@ -13,7 +14,7 @@ const log = getLogger('signer:subscriber');
 
 // We keep track of the current user address to avoid
 // querying the API for transactions if the address is the same.
-let currentUserAddress = '';
+let currentUserAddress: Address = constants.AddressZero;
 
 /**
  * Subscribe to signer changes.
@@ -25,7 +26,7 @@ export async function subscribeToSigner(newSigner: Signer | null) {
   if (newSigner) {
     log('New signer set', newSigner);
 
-    const userAddress = await newSigner.getAddress();
+    const userAddress = (await newSigner.getAddress()) as Address;
 
     // We actually don't want to run all this if the
     // new address is the same as the previous one, since it
@@ -84,7 +85,7 @@ export async function subscribeToSigner(newSigner: Signer | null) {
   } else {
     log('Signer deleted');
 
-    currentUserAddress = '';
+    currentUserAddress = constants.AddressZero;
     transactions.set([]);
     userTokens.set([]);
     paginationInfo.set(null);
diff --git a/packages/bridge-ui/src/storage/StorageService.spec.ts b/packages/bridge-ui/src/storage/StorageService.spec.ts
index 0738dc59af6..d079c5f4a52 100644
--- a/packages/bridge-ui/src/storage/StorageService.spec.ts
+++ b/packages/bridge-ui/src/storage/StorageService.spec.ts
@@ -1,4 +1,4 @@
-import { BigNumber, BigNumberish, ethers } from 'ethers';
+import { BigNumber } from 'ethers';
 
 import { L1_CHAIN_ID, L2_CHAIN_ID } from '../constants/envVars';
 import { MessageStatus } from '../domain/message';
diff --git a/packages/bridge-ui/src/storage/StorageService.ts b/packages/bridge-ui/src/storage/StorageService.ts
index 3c868899506..5265186c2da 100644
--- a/packages/bridge-ui/src/storage/StorageService.ts
+++ b/packages/bridge-ui/src/storage/StorageService.ts
@@ -1,8 +1,9 @@
+import type { Address } from '@wagmi/core';
 import { BigNumber, Contract, ethers } from 'ethers';
 
 import { chains } from '../chain/chains';
 import { bridgeABI, erc20ABI, tokenVaultABI } from '../constants/abi';
-import type { Address, ChainID } from '../domain/chain';
+import type { ChainID } from '../domain/chain';
 import { MessageStatus } from '../domain/message';
 import type { BridgeTransaction, Transactioner } from '../domain/transaction';
 import { jsonParseOrEmptyArray } from '../utils/jsonParseOrEmptyArray';
@@ -111,7 +112,7 @@ export class StorageService implements Transactioner {
     this.providers = providers;
   }
 
-  private _getTransactionsFromStorage(address: string): BridgeTransaction[] {
+  private _getTransactionsFromStorage(address: Address): BridgeTransaction[] {
     const existingTransactions = this.storage.getItem(
       `${STORAGE_PREFIX}-${address.toLowerCase()}`,
     );
@@ -119,7 +120,7 @@ export class StorageService implements Transactioner {
     return jsonParseOrEmptyArray<BridgeTransaction>(existingTransactions);
   }
 
-  async getAllByAddress(address: string): Promise<BridgeTransaction[]> {
+  async getAllByAddress(address: Address): Promise<BridgeTransaction[]> {
     const txs = this._getTransactionsFromStorage(address);
 
     log('Transactions from storage', txs);
@@ -231,7 +232,7 @@ export class StorageService implements Transactioner {
   }
 
   async getTransactionByHash(
-    address: string,
+    address: Address,
     hash: string,
   ): Promise<BridgeTransaction | undefined> {
     const txs = this._getTransactionsFromStorage(address);
@@ -329,7 +330,7 @@ export class StorageService implements Transactioner {
     return bridgeTx;
   }
 
-  updateStorageByAddress(address: string, txs: BridgeTransaction[] = []) {
+  updateStorageByAddress(address: Address, txs: BridgeTransaction[] = []) {
     log('Updating storage with transactions', txs);
     this.storage.setItem(
       `${STORAGE_PREFIX}-${address.toLowerCase()}`,
diff --git a/packages/bridge-ui/src/utils/addressAvatar.spec.ts b/packages/bridge-ui/src/utils/addressAvatar.spec.ts
index 4eae55f49da..30ac9f5c19f 100644
--- a/packages/bridge-ui/src/utils/addressAvatar.spec.ts
+++ b/packages/bridge-ui/src/utils/addressAvatar.spec.ts
@@ -15,7 +15,5 @@ it('should return a base64 avatar string', () => {
 });
 
 it('should return default base64 avatar when no address is passed', () => {
-  const dummyAddress = '';
-
   expect(getAddressAvatarFromIdenticon('')).toStrictEqual(DEFAULT_IDENTICON);
 });
diff --git a/packages/bridge-ui/src/utils/addressSubsection.spec.ts b/packages/bridge-ui/src/utils/addressSubsection.spec.ts
index 25b0a0cc186..03f371d1a8c 100644
--- a/packages/bridge-ui/src/utils/addressSubsection.spec.ts
+++ b/packages/bridge-ui/src/utils/addressSubsection.spec.ts
@@ -7,7 +7,5 @@ it('should return string with prefix and suffix', () => {
 });
 
 it('should return 0x if empty', () => {
-  const dummyAddress = '';
-
   expect(addressSubsection('')).toStrictEqual('0x');
 });
diff --git a/packages/bridge-ui/src/utils/checkIfTokenIsDeployedCrossChain.spec.ts b/packages/bridge-ui/src/utils/checkIfTokenIsDeployedCrossChain.spec.ts
index b447f39008c..d60c755d420 100644
--- a/packages/bridge-ui/src/utils/checkIfTokenIsDeployedCrossChain.spec.ts
+++ b/packages/bridge-ui/src/utils/checkIfTokenIsDeployedCrossChain.spec.ts
@@ -30,7 +30,7 @@ describe('checkIfTokenIsDeployedCrossChain', () => {
     name: 'MyChain',
     rpc: 'http://mychain.rpc.com',
     explorerUrl: '',
-    signalServiceAddress: '',
+    signalServiceAddress: '0x1234567890',
     bridgeAddress: '0x1234567890',
     crossChainSyncAddress: '0x0987654321',
   };
@@ -40,7 +40,7 @@ describe('checkIfTokenIsDeployedCrossChain', () => {
     name: 'SomeOtherChain',
     rpc: 'http://otherchain.rpc.com',
     explorerUrl: '',
-    signalServiceAddress: '',
+    signalServiceAddress: '0x9876543210',
     bridgeAddress: '0x9876543210',
     crossChainSyncAddress: '0x0123456789',
   };
diff --git a/packages/bridge-ui/src/utils/getAddressForToken.ts b/packages/bridge-ui/src/utils/getAddressForToken.ts
index da6aece6fe8..cd5409ecfda 100644
--- a/packages/bridge-ui/src/utils/getAddressForToken.ts
+++ b/packages/bridge-ui/src/utils/getAddressForToken.ts
@@ -1,7 +1,8 @@
+import type { Address } from '@wagmi/core';
 import { Contract, Signer } from 'ethers';
 
 import { tokenVaultABI } from '../constants/abi';
-import type { Address, Chain } from '../domain/chain';
+import type { Chain } from '../domain/chain';
 import type { Token } from '../domain/token';
 import { isETH } from '../token/tokens';
 import { tokenVaults } from '../vault/tokenVaults';
diff --git a/packages/bridge-ui/src/utils/isOnCorrectChain.spec.ts b/packages/bridge-ui/src/utils/isOnCorrectChain.spec.ts
index c06ed481d07..82f37e2da10 100644
--- a/packages/bridge-ui/src/utils/isOnCorrectChain.spec.ts
+++ b/packages/bridge-ui/src/utils/isOnCorrectChain.spec.ts
@@ -1,4 +1,4 @@
-import { ethers, Wallet } from 'ethers';
+import { Wallet } from 'ethers';
 
 import { taikoChain } from '../chain/chains';
 import { isOnCorrectChain } from './isOnCorrectChain';
diff --git a/packages/bridge-ui/src/utils/recommendProcessingFee.spec.ts b/packages/bridge-ui/src/utils/recommendProcessingFee.spec.ts
index 862ea106311..6f99084f8d1 100644
--- a/packages/bridge-ui/src/utils/recommendProcessingFee.spec.ts
+++ b/packages/bridge-ui/src/utils/recommendProcessingFee.spec.ts
@@ -7,7 +7,7 @@ import { ProcessingFeeMethod } from '../domain/fee';
 import type { Token } from '../domain/token';
 import { providers } from '../provider/providers';
 import { signer } from '../store/signer';
-import { ETHToken, testERC20Tokens, TKOToken } from '../token/tokens';
+import { ETHToken, testERC20Tokens } from '../token/tokens';
 import {
   erc20DeployedGasLimit,
   erc20NotDeployedGasLimit,
@@ -51,6 +51,7 @@ const mockToken = {
   ],
   decimals: 18,
   symbol: 'MKT',
+  logoComponent: null,
 } as Token;
 
 describe('recommendProcessingFee()', () => {
diff --git a/packages/bridge-ui/src/utils/truncateString.spec.ts b/packages/bridge-ui/src/utils/truncateString.spec.ts
index 561e6132a92..4303ea32e36 100644
--- a/packages/bridge-ui/src/utils/truncateString.spec.ts
+++ b/packages/bridge-ui/src/utils/truncateString.spec.ts
@@ -14,7 +14,5 @@ it('should return string when < maxLength', () => {
 });
 
 it('should return empty string if empty', () => {
-  const dummyAddress = '';
-
   expect(truncateString('')).toStrictEqual('');
 });
diff --git a/packages/bridge-ui/src/vault/tokenVaults.ts b/packages/bridge-ui/src/vault/tokenVaults.ts
index bc90c77d334..6d8d8d86e07 100644
--- a/packages/bridge-ui/src/vault/tokenVaults.ts
+++ b/packages/bridge-ui/src/vault/tokenVaults.ts
@@ -1,10 +1,12 @@
+import type { Address } from '@wagmi/core';
+
 import {
   L1_CHAIN_ID,
   L1_TOKEN_VAULT_ADDRESS,
   L2_CHAIN_ID,
   L2_TOKEN_VAULT_ADDRESS,
 } from '../constants/envVars';
-import type { Address, ChainID } from '../domain/chain';
+import type { ChainID } from '../domain/chain';
 
 export const tokenVaults: Record<ChainID, Address> = {
   [L1_CHAIN_ID]: L1_TOKEN_VAULT_ADDRESS,