Signers: put console.log behind debug flag (#533)

* put console.log behind debug flag

* more chains

Author: barnjamin

platforms/algorand/src/signer.ts

@@ -63,12 +63,14 @@ export class AlgorandSigner<N extends Network, C extends AlgorandChains>
       }
 
       if (signer) {
-        console.log(
-          `Signing: ${description} with signer ${signer.address} for address ${this.address()}`,
-        );
+        if (this._debug)
+          console.log(
+            `Signing: ${description} with signer ${signer.address} for address ${this.address()}`,
+          );
         signed.push(await signer.signTxn(tx));
       } else {
-        console.log(`Signing: ${description} without signer for address ${this.address()}`);
+        if (this._debug)
+          console.log(`Signing: ${description} without signer for address ${this.address()}`);
         signed.push(tx.signTxn(this._account.sk));
       }
     }

platforms/aptos/src/signer.ts

@@ -28,6 +28,7 @@ export class AptosSigner<N extends Network, C extends AptosChains>
     private _chain: C,
     private _account: AptosAccount,
     private _rpc: AptosClient,
+    private _debug?: boolean,
   ) {}
 
   chain() {
@@ -45,7 +46,7 @@ export class AptosSigner<N extends Network, C extends AptosChains>
         description: string;
         transaction: Types.EntryFunctionPayload;
       };
-      console.log(`Signing: ${description} for ${this.address()}`);
+      if (this._debug) console.log(`Signing: ${description} for ${this.address()}`);
 
       // overwriting `max_gas_amount` and `gas_unit_price` defaults
       // rest of defaults are defined here: https://aptos-labs.github.io/ts-sdk-doc/classes/AptosClient.html#generateTransaction

platforms/cosmwasm/src/signer.ts

@@ -61,6 +61,7 @@ export class CosmwasmSigner<N extends Network, C extends CosmwasmChains>
     private _chain: C,
     private _signer: SigningCosmWasmClient,
     private _account: string,
+    private _debug?: string,
   ) {}
 
   chain(): C {
@@ -75,9 +76,10 @@ export class CosmwasmSigner<N extends Network, C extends CosmwasmChains>
     const signed = [];
     for (const txn of tx) {
       const { description, transaction } = txn as CosmwasmUnsignedTransaction<N, C>;
-      console.log(`Signing: ${description} for ${this.address()}`);
-
-      console.log(transaction.msgs, transaction.fee, transaction.memo);
+      if (this._debug) {
+        console.log(`Signing: ${description} for ${this.address()}`);
+        console.log(transaction.msgs, transaction.fee, transaction.memo);
+      }
 
       const txRaw = await this._signer.sign(
         this.address(),

platforms/evm/src/signer.ts

@@ -26,6 +26,7 @@ export async function getEvmSigner(
   opts?: {
     maxGasLimit?: bigint;
     chain?: EvmChains;
+    debug?: boolean;
   },
 ): Promise<Signer> {
   const signer: EthersSigner =
@@ -74,7 +75,7 @@ export class EvmNativeSigner<N extends Network, C extends EvmChains = EvmChains>
     _chain: C,
     _address: string,
     _signer: EthersSigner,
-    readonly opts?: { maxGasLimit?: bigint },
+    readonly opts?: { maxGasLimit?: bigint; debug?: boolean },
   ) {
     super(_chain, _address, _signer);
   }
@@ -105,7 +106,8 @@ export class EvmNativeSigner<N extends Network, C extends EvmChains = EvmChains>
 
     for (const txn of tx) {
       const { transaction, description } = txn;
-      console.log(`Signing: ${description} for ${this.address()}`);
+      if (this.opts?.debug)
+        console.log(`Signing: ${description} for ${this.address()}`);
 
       const t: TransactionRequest = {
         ...transaction,

platforms/solana/src/signer.ts

@@ -102,7 +102,8 @@ export class SolanaSigner<N extends Network, C extends SolanaChains = 'Solana'>
         transaction: { transaction, signers: extraSigners },
       } = txn;
 
-      console.log(`Signing: ${description} for ${this.address()}`);
+      if (this._debug)
+        console.log(`Signing: ${description} for ${this.address()}`);
 
       if (this._debug) logTxDetails(transaction);
 
@@ -190,7 +191,9 @@ export class SolanaSendSigner<
         description,
         transaction: { transaction, signers: extraSigners },
       } = txn as SolanaUnsignedTransaction<N, C>;
-      console.log(`Signing: ${description} for ${this.address()}`);
+
+      if (this._debug)
+        console.log(`Signing: ${description} for ${this.address()}`);
 
       let priorityFeeIx: TransactionInstruction[] | undefined;
       if (this._priorityFeePercentile && this._priorityFeePercentile > 0)

platforms/sui/src/signer.ts

@@ -22,6 +22,7 @@ export class SuiSigner<N extends Network, C extends SuiChains> implements SignAn
     private _chain: C,
     private _client: SuiClient,
     private _signer: Ed25519Keypair,
+    private _debug?: boolean,
   ) {}
 
   chain(): C {
@@ -36,7 +37,7 @@ export class SuiSigner<N extends Network, C extends SuiChains> implements SignAn
     const txids: TxHash[] = [];
     for (const tx of txns) {
       const { description, transaction } = tx as SuiUnsignedTransaction<N, C>;
-      console.log(`Signing ${description} for ${this.address()}`);
+      if (this._debug) console.log(`Signing ${description} for ${this.address()}`);
 
       const result = await this._client.signAndExecuteTransactionBlock({
         transactionBlock: transaction,