Make TransactionValidationOutcome generic over error type

[emhane]

Describe the feature

Make generic over invalid transaction error type with default being E = InvalidPoolTransactionError

reth/crates/transaction-pool/src/validate/mod.rs

Lines 31 to 55 in 42dc1ed

	/// A Result type returned after checking a transaction's validity.
	#[derive(Debug)]
	pub enum TransactionValidationOutcome<T: PoolTransaction> {
	/// The transaction is considered _currently_ valid and can be inserted into the pool.
	Valid {
	/// Balance of the sender at the current point.
	balance: U256,
	/// Current nonce of the sender.
	state_nonce: u64,
	/// The validated transaction.
	///
	/// See also [`ValidTransaction`].
	///
	/// If this is a _new_ EIP-4844 blob transaction, then this must contain the extracted
	/// sidecar.
	transaction: ValidTransaction<T>,
	/// Whether to propagate the transaction to the network.
	propagate: bool,
	},
	/// The transaction is considered invalid indefinitely: It violates constraints that prevent
	/// this transaction from ever becoming valid.
	Invalid(T, InvalidPoolTransactionError),
	/// An error occurred while trying to validate the transaction
	Error(TxHash, Box<dyn core::error::Error + Send + Sync>),
	}

This is necessary so that rollups can match on their own variants of invalid transaction errors.

Additional context

No response

[PanGan21]

Hey could I work on this?

[PanGan21]

Hey @emhane I prepared #14046 for this issue. Let me know if it requires something more.