Skip to content
This repository was archived by the owner on Oct 25, 2024. It is now read-only.

Conversation

@dvush
Copy link
Contributor

@dvush dvush commented Jun 2, 2023

📝 Summary

Implements mev-share builder.

  • in the block building
  • simulation method

📚 References


dvush added 6 commits June 1, 2023 17:48
* basic sbundle

* sbundle pool

* sbundle api

* local builder

* move sim bundle to core

* working builder

* db for sbundles

* report sbundle stat

* mev_simBundle nested logs

* refundConfig

* pay kickback from refundable value

* lints

* percentof

* sbundle pool with separate lock

* don't wait for error when adding sbundle
* don't touch header in SimBundle

* fix consolecmd test

* fix log extraction
* control inclusion in sim bundle
* use mevGasPrice in mev_simBundle
TotalProfit *big.Int
RefundableValue *big.Int
GasUsed uint64
MevGasPrice *big.Int
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we add a small, concise godoc on what MevGasPrice means?

ErrIncorrectRefundConfig = errors.New("incorrect refund config")
)

type SBundle struct {
Copy link
Contributor

Choose a reason for hiding this comment

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

could we add a small concise godoc on SBundle ?

Specifically what the S denotes, and how it's different from a regular bundle or transaction

@dvush dvush merged commit b6a6dd3 into main Jun 7, 2023
@dvush dvush deleted the mev-share branch June 7, 2023 16:19
avalonche pushed a commit that referenced this pull request Jul 6, 2023
New kind of bundle added. (`SBundle`) This bundle supports validity conditions such as refunds.
* bundle is supported in the builder, txpool
* api `mev_sendBundle` is used to accept this bundles
* api `mev_cancelBundleByHash` is used to remove bundle from pool
* api `mev_simBundle` is used to simulate this kind of bundle
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants