Skip to content

refactor(mev): drop pre-submit simulation in BloxrouteBackrunmeSender.SendBackrunBundle#210

Merged
datluongductuan merged 1 commit into
mainfrom
chore/remove-presubmit-sim-backrunme
Jun 17, 2026
Merged

refactor(mev): drop pre-submit simulation in BloxrouteBackrunmeSender.SendBackrunBundle#210
datluongductuan merged 1 commit into
mainfrom
chore/remove-presubmit-sim-backrunme

Conversation

@datluongductuan

Copy link
Copy Markdown
Member

What

BloxrouteBackrunmeSender.SendBackrunBundle called MevSimulateBundle (simulate_arb_only_bundle) before submitting, and aborted the send if it errored. This drops that pre-submit simulation — SendBackrunBundle now goes straight to submit_arb_only_bundle (with coinbase_profit).

-	// First, simulate the bundle using MevSimulateBundle
-	_, err := s.MevSimulateBundle(ctx, blockNumber, pendingTxHashes[0], txs[0])
-	if err != nil {
-		return SendBundleResponse{}, fmt.Errorf("simulate bundle failed: %w", err)
-	}
-
-	// If simulation passed, proceed with submission
	// Build request params
	params := backrunmeRequestParams{ ... }

Why

The pre-submit simulate round-trip is unnecessary on the send path and adds latency. MevSimulateBundle stays on the type (still part of IBackrunSender) for callers that want to simulate explicitly — only the implicit call inside SendBackrunBundle is removed.

Also resolves the earlier review note that coinbase_profit was carried by submit but not by the in-send simulate: there is no in-send simulate anymore, so the two can't diverge.

Build + vet clean; no signature change.

🤖 Generated with Claude Code

….SendBackrunBundle

SendBackrunBundle no longer calls MevSimulateBundle before submitting the arb-only
bundle — the pre-submit simulate_arb_only_bundle round-trip is unnecessary and adds
latency to the send path. MevSimulateBundle remains on the type for callers that want to
simulate explicitly. submit_arb_only_bundle (with coinbase_profit) is sent directly.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@datluongductuan datluongductuan merged commit bbca10e into main Jun 17, 2026
3 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