Skip to content

Conversation

Rjected
Copy link
Member

@Rjected Rjected commented Oct 15, 2022

DO NOT MERGE YET - depends on ethers-rs#1786, currently patched to get it to compile.

PR is merged so this is ready.

 * uses patch for now
@gakonst gakonst merged commit 07a83b1 into main Oct 15, 2022
@gakonst gakonst deleted the dan/add-h128 branch October 15, 2022 03:06
yutianwu pushed a commit to yutianwu/reth that referenced this pull request Jul 30, 2024
* fix: read sidecars from table in `get_take_block_range`

* fix lint issue
PeaStew added a commit to NeuraProtocol/reth-new-consensus that referenced this pull request Jul 7, 2025
…ot in canonical state

- Blocks are successfully created and persisted to database (up to block paradigmxyz#79 in logs)
- eth_blockNumber RPC returns 0x0 because it reads from canonical_in_memory_state
- Root cause: Our custom consensus bypasses Reth's engine API flow
- Blocks are written to DB but not to the canonical in-memory state
- BlockchainProvider uses canonical_in_memory_state, not direct DB reads
- Proper fix requires integrating with engine API or canonical state notifications
- Updated CLAUDE.md to document this critical issue

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
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