Skip to content

Comments

feat: add scenario labels#456

Merged
zeroXbrock merged 5 commits intoflashbots:mainfrom
0xl3on:scenario-contract-labels
Feb 18, 2026
Merged

feat: add scenario labels#456
zeroXbrock merged 5 commits intoflashbots:mainfrom
0xl3on:scenario-contract-labels

Conversation

@0xl3on
Copy link
Contributor

@0xl3on 0xl3on commented Feb 18, 2026

Motivation

Closes stale issue at #436

Solution

  • Contract names are suffixed with _{label} when stored during deploy (insert_named_txs) and when looked up during template resolution (get_named_tx).
  • Added scenario_label: Option to TestScenarioParams and TestScenario
  • Added get_scenario_label() to the Generator trait
  • Threaded scenario_label into Templater methods (find_placeholder_values, find_fncall_placeholders, find_create_placeholders) and DeployContractParams
  • Exposed scenario_label on ContenderCtxBuilder for library users

PR Checklist

  • Added Tests
  • Added Documentation
  • Ran cargo +nightly clippy --workspace --lib --examples --tests --benches --all-features --locked --fix
  • Ran cargo fmt --all
  • Note breaking changes in PR description, if applicable
  • update changelogs
    • Update CHANGELOG.md in each affected crate
    • add a high-level description in the root changelog

@0xl3on 0xl3on requested a review from zeroXbrock as a code owner February 18, 2026 02:58
@0xl3on
Copy link
Contributor Author

0xl3on commented Feb 18, 2026

Hey @zeroXbrock !

Noticed that this issue was stale for a long time so picked it up. Hope that's fine!

Copy link
Member

@zeroXbrock zeroXbrock left a comment

Choose a reason for hiding this comment

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

Nicely done, thank you @0xl3on! I just made a couple minor changes. I'll also resolve the conflicts.

@zeroXbrock zeroXbrock merged commit 0810d55 into flashbots:main Feb 18, 2026
7 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