Skip to content

chore: bump reth deps to latest main#17

Merged
figtracer merged 1 commit intoparadigmxyz:mainfrom
figtracer:gustavo/bump-reth-main
Apr 5, 2026
Merged

chore: bump reth deps to latest main#17
figtracer merged 1 commit intoparadigmxyz:mainfrom
figtracer:gustavo/bump-reth-main

Conversation

@figtracer
Copy link
Copy Markdown
Collaborator

@figtracer figtracer commented Apr 4, 2026

Summary

  • Bumps all reth deps from v1.11.0 to adc960162f (latest main)
  • reth-primitives-traits moved to crates.io v0.1.0
  • Removed serde-bincode-compat feature (no longer exists)
  • Reimplemented reveal_witness locally (removed upstream in reth#22021)
  • Updated check_valid_account_witnessis_account_revealed
  • Bumped alloy 1.51.8, revm 3436

Unblocks

@figtracer figtracer force-pushed the gustavo/bump-reth-main branch from ad84a2b to 67770cf Compare April 4, 2026 20:14
@figtracer figtracer marked this pull request as draft April 4, 2026 20:19
@figtracer figtracer force-pushed the gustavo/bump-reth-main branch 6 times, most recently from 1f02dab to f207da8 Compare April 4, 2026 21:23
@figtracer figtracer force-pushed the gustavo/bump-reth-main branch from f207da8 to 1184931 Compare April 4, 2026 21:27
nodes_by_hash: &B256IndexMap<alloy_primitives::Bytes>,
) -> Result<DecodedMultiProofV2, alloy_rlp::Error> {
#[allow(clippy::disallowed_types)]
use alloy_primitives::map::B256Map;
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should decide on whether to use B256IndexMap or B256Map (...)) on Reth, otherwise this will get very messy


env:
CARGO_TERM_COLOR: always
LIBZ_SYS_STATIC: 0
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added since rocksdb is non-optional now

///
/// This replicates the old `SparseStateTrie::reveal_witness` logic, but outputs the V2 multiproof
/// structure needed by the current API.
fn build_multiproof_from_witness(
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we could add this in Reth

@figtracer figtracer requested a review from mattsse April 5, 2026 08:28
@figtracer figtracer marked this pull request as ready for review April 5, 2026 08:28
@jsign
Copy link
Copy Markdown
Contributor

jsign commented Apr 5, 2026

@figtracer, yeah, merging this PR will help to try to refloat #8 since I've to rebase paradigmxyz/reth#22289 on top of main.

@figtracer
Copy link
Copy Markdown
Collaborator Author

@figtracer, yeah, merging this PR will help to try to refloat #8 since I've to rebase paradigmxyz/reth#22289 on top of main.

will get this merged asap

@figtracer figtracer merged commit 8f8c9dc into paradigmxyz:main Apr 5, 2026
4 checks passed
@figtracer figtracer deleted the gustavo/bump-reth-main branch April 5, 2026 23:45
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.

2 participants