Skip to content

fix(forge): disable synthetic Tempo fees - #16218

Closed
mattsse wants to merge 1 commit into
masterfrom
mattsse/disable-synthetic-tempo-fees
Closed

fix(forge): disable synthetic Tempo fees#16218
mattsse wants to merge 1 commit into
masterfrom
mattsse/disable-synthetic-tempo-fees

Conversation

@mattsse

@mattsse mattsse commented Aug 17, 2026

Copy link
Copy Markdown
Member

Forge validates synthetic script and test transactions with a zero gas price so network balances cannot block local execution, then the cheatcode inspector restores the configured gas price. Tempo observes that restored price during post-transaction fee handling and attempts to refund a fee that was never collected, causing a TIP20 insufficient-balance error during script setup.

This marks synthetic test environments with disable_fee_charge while preserving the restored tx.gasprice semantics visible to Solidity. It also adds an executor invariant and a forked Tempo script regression using a non-zero gas price and maximum block gas limit, with local fee liquidity cleared so the original failure is reproduced.

This PR was written with Codex, including investigation, implementation, regression tests, and PR text.

Synthetic script and test transactions validate with a zero gas price before the cheatcode inspector restores the configured value. Mark these environments as fee-free so Tempo does not attempt to collect or refund real TIP20 fees during local execution.
@github-actions

Copy link
Copy Markdown
Contributor

✅ Changelog found

The deterministic check will validate the changed entry.

@mablr mablr 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.

#16217 is narrower, and prevents breaking testdata snapshot.

@mablr mablr closed this Aug 17, 2026
@github-project-automation github-project-automation Bot moved this to Done in Foundry Aug 17, 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.

2 participants