Skip to content

fix: sync Vm interface IDs - #901

Merged
DaniPopes merged 2 commits into
foundry-rs:masterfrom
decofe:centaur/sync-vm-interface-ids-1786964997
Aug 17, 2026
Merged

fix: sync Vm interface IDs#901
DaniPopes merged 2 commits into
foundry-rs:masterfrom
decofe:centaur/sync-vm-interface-ids-1786964997

Conversation

@decofe

@decofe decofe commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • calculate the Vm and VmSafe interface IDs from the generated cheatcode selectors
  • update the assertions in test/Vm.t.sol whenever scripts/vm.py runs
  • fail clearly if either expected assertion cannot be found

Testing

  • forge fmt --check
  • forge test --match-contract VmTest -vvv
  • ran uv run python scripts/vm.py against the current Foundry cheatcodes input in a temporary checkout, then reran VmTest

Prompted by: @DaniPopes

Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
@DaniPopes
DaniPopes enabled auto-merge (squash) August 17, 2026 11:32
@DaniPopes
DaniPopes merged commit 680ee66 into foundry-rs:master Aug 17, 2026
20 checks passed
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