Skip to content

fix: restore L1 integration tests with PeerDAS-compatible SDK - #525

Merged
ping-ke merged 2 commits into
mainfrom
fix-action-it
Nov 3, 2026
Merged

ping-ke merged 2 commits into
mainfrom
fix-action-it

Conversation

@ping-ke

@ping-ke ping-ke commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

Motivation

The L1 integration test run timed out and was canceled while uploading blobs. The RPC repeatedly rejected eth_sendRawTransaction with:

Wrong sidecar format: unexpected sidecar version, want: 1, got: 0

The older SDK generated version 0 blob sidecars, while Geth required version 1 sidecars with PeerDAS cell proofs. Failed uploads kept retrying without making progress until the job timed out.

This change upgrades the SDK and aligns the integration test verifier with its blob encoding.

Summary

  • Upgrade ethstorage-sdk to 4.2.5 and ethers to 6.17.0 for version 1 sidecar support.
  • Check the SDK's { hash, success } return value before recording uploaded data.
  • Close SDK resources after uploading.
  • Use the existing Optimism encoder in the Go verifier to match the updated SDK.

Validation

  • Build: go build ./cmd/integration-test-server
  • Test: go test ./cmd/es-utils/utils ./cmd/integration-test-server
  • JavaScript syntax: node --check integration_tests/scripts/ituploader.js
  • Action: L1 integration test — in progress.

@ping-ke
ping-ke marked this pull request as draft October 28, 2026 02:06
@ping-ke
ping-ke marked this pull request as ready for review November 2, 2026 02:09
@ping-ke
ping-ke merged commit e4bb06e into main Nov 3, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants