feat(primitives): add Receipts to encapsulate Vec<Vec<Option<Receipt>>

[tcoratger]

Should resolve #4621.

[codecov]

Codecov Report

Merging #4626 (3c55fb8) into main (b994d15) will increase coverage by 0.02%.
The diff coverage is 50.60%.

Files	Coverage Δ
bin/reth/src/init.rs	97.54% <100.00%> (ø)
crates/primitives/src/lib.rs	100.00% <ø> (ø)
crates/primitives/src/prune/part.rs	100.00% <ø> (ø)
crates/storage/provider/src/chain.rs	74.24% <100.00%> (ø)
...torage/provider/src/providers/database/provider.rs	78.90% <100.00%> (ø)
crates/storage/provider/src/test_utils/blocks.rs	79.75% <100.00%> (ø)
crates/revm/src/processor.rs	84.34% <33.33%> (+2.14%)	⬆️
...der/src/bundle_state/bundle_state_with_receipts.rs	92.54% <84.61%> (+0.36%)	⬆️
crates/rpc/rpc/src/eth/api/pending_block.rs	0.00% <0.00%> (ø)
crates/payload/basic/src/lib.rs	0.00% <0.00%> (ø)
... and 1 more

... and 10 files with indirect coverage changes

Flag	Coverage Δ
integration-tests	16.17% <0.00%> (-0.01%)	⬇️
unit-tests	63.29% <50.60%> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
reth binary	32.11% <100.00%> (ø)
blockchain tree	83.69% <ø> (ø)
pipeline	88.45% <ø> (ø)
storage (db)	73.49% <90.00%> (+0.01%)	⬆️
trie	94.48% <ø> (ø)
txpool	49.97% <ø> (+0.46%)	⬆️
networking	76.42% <ø> (-0.06%)	⬇️
rpc	57.73% <0.00%> (+<0.01%)	⬆️
consensus	62.59% <ø> (+0.12%)	⬆️
revm	28.46% <33.33%> (+0.12%)	⬆️
payload builder	8.16% <0.00%> (-0.05%)	⬇️
primitives	86.24% <46.80%> (-0.12%)	⬇️

[mattsse]

smol nit

[rakita]

Just one more code blocks that can be moved to a function inside Receipts

reth/crates/revm/src/processor.rs

Lines 329 to 347 in 8baf234

	gas_spent_by_tx: self
	.receipts
	.last()
	.map(|block_r| {
	block_r
	.iter()
	.enumerate()
	.map(|(id, tx_r)| {
	(
	id as u64,
	tx_r.as_ref()
	.expect("receipts have not been pruned")
	.cumulative_gas_used,
	)
	})
	.collect()
	})
	.unwrap_or_default(),
	}

[tcoratger]

Just one more code blocks that can be moved to a function inside Receipts

reth/crates/revm/src/processor.rs

Lines 329 to 347 in 8baf234

	gas_spent_by_tx: self
	.receipts
	.last()
	.map(|block_r| {
	block_r
	.iter()
	.enumerate()
	.map(|(id, tx_r)| {
	(
	id as u64,
	tx_r.as_ref()
	.expect("receipts have not been pruned")
	.cumulative_gas_used,
	)
	})
	.collect()
	})
	.unwrap_or_default(),
	}

@rakita Done

[mattsse]

lgtm, pending @rakita

and conflicts

[tcoratger]

lgtm, pending @rakita

and conflicts

@mattsse Conflicts are resolved :)

[rakita]

Have a request for change here: #4626 (comment)

[rakita]

lgtm

[mattsse]

lgtm

[rakita]

@tcoratger there is one conflict at crates/payload/basic/src/lib.rs

[tcoratger]

@tcoratger there is one conflict at crates/payload/basic/src/lib.rs

@rakita Just solved :)

unblock merge