Skip to content

Conversation

@zeroXbrock
Copy link
Member

@zeroXbrock zeroXbrock commented May 22, 2025

Motivation

support this bundle type.

Solution

  • generalized bundle input to provider's send_bundle
  • new TypedBundle enum to hold different bundle types & send them with a given provider
  • new cli flag --bundle-type (--bt) to choose types (l1 or no-revert)

I also removed the --gas-price-percent-add (-g) flag bc it seemed unnecessary now that we have an automatic gas adder, and --skip-deploy-prompt since there's no more deploy prompt (builtin contracts are always deployed before spamming).

  • RevertProtectBundle encoding uses only a single tx
  • when sending multiple txs via RevertProtectBundleRequest, each tx gets its own bundle (for now)
cargo run -- spam ./scenarios/bundles.toml \
    -r $UNI_URL \
    --builder-url $UNI_URL \
    --bundle-type no-revert \
    -p $PRK \
    --min-balance 0.41eth \
    --tps 5

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

@github-actions
Copy link
Contributor

github-actions bot commented May 22, 2025

Preview Report

An example contender report preview is available here.

@zeroXbrock zeroXbrock changed the title revert-protect bundles 2 revert-protect bundles May 28, 2025
@zeroXbrock zeroXbrock changed the title revert-protect bundles revert-protect bundle support May 28, 2025
@zeroXbrock zeroXbrock merged commit 5ae0deb into main May 28, 2025
8 checks passed
@zeroXbrock zeroXbrock deleted the feat/revert-protect-bundles-2 branch May 28, 2025 20:37
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.

1 participant