feat(primitives): Remove Attributes

[refcell]

Description

Removes the attributes module from kona-primitives, re-using payload attributes types defined in alloy-rpc-types-engine and op-alloy-rpc-types-engine.

Closes #510

[refcell]

• feat(primitives): Remove Attributes #529 👈
• feat: Dependency Update #528
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @refcell and the rest of your teammates on Graphite

[codecov]

Codecov Report

Attention: Patch coverage is 97.75785% with 5 lines in your changes missing coverage. Please review.

Project coverage is 76.9%. Comparing base (f639744) to head (7f3ade4).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/derive/src/stages/attributes_queue.rs	94.5%	2 Missing ⚠️
crates/executor/src/lib.rs	98.4%	2 Missing ⚠️
crates/derive/src/pipeline/core.rs	0.0%	1 Missing ⚠️

Additional details and impacted files

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[clabby]

Looks good, but I'd prefer if we changed OptimismPayloadAttributes to not have Options for transactions, no_tx_pool. These should always be there, since the type is not intended to be backwards-compat w/ L1 payload attributes.

[refcell]

Looks good, but I'd prefer if we changed OptimismPayloadAttributes to not have Options for transactions, no_tx_pool. These should always be there, since the type is not intended to be backwards-compat w/ L1 payload attributes.

They're all marked omitempty in the reference implementation: https://github.com/ethereum-optimism/optimism/blob/8dd6fb36aad7a1cd5c9e7e877b340b2e0c5cc14c/op-service/eth/types.go#L326-L331

Do you want to diverge from this for any specific reason?