chore: remove dead code (#770)

* chore: remove dead code

* bump version

Author: mikhailUshakoff

Cargo.lock

@@ -12,7 +12,7 @@ resolver = "2"
 default-members = ["node"]
 
 [workspace.package]
-version = "1.23.31"
+version = "1.23.32"
 edition = "2024"
 repository = "https://github.com/NethermindEth/Catalyst"
 license = "MIT"

Cargo.toml

@@ -4,8 +4,6 @@ use taiko_bindings::inbox::IInbox::Config;
 #[derive(Clone, Default)]
 pub struct ProtocolConfig {
     basefee_sharing_pctg: u8,
-    // TODO initialize these values correctly
-    _min_anchor_offset: u64,
     max_anchor_offset: u64,
     codec_address: Address,
 }
@@ -14,7 +12,6 @@ impl ProtocolConfig {
     pub fn from(shasta_config: &Config) -> Self {
         Self {
             basefee_sharing_pctg: shasta_config.basefeeSharingPctg,
-            _min_anchor_offset: 2, // https://github.com/taikoxyz/taiko-mono/blob/main/packages/protocol/docs/Derivation.md#constants
             max_anchor_offset: 100, // 128 by document
             codec_address: shasta_config.codec,
         }
@@ -24,11 +21,6 @@ impl ProtocolConfig {
         self.basefee_sharing_pctg
     }
 
-    #[allow(dead_code)]
-    pub fn get_min_anchor_height_offset(&self) -> u64 {
-        self._min_anchor_offset
-    }
-
     pub fn get_max_anchor_height_offset(&self) -> u64 {
         self.max_anchor_offset
     }

shasta/src/l1/protocol_config.rs

@@ -278,8 +278,7 @@ impl Taiko {
             block_number: l2_slot_context.info.parent_id() + 1,
             extra_data: format!("0x{:04x}", extra_data),
             fee_recipient: l2_block_payload.coinbase.to_string(),
-            gas_limit: l2_slot_context.info.parent_gas_limit_without_anchor()
-                + ANCHOR_V3_V4_GAS_LIMIT,
+            gas_limit: l2_block_payload.gas_limit_without_anchor + ANCHOR_V3_V4_GAS_LIMIT,
             parent_hash: format!("0x{}", hex::encode(l2_slot_context.info.parent_hash())),
             timestamp: l2_block_payload.timestamp_sec,
             transactions: format!("0x{}", hex::encode(tx_list_bytes)),

shasta/src/l2/taiko.rs

@@ -1,7 +1,5 @@
 mod chain_monitor;
-#[allow(dead_code)] // TODO: remove this once we have a used create_shasta_node function
 mod node;
-#[allow(dead_code)] // TODO: remove this once we have a used create_shasta_node function
 mod utils;
 
 mod forced_inclusion;

shasta/src/lib.rs

@@ -106,10 +106,6 @@ impl BatchBuilder {
         });
     }
 
-    pub fn remove_current_proposal(&mut self) {
-        self.current_proposal = None;
-    }
-
     pub fn add_l2_draft_block(
         &mut self,
         l2_draft_block: L2BlockV2Draft,
@@ -417,10 +413,6 @@ impl BatchBuilder {
             }
     }
 
-    pub fn get_number_of_batches_ready_to_send(&self) -> u64 {
-        self.proposals_to_send.len() as u64
-    }
-
     /// Alias for `take_proposals_to_send` for compatibility
     pub fn take_batches_to_send(&mut self) -> VecDeque<Proposal> {
         std::mem::take(&mut self.proposals_to_send)
@@ -495,8 +487,4 @@ impl BatchBuilder {
             .map(|proposal| proposal.num_forced_inclusion += 1)
             .ok_or_else(|| anyhow::anyhow!("No current proposal to add forced inclusion to"))
     }
-
-    pub fn get_current_proposal(&self) -> Option<&Proposal> {
-        self.current_proposal.as_ref()
-    }
 }

shasta/src/node/proposal_manager/batch_builder.rs

@@ -67,41 +67,6 @@ impl Proposal {
         self.total_bytes = manifest_data.len() as u64;
     }
 
-    pub fn get_last_block_timestamp(&self) -> Result<u64, anyhow::Error> {
-        self.l2_blocks
-            .last()
-            .map(|block| block.timestamp_sec)
-            .ok_or_else(|| anyhow::anyhow!("No L2 blocks in proposal"))
-    }
-
-    pub fn has_only_one_common_block(&self) -> bool {
-        self.num_forced_inclusion == 0 && self.l2_blocks.len() == 1
-    }
-
-    pub fn is_empty(&self) -> bool {
-        self.num_forced_inclusion == 0 && self.l2_blocks.is_empty()
-    }
-
-    pub fn get_last_block_tx_list_copy(
-        &self,
-    ) -> Result<Vec<alloy::rpc::types::Transaction>, anyhow::Error> {
-        self.l2_blocks
-            .last()
-            .map(|block| block.prebuilt_tx_list.tx_list.clone())
-            .ok_or_else(|| anyhow::anyhow!("No L2 blocks in proposal"))
-    }
-
-    pub fn get_last_block_tx_len(&self) -> Result<usize, anyhow::Error> {
-        self.l2_blocks
-            .last()
-            .map(|block| block.prebuilt_tx_list.tx_list.len())
-            .ok_or_else(|| anyhow::anyhow!("No L2 blocks in proposal"))
-    }
-
-    pub fn total_bytes(&self) -> u64 {
-        self.total_bytes
-    }
-
     fn create_block_from_draft(&mut self, l2_draft_block: L2BlockV2Draft) -> L2BlockV2 {
         L2BlockV2::new_from(
             l2_draft_block.prebuilt_tx_list,

shasta/src/node/proposal_manager/proposal.rs

@@ -1,11 +1,6 @@
 use common::config::ConfigTrait;
 use tracing::warn;
 
-#[derive(Debug, Clone)]
-pub struct L1ContractAddresses {
-    pub shasta_inbox: String,
-}
-
 #[derive(Debug, Clone)]
 pub struct ShastaConfig {
     pub shasta_inbox: String,