From 5e1d1c31a9d29cb5312a94e8311616d592e74bea Mon Sep 17 00:00:00 2001 From: Vitaly Drogan Date: Fri, 24 Oct 2025 14:36:37 +0200 Subject: [PATCH 1/2] restore_landed_orders: fix restore for bundle with refund --- .../restore_landed_orders/find_landed_orders.rs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/crates/rbuilder/src/backtest/restore_landed_orders/find_landed_orders.rs b/crates/rbuilder/src/backtest/restore_landed_orders/find_landed_orders.rs index 7ddd21a9d..2ab1c90ce 100644 --- a/crates/rbuilder/src/backtest/restore_landed_orders/find_landed_orders.rs +++ b/crates/rbuilder/src/backtest/restore_landed_orders/find_landed_orders.rs @@ -46,7 +46,7 @@ impl SimplifiedOrder { )], ), Order::Bundle(bundle) => { - let (refund_percent, receiver_hash) = if let Some(refund) = &bundle.refund { + let (refund_percent, refund_payer_hash) = if let Some(refund) = &bundle.refund { (refund.percent as usize, Some(refund.tx_hash)) } else { (0, None) @@ -55,10 +55,10 @@ impl SimplifiedOrder { .list_txs_revert() .into_iter() .map(|(tx, revert)| { - let tx_refund_percent = if Some(tx.hash()) == receiver_hash { - 0 - } else { + let tx_refund_percent = if Some(tx.hash()) == refund_payer_hash { refund_percent + } else { + 0 }; OrderTxData::new(tx.hash(), revert, tx_refund_percent) }) @@ -1000,7 +1000,7 @@ mod tests { refund: Some(BundleRefund { percent: 10, recipient: Default::default(), - tx_hash: hash(0x01), + tx_hash: hash(0x02), delayed: false, }), refund_identity: None, From 86afa8d6a0a5020989a35ed0b33f2e42c7b4faf4 Mon Sep 17 00:00:00 2001 From: Vitaly Drogan Date: Mon, 27 Oct 2025 11:32:59 +0100 Subject: [PATCH 2/2] redistribution: downgrade legacy warnings --- crates/rbuilder/src/backtest/redistribute/mod.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/rbuilder/src/backtest/redistribute/mod.rs b/crates/rbuilder/src/backtest/redistribute/mod.rs index d2cbd73b1..9400a1e9a 100644 --- a/crates/rbuilder/src/backtest/redistribute/mod.rs +++ b/crates/rbuilder/src/backtest/redistribute/mod.rs @@ -332,7 +332,7 @@ where warn!("Block has no bundles"); } if share_bundles == 0 { - warn!("Block has no share bundles"); + debug!("Block has no share bundles"); } let block_profit = if built_block_data.profit.is_positive() { @@ -591,7 +591,7 @@ fn split_orders_by_identities( } if !protect_signer_seen { - warn!("No orders from protect signer"); + debug!("No orders from protect signer"); } let mut included_orders_by_address: Vec<(Address, Vec)> =