fix: min bump calculation

Author: MarcosNicolau

crates/batcher/src/lib.rs

@@ -1011,7 +1011,8 @@ impl Batcher {
         // TODO: move this to a config file
         let original_max_fee = entry.nonced_verification_data.max_fee;
         let bump_factor_percentage = 10;
-        let min_bump = original_max_fee * U256::from(bump_factor_percentage) / U256::from(100);
+        let min_bump = original_max_fee
+            + (original_max_fee * U256::from(bump_factor_percentage)) / U256::from(100);
         if replacement_max_fee < min_bump {
             std::mem::drop(batch_state_lock);
             warn!("Invalid replacement message for address {addr}, had max fee: {original_max_fee:?}, received fee: {replacement_max_fee:?}");

crates/sdk/src/communication/messaging.rs

@@ -267,7 +267,7 @@ async fn handle_batcher_response(msg: Message) -> Result<BatchInclusionData, Sub
         }
         Ok(SubmitProofResponseMessage::UnderpricedProof) => {
             error!("Batcher responded with error: proof underpriced. Funds have not been spent.");
-            Err(SubmitError::BatchQueueLimitExceededError)
+            Err(SubmitError::InvalidMaxFee)
         }
         Err(e) => {
             error!(

crates/sdk/src/verification_layer/mod.rs

@@ -202,12 +202,8 @@ pub fn compute_fee_per_proof_formula(num_proofs_in_batch: usize, gas_price: U256
         + ADDITIONAL_SUBMISSION_GAS_COST_PER_PROOF * num_proofs_in_batch as u128)
         / num_proofs_in_batch as u128;
 
-    let fee_per_proof = (U256::from(estimated_gas_per_proof)
-        * gas_price
-        * U256::from(GAS_PRICE_PERCENTAGE_MULTIPLIER))
-        / U256::from(PERCENTAGE_DIVIDER);
-
-    fee_per_proof
+    (U256::from(estimated_gas_per_proof) * gas_price * U256::from(GAS_PRICE_PERCENTAGE_MULTIPLIER))
+        / U256::from(PERCENTAGE_DIVIDER)
 }
 
 async fn fetch_gas_price(