feat(mbe): fix comments

Author: cpatino-intive

src/api/master/handlers/recoveryConsolidationsWallet.ts

@@ -1,7 +1,7 @@
 import { MasterApiSpecRouteRequest } from '../routers/masterApiSpec';
 import logger from '../../../logger';
-import { isSolCoin, isTrxCoin } from '../../../shared/coinUtils';
-import { MPCSweepTxs, MPCTx, MPCTxs } from 'bitgo';
+import { isSolCoin } from '../../../shared/coinUtils';
+import { MPCTx } from 'bitgo';
 import { RecoveryTransaction } from '@bitgo/sdk-coin-trx';
 
 // Handler for recovery from receive addresses (consolidation sweeps)
@@ -16,34 +16,32 @@ export async function handleRecoveryConsolidationsOnPrem(
 
   const sdkCoin = bitgo.coin(coin);
   let txs: MPCTx[] | RecoveryTransaction[] = [];
-
-  logger.debug(
-    `${isSolCoin(sdkCoin) ? 'Solana' : isTrxCoin(sdkCoin) ? 'Tron' : 'Unknown'} recovery`,
-  );
-
   // 1. Build unsigned consolidations
-  if (isSolCoin(sdkCoin)) {
-    const result = await sdkCoin.recoverConsolidations({
-      ...req.decoded,
-      userKey: userPub,
-      backupKey: backupPub,
-      bitgoKey,
-      durableNonces: req.decoded.durableNonces!,
-    });
+  if (isSolCoin(sdkCoin) && !req.decoded.durableNonces) {
+    throw new Error('durableNonces is required for Solana consolidation recovery');
+  }
+
+  if (typeof (sdkCoin as any).recoverConsolidations !== 'function') {
+    throw new Error(`recoverConsolidations is not supported for coin: ${coin}`);
+  }
 
-    txs = (result as MPCTxs).transactions || (result as MPCSweepTxs).txRequests || [];
-  } else if (isTrxCoin(sdkCoin)) {
-    const result = await sdkCoin.recoverConsolidations({
-      ...req.decoded,
-      userKey: userPub,
-      backupKey: backupPub,
-      bitgoKey,
-    });
+  // Use type assertion to access recoverConsolidations
+  const result = await (sdkCoin as any).recoverConsolidations({
+    ...req.decoded,
+    userKey: userPub,
+    backupKey: backupPub,
+    bitgoKey,
+    durableNonces: req.decoded.durableNonces,
+  });
 
+  if ('transactions' in result) {
     txs = result.transactions;
+  } else if ('txRequests' in result) {
+    txs = result.txRequests;
+  } else {
+    throw new Error('recoverConsolidations did not return expected transactions');
   }
 
-  console.log(txs);
   logger.debug(`Found ${txs.length} unsigned consolidation transactions`);
 
   // 2. For each unsigned sweep, get it signed by EBE (using recoveryMultisig)

src/api/master/routers/masterApiSpec.ts

@@ -306,8 +306,8 @@ export const MasterApiSpec = apiSpec({
       }),
       response: RecoveryConsolidationsResponse,
       description: 'Recover consolidations from receive addresses',
-    })
-  }
+    }),
+  },
 });
 
 export type MasterApiSpec = typeof MasterApiSpec;