fix(sdk-coin-stx): return required fields for unsigned sweep

Ticket: COIN-3893

Author: ravibitgo

modules/sdk-coin-stx/src/lib/iface.ts

@@ -1,5 +1,6 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import { ClarityValue, PayloadType } from '@stacks/transactions';
+import { TransactionExplanation as BaseTransactionExplanation } from '@bitgo/sdk-core';
 
 export interface TxData {
   id: string;
@@ -107,6 +108,15 @@ export interface RecoveryOptions {
   contractId?: string;
 }
 
+export interface RecoveryInfo extends BaseTransactionExplanation {
+  txHex: string;
+  feeInfo?: {
+    fee: string;
+  };
+  coin?: string;
+}
+
 export interface RecoveryTransaction {
   txHex: string;
+  coin?: string;
 }

modules/sdk-coin-stx/src/stx.ts

@@ -42,11 +42,13 @@ import { findContractTokenNameUsingContract, findTokenNameByContract, getAddress
 import {
   AddressDetails,
   NativeStxBalance,
+  RecoveryInfo,
   RecoveryOptions,
   RecoveryTransaction,
   SingleFungibleTokenBalance,
   StxNonceResponse,
   StxTxnFeeEstimationResponse,
+  TxData,
 } from './lib/iface';
 import { TransferBuilder } from './lib/transferBuilder';
 import { FungibleTokenTransferBuilder } from './lib/fungibleTokenTransferBuilder';
@@ -667,9 +669,9 @@ export class Stx extends BaseCoin {
    * @param {String} params.bitgoKey - encrypted bitgo public key
    * @param {String} params.walletPassphrase - wallet password
    * @param {String} params.contractId - contract id of the token (mandatory for token recovery)
-   * @returns {Promise<RecoveryTransaction>} RecoveryTransaction.txHex - hex of serialized transaction (signed or unsigned)
+   * @returns {Promise<RecoveryInfo|RecoveryTransaction>} RecoveryTransaction.txHex - hex of serialized transaction (signed or unsigned)
    */
-  async recover(params: RecoveryOptions): Promise<RecoveryTransaction> {
+  async recover(params: RecoveryOptions): Promise<RecoveryInfo | RecoveryTransaction> {
     if (!this.isValidAddress(params.rootAddress)) {
       throw new Error('invalid root address!');
     }
@@ -716,9 +718,15 @@ export class Stx extends BaseCoin {
     const serializedTx: string = tx.toBroadcastFormat();
 
     if (isUnsignedSweep) {
-      return {
+      const txJson: TxData = tx.toJson();
+      const transactionExplanation: RecoveryInfo = (await this.explainTransaction({
         txHex: serializedTx,
-      };
+        feeInfo: { fee: txJson.fee },
+      })) as RecoveryInfo;
+      transactionExplanation.coin = this.getChain();
+      transactionExplanation.feeInfo = { fee: txJson.fee };
+      transactionExplanation.txHex = serializedTx;
+      return transactionExplanation;
     }
     // check the private key & sign
     if (!keys[0].privateKey) {

modules/sdk-coin-stx/test/unit/sip10Token.ts

@@ -311,7 +311,10 @@ describe('Sip10Token:', function () {
       };
       const response: RecoveryTransaction = await basecoin.recover(recoveryOptions);
       response.should.have.property('txHex');
+      response.should.have.property('coin');
+      response.should.have.property('feeInfo');
       assert.deepEqual(response.txHex, testData.COLD_WALLET_TOKEN_UNSIGNED_SWEEP_TX_HEX, 'tx hex not matching!');
+      assert.deepEqual(response.coin, 'tstx:tsip6dp', 'coin not matching!');
     });
 
     it('should fail with insufficient balance when native stx balance is lower than fee for sip10', async () => {

modules/sdk-coin-stx/test/unit/stx.ts

@@ -401,7 +401,10 @@ describe('STX:', function () {
       };
       const response: RecoveryTransaction = await basecoin.recover(recoveryOptions);
       response.should.have.property('txHex');
+      response.should.have.property('coin');
+      response.should.have.property('feeInfo');
       assert.deepEqual(response.txHex, testData.COLD_WALLET_UNSIGNED_SWEEP_TX_HEX, 'tx hex not matching!');
+      assert.deepEqual(response.coin, 'tstx', 'coin not matching!');
     });
 
     it('should throw invalid root address when root address is missing or invalid', async function () {