fix: auto-register credit managers in v310 partial liquidator contracts

Author: doomsower

src/services/liquidate/partial/AbstractPartialLiquidatorContract.ts

@@ -72,11 +72,11 @@ export abstract class AbstractPartialLiquidatorContract
     this.logger = DI.create(DI.Logger, this.name.replaceAll(" ", ""));
   }
 
-  public addCreditManager(cm: CreditSuite): void {
+  public queueCreditManagerRegistration(cm: CreditSuite): void {
+    this.#pendingCreditManagers.push(cm);
     this.logger.debug(
-      `adding credit manager ${cm.creditManager.name} (${cm.creditManager.address})`,
+      `queued credit manager ${cm.creditManager.name} (${cm.creditManager.address})`,
     );
-    this.#pendingCreditManagers.push(cm);
   }
 
   public async syncState(): Promise<void> {

src/services/liquidate/partial/PartialContractsDeployer.ts

@@ -80,7 +80,9 @@ export class PartialContractsDeployer extends SDKConstruct {
         liquidatorForCM =
           this.#uniqueContracts[liquidatorForCM.name] ?? liquidatorForCM;
         this.#uniqueContracts[liquidatorForCM.name] = liquidatorForCM;
-        this.#uniqueContracts[liquidatorForCM.name].addCreditManager(cm);
+        this.#uniqueContracts[
+          liquidatorForCM.name
+        ].queueCreditManagerRegistration(cm);
         this.#liquidatorForCM.upsert(cm.creditManager.address, liquidatorForCM);
         this.logger?.debug(
           `will use partial liquidator contract for ${cm.creditManager.name}: ${liquidatorForCM.name}`,

src/services/liquidate/partial/types.ts

@@ -28,7 +28,12 @@ export interface IPartialLiquidatorContract {
   name: string;
   version: number;
   envVariables: Record<string, string>;
-  addCreditManager: (cm: CreditSuite) => void;
+  /**
+   * Queues credit manager that uses this partial lqiuidation/deleverage contract for registration
+   * Registraction will be performed during next syncState
+   * @param cm
+   */
+  queueCreditManagerRegistration: (cm: CreditSuite) => void;
   /**
    * Performs all actions necessary to make the contract ready for use
    * This includes deploying contracts if necessary, setting router, registering credit managers, etc.

src/services/liquidate/partial/v310/PartialLiquidatorV310Contract.ts

@@ -39,6 +39,14 @@ export default abstract class PartialLiquidatorV310Contract extends AbstractPart
     this.deployer = new Create2Deployer(this.sdk, this.client.wallet);
   }
 
+  public override queueCreditManagerRegistration(cm: CreditSuite): void {
+    // For v310, credit managers are registered automatically, unless they have degen NFT
+    if (cm.creditFacade.degenNFT === ADDRESS_0X0) {
+      return;
+    }
+    super.queueCreditManagerRegistration(cm);
+  }
+
   /**
    * Registers router, partial liquidation bot and credit manager addresses in liquidator contract if necessary
    */