Skip to content

feat: add bid-options-slash-enabled flag to oracle#909

Merged
kant777 merged 3 commits intomainfrom
disable-bid-options-slash
Mar 23, 2026
Merged

feat: add bid-options-slash-enabled flag to oracle#909
kant777 merged 3 commits intomainfrom
disable-bid-options-slash

Conversation

@kant777
Copy link
Contributor

@kant777 kant777 commented Mar 23, 2026

Summary

  • Comments out bid options slashing logic in the oracle updater (oracle/pkg/updater/updater.go)
  • Disables: bid options unmarshaling, position constraint checking, shutterised bid option exception, and the "unsatisfied position constraints" slash path
  • Bundle atomicity slashing is preserved — the oracle still slashes if committed transactions are missing/out-of-order/failed

Test plan

  • Verify oracle builds cleanly (go build ./oracle/...)
  • Confirm oracle no longer slashes based on unsatisfied position constraints
  • Confirm bundle atomicity slashing still works as expected

kant777 added 3 commits March 23, 2026 10:33
Comment out bid options unmarshaling, position constraint
checking, and shutterised bid option logic in the oracle
updater. Bundle atomicity slashing is preserved.
Add a --bid-options-slash-enabled CLI flag (env: MEV_ORACLE_BID_OPTIONS_SLASH_ENABLED)
that gates bid options slashing in the oracle updater. When disabled (default),
the oracle skips position constraint checking, shutterised bid option handling,
and unsatisfied constraint slashing. Bundle atomicity slashing is always active.
…ions

Split TestBidOptions into subtests for enabled/disabled flag.
When disabled, all commitments settle as reward with no bid
options slashing.
@kant777 kant777 changed the title disable: bid options slashing in oracle feat: add bid-options-slash-enabled flag to oracle Mar 23, 2026
@kant777 kant777 merged commit 4d0abed into main Mar 23, 2026
7 of 8 checks passed
@kant777 kant777 deleted the disable-bid-options-slash branch March 23, 2026 18:00
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