fix: lazy sync for botlist state

Author: doomsower

src/sdk/GearboxSDK.ts

@@ -173,7 +173,6 @@ export class GearboxSDK {
     // Attaching bot list contract
     const botListAddress = this.#addressProvider.getAddress(AP_BOT_LIST, 300);
     this.#botListContract = new BotListContract(this, botListAddress);
-    await this.#botListContract.fetchState(this.currentBlock);
 
     // Attaching gear staking contract
     this.#gear = this.#addressProvider.getAddress(AP_GEAR_TOKEN);

src/sdk/constants/address-provider.ts

@@ -41,3 +41,13 @@ export const ADDRESS_PROVIDER: Record<NetworkType, Address> = {
   Optimism: "0x3761ca4BFAcFCFFc1B8034e69F19116dD6756726",
   Base: NOT_DEPLOYED,
 };
+
+/**
+ * Block number when address provider was deployed
+ */
+export const ADDRESS_PROVIDER_BLOCK: Record<NetworkType, bigint> = {
+  Mainnet: 18433056n,
+  Arbitrum: 184650310n,
+  Optimism: 118410666n,
+  Base: 0n,
+};

src/sdk/core/BotListV3Contract.ts

@@ -7,17 +7,19 @@ import type {
 
 import { botListV3Abi } from "../abi";
 import { BaseContract } from "../base";
-import { botPermissionsToString } from "../constants";
+import { ADDRESS_PROVIDER_BLOCK, botPermissionsToString } from "../constants";
 import type { GearboxSDK } from "../GearboxSDK";
 import type { BotListStateHuman } from "../types";
 
 type abi = typeof botListV3Abi;
 
 export class BotListContract extends BaseContract<abi> {
-  approvedCreditManagers: Set<Address> = new Set();
+  #approvedCreditManagers?: Set<Address>;
+  #currentBlock: bigint;
 
   constructor(sdk: GearboxSDK, address: Address) {
     super(sdk, { addr: address, name: "BotListV3", abi: botListV3Abi });
+    this.#currentBlock = ADDRESS_PROVIDER_BLOCK[sdk.provider.networkType];
   }
 
   public parseFunctionParams(
@@ -41,15 +43,15 @@ export class BotListContract extends BaseContract<abi> {
     }
   }
 
-  public async fetchState(toBlock: bigint): Promise<void> {
+  public async syncState(toBlock: bigint): Promise<void> {
     const logs = await this.provider.publicClient.getContractEvents({
       address: this.address,
       abi: this.abi,
-      fromBlock: 0n,
+      fromBlock: this.#currentBlock,
       toBlock,
     });
-
     logs.forEach(e => this.processLog(e));
+    this.#currentBlock = toBlock;
   }
 
   public override processLog(
@@ -85,6 +87,15 @@ export class BotListContract extends BaseContract<abi> {
     }
   }
 
+  public get approvedCreditManagers(): Set<Address> {
+    if (!this.#approvedCreditManagers) {
+      throw new Error(
+        "BotListContract state needs to be synced to load approvedCreditManagers",
+      );
+    }
+    return this.#approvedCreditManagers;
+  }
+
   public override stateHuman(raw = true): BotListStateHuman {
     return super.stateHuman(raw);
   }