refactor: Rename optimize() -> minimize_tx_size()

Author: snawaz

magicblock-committor-service/src/tasks/args_task.rs

@@ -84,7 +84,7 @@ impl BaseTask for ArgsTask {
         }
     }
 
-    fn optimize(
+    fn minimize_tx_size(
         self: Box<Self>,
     ) -> Result<Box<dyn BaseTask>, Box<dyn BaseTask>> {
         match self.task_type {

magicblock-committor-service/src/tasks/buffer_task.rs

@@ -69,7 +69,7 @@ impl BaseTask for BufferTask {
     }
 
     /// No further optimizations
-    fn optimize(
+    fn minimize_tx_size(
         self: Box<Self>,
     ) -> Result<Box<dyn BaseTask>, Box<dyn BaseTask>> {
         // Since the buffer in BufferTask doesn't contribute to the size of

magicblock-committor-service/src/tasks/mod.rs

@@ -69,8 +69,9 @@ pub trait BaseTask: Send + Sync + DynClone {
     /// Gets instruction for task execution
     fn instruction(&self, validator: &Pubkey) -> Instruction;
 
-    /// Optimizes Task strategy if possible, otherwise returns itself
-    fn optimize(
+    /// Optimize for transaction size so that more instructions can be buddled together in a single
+    /// transaction. Return Ok(new_tx_optimized_task), else Err(self) if task cannot be optimized.
+    fn minimize_tx_size(
         self: Box<Self>,
     ) -> Result<Box<dyn BaseTask>, Box<dyn BaseTask>>;
 

magicblock-committor-service/src/tasks/task_strategist.rs

@@ -53,7 +53,8 @@ impl TaskStrategist {
         persistor: &Option<P>,
     ) -> TaskStrategistResult<TransactionStrategy> {
         // Attempt optimizing tasks themselves(using buffers)
-        if Self::optimize_strategy(&mut tasks)? <= MAX_ENCODED_TRANSACTION_SIZE
+        if Self::minimize_tx_size_if_needed(&mut tasks)?
+            <= MAX_ENCODED_TRANSACTION_SIZE
         {
             // Persist tasks strategy
             if let Some(persistor) = persistor {
@@ -152,7 +153,7 @@ impl TaskStrategist {
 
     /// Optimizes set of [`TaskDeliveryStrategy`] to fit [`MAX_ENCODED_TRANSACTION_SIZE`]
     /// Returns size of tx after optimizations
-    fn optimize_strategy(
+    fn minimize_tx_size_if_needed(
         tasks: &mut [Box<dyn BaseTask>],
     ) -> Result<usize, SignerError> {
         // Get initial transaction size
@@ -207,7 +208,7 @@ impl TaskStrategist {
                 let tmp_task = Box::new(tmp_task) as Box<dyn BaseTask>;
                 std::mem::replace(&mut tasks[index], tmp_task)
             };
-            match task.optimize() {
+            match task.minimize_tx_size() {
                 // If we can decrease:
                 // 1. Calculate new tx size & ix size
                 // 2. Insert item's data back in the heap
@@ -456,7 +457,7 @@ mod tests {
             Box::new(create_test_commit_task(3, 1000)) as Box<dyn BaseTask>, // Larger task
         ];
 
-        let _ = TaskStrategist::optimize_strategy(&mut tasks);
+        let _ = TaskStrategist::minimize_tx_size_if_needed(&mut tasks);
         // The larger task should have been optimized first
         assert!(matches!(tasks[0].strategy(), TaskStrategy::Args));
         assert!(matches!(tasks[1].strategy(), TaskStrategy::Buffer));