Fix console.log spam

Author: mikemaccana

src/lib/transaction.ts

@@ -220,7 +220,6 @@ export async function sendTransaction(
   const computeUnitBuffer = userComputeBuffer ?? { multiplier: 1.1 };
 
   if (transaction.recentBlockhash === undefined) {
-    console.log("No blockhash provided. Setting recent blockhash");
     const { blockhash } = await connection.getLatestBlockhash(commitment);
     transaction.recentBlockhash = blockhash;
   }
@@ -233,7 +232,6 @@ export async function sendTransaction(
 
   // Skip compute preparation if transaction is already signed or has compute instructions
   if (transaction.signatures.length > 0) {
-    console.log("Transaction already signed, skipping compute preparation");
     return sendRawTransactionWithRetry(connection, transaction.serialize() as Uint8Array, {
       commitment,
       ...sendOptions,
@@ -383,8 +381,6 @@ export async function addComputeInstructions(
       throw new Error("Failed to simulate compute units");
     }
 
-    console.log("Simulated compute units", simulatedCompute);
-
     // Apply buffer to compute units
     let finalComputeUnits = simulatedCompute;
     if (computeUnitBuffer.multiplier) {
@@ -394,8 +390,6 @@ export async function addComputeInstructions(
       finalComputeUnits += computeUnitBuffer.fixed;
     }
 
-    console.log("Final compute units (with buffer)", finalComputeUnits);
-
     instructions.push(
       ComputeBudgetProgram.setComputeUnitLimit({
         units: finalComputeUnits,
@@ -448,7 +442,7 @@ async function sendRawTransactionWithRetry(
     initialDelayMs = DEFAULT_SEND_OPTIONS.initialDelayMs,
     commitment = DEFAULT_SEND_OPTIONS.commitment,
     skipPreflight = DEFAULT_SEND_OPTIONS.skipPreflight,
-    onStatusUpdate = (status) => console.log("Transaction status:", status),
+    onStatusUpdate,
   }: SendTransactionOptions = {},
 ): Promise<string> {
   onStatusUpdate?.({ status: "created" });
@@ -553,8 +547,6 @@ export async function createLookupTable(
   // Need to wait until the lookup table is active
   await confirmTransaction(connection, lookupTableInstructionsSignature, "finalized");
 
-  console.log("Lookup table instructions signature", lookupTableInstructionsSignature);
-
   const lookupTableAccount = (
     await connection.getAddressLookupTable(lookupTableAddress, {
       commitment: "confirmed",

tests/src/idl.test.ts

@@ -27,7 +27,6 @@ describe("confirmTransaction", () => {
       "51f5WEqS7VSaeyg1o7qSY4oVqbuzxm4tCe7woSBKdSgxqkp9h1ebJLV5nKhaLHYohkSaV4Kaccs1ye8CexhBhgG6",
       provider,
     );
-    console.log("Events", JSON.stringify(events, null, 2));
     assert.equal(events.length, 1, "Should have two events");
     assert.equal(events[0].name, "otterVerifyEvent", "Its a otterVerifyEvent");
   });
@@ -49,10 +48,6 @@ describe("confirmTransaction", () => {
       provider,
       programId,
     );
-    console.log(
-      "Counter increase transaction",
-      JSON.stringify(counterTransaction, null, 2),
-    );
 
     // Assert transaction structure
     assert.equal(
@@ -104,10 +99,6 @@ describe("confirmTransaction", () => {
       provider,
       programId,
     );
-    console.log(
-      "VerifyTransaction",
-      JSON.stringify(versionedTransactionDecoded, null, 2),
-    );
 
     // Assert transaction structure
     assert.equal(
@@ -150,7 +141,6 @@ describe("confirmTransaction", () => {
       new PublicKey("NRBNcTmfRkZWCLwnd6ygiz8CYerneu6m5Hcchx8RbFD"),
       provider,
     );
-    console.log("Account data", JSON.stringify(accountData, null, 2));
 
     // Type assertion for better IDE support
     const buildParams = accountData as {

tests/src/transaction.test.ts

@@ -128,7 +128,6 @@ describe("Transaction utilities", () => {
       {
         onStatusUpdate: (status) => {
           statusUpdates.push(status);
-          console.log("status", status);
         },
       },
     );
@@ -221,7 +220,6 @@ describe("Transaction utilities", () => {
         computeUnitBuffer: { multiplier: 1.1 },
         onStatusUpdate: (status) => {
           statusUpdates.push(status);
-          console.log("status", status);
         },
       },
     );
@@ -232,7 +230,6 @@ describe("Transaction utilities", () => {
       ["created", "signed", "sent", "confirmed"],
     );
 
-    console.log("Versioned transaction signature", signature);
   });
 
   test("send versioned transaction with already setComputeLimit instructions", async () => {
@@ -268,7 +265,6 @@ describe("Transaction utilities", () => {
         computeUnitBuffer: { multiplier: 1.1 },
         onStatusUpdate: (status) => {
           statusUpdates.push(status);
-          console.log("status", status);
         },
       },
     );
@@ -279,7 +275,6 @@ describe("Transaction utilities", () => {
       ["created", "signed", "sent", "confirmed"],
     );
 
-    console.log("Versioned transaction signature", signature);
   });
 
   test("send versioned transaction with lookup table", async () => {
@@ -335,10 +330,6 @@ describe("Transaction utilities", () => {
 
     assert.ok(signature);
 
-    console.log(
-      "Versioned transaction with lookup tables signature",
-      signature,
-    );
   });
 
   test("sendTransaction should prepare and send a transaction and add priority fee instructions", async () => {
@@ -370,7 +361,6 @@ describe("Transaction utilities", () => {
         computeUnitBuffer: { multiplier: 1.1 },
         onStatusUpdate: (status) => {
           statusUpdates.push(status);
-          console.log("status", status);
         },
       },
     );
@@ -421,7 +411,6 @@ describe("Transaction utilities", () => {
         computeUnitBuffer: { multiplier: 1.1 },
         onStatusUpdate: (status) => {
           statusUpdates.push(status);
-          console.log("status", status);
         },
       },
     );
@@ -472,7 +461,6 @@ describe("Transaction utilities", () => {
         computeUnitBuffer: { multiplier: 1.1 },
         onStatusUpdate: (status) => {
           statusUpdates.push(status);
-          console.log("status", status);
         },
       },
     );