Maybe no need to remove transactions manually after producing L2 block

[yaziciahmet]

We have this following code piece in produce_l2_block method of sequencer:

                let mut txs_to_remove = self.db_provider.last_block_tx_hashes()?;
                txs_to_remove.extend(l1_fee_failed_txs);

                self.mempool.remove_transactions(txs_to_remove.clone());
                SEQUENCER_METRICS.mempool_txs.set(self.mempool.len() as f64);

                let account_updates = self.get_account_updates()?;

                self.mempool.update_accounts(account_updates);

As far as I checked the reth mempool implementation, update_accounts method already removes the transactions since the transactions will already be dropped due to updated nonce of the account. But there is also the fact that we also need to remove l1_fee_failed_txs, which is not going to be removed by update_accounts method.

This is a reminder issue to double-check reth and potentially find ways to improve this part of the code.