[Feature] Check empty EL reqs newPayloadV4

[emhane]

Component

provider

Describe the feature you would like

Check that execution_requests are always empty

op-alloy/crates/provider/src/ext/engine.rs

Lines 210 to 225 in b506e16

	async fn new_payload_v4(
	&self,
	payload: ExecutionPayloadV3,
	parent_beacon_block_root: B256,
	execution_requests: Vec<Bytes>,
	) -> TransportResult<PayloadStatus> {
	// Note: The `versioned_hashes` parameter is always an empty array for OP chains.
	let versioned_hashes: Vec<B256> = vec![];
	self.client()
	.request(
	"engine_newPayloadV4",
	(payload, versioned_hashes, parent_beacon_block_root, execution_requests),
	)
	.await
	}

Update docs accordingly

op-alloy/crates/provider/src/ext/engine.rs

Line 52 in b506e16

/// OP modifications: TODO

Additional context

ethereum-optimism/specs#506

[emhane]

matching issue to the spec if you like to work on it @meyer9

[emhane]

inclusion of this in op-reth is blocked by paradigmxyz/reth#13831 cc @mattsse

[meyer9]

sounds good, will do!

[emhane]

@meyer9 this is low prio, tackling this from another easier end for now paradigmxyz/reth#13908

[mattsse]

I think this argument should probably be removed from the function itself?
like we do with let versioned_hashes: Vec<B256> = vec![];