Skip to content

refactor(tempo): derive broadcast fee token - #16222

Merged
figtracer merged 5 commits into
masterfrom
mablr/lean-tempo-fee-token
Aug 18, 2026
Merged

refactor(tempo): derive broadcast fee token#16222
figtracer merged 5 commits into
masterfrom
mablr/lean-tempo-fee-token

Conversation

@mablr

@mablr mablr commented Aug 17, 2026

Copy link
Copy Markdown
Member

Description

This simplification was found while preparing #16217.

This removes duplicate Tempo fee-token state from CheatsConfig and instead derives the token from the active typed transaction environment when broadcast calls and contract creations are captured. This keeps the transaction environment as the single source of truth while preserving explicit --tempo.fee-token selections through simulation and broadcast. The regression coverage verifies that both CREATE and CALL entries persist the selected token in the broadcast artifact.

This change was developed and reviewed with AI assistance.

Use the active typed transaction environment as the source of fee-token
data when cheatcodes capture broadcast calls and creates. This removes
duplicate Tempo state from CheatsConfig while preserving explicit
fee-token propagation for simulation and broadcast.
@github-actions

Copy link
Copy Markdown
Contributor

✅ Changelog found

The deterministic check will validate the changed entry.

mattsse
mattsse previously approved these changes Aug 17, 2026

@mattsse mattsse left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

Comment thread crates/cheatcodes/src/inspector.rs
@figtracer
figtracer enabled auto-merge (squash) August 18, 2026 14:28
@figtracer
figtracer merged commit 8ac81ec into master Aug 18, 2026
42 checks passed
@figtracer
figtracer deleted the mablr/lean-tempo-fee-token branch August 18, 2026 16:57
@github-project-automation github-project-automation Bot moved this to Done in Foundry Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants