chore(consensus): Cut down on reth-primitives::TransactionSigned usage (#13659)

Author: hoank101

Cargo.lock

@@ -25,6 +25,7 @@ alloy-eips.workspace = true
 [dev-dependencies]
 alloy-consensus.workspace = true
 reth-storage-api.workspace = true
+reth-ethereum-primitives.workspace = true
 rand.workspace = true
 mockall = "0.13"
 

crates/consensus/common/Cargo.toml

@@ -347,7 +347,8 @@ mod tests {
     use mockall::mock;
     use rand::Rng;
     use reth_chainspec::ChainSpecBuilder;
-    use reth_primitives::{proofs, Account, BlockBody, Transaction, TransactionSigned};
+    use reth_ethereum_primitives::{Transaction, TransactionSigned};
+    use reth_primitives::{proofs, Account, BlockBody};
     use reth_storage_api::{
         errors::provider::ProviderResult, AccountReader, HeaderProvider, WithdrawalsProvider,
     };

crates/consensus/common/src/validation.rs

@@ -290,6 +290,15 @@ impl PartialEq for TransactionSigned {
     }
 }
 
+impl TransactionSigned {
+    /// Creates a new signed transaction from the given transaction and signature without the hash.
+    ///
+    /// Note: this only calculates the hash on the first [`TransactionSigned::hash`] call.
+    pub fn new_unhashed(transaction: Transaction, signature: Signature) -> Self {
+        Self { hash: Default::default(), signature, transaction }
+    }
+}
+
 impl Typed2718 for TransactionSigned {
     fn ty(&self) -> u8 {
         self.transaction.ty()