diff --git a/crates/autopilot/src/boundary/order.rs b/crates/autopilot/src/boundary/order.rs index 87c9c58b5c..708ec79396 100644 --- a/crates/autopilot/src/boundary/order.rs +++ b/crates/autopilot/src/boundary/order.rs @@ -40,7 +40,6 @@ pub fn to_domain( .collect(), sell_token_balance: order.data.sell_token_balance.into(), buy_token_balance: order.data.buy_token_balance.into(), - class: order.metadata.class.into(), app_data: order.data.app_data.into(), signature: order.signature.into(), quote, diff --git a/crates/autopilot/src/domain/auction/order.rs b/crates/autopilot/src/domain/auction/order.rs index 4b1a3a8198..e5acf8d17a 100644 --- a/crates/autopilot/src/domain/auction/order.rs +++ b/crates/autopilot/src/domain/auction/order.rs @@ -14,7 +14,6 @@ pub struct Order { pub buy: eth::Asset, pub protocol_fees: Vec, pub side: Side, - pub class: Class, pub created: u32, pub valid_to: u32, pub receiver: Option, @@ -32,18 +31,6 @@ pub struct Order { pub quote: Option, } -impl Order { - pub fn is_limit_order(&self) -> bool { - matches!(self.class, Class::Limit) - } - - /// For some orders the protocol doesn't precompute a fee. Instead solvers - /// are supposed to compute a reasonable fee themselves. - pub fn solver_determines_fee(&self) -> bool { - self.is_limit_order() - } -} - // uid as 56 bytes: 32 for orderDigest, 20 for ownerAddress and 4 for validTo #[derive(Copy, Clone, PartialEq, Hash, Eq)] pub struct OrderUid(pub [u8; 56]); @@ -87,13 +74,6 @@ pub enum Side { Sell, } -#[derive(Clone, Debug, PartialEq)] -pub enum Class { - Market, - Limit, - Liquidity, -} - #[derive(Clone, Debug, PartialEq)] pub struct Interaction { pub target: H160, diff --git a/crates/autopilot/src/infra/persistence/dto/order.rs b/crates/autopilot/src/infra/persistence/dto/order.rs index 67639a2a09..1ffdab1843 100644 --- a/crates/autopilot/src/infra/persistence/dto/order.rs +++ b/crates/autopilot/src/infra/persistence/dto/order.rs @@ -65,7 +65,7 @@ pub fn from_domain(order: domain::Order) -> Order { .collect(), sell_token_balance: order.sell_token_balance.into(), buy_token_balance: order.buy_token_balance.into(), - class: order.class.into(), + class: boundary::OrderClass::Limit, app_data: order.app_data.into(), signature: order.signature.into(), quote: order.quote.map(Into::into), @@ -99,7 +99,6 @@ pub fn to_domain(order: Order) -> domain::Order { .collect(), sell_token_balance: order.sell_token_balance.into(), buy_token_balance: order.buy_token_balance.into(), - class: order.class.into(), app_data: order.app_data.into(), signature: order.signature.into(), quote: order.quote.map(|q| q.to_domain(order.uid.into())), @@ -136,26 +135,6 @@ impl From for domain::auction::order::Side { } } -impl From for boundary::OrderClass { - fn from(class: domain::auction::order::Class) -> Self { - match class { - domain::auction::order::Class::Limit => Self::Limit, - domain::auction::order::Class::Market => Self::Market, - domain::auction::order::Class::Liquidity => Self::Liquidity, - } - } -} - -impl From for domain::auction::order::Class { - fn from(class: boundary::OrderClass) -> Self { - match class { - boundary::OrderClass::Limit => Self::Limit, - boundary::OrderClass::Market => Self::Market, - boundary::OrderClass::Liquidity => Self::Liquidity, - } - } -} - impl From for boundary::InteractionData { fn from(interaction: domain::auction::order::Interaction) -> Self { Self { diff --git a/crates/autopilot/src/run_loop.rs b/crates/autopilot/src/run_loop.rs index 8d7a85bda8..8631c3e754 100644 --- a/crates/autopilot/src/run_loop.rs +++ b/crates/autopilot/src/run_loop.rs @@ -4,7 +4,6 @@ use { database::competition::Competition, domain::{ self, - auction::order::Class, competition::{self, SolutionError, TradedAmounts}, OrderUid, }, @@ -169,11 +168,7 @@ impl RunLoop { } }; - if auction.orders.iter().all(|order| match order.class { - Class::Market => false, - Class::Liquidity => true, - Class::Limit => false, - }) { + if auction.orders.is_empty() { // Updating liveness probe to not report unhealthy due to this optimization self.liveness.auction(); tracing::debug!("skipping empty auction"); diff --git a/crates/autopilot/src/shadow.rs b/crates/autopilot/src/shadow.rs index 13e62c9b31..6e1d219631 100644 --- a/crates/autopilot/src/shadow.rs +++ b/crates/autopilot/src/shadow.rs @@ -10,7 +10,7 @@ use { crate::{ arguments::RunLoopMode, - domain::{self, auction::order::Class}, + domain, infra::{ self, solvers::dto::{reveal, solve}, @@ -102,16 +102,7 @@ impl RunLoop { return None; } - if auction - .auction - .orders - .iter() - .all(|order| match order.class { - Class::Market => false, - Class::Liquidity => true, - Class::Limit => false, - }) - { + if auction.auction.orders.is_empty() { tracing::trace!("skipping empty auction"); return None; }