refactor: unneeded changes

Author: obatirou

coprocessor/fhevm-engine/transaction-sender/src/ops/add_ciphertext.rs

@@ -146,17 +146,19 @@ where
                 handle = h,
                 "addCiphertext txn failed"
             );
+
             self.increment_txn_limited_retries_count(
                 handle,
                 "receipt status = false",
                 current_limited_retries_count,
             )
             .await?;
+
             return Err(anyhow::anyhow!(
                 "Transaction {} failed with status {}, handle: {}",
                 receipt.transaction_hash,
                 receipt.status(),
-                h
+                h,
             ));
         }
         Ok(())

coprocessor/fhevm-engine/transaction-sender/src/ops/allow_handle.rs

@@ -172,14 +172,20 @@ where
                 handle = h,
                 "allowAccount txn failed"
             );
-            let reason = format!(
-                "Transaction {} failed with status {}",
+
+            self.increment_txn_limited_retries_count(
+                key,
+                "receipt status = false",
+                current_limited_retries_count,
+            )
+            .await?;
+
+            return Err(anyhow::anyhow!(
+                "Transaction {} failed with status {}, handle: {}",
                 receipt.transaction_hash,
                 receipt.status(),
-            );
-            self.increment_txn_limited_retries_count(key, &reason, current_limited_retries_count)
-                .await?;
-            return Err(anyhow::anyhow!("{reason}, handle: {h}",));
+                h,
+            ));
         }
         Ok(())
     }
@@ -374,8 +380,8 @@ where
         let rows = sqlx::query!(
             "
             SELECT handle, account_address, event_type, txn_limited_retries_count, txn_unlimited_retries_count, transaction_id
-            FROM allowed_handles 
-            WHERE txn_is_sent = false 
+            FROM allowed_handles
+            WHERE txn_is_sent = false
             AND txn_limited_retries_count < $1
             LIMIT $2;
             ",

coprocessor/fhevm-engine/transaction-sender/src/ops/delegate_user_decrypt.rs

@@ -192,7 +192,10 @@ impl<P: Provider<Ethereum> + Clone + 'static> DelegateUserDecryptOperation<P> {
                 "{operation} txn failed"
             );
             TxResult::OtherError(format!(
-                "Transaction {transaction_hash} failed with status false"
+                "Transaction {} failed with status {}, Delegation: {:?}",
+                transaction_hash,
+                receipt.status(),
+                delegation,
             ))
         }
     }

coprocessor/fhevm-engine/transaction-sender/src/ops/verify_proof.rs

@@ -213,14 +213,18 @@ where
                 status = receipt.status(),
                 "Transaction failed"
             );
-            let reason = format!(
-                "Transaction {} failed with status {}",
+            self.update_retry_count_by_proof_id(
+                txn_request.0,
+                current_retry_count,
+                "receipt status = false",
+            )
+            .await?;
+            return Err(anyhow::anyhow!(
+                "Transaction {} for zk_proof_id {} failed with status {}",
                 receipt.transaction_hash,
+                txn_request.0,
                 receipt.status(),
-            );
-            self.update_retry_count_by_proof_id(txn_request.0, current_retry_count, &reason)
-                .await?;
-            return Err(anyhow::anyhow!("{reason}, zk_proof_id: {}", txn_request.0,));
+            ));
         }
         Ok(())
     }