fix: Handle error message for mined transactions (#637)

This commit adds support for handling error messages in mined transactions. The `MinedTransaction` type in `types.ts` now includes an optional `errorMessage` field. In the `mineTransactionWorker.ts` file, the code has been updated to retrieve and store the error message if the `userOpReceipt` is not successful. This allows for better error handling and reporting for mined transactions.

Author: d4mr

src/utils/transaction/types.ts

@@ -81,6 +81,9 @@ export type MinedTransaction = (
   gasUsed: bigint;
   effectiveGasPrice?: bigint;
   cumulativeGasUsed?: bigint;
+
+  // mined transactions can have an error message if they revert
+  errorMessage?: string;
 };
 
 // ErroredTransaction received an error before or while sending to RPC.

src/worker/tasks/mineTransactionWorker.ts

@@ -8,7 +8,7 @@ import {
   getRpcClient,
 } from "thirdweb";
 import { stringify } from "thirdweb/utils";
-import { getUserOpReceiptRaw } from "thirdweb/wallets/smart";
+import { getUserOpReceipt, getUserOpReceiptRaw } from "thirdweb/wallets/smart";
 import { TransactionDB } from "../../db/transactions/db";
 import { recycleNonce, removeSentNonce } from "../../db/wallets/walletNonce";
 import { getBlockNumberish } from "../../utils/block";
@@ -190,6 +190,7 @@ const _mineUserOp = async (
     chain,
     userOpHash,
   });
+
   if (!userOpReceiptRaw) {
     return null;
   }
@@ -205,6 +206,28 @@ const _mineUserOp = async (
     hash: transaction.hash,
   });
 
+  let errorMessage: string | undefined;
+
+  // if the userOpReceipt is not successful, try to get the parsed userOpReceipt
+  // we expect this to fail, but we want the error message if it does
+  if (!userOpReceiptRaw.success) {
+    try {
+      const userOpReceipt = await getUserOpReceipt({
+        client: thirdwebClient,
+        chain,
+        userOpHash,
+      });
+      await job.log(`Found userOpReceipt: ${userOpReceipt}`);
+    } catch (e) {
+      if (e instanceof Error) {
+        errorMessage = e.message;
+        await job.log("Failed to get userOpReceipt: " + e.message);
+      } else {
+        throw e;
+      }
+    }
+  }
+
   return {
     ...sentTransaction,
     status: "mined",
@@ -219,6 +242,7 @@ const _mineUserOp = async (
     cumulativeGasUsed: receipt.cumulativeGasUsed,
     sender: userOpReceiptRaw.sender as Address,
     nonce: userOpReceiptRaw.nonce.toString(),
+    errorMessage,
   };
 };