Modified comments and improved the error thrown by getTransactionWithRetry

Author: julianlen

tool/pmt-builder-utils.js

@@ -74,10 +74,10 @@ const getTransactionWithRetry = async (transactionsClient, txId, retries = 0) =>
     try {
         return await transactionsClient.getTxHex({txid: txId});
     } catch (error) {
-        // mempool.js might wrap the error, so we check for common indicators of 429
+        // mempool.js might wrap the error, so we check for common indicators of too many requests error code
         const isRateLimitError = error.response && error.response.status === TOO_MANY_REQUESTS_ERROR_CODE;
 
-        // Sometimes the error message might contain clues if status is not directly 429
+        // Sometimes the error message might contain clues if status code is not directly 429
         const isMempoolSpecificRateLimit = error.message && error.message.includes('Too Many Requests');
 
         if ((isRateLimitError || isMempoolSpecificRateLimit) && retries < MAX_RETRIES) {
@@ -87,9 +87,7 @@ const getTransactionWithRetry = async (transactionsClient, txId, retries = 0) =>
             return getTransactionWithRetry(transactionsClient, txId, retries + 1);
         }
 
-        throw new Error(`Failed to fetch transaction details for txId: ${txId} after ${retries} retries. Error: ${error.message}`,
-            { cause: error }
-        );
+        throw new Error(`Failed to fetch transaction details for txId: ${txId} after ${retries} retries. Error: ${error.message}`);
     }
 };