Skip to content

Conversation

CPerezz
Copy link
Collaborator

@CPerezz CPerezz commented Sep 30, 2025

Skip DumpToCollector entirely for Binary Trie mode and use only
DumpBinTrieLeaves, which correctly handles all leaf types in the unified tree.

  • Modified transition.go to conditionally skip DumpToCollector for Binary Trie
  • Only use DumpBinTrieLeaves when in Binary Trie mode (IsVerkle check)

All 52 tests in eip6800_genesis_verkle_tree now PASSIN

This fix resolves the critical multi-block persistence issues that were
preventing Binary Trie state from being correctly reconstructed between blocks.

@CPerezz CPerezz changed the title feat(dump): implement Binary Trie state dumping for t8n tool feat(cmd/evm/internal/t8ntool): Add Binary Trie state dumping for t8n tool for multi-block persistent tests Sep 30, 2025
@gballet gballet changed the title feat(cmd/evm/internal/t8ntool): Add Binary Trie state dumping for t8n tool for multi-block persistent tests cmd/evm/internal/t8ntool: Add Binary Trie state dumping for t8n tool for multi-block persistent tests Sep 30, 2025
@CPerezz CPerezz requested a review from gballet as a code owner September 30, 2025 15:26
Skip DumpToCollector entirely for Binary Trie mode and use only
DumpBinTrieLeaves, which correctly handles all leaf types in the unified tree.

- Modified transition.go to conditionally skip DumpToCollector for Binary Trie
- Only use DumpBinTrieLeaves when in Binary Trie mode (IsVerkle check)

All 52 tests in eip6800_genesis_verkle_tree now PASSIN

This fix resolves the critical multi-block persistence issues that were
preventing Binary Trie state from being correctly reconstructed between blocks.
@CPerezz CPerezz force-pushed the fix/multiblock-dump-in-bintrie branch from ee53428 to 68f8dc9 Compare September 30, 2025 21:25
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.

1 participant