Skip to content

replace deleted SDK alias SealedHeader and bump version#61

Merged
Fraser999 merged 1 commit into
mainfrom
fraser/eng-2240/sealed-header
May 8, 2026
Merged

replace deleted SDK alias SealedHeader and bump version#61
Fraser999 merged 1 commit into
mainfrom
fraser/eng-2240/sealed-header

Conversation

@Fraser999
Copy link
Copy Markdown
Contributor

@Fraser999 Fraser999 commented May 8, 2026

Summary

  • Bump all storage crates from 0.7.2 to 0.8.0
  • Upgrade signet-sdk dependencies from 0.16.0-rc.11 to 0.17.1
  • Provide SealedHeader as a local type alias (Sealed<Header>) in signet-storage-types instead of re-exporting from signet-types, which no longer exports it

Motivation

signet-sdk 0.17.1 (init4tech/signet-sdk#232) removes the SealedHeader type alias from signet_types::primitives in favour of the validated SignetHeaderV1/SignetHeaderV2 wrapper types. Storage doesn't need the signet-specific header validation - it stores plain alloy::consensus::Header values - so we define the same Sealed<Header> alias locally and keep the rest of the codebase unchanged.

Note

The PR will only pass CI once init4tech/signet-sdk#232 is merged and the new version published.

Test plan

  • cargo t --workspace - all tests pass
  • cargo clippy --workspace --all-targets --all-features
  • cargo clippy --workspace --all-targets --no-default-features
  • RUSTDOCFLAGS="-D warnings" cargo doc --workspace --no-deps

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Fraser999 Fraser999 marked this pull request as draft May 8, 2026 16:18
@Fraser999 Fraser999 requested review from Evalir and prestwich May 8, 2026 16:20
@prestwich
Copy link
Copy Markdown
Member

approval for concept ACK

@Fraser999 Fraser999 marked this pull request as ready for review May 8, 2026 18:55
@Fraser999 Fraser999 merged commit b136c39 into main May 8, 2026
9 of 15 checks passed
@Fraser999 Fraser999 deleted the fraser/eng-2240/sealed-header branch May 8, 2026 19:06
@Evalir Evalir mentioned this pull request May 11, 2026
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