replace block fields with BlockBody

[mattsse]

Describe the feature

currently Block has a flattened blockbody as its fields:

reth/crates/primitives/src/block.rs

Lines 40 to 47 in 97cc9dd

	/// Transactions in this block.
	pub body: Vec<TransactionSigned>,
	/// Ommers/uncles header.
	pub ommers: Vec<Header>,
	/// Block withdrawals.
	pub withdrawals: Option<Withdrawals>,
	/// Block requests.
	pub requests: Option<Requests>,

reth/crates/primitives/src/block.rs

Lines 557 to 566 in 97cc9dd

	pub struct BlockBody {
	/// Transactions in the block
	pub transactions: Vec<TransactionSigned>,
	/// Uncle headers for the given block
	pub ommers: Vec<Header>,
	/// Withdrawals in the block.
	pub withdrawals: Option<Withdrawals>,
	/// Requests in the block.
	pub requests: Option<Requests>,
	}

if we want to abstract a block as

trait Block {type Body;}

then we need to use the container type

see also:

reth/crates/primitives/src/block.rs

Lines 301 to 306 in 97cc9dd

	/// Create a new sealed block instance using the sealed header and block body.
	#[inline]
	pub fn new(header: SealedHeader, body: BlockBody) -> Self {
	let BlockBody { transactions, ommers, withdrawals, requests } = body;
	Self { header, body: transactions, ommers, withdrawals, requests }
	}

TODO

• replace fields with BlockBody

Additional context

fyi @refcell

alloy-rs/alloy#1290

[roynalnaruto]

@mattsse can I take this up?

In addition to the above, may be some methods can be moved to the BlockBody, for instance has_blob_transactions, has_eip7702_transactions, and a few others. With Block dereferencing to BlockBody for those ?

[PoulavBhowmick03]

@mattsse I am a new contributor to reth. I would love to work on this!

[roynalnaruto]

@mattsse I have some preliminary work here: #10913, let me know if it generally looks good