feat(taiko-client): validate the payload before caching it && only cache the preconf block request after sent

[davidtaikocha]

No description provided.

[Copilot]

Pull Request Overview

Adds upfront payload validation in the L2 payload handlers and enhances transaction decoding logs

• Moves ValidateExecutionPayload checks to the start of OnUnsafeL2Payload and OnUnsafeL2Response
• Removes redundant validation blocks further down in each handler
• Expands the log.Info call in ValidateExecutionPayload to include parentHash, timestamp, and coinbase

Comments suppressed due to low confidence (2)

packages/taiko-client/driver/preconf_blocks/server.go:228

• Add unit or integration tests for the new early-return validation in OnUnsafeL2Payload, ensuring that invalid payloads increment DriverPreconfP2PInvalidEnvelopeCounter and that no further processing occurs.

if err := s.ValidateExecutionPayload(msg.ExecutionPayload); err != nil {

packages/taiko-client/driver/preconf_blocks/server.go:320

• [nitpick] The warning message reads slightly awkwardly; consider rephrasing to something like "Invalid preconfirmation block response payload" for clarity.

"Invalid preconfirmation block payload response",

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 34 lines in your changes missing coverage. Please review.

Project coverage is 32.40%. Comparing base (5c32d6a) to head (b4a827c).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
...kages/taiko-client/driver/preconf_blocks/server.go	0.00%	34 Missing ⚠️

Additional details and impacted files

Files with missing lines	Coverage Δ
...kages/taiko-client/driver/preconf_blocks/server.go	10.23% <0.00%> (-0.10%)	⬇️

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c32d6a...b4a827c. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.