Skip to content

fix(cheatcodes): read broadcasts with the active network#14388

Merged
zerosnacks merged 1 commit intofoundry-rs:masterfrom
solanaXpeter:fix-cheatcodes-active-network-broadcasts
Apr 21, 2026
Merged

fix(cheatcodes): read broadcasts with the active network#14388
zerosnacks merged 1 commit intofoundry-rs:masterfrom
solanaXpeter:fix-cheatcodes-active-network-broadcasts

Conversation

@solanaXpeter
Copy link
Copy Markdown
Contributor

Motivation

The filesystem cheatcodes already run under generic FoundryEvmNetwork, but the broadcast and deployment readers still deserialize artifacts as Ethereum. That rejects valid Tempo broadcast data even though the surrounding execution path is already network-generic.

Solution

Thread the active FoundryEvmNetwork::Network through getBroadcast*, getDeployment*, getDeployments, and latest_broadcast instead of hardcoding Ethereum. Add a unit test with a minimal Tempo broadcast artifact to verify the latest sequence is parsed through the generic path.

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

Copy link
Copy Markdown
Collaborator

@mablr mablr left a comment

Choose a reason for hiding this comment

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

Good catch! Tysm!

Copy link
Copy Markdown
Member

@zerosnacks zerosnacks left a comment

Choose a reason for hiding this comment

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

Thanks! Good catch

@zerosnacks zerosnacks merged commit f609179 into foundry-rs:master Apr 21, 2026
16 checks passed
@github-project-automation github-project-automation Bot moved this to Done in Foundry Apr 21, 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