Skip to content

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

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

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

Conversation

@sduchesneau

Copy link
Copy Markdown

Moves the streamingfast/reth pin from v2.3.0-fh-5 to v2.3.0-fh-7, picking up two Firehose balance-change fixes:

  • fh-7 (fix(firehose): credit selfdestruct refund in post-tx balance resolver reth#22): the SELFDESTRUCT refund is now included when resolving an account's post-transaction balance. On the truly-destroyed path (EIP-6780) revm credits the beneficiary in place and records the move only inside its AccountDestroyed journal entry — no BalanceTransfer — so a coinbase, sender or fee vault that received a suicide refund reported a RewardTransactionFee / GasRefund old_balance contradicting the SuicideRefund event emitted moments earlier.
  • fh-6: value-transfer balance changes are emitted when a transaction sends value to a precompile and then fails; the reverted callee had its BalanceTransfer journal entry truncated before the journal walk ran.

fh-5 → 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 --workspace --all-targets passes.

🤖 Generated with Claude Code

@sduchesneau
sduchesneau requested a review from maoueh August 10, 2026 17:41
Picks up the selfdestruct-refund post-tx balance fix (fh-7) and the
reverted-precompile value-transfer balance changes (fh-6).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@sduchesneau
sduchesneau force-pushed the fix/bump-reth-firehose-fh-7 branch from 8afc495 to 4b3e490 Compare August 10, 2026 17:56
@sduchesneau
sduchesneau merged commit 84633f3 into release/1.x Aug 10, 2026
3 checks passed
@sduchesneau
sduchesneau deleted the fix/bump-reth-firehose-fh-7 branch August 10, 2026 20:24
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