Skip to content

feat(submit): pylon blob preloading integration#215

Draft
Evalir wants to merge 6 commits intomainfrom
evalir/pylon-integration
Draft

feat(submit): pylon blob preloading integration#215
Evalir wants to merge 6 commits intomainfrom
evalir/pylon-integration

Conversation

@Evalir
Copy link
Member

@Evalir Evalir commented Feb 5, 2026

No description provided.

Copy link
Member Author

Evalir commented Feb 5, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Member

@prestwich prestwich left a comment

Choose a reason for hiding this comment

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

i think the task is too heavy. we just need a fire-and-forget dispatch in the flashbots spawned future. I also think we're duplicating the sidecar and passing it around too much?

/// Builds a MEV bundle from a signed transaction envelope and simulation result.
fn build_bundle(&self, envelope: &TxEnvelope, sim_result: &SimResult) -> EthSendBundle {
let tx_bytes: Bytes = envelope.encoded_2718().into();
let txs = self.build_bundle_body(sim_result, tx_bytes);
Copy link
Member

Choose a reason for hiding this comment

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

refactor while we're here: the build_bundle_body method doesn't use self. which implies that it is on the wrong type. what type should it be on?

Copy link
Member Author

@Evalir Evalir Feb 5, 2026

Choose a reason for hiding this comment

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

this is something i was kinda pondering before, along with sidecar things. both building the sidecar, and the bundle body, could live on SimResult? it doesn't really change once returned, so it could just be memoized

Copy link
Member

Choose a reason for hiding this comment

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

that seems fine

@Evalir Evalir requested a review from prestwich February 6, 2026 13:39
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