Skip to content

fix(anvil): use fee manager as Tempo fork coinbase#14964

Open
BROCCOLO1D wants to merge 7 commits into
foundry-rs:masterfrom
BROCCOLO1D:fix/tempo-fork-coinbase
Open

fix(anvil): use fee manager as Tempo fork coinbase#14964
BROCCOLO1D wants to merge 7 commits into
foundry-rs:masterfrom
BROCCOLO1D:fix/tempo-fork-coinbase

Conversation

@BROCCOLO1D
Copy link
Copy Markdown
Contributor

@BROCCOLO1D BROCCOLO1D commented May 29, 2026

Summary

  • Defaults Anvil Tempo forks to the Tempo fee manager address as block.coinbase/beneficiary when no genesis coinbase is supplied.
  • Applies the same default when anvil_reset starts forking from a non-forked Tempo node.
  • Keeps explicit genesis coinbase configuration unchanged.

Why

Tempo mainnet maps the zero-address validator token to a DONOTUSE sentinel. In fork mode, Anvil previously inherited the default zero beneficiary, so state-changing transactions could hit Tempo fee collection failures and then appear as missing receipts indefinitely. Tempo node-side simulation already uses the fee-manager sentinel to fall back to PathUSD.

Changes

  • Centralized the Tempo fork beneficiary default in NodeConfig.
  • Applied the default during Tempo fork startup and during anvil_reset when it initializes a fork from a non-forked node.
  • Added regression coverage for both startup forks and reset-to-fork flows.

Validation

  • git diff --check
  • cargo fmt --check
  • cargo test -p anvil --test it tempo::test_tempo

Scope

Small Anvil Tempo fork/default-beneficiary behavior only; does not alter non-Tempo forks, standalone Tempo nodes, or explicit genesis coinbase settings.

Refs #14866

Copy link
Copy Markdown
Collaborator

@figtracer figtracer left a comment

Choose a reason for hiding this comment

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

two findings.

Comment thread crates/anvil/tests/it/tempo.rs
Comment thread crates/anvil/src/config.rs Outdated
@BROCCOLO1D BROCCOLO1D force-pushed the fix/tempo-fork-coinbase branch from f5bc16a to 7c0b0dc Compare May 29, 2026 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants