Skip to content

fix: bump streamingfast/reth to v2.3.0-fh-7 - #10

Merged
sduchesneau merged 1 commit into
release/world-chain-2.xfrom
fix/bump-reth-firehose-fh-7-world-chain
Aug 10, 2026
Merged

fix: bump streamingfast/reth to v2.3.0-fh-7#10
sduchesneau merged 1 commit into
release/world-chain-2.xfrom
fix/bump-reth-firehose-fh-7-world-chain

Conversation

@sduchesneau

Copy link
Copy Markdown

Moves the streamingfast/reth pin from v2.3.0-fh-6 to v2.3.0-fh-7 (streamingfast/reth#22).

On the truly-destroyed SELFDESTRUCT path (EIP-6780) revm credits the beneficiary in place and records the move only inside its AccountDestroyed journal entry — no BalanceTransfer — so resolve_post_tx_balance missed it. A coinbase, sender or fee vault that received a suicide refund reported a RewardTransactionFee / GasRefund old_balance contradicting the SuicideRefund event emitted moments earlier. The same resolver backs the OP fee-vault credits via post_tx_balance.

fh-6 → fh-7 touches only crates/firehose; no reth, revm or alloy-evm version moves, so the Cargo.lock diff is purely the git tag/rev for the streamingfast/reth source. cargo check -p reth-optimism-firehose --all-targets and cargo check -p op-reth --all-targets both pass.

🤖 Generated with Claude Code

Picks up the selfdestruct-refund post-tx balance fix; the resolver also
backs the OP fee-vault credits.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@sduchesneau
sduchesneau changed the base branch from firehose/world-chain-2.x to release/world-chain-2.x August 10, 2026 17:56
@sduchesneau
sduchesneau requested a review from maoueh August 10, 2026 18:04
@sduchesneau
sduchesneau merged commit 1df6488 into release/world-chain-2.x Aug 10, 2026
sduchesneau added a commit to streamingfast/world-chain that referenced this pull request Aug 10, 2026
streamingfast/optimism#10 merged, so release/world-chain-2.x now pins fh-7
and the reth graph resolves to a single copy again.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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