Skip to content

fix(forge): isolate overloaded fuzz paths - #16307

Open
stevencartavia wants to merge 9 commits into
masterfrom
steven/overloaded-fuzz-paths
Open

fix(forge): isolate overloaded fuzz paths#16307
stevencartavia wants to merge 9 commits into
masterfrom
steven/overloaded-fuzz-paths

Conversation

@stevencartavia

Copy link
Copy Markdown
Member

Uses selector-suffixed persistence paths for overloaded fuzz tests, preventing their failures, corpora, and showmap outputs from overwriting each other. Legacy failures remain replayable when their calldata selector matches the requested overload.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Changelog found

The deterministic check will validate the changed entry.

mablr
mablr previously approved these changes Aug 21, 2026

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

SGTM code wise, do we need a benchmark ?

@stevencartavia

Copy link
Copy Markdown
Member Author

SGTM code wise, do we need a benchmark ?

I don’t think so, this is a correctness fix outside the fuzzing hot loop.

Comment thread crates/forge/src/runner.rs Outdated
Comment thread crates/forge/src/runner.rs
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.

3 participants