chore(core,clippy): remove redundant .clone() in rollout items

Replace unnecessary .clone() calls when pushing ResponseItem instances in sample_rollout test. Moves user and assistant values into RolloutItem::ResponseItem (user1, assistant1, user2, assistant2) instead of cloning them, eliminating needless clones

Author: ignatremizov

codex-rs/core/src/codex_tests.rs

@@ -4465,7 +4465,7 @@ async fn sample_rollout(
         std::iter::once(&user1),
         reconstruction_turn.truncation_policy,
     );
-    rollout_items.push(RolloutItem::ResponseItem(user1.clone()));
+    rollout_items.push(RolloutItem::ResponseItem(user1));
 
     let assistant1 = ResponseItem::Message {
         id: None,
@@ -4480,7 +4480,7 @@ async fn sample_rollout(
         std::iter::once(&assistant1),
         reconstruction_turn.truncation_policy,
     );
-    rollout_items.push(RolloutItem::ResponseItem(assistant1.clone()));
+    rollout_items.push(RolloutItem::ResponseItem(assistant1));
 
     let summary1 = "summary one";
     let snapshot1 = live_history
@@ -4507,7 +4507,7 @@ async fn sample_rollout(
         std::iter::once(&user2),
         reconstruction_turn.truncation_policy,
     );
-    rollout_items.push(RolloutItem::ResponseItem(user2.clone()));
+    rollout_items.push(RolloutItem::ResponseItem(user2));
 
     let assistant2 = ResponseItem::Message {
         id: None,
@@ -4522,7 +4522,7 @@ async fn sample_rollout(
         std::iter::once(&assistant2),
         reconstruction_turn.truncation_policy,
     );
-    rollout_items.push(RolloutItem::ResponseItem(assistant2.clone()));
+    rollout_items.push(RolloutItem::ResponseItem(assistant2));
 
     let summary2 = "summary two";
     let snapshot2 = live_history