fix: update types & drop bundle support

bolt-boost/src/server.rs

@@ -146,7 +146,7 @@ async fn delegate(
     State(state): State<PbsState<BuilderState>>,
     Json(delegation): Json<SignedDelegation>,
 ) -> Result<impl IntoResponse, PbsClientError> {
-    info!(pubkey = %delegation.message.pubkey, validator_index = delegation.message.validator_index, "Delegating signing rights");
+    info!(delegatee = %delegation.message.delegatee_pubkey, validator = %delegation.message.validator_pubkey, "Delegating signing rights");
     post_request(state, DELEGATE_PATH, &delegation).await?;
     Ok(StatusCode::OK)
 }
@@ -158,7 +158,7 @@ async fn revoke(
     State(state): State<PbsState<BuilderState>>,
     Json(revocation): Json<SignedRevocation>,
 ) -> Result<impl IntoResponse, PbsClientError> {
-    info!(pubkey = %revocation.message.pubkey, validator_index = revocation.message.validator_index, "Revoking signing rights");
+    info!(delegatee = %revocation.message.delegatee_pubkey, validator = %revocation.message.validator_pubkey, "Revoking signing rights");
     post_request(state, REVOKE_PATH, &revocation).await?;
     Ok(StatusCode::OK)
 }

bolt-boost/src/types.rs

@@ -114,8 +114,8 @@ pub struct SignedDelegation {
 
 #[derive(Debug, Clone, Serialize, Deserialize, Encode, Decode)]
 pub struct DelegationMessage {
-    pub validator_index: u64,
-    pub pubkey: BlsPublicKey,
+    pub validator_pubkey: BlsPublicKey,
+    pub delegatee_pubkey: BlsPublicKey,
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize, Encode, Decode)]
@@ -126,8 +126,8 @@ pub struct SignedRevocation {
 
 #[derive(Debug, Clone, Serialize, Deserialize, Encode, Decode)]
 pub struct RevocationMessage {
-    pub validator_index: u64,
-    pub pubkey: BlsPublicKey,
+    pub validator_pubkey: BlsPublicKey,
+    pub delegatee_pubkey: BlsPublicKey,
 }
 
 pub type GetHeaderWithProofsResponse = VersionedResponse<SignedExecutionPayloadHeaderWithProofs>;

bolt-sidecar/src/driver.rs

@@ -220,21 +220,26 @@ impl<C: StateFetcher, BLS: SignerBLS, ECDSA: SignerECDSA> SidecarDriver<C, BLS,
 
         // parse the request into constraints and sign them
         let slot = inclusion_request.slot;
-        let message = ConstraintsMessage::build(validator_pubkey, inclusion_request);
-        let signed_constraints = match self.constraint_signer.sign(&message.digest()).await {
-            Ok(signature) => SignedConstraints { message, signature },
-            Err(err) => {
-                error!(?err, "Failed to sign constraints");
-                let _ = response.send(Err(CommitmentError::Internal));
-                return;
-            }
-        };
 
-        // Track the number of transactions preconfirmed considering their type
-        signed_constraints.message.transactions.iter().for_each(|full_tx| {
-            ApiMetrics::increment_transactions_preconfirmed(full_tx.tx_type());
-        });
-        self.execution.add_constraint(slot, signed_constraints);
+        // NOTE: we iterate over the transactions in the request and generate a signed constraint for each one. This is because
+        // the transactions in the commitment request are not supposed to be treated as a relative-ordering bundle, but a batch
+        // with no ordering guarantees.
+        for tx in inclusion_request.txs {
+            let tx_type = tx.tx_type();
+            let message = ConstraintsMessage::from_transaction(validator_pubkey.clone(), slot, tx);
+
+            let signed_constraints = match self.constraint_signer.sign(&message.digest()).await {
+                Ok(signature) => SignedConstraints { message, signature },
+                Err(err) => {
+                    error!(?err, "Failed to sign constraints");
+                    let _ = response.send(Err(CommitmentError::Internal));
+                    return;
+                }
+            };
+
+            ApiMetrics::increment_transactions_preconfirmed(tx_type);
+            self.execution.add_constraint(slot, signed_constraints);
+        }
 
         // Create a commitment by signing the request
         match request.commit_and_sign(&self.commitment_signer).await {

bolt-sidecar/src/primitives/constraint.rs

@@ -70,6 +70,11 @@ impl ConstraintsMessage {
 
         Self { pubkey, slot: request.slot, top: false, transactions }
     }
+
+    /// Builds a constraints message from a single transaction.
+    pub fn from_transaction(pubkey: BlsPublicKey, slot: u64, transaction: FullTransaction) -> Self {
+        Self { pubkey, slot, top: false, transactions: vec![transaction] }
+    }
 }
 
 impl SignableBLS for ConstraintsMessage {

bolt-sidecar/src/state/execution.rs

@@ -531,7 +531,7 @@ mod tests {
     use std::{num::NonZero, str::FromStr, time::Duration};
 
     use alloy::{
-        consensus::constants::{ETH_TO_WEI, LEGACY_TX_TYPE_ID},
+        consensus::constants::ETH_TO_WEI,
         eips::eip2718::Encodable2718,
         network::EthereumWallet,
         primitives::{uint, Uint},
@@ -540,7 +540,6 @@ mod tests {
     };
     use fetcher::{StateClient, StateFetcher};
     use reth_primitives::constants::GWEI_TO_WEI;
-    use tracing::info;
 
     use crate::{
         crypto::{bls::Signer, SignableBLS, SignerBLS},