[Feature] Deduplicate consensus and rpc types AccessList

[DaniPopes]

Component

consensus, eips, genesis, network, json-rpc

Describe the feature you would like

alloy/crates/eips/src/eip2930.rs

Line 47 in b370cf3

pub struct AccessList(

alloy/crates/rpc-types/src/eth/transaction/access_list.rs

Line 17 in b370cf3

pub struct AccessList(pub Vec<AccessListItem>);

Additional context

No response

[mattsse]

from reth POV I believe there's a similar problem with the Withdrawal type,

so I think we should find a way to extract the Withdrawal type from rpc and perhaps also move to eip crate or find a better location for these types that have the same representation in rpc and consensus

[sambacha]

Checking out the access list generation, it seems there are no checks against preventing that `tx sender`, `tx recipient`, `block producer`, created contracts and precompiles  should *not* be included in access lists as they are considered warm. Should I make a separate issue?

[DaniPopes]

I don't fully understand your comment but that should definitely be in a different issue

[mattsse]

from,to are excluded

https://github.com/paradigmxyz/evm-inspectors/blob/d7317c7fbf38d5e5df2393beee0414c987ea440a/src/access_list.rs#L31

[mattsse]

AccessList+Withdrawals should be moved to eips crate

[JackG-eth]

Happy to pick this up