Update BlindedBlobsBundle SSZ list max length and update builder tests

[jimmygchen]

Issue Addressed

Addresses #4665

Proposed Changes

• Use MAX_BLOB_COMMITMENTS_PER_BLOCK as max length for BlindedBlobsBundle SSZ lists. This removes the BuilderKzgCommitments type and replace its usage with BlockBodyKzgCommitments everywhere, and renaming BlockBodyKzgCommitments back to just KzgCommitments. Builder spec PR:
  • RFC: Use MAX_BLOB_COMMITMENTS_PER_BLOCK as max length for BlindedBlobsBundle SSZ lists ethereum/builder-specs#87
• Testing:
  • Update mev-rs and ethereum-consensus dependencies to support latest deneb spec.
  • Update mock_builder::open_bid to convert response to mev-rs ExecutionPayload type correctly
  • Update local testnet scripts and README to support builder testing.

Issues

There's an issue with proposer prep that's currently being fixed in #4703, and the fix is required before continuing with testing.

[jimmygchen]

I haven't got mock builder fully working with mock relay yet, but I think these changes are good to go.

I think we need a modified mock_builder to make it work with mock-relay unfortunately. The payload header modifications in mock builder invalidates the block hash, which likely means the block hash needs to be recomputed and cached separately within the mock builder.

[jimmygchen]

I think our local testing with mock-relay is good enough now (able to cover fetch bid, reveal payload and construct a full block and publish), and happy to continue testing when we have mev infra in devnets. Unfortunately it's a bit tricky to build valid payload using mock/noop builder, and probably not going to help us much even if we get it to work.

Tested using this version of mock-relay: realbigsean/mock-relay#7

[realbigsean]

Nice! looks good to me