fix: return connect bot calls without ca

Author: essserrr

src/sdk/accounts/CreditAccountsServiceV310.ts

@@ -7,6 +7,7 @@ import { PERMISSION_BY_TYPE } from "./constants.js";
 import type {
   ClaimFarmRewardsProps,
   CreditAccountOperationResult,
+  CreditManagerOperationResult,
   ICreditAccountsService,
   RepayAndLiquidateCreditAccountProps,
   RepayCreditAccountProps,
@@ -26,17 +27,24 @@ export class CreditAccountServiceV310
     botBaseType,
     stopBot,
 
-    creditAccount: ca,
-  }: SetBotProps): Promise<CreditAccountOperationResult> {
-    const cm = this.sdk.marketRegister.findCreditManager(ca.creditManager);
+    targetContract,
+  }: SetBotProps): Promise<
+    CreditAccountOperationResult | CreditManagerOperationResult
+  > {
+    const cm = this.sdk.marketRegister.findCreditManager(
+      targetContract.creditManager,
+    );
 
-    const priceUpdatesCalls = await this.getPriceUpdatesForFacade({
-      creditManager: ca.creditManager,
-      creditAccount: ca,
-    });
+    const priceUpdatesCalls =
+      targetContract.type === "creditAccount"
+        ? await this.getPriceUpdatesForFacade({
+            creditManager: targetContract.creditManager,
+            creditAccount: targetContract,
+          })
+        : [];
 
     const addBotCall: MultiCall = {
-      target: ca.creditFacade,
+      target: cm.creditFacade.address,
       callData: encodeFunctionData({
         abi: iCreditFacadeMulticallV310Abi,
         functionName: "setBotPermissions",
@@ -46,7 +54,10 @@ export class CreditAccountServiceV310
 
     const calls = [...priceUpdatesCalls, addBotCall];
 
-    const tx = cm.creditFacade.multicall(ca.creditAccount, calls);
+    const tx =
+      targetContract.type === "creditAccount"
+        ? cm.creditFacade.multicall(targetContract.creditAccount, calls)
+        : undefined;
 
     return { tx, calls, creditFacade: cm.creditFacade };
   }

src/sdk/accounts/types.ts

@@ -80,6 +80,11 @@ export interface CreditAccountOperationResult {
   creditFacade: CreditSuite["creditFacade"];
 }
 
+export interface CreditManagerOperationResult {
+  calls: Array<MultiCall>;
+  creditFacade: CreditSuite["creditFacade"];
+}
+
 export type CloseOptions = "close" | "zeroDebt";
 
 export interface CloseCreditAccountProps {
@@ -443,6 +448,12 @@ export type LiquidationBotType = Exclude<
 >;
 export type { MigrationBotType };
 
+interface CMSlice {
+  creditManager: Address;
+  creditFacade: Address;
+  type: "creditManager";
+}
+
 export interface SetBotProps {
   /**
    * Address of a bot that is being updated
@@ -457,9 +468,11 @@ export interface SetBotProps {
    */
   stopBot: boolean;
   /**
-   * Minimal credit account data {@link RouterCASlice} on which operation is performed
+   * Minimal credit account data {@link RouterCASlice} on which operation is performed; if omitted, credit manager data is used
+   * Minimal credit manager data {@link CMSlice} on which operation is performed; used only if credit account is omitted
+   * At least one of credit account or credit manager must be provided
    */
-  creditAccount: RouterCASlice;
+  targetContract: (RouterCASlice & { type: "creditAccount" }) | CMSlice;
 }
 
 export type GetConnectedBotsResult = Array<
@@ -546,7 +559,9 @@ export interface ICreditAccountsService extends SDKConstruct {
    * @param props - {@link SetBotProps}
    * @return All necessary data to execute the transaction (call, credit facade)
    */
-  setBot: (props: SetBotProps) => Promise<CreditAccountOperationResult>;
+  setBot: (
+    props: SetBotProps,
+  ) => Promise<CreditAccountOperationResult | CreditManagerOperationResult>;
 
   /**
    * Generates transaction to liquidate credit account