feat(taiko-client): introduce payloadQueue for P2P gossip messages

[davidtaikocha]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces a payload queue for managing P2P gossip messages and refactors several methods to handle batches of execution payloads.

• Added a new payload queue to temporarily cache payloads for missing block imports.
• Updated API and block insertion logic to support batch processing via new methods.
• Introduced tests to validate scenarios with missing ancients.

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
packages/taiko-client/driver/preconf_blocks/server.go	Updated payload processing logic to support batch block insertion and caching of missing ancestors.
packages/taiko-client/driver/preconf_blocks/queue.go	Added a new payload queue implementation for managing P2P payloads.
packages/taiko-client/driver/preconf_blocks/api.go	Modified API to use the new batch insertion method and remove decompression logic.
packages/taiko-client/driver/driver_test.go	Added tests covering payloads with missing ancients.
packages/taiko-client/driver/chain_syncer/blob/blocks_inserter/pacaya.go	Refactored block insertion to support batch processing and updated error messages.
.github/workflows/taiko-client--docker.yml	Updated branch trigger list for CI builds.

Comments suppressed due to low confidence (2)

packages/taiko-client/driver/preconf_blocks/queue.go:52

• In the 'get' function, the loop exits upon encountering the first nil item. If the underlying slice becomes non-contiguous due to evictions or other operations, this may bypass valid entries later in the slice. Consider iterating over the entire slice instead of stopping at the first nil.

if item == nil { return nil // no more items }

packages/taiko-client/driver/chain_syncer/blob/blocks_inserter/pacaya.go:276

• [nitpick] The parameter name 'executableData' used in this function is inconsistent with the rest of the code which uses 'executionPayloads'. Renaming it to 'payload' (or 'executionPayload') would improve clarity and consistency.

func (i *BlocksInserterPacaya) insertPreconfBlockFromExecutionPayload(

[davidtaikocha]

Deploying this on helder to test, will approve/merge if works.

FYI also added more checks for P2P messages: 9705881 , not much changes but may need to build again 🙏