feat: update bots to accommodate mode (#1528)

* update finalizer to accommodate mode Signed-off-by: Bennett <[email protected]> * fix multicall3 address and incorrect mapping Signed-off-by: Bennett <[email protected]> * refactor weth abi for op stack networks Signed-off-by: Bennett <[email protected]> * condense op standard bridge abi Signed-off-by: Bennett <[email protected]> * update fee query and whitelisted tokens Signed-off-by: Bennett <[email protected]> * do not prioritize repayment on Mode Signed-off-by: Bennett <[email protected]> * add WBTC to mode adapter client Signed-off-by: Bennett <[email protected]> * Update contracts-v3 * Update sendTokens.ts * Bump eth-optimism sdk * wip * Update OpStackAdapter.ts * Update package.json * Update OpStackAdapter.ts * fix bundle data client * add new atomic depositor address * Update yarn.lock * chore(lint): fix json Signed-off-by: james-a-morris <[email protected]> * refactor ovm standard bridge ABI Signed-off-by: Bennett <[email protected]> * Update OpStackAdapter.ts --------- Signed-off-by: Bennett <[email protected]> Signed-off-by: james-a-morris <[email protected]> Co-authored-by: nicholaspai <[email protected]> Co-authored-by: james-a-morris <[email protected]>
across-protocol · May 23, 2024 · b03e3db · b03e3db
1 parent c8a5af8
commit b03e3db
Show file tree

Hide file tree

Showing 20 changed files with 338 additions and 326 deletions.
diff --git a/contracts/AtomicWethDepositor.sol b/contracts/AtomicWethDepositor.sol
@@ -46,6 +46,7 @@ interface LineaL1MessageService {
 contract AtomicWethDepositor {
     Weth public immutable weth = Weth(0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2);
     OvmL1Bridge public immutable optimismL1Bridge = OvmL1Bridge(0x99C9fc46f92E8a1c0deC1b1747d010903E884bE1);
+    OvmL1Bridge public immutable modeL1Bridge = OvmL1Bridge(0x735aDBbE72226BD52e818E7181953f42E3b0FF21);
     OvmL1Bridge public immutable bobaL1Bridge = OvmL1Bridge(0xdc1664458d2f0B6090bEa60A8793A4E66c2F1c00);
     OvmL1Bridge public immutable baseL1Bridge = OvmL1Bridge(0x3154Cf16ccdb4C6d922629664174b904d80F2C35);
     PolygonL1Bridge public immutable polygonL1Bridge = PolygonL1Bridge(0xA0c68C638235ee32657e8f720a23ceC1bFc77C77);
@@ -55,6 +56,7 @@ contract AtomicWethDepositor {
 
     event ZkSyncEthDepositInitiated(address indexed from, address indexed to, uint256 amount);
     event LineaEthDepositInitiated(address indexed from, address indexed to, uint256 amount);
+    event OvmEthDepositInitiated(uint256 indexed chainId, address indexed from, address indexed to, uint256 amount);
 
     function bridgeWethToOvm(address to, uint256 amount, uint32 l2Gas, uint256 chainId) public {
         weth.transferFrom(msg.sender, address(this), amount);
@@ -64,11 +66,15 @@ contract AtomicWethDepositor {
             optimismL1Bridge.depositETHTo{ value: amount }(to, l2Gas, "");
         } else if (chainId == 8453) {
             baseL1Bridge.depositETHTo{ value: amount }(to, l2Gas, "");
+        } else if (chainId == 34443) {
+            modeL1Bridge.depositETHTo{ value: amount }(to, l2Gas, "");
         } else if (chainId == 288) {
             bobaL1Bridge.depositETHTo{ value: amount }(to, l2Gas, "");
         } else {
             revert("Invalid OVM chainId");
         }
+
+        emit OvmEthDepositInitiated(chainId, msg.sender, to, amount);
     }
 
     function bridgeWethToPolygon(address to, uint256 amount) public {

diff --git a/deploy/001_deploy_atomic_depositor.ts b/deploy/001_deploy_atomic_depositor.ts
@@ -11,7 +11,7 @@ const func: DeployFunction = async function (hre: HardhatRuntimeEnvironment) {
   await deploy("AtomicWethDepositor", {
     from: deployer,
     log: true,
-    skipIfAlreadyDeployed: true,
+    skipIfAlreadyDeployed: false,
   });
 };
 module.exports = func;

diff --git a/deployments/mainnet/AtomicWethDepositor.json b/deployments/mainnet/AtomicWethDepositor.json
diff --git a/deployments/mainnet/solcInputs/5700d0188c8ea770a99799abe93c0e83.json b/deployments/mainnet/solcInputs/5700d0188c8ea770a99799abe93c0e83.json
@@ -0,0 +1,21 @@
+{
+  "language": "Solidity",
+  "sources": {
+    "contracts/AtomicWethDepositor.sol": {
+      "content": "// SPDX-License-Identifier: GPL-3.0-only\npragma solidity ^0.8.0;\n\ninterface Weth {\n    function withdraw(uint256 _wad) external;\n\n    function transferFrom(address _from, address _to, uint256 _wad) external;\n}\n\ninterface OvmL1Bridge {\n    function depositETHTo(address _to, uint32 _l2Gas, bytes calldata _data) external payable;\n}\n\ninterface PolygonL1Bridge {\n    function depositEtherFor(address _to) external payable;\n}\n\ninterface ZkSyncL1Bridge {\n    function requestL2Transaction(\n        address _contractL2,\n        uint256 _l2Value,\n        bytes calldata _calldata,\n        uint256 _l2GasLimit,\n        uint256 _l2GasPerPubdataByteLimit,\n        bytes[] calldata _factoryDeps,\n        address _refundRecipient\n    ) external payable;\n\n    function l2TransactionBaseCost(\n        uint256 _gasPrice,\n        uint256 _l2GasLimit,\n        uint256 _l2GasPerPubdataByteLimit\n    ) external pure returns (uint256);\n}\n\ninterface LineaL1MessageService {\n    function sendMessage(address _to, uint256 _fee, bytes calldata _calldata) external payable;\n}\n\n/**\n * @notice Contract deployed on Ethereum helps relay bots atomically unwrap and bridge WETH over the canonical chain\n * bridges for Optimism, Base, Boba, ZkSync, Linea, and Polygon. Needed as these chains only support bridging of ETH,\n * not WETH.\n */\n\ncontract AtomicWethDepositor {\n    Weth public immutable weth = Weth(0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2);\n    OvmL1Bridge public immutable optimismL1Bridge = OvmL1Bridge(0x99C9fc46f92E8a1c0deC1b1747d010903E884bE1);\n    OvmL1Bridge public immutable modeL1Bridge = OvmL1Bridge(0x735aDBbE72226BD52e818E7181953f42E3b0FF21);\n    OvmL1Bridge public immutable bobaL1Bridge = OvmL1Bridge(0xdc1664458d2f0B6090bEa60A8793A4E66c2F1c00);\n    OvmL1Bridge public immutable baseL1Bridge = OvmL1Bridge(0x3154Cf16ccdb4C6d922629664174b904d80F2C35);\n    PolygonL1Bridge public immutable polygonL1Bridge = PolygonL1Bridge(0xA0c68C638235ee32657e8f720a23ceC1bFc77C77);\n    ZkSyncL1Bridge public immutable zkSyncL1Bridge = ZkSyncL1Bridge(0x32400084C286CF3E17e7B677ea9583e60a000324);\n    LineaL1MessageService public immutable lineaL1MessageService =\n        LineaL1MessageService(0xd19d4B5d358258f05D7B411E21A1460D11B0876F);\n\n    event ZkSyncEthDepositInitiated(address indexed from, address indexed to, uint256 amount);\n    event LineaEthDepositInitiated(address indexed from, address indexed to, uint256 amount);\n    event OvmEthDepositInitiated(uint256 indexed chainId, address indexed from, address indexed to, uint256 amount);\n\n    function bridgeWethToOvm(address to, uint256 amount, uint32 l2Gas, uint256 chainId) public {\n        weth.transferFrom(msg.sender, address(this), amount);\n        weth.withdraw(amount);\n\n        if (chainId == 10) {\n            optimismL1Bridge.depositETHTo{ value: amount }(to, l2Gas, \"\");\n        } else if (chainId == 8453) {\n            baseL1Bridge.depositETHTo{ value: amount }(to, l2Gas, \"\");\n        } else if (chainId == 34443) {\n            modeL1Bridge.depositETHTo{ value: amount }(to, l2Gas, \"\");\n        } else if (chainId == 288) {\n            bobaL1Bridge.depositETHTo{ value: amount }(to, l2Gas, \"\");\n        } else {\n            revert(\"Invalid OVM chainId\");\n        }\n\n        emit OvmEthDepositInitiated(chainId, msg.sender, to, amount);\n    }\n\n    function bridgeWethToPolygon(address to, uint256 amount) public {\n        weth.transferFrom(msg.sender, address(this), amount);\n        weth.withdraw(amount);\n        polygonL1Bridge.depositEtherFor{ value: amount }(to);\n    }\n\n    function bridgeWethToLinea(address to, uint256 amount) public payable {\n        weth.transferFrom(msg.sender, address(this), amount);\n        weth.withdraw(amount);\n        lineaL1MessageService.sendMessage{ value: amount + msg.value }(to, msg.value, \"\");\n        // Emit an event that we can easily track in the Linea-related adapters/finalizers\n        emit LineaEthDepositInitiated(msg.sender, to, amount);\n    }\n\n    function bridgeWethToZkSync(\n        address to,\n        uint256 amount,\n        uint256 l2GasLimit,\n        uint256 l2GasPerPubdataByteLimit,\n        address refundRecipient\n    ) public {\n        // The ZkSync Mailbox contract checks that the msg.value of the transaction is enough to cover the transaction base\n        // cost. The transaction base cost can be queried from the Mailbox by passing in an L1 \"executed\" gas price,\n        // which is the priority fee plus base fee. This is the same as calling tx.gasprice on-chain as the Mailbox\n        // contract does here:\n        // https://github.com/matter-labs/era-contracts/blob/3a4506522aaef81485d8abb96f5a6394bd2ba69e/ethereum/contracts/zksync/facets/Mailbox.sol#L287\n        uint256 l2TransactionBaseCost = zkSyncL1Bridge.l2TransactionBaseCost(\n            tx.gasprice,\n            l2GasLimit,\n            l2GasPerPubdataByteLimit\n        );\n        uint256 valueToSubmitXChainMessage = l2TransactionBaseCost + amount;\n        weth.transferFrom(msg.sender, address(this), valueToSubmitXChainMessage);\n        weth.withdraw(valueToSubmitXChainMessage);\n        zkSyncL1Bridge.requestL2Transaction{ value: valueToSubmitXChainMessage }(\n            to,\n            amount,\n            \"\",\n            l2GasLimit,\n            l2GasPerPubdataByteLimit,\n            new bytes[](0),\n            refundRecipient\n        );\n\n        // Emit an event that we can easily track in the ZkSyncAdapter because otherwise there is no easy event to\n        // track ETH deposit initiations.\n        emit ZkSyncEthDepositInitiated(msg.sender, to, amount);\n    }\n\n    fallback() external payable {}\n\n    // Included to remove a compilation warning.\n    // NOTE: this should not affect behavior.\n    receive() external payable {}\n}\n"
+    }
+  },
+  "settings": {
+    "optimizer": {
+      "enabled": true,
+      "runs": 1000000
+    },
+    "viaIR": true,
+    "outputSelection": {
+      "*": {
+        "*": ["abi", "evm.bytecode", "evm.deployedBytecode", "evm.methodIdentifiers", "metadata"],
+        "": ["ast"]
+      }
+    }
+  }
+}
diff --git a/hardhat.config.ts b/hardhat.config.ts
@@ -23,9 +23,20 @@ dotenv.config();
 const solcVersion = "0.8.23";
 const mnemonic = getMnemonic();
 
+const LARGE_CONTRACT_COMPILER_SETTINGS = {
+  version: solcVersion,
+  settings: {
+    optimizer: { enabled: true, runs: 1000000 },
+    viaIR: true,
+  },
+};
+
 const config: HardhatUserConfig = {
   solidity: {
     compilers: [{ version: solcVersion, settings: { optimizer: { enabled: true, runs: 1 }, viaIR: true } }],
+    overrides: {
+      "contracts/AtomicWethDepositor.sol": LARGE_CONTRACT_COMPILER_SETTINGS,
+    },
   },
   networks: {
     hardhat: { accounts: { accountsBalance: "1000000000000000000000000" } },

diff --git a/package.json b/package.json
@@ -10,13 +10,13 @@
     "node": ">=16.18.0"
   },
   "dependencies": {
-    "@across-protocol/constants-v2": "1.0.25",
-    "@across-protocol/contracts-v2": "2.5.6",
-    "@across-protocol/sdk-v2": "0.24.0",
+    "@across-protocol/constants-v2": "1.0.26",
+    "@across-protocol/contracts-v2": "2.5.9",
+    "@across-protocol/sdk-v2": "0.24.2",
     "@arbitrum/sdk": "^3.1.3",
     "@consensys/linea-sdk": "^0.2.1",
     "@defi-wonderland/smock": "^2.3.5",
-    "@eth-optimism/sdk": "^3.2.2",
+    "@eth-optimism/sdk": "^3.3.1",
     "@ethersproject/abi": "^5.7.0",
     "@ethersproject/abstract-provider": "^5.7.0",
     "@ethersproject/abstract-signer": "^5.7.0",

diff --git a/scripts/sendTokens.ts b/scripts/sendTokens.ts
@@ -62,7 +62,15 @@ export async function run(): Promise<void> {
       return;
     }
     console.log("sending...");
-    const tx = await erc20.transfer(recipient, args.amount);
+    const tx = await erc20.transfer(recipient, args.amount, {
+      maxFeePerGas: 150000000000,
+      maxPriorityFeePerGas: 40000000000,
+    });
+    console.log(
+      `submitted with max fee per gas ${tx.maxFeePerGas.toString()} and priority fee ${tx.maxPriorityFeePerGas.toString()} at nonce ${
+        tx.nonce
+      }`
+    );
     const receipt = await tx.wait();
     console.log("Transaction hash:", receipt.transactionHash);
   }

diff --git a/src/clients/BundleDataClient.ts b/src/clients/BundleDataClient.ts
@@ -414,7 +414,10 @@ export class BundleDataClient {
     // If a chain is disabled or doesn't have a spoke pool client, return a range of 0
     function getBlockRangeDelta(_pendingBlockRanges: number[][]): number[][] {
       return widestBundleBlockRanges.map((blockRange, index) => {
-        const initialBlockRange = _pendingBlockRanges[index];
+        // If pending block range doesn't have an entry for the widest range, which is possible when a new chain
+        // is added to the CHAIN_ID_INDICES list, then simply set the initial block range to the widest block range.
+        // This will produce a block range delta of 0 where the returned range for this chain is [widest[1], widest[1]].
+        const initialBlockRange = _pendingBlockRanges[index] ?? blockRange;
         // If chain is disabled, return disabled range
         if (initialBlockRange[0] === initialBlockRange[1]) {
           return initialBlockRange;

diff --git a/src/clients/bridges/AdapterManager.ts b/src/clients/bridges/AdapterManager.ts
@@ -8,6 +8,7 @@ import {
   ZKSyncAdapter,
   BaseChainAdapter,
   LineaAdapter,
+  ModeAdapter,
 } from "./";
 import { InventoryConfig, OutstandingTransfers } from "../../interfaces";
 import { utils } from "@across-protocol/sdk-v2";
@@ -62,6 +63,9 @@ export class AdapterManager {
     if (this.spokePoolClients[59144] !== undefined) {
       this.adapters[59144] = new LineaAdapter(logger, spokePoolClients, filterMonitoredAddresses(59144));
     }
+    if (this.spokePoolClients[34443] !== undefined) {
+      this.adapters[34443] = new ModeAdapter(logger, spokePoolClients, filterMonitoredAddresses(34443));
+    }
 
     logger.debug({
       at: "AdapterManager#constructor",

diff --git a/src/clients/bridges/index.ts b/src/clients/bridges/index.ts
@@ -2,6 +2,7 @@ export * from "./BaseAdapter";
 export * from "./AdapterManager";
 export * from "./op-stack/optimism/OptimismAdapter";
 export * from "./op-stack/base/BaseChainAdapter";
+export * from "./op-stack/mode/ModeAdapter";
 export * from "./ArbitrumAdapter";
 export * from "./PolygonAdapter";
 export * from "./CrossChainTransferClient";

diff --git a/src/clients/bridges/op-stack/OpStackAdapter.ts b/src/clients/bridges/op-stack/OpStackAdapter.ts
@@ -47,9 +47,13 @@ export class OpStackAdapter extends BaseAdapter {
       );
     }
 
-    // We should manually override the bridge for USDC to use CCTP.
+    // We should manually override the bridge for USDC to use CCTP if this chain has a Native USDC entry. We can
+    // assume that all Op Stack chains will have a bridged USDC.e variant that uses the OVM standard bridge, so we
+    // only need to check if a native USDC exists for this chain. If so, then we'll use the TokenSplitter bridge
+    // which maps to either the CCTP or OVM Standard bridge depending on the request.
     const usdcAddress = TOKEN_SYMBOLS_MAP._USDC.addresses[this.hubChainId];
-    if (usdcAddress) {
+    const l2NativeUsdcAddress = TOKEN_SYMBOLS_MAP._USDC.addresses[this.chainId];
+    if (usdcAddress && l2NativeUsdcAddress) {
       this.customBridges[usdcAddress] = new UsdcTokenSplitterBridge(
         this.chainId,
         this.hubChainId,
@@ -151,7 +155,6 @@ export class OpStackAdapter extends BaseAdapter {
     simMode = false
   ): Promise<TransactionResponse | null> {
     const { chainId } = this;
-    assert([10, 8453].includes(chainId), `chainId ${chainId} is not supported`);
 
     const ovmWeth = CONTRACT_ADDRESSES[this.chainId].weth;
     const ethBalance = await this.getSigner(chainId).getBalance();

diff --git a/src/clients/bridges/op-stack/index.ts b/src/clients/bridges/op-stack/index.ts
@@ -2,3 +2,4 @@
 export * from "./OpStackAdapter";
 export * from "./optimism";
 export * from "./base";
+export * from "./mode";
diff --git a/src/clients/bridges/op-stack/mode/ModeAdapter.ts b/src/clients/bridges/op-stack/mode/ModeAdapter.ts
@@ -0,0 +1,21 @@
+import { winston } from "../../../../utils";
+import { SpokePoolClient } from "../../..";
+import { OpStackAdapter } from "../OpStackAdapter";
+
+export class ModeAdapter extends OpStackAdapter {
+  constructor(
+    logger: winston.Logger,
+    readonly spokePoolClients: { [chainId: number]: SpokePoolClient },
+    monitoredAddresses: string[]
+  ) {
+    super(
+      34443,
+      // Custom Bridges
+      {},
+      logger,
+      ["ETH", "WETH", "USDC", "USDT", "WBTC"],
+      spokePoolClients,
+      monitoredAddresses
+    );
+  }
+}
diff --git a/src/clients/bridges/op-stack/mode/index.ts b/src/clients/bridges/op-stack/mode/index.ts
@@ -0,0 +1 @@
+export * from "./ModeAdapter";
diff --git a/src/common/Constants.ts b/src/common/Constants.ts
@@ -18,6 +18,7 @@ export const DATAWORKER_FAST_LOOKBACK: { [chainId: number]: number } = {
   288: 11520,
   324: 4 * 24 * 60 * 60,
   8453: 172800, // Same as Optimism.
+  34443: 172800, // Same as Optimism.
   42161: 1382400,
   59144: 115200, // 1 block every 3 seconds
 };
@@ -54,12 +55,14 @@ export const DEFAULT_MIN_DEPOSIT_CONFIRMATIONS = {
   288: 0,
   324: 120,
   8453: 120,
+  34443: 120,
   42161: 0,
   59144: 30,
   // Testnets:
   5: 0,
   280: 0,
   420: 0,
+  919: 0,
   59140: 0,
   80001: 0,
   84531: 0,
@@ -77,12 +80,14 @@ export const MIN_DEPOSIT_CONFIRMATIONS: { [threshold: number | string]: { [chain
     288: 0,
     324: 0,
     8453: 60,
+    34443: 60,
     42161: 0,
     59144: 1,
     // Testnets:
     5: 0,
     280: 0,
     420: 0,
+    919: 0,
     59140: 0,
     80001: 0,
     84531: 0,
@@ -95,12 +100,14 @@ export const MIN_DEPOSIT_CONFIRMATIONS: { [threshold: number | string]: { [chain
     288: 0,
     324: 0,
     8453: 60,
+    34443: 60,
     42161: 0,
     59144: 1,
     // Testnets:
     5: 0,
     280: 0,
     420: 0,
+    919: 0,
     59140: 0,
     80001: 0,
     84531: 0,
@@ -122,12 +129,14 @@ export const CHAIN_MAX_BLOCK_LOOKBACK = {
   288: 4990,
   324: 10000,
   8453: 1500,
+  34443: 1500,
   42161: 10000,
   59144: 5000,
   // Testnets:
   5: 10000,
   280: 10000,
   420: 10000,
+  919: 10000,
   59140: 10000,
   80001: 10000,
   84531: 10000,
@@ -149,12 +158,14 @@ export const BUNDLE_END_BLOCK_BUFFERS = {
   288: 0, // **UPDATE** 288 is disabled so there should be no buffer.
   324: 120, // ~1s/block. ZkSync is a centralized sequencer but is relatively unstable so this is kept higher than 0
   8453: 60, // 2s/block. Same finality profile as Optimism
+  34443: 60, // 2s/block. Same finality profile as Optimism
   42161: 240, // ~0.25s/block. Arbitrum is a centralized sequencer
   59144: 40, // At 3s/block, 2 mins = 40 blocks.
   // Testnets:
   5: 0,
   280: 0,
   420: 0,
+  919: 0,
   59140: 0,
   80001: 0,
   84531: 0,
@@ -194,13 +205,15 @@ export const CHAIN_CACHE_FOLLOW_DISTANCE: { [chainId: number]: number } = {
   288: 0,
   324: 512,
   8453: 120,
+  34443: 120,
   42161: 32,
   59144: 100, // Linea has a soft-finality of 1 block. This value is padded - but at 3s/block the padding is 5 minutes
   534352: 0,
   // Testnets:
   5: 0,
   280: 0,
   420: 0,
+  919: 0,
   59140: 0,
   80001: 0,
   84531: 0,
@@ -222,6 +235,7 @@ export const DEFAULT_NO_TTL_DISTANCE: { [chainId: number]: number } = {
   288: 86400,
   324: 172800,
   8453: 86400,
+  34443: 86400,
   59144: 57600,
   42161: 691200,
   534352: 57600,
@@ -234,6 +248,7 @@ export const DEFAULT_GAS_FEE_SCALERS: {
   1: { maxFeePerGasScaler: 3, maxPriorityFeePerGasScaler: 1.2 },
   10: { maxFeePerGasScaler: 2, maxPriorityFeePerGasScaler: 1 },
   8453: { maxFeePerGasScaler: 2, maxPriorityFeePerGasScaler: 1 },
+  34443: { maxFeePerGasScaler: 2, maxPriorityFeePerGasScaler: 1 },
 };
 
 // This is how many seconds stale the block number can be for us to use it for evaluating the reorg distance in the cache provider.
@@ -251,6 +266,7 @@ export const multicall3Addresses = {
   288: "0xcA11bde05977b3631167028862bE2a173976CA11",
   324: "0xF9cda624FBC7e059355ce98a31693d299FACd963",
   8453: "0xcA11bde05977b3631167028862bE2a173976CA11",
+  34443: "0xcA11bde05977b3631167028862bE2a173976CA11",
   42161: "0xcA11bde05977b3631167028862bE2a173976CA11",
   59144: "0xcA11bde05977b3631167028862bE2a173976CA11",
   534352: "0xcA11bde05977b3631167028862bE2a173976CA11",
@@ -273,7 +289,7 @@ export type Multicall2Call = {
 
 // These are the spokes that can hold both ETH and WETH, so they should be added together when caclulating whether
 // a bundle execution is possible with the funds in the pool.
-export const spokesThatHoldEthAndWeth = [10, 324, 8453, 59144];
+export const spokesThatHoldEthAndWeth = [10, 324, 8453, 34443, 59144];
 
 /**
  * An official mapping of chain IDs to CCTP domains. This mapping is separate from chain identifiers
@@ -317,5 +333,6 @@ export const RELAYER_DEFAULT_SPOKEPOOL_INDEXER = "./dist/src/libexec/RelayerSpok
 
 export const DEFAULT_ARWEAVE_GATEWAY = { url: "arweave.net", port: 443, protocol: "https" };
 
-// Chains with slow (> 2 day liveness) canonical L2-->L1 bridges.
+// Chains with slow (> 2 day liveness) canonical L2-->L1 bridges that we prioritize taking repayment on.
+// This does not include all  7-day withdrawal chains because we don't necessarily prefer being repaid on some of these 7-day chains, like Mode.
 export const SLOW_WITHDRAWAL_CHAINS = [CHAIN_IDs.BASE, CHAIN_IDs.ARBITRUM, CHAIN_IDs.OPTIMISM];