fix: resolve clippy warnings for Rust 1.93.0

- Remove unused imports across test modules and examples
- Fix duplicated #[test] attributes in config.rs
- Replace filter_map with map where all branches return Some
- Replace vec![] with array literals for non-growing collections
- Use rfind instead of filter().last() on DoubleEndedIterator
- Allow dead_code on ToolEntry.description (stored for future use)
- Allow clippy::large_enum_variant on SandboxState
- Allow clippy::assertions_on_constants in compaction tests
- Use or_default() instead of or_insert_with(VecDeque::new)
- Remove needless borrows for generic args

Author: ZhiXiao-Lin

core/examples/default_implementations.rs

@@ -49,7 +49,7 @@ async fn main() -> anyhow::Result<()> {
     println!("  - Max file size: 1MB\n");
 
     // 4. Set up HITL confirmation
-    let (event_tx, mut event_rx) = broadcast::channel(256);
+    let (event_tx, _event_rx) = broadcast::channel(256);
     let hitl_policy = ConfirmationPolicy::enabled()
         .with_timeout(30_000, a3s_code_core::hitl::TimeoutAction::Reject);
     let confirmation_manager = Arc::new(ConfirmationManager::new(hitl_policy, event_tx.clone()));
@@ -60,7 +60,7 @@ async fn main() -> anyhow::Result<()> {
     println!("  - Timeout action: Reject\n");
 
     // 5. Create session with all default implementations
-    let session = agent.session(
+    let _session = agent.session(
         ".",
         Some(
             SessionOptions::new()

core/examples/sdk_chat.rs

@@ -108,22 +108,22 @@ async fn main() -> anyhow::Result<()> {
         let preview = msg
             .content
             .iter()
-            .filter_map(|b| match b {
+            .map(|b| match b {
                 ContentBlock::Text { text } => {
                     let trimmed = text.trim();
                     if trimmed.len() > 60 {
-                        Some(format!("{}...", &trimmed[..60]))
+                        format!("{}...", &trimmed[..60])
                     } else {
-                        Some(trimmed.to_string())
+                        trimmed.to_string()
                     }
                 }
-                ContentBlock::ToolUse { name, .. } => Some(format!("[tool_use: {}]", name)),
+                ContentBlock::ToolUse { name, .. } => format!("[tool_use: {}]", name),
                 ContentBlock::ToolResult { content, .. } => {
                     let text = content.as_text();
                     let truncated = &text[..text.len().min(40)];
-                    Some(format!("[tool_result: {}]", truncated))
+                    format!("[tool_result: {}]", truncated)
                 }
-                ContentBlock::Image { .. } => Some("[image]".to_string()),
+                ContentBlock::Image { .. } => "[image]".to_string(),
             })
             .collect::<Vec<_>>()
             .join(" | ");

core/examples/test_auto_compact.rs

@@ -5,7 +5,7 @@
 //!
 //! Run with: cargo run --example test_auto_compact
 
-use a3s_code_core::{Agent, AgentEvent, SessionOptions};
+use a3s_code_core::{Agent, SessionOptions};
 use anyhow::Result;
 use std::path::PathBuf;
 

core/examples/test_parallel_processing.rs

@@ -49,7 +49,7 @@ async fn main() -> Result<()> {
         let agent = Agent::new(config_path.to_str().unwrap()).await?;
         let session = agent.session(".", None)?;
 
-        let tasks = vec![
+        let tasks = [
             "Count the number of Rust files in this project",
             "Find all TODO comments in Rust files",
             "List all public functions in src/lib.rs",
@@ -97,7 +97,7 @@ async fn main() -> Result<()> {
 
         let session = agent.session(".", Some(opts))?;
 
-        let tasks = vec![
+        let tasks = [
             "Count the number of Rust files in this project",
             "Find all TODO comments in Rust files",
             "List all public functions in src/lib.rs",
@@ -178,7 +178,7 @@ async fn main() -> Result<()> {
         let start = Instant::now();
 
         // Mix of different task types
-        let tasks = vec![
+        let tasks = [
             ("Query", "How many Rust files are there?"),
             ("Query", "What is the project structure?"),
             ("Query", "List all dependencies"),
@@ -245,7 +245,7 @@ async fn main() -> Result<()> {
         println!("  Initial delay: 100ms");
         println!("  Strategy: exponential");
 
-        let tasks = vec![
+        let tasks = [
             "Analyze the main function",
             "Find all error types",
             "List all test functions",

core/examples/test_task_priority.rs

@@ -357,8 +357,7 @@ async fn test_late_urgent_insertion(agent: &Agent) -> Result<()> {
         .map(|r| r.completed_at);
     let last_exec_completed = sorted
         .iter()
-        .filter(|r| r.lane.contains("P2"))
-        .last()
+        .rfind(|r| r.lane.contains("P2"))
         .map(|r| r.completed_at);
 
     if let (Some(urgent), Some(last_exec)) = (urgent_completed, last_exec_completed) {

core/examples/test_vector_rag.rs

@@ -5,13 +5,9 @@
 //!
 //! Run with: cargo run --example test_vector_rag
 
-use a3s_code_core::context::embedding::OpenAiEmbeddingProvider;
-use a3s_code_core::context::vector_store::InMemoryVectorStore;
-use a3s_code_core::context::{VectorContextConfig, VectorContextProvider};
 use a3s_code_core::{Agent, SessionOptions};
 use anyhow::Result;
 use std::path::PathBuf;
-use std::sync::Arc;
 use tempfile::TempDir;
 
 fn find_config() -> PathBuf {

core/src/agent.rs

@@ -4340,7 +4340,6 @@ mod tests {
 mod extra_agent_tests {
     use super::*;
     use crate::agent::tests::MockLlmClient;
-    use crate::llm::{ContentBlock, StreamEvent};
     use crate::queue::SessionQueueConfig;
     use crate::tools::ToolExecutor;
     use std::path::PathBuf;

core/src/config.rs

@@ -792,7 +792,6 @@ mod tests {
         assert_eq!(models.len(), 3);
     }
 
-    #[test]
     #[test]
     fn test_config_from_file_not_found() {
         let result = CodeConfig::from_file(Path::new("/nonexistent/config.json"));
@@ -1096,7 +1095,6 @@ mod tests {
         );
     }
 
-    #[test]
     #[test]
     fn test_code_config_default_provider_config() {
         let config = CodeConfig {

core/src/llm/tests.rs

@@ -4,8 +4,6 @@ mod tests {
     use crate::llm::http::normalize_base_url;
     use crate::llm::openai::*;
     use crate::llm::*;
-    use crate::retry::RetryConfig;
-    use std::sync::Arc;
 
     #[test]
     fn test_secret_string_redacts_debug() {
@@ -501,8 +499,6 @@ mod extra_llm_tests {
     use crate::llm::http::normalize_base_url;
     use crate::llm::openai::*;
     use crate::llm::*;
-    use crate::retry::RetryConfig;
-    use std::sync::Arc;
 
     #[test]
     fn test_message_assistant_text() {
@@ -2026,7 +2022,6 @@ mod extra_llm_tests3 {
     use crate::llm::openai::*;
     use crate::llm::*;
     use crate::retry::RetryConfig;
-    use std::sync::Arc;
 
     // ========================================================================
     // OpenAiClient convert_messages - Additional Coverage
@@ -2776,7 +2771,7 @@ mod multimodal_tests {
     fn test_attachment_from_file_jpeg() {
         let dir = tempfile::tempdir().unwrap();
         let path = dir.path().join("test.jpg");
-        std::fs::write(&path, &[0xFF, 0xD8, 0xFF]).unwrap();
+        std::fs::write(&path, [0xFF, 0xD8, 0xFF]).unwrap();
         let a = Attachment::from_file(&path).unwrap();
         assert_eq!(a.media_type, "image/jpeg");
         assert_eq!(a.data, vec![0xFF, 0xD8, 0xFF]);
@@ -2786,7 +2781,7 @@ mod multimodal_tests {
     fn test_attachment_from_file_png() {
         let dir = tempfile::tempdir().unwrap();
         let path = dir.path().join("test.png");
-        std::fs::write(&path, &[0x89, 0x50]).unwrap();
+        std::fs::write(&path, [0x89, 0x50]).unwrap();
         let a = Attachment::from_file(&path).unwrap();
         assert_eq!(a.media_type, "image/png");
     }
@@ -2795,7 +2790,7 @@ mod multimodal_tests {
     fn test_attachment_from_file_unknown_ext() {
         let dir = tempfile::tempdir().unwrap();
         let path = dir.path().join("test.bin");
-        std::fs::write(&path, &[0x00]).unwrap();
+        std::fs::write(&path, [0x00]).unwrap();
         let a = Attachment::from_file(&path).unwrap();
         assert_eq!(a.media_type, "application/octet-stream");
     }

core/src/sandbox.rs

@@ -103,6 +103,7 @@ mod box_impl {
     use std::sync::Arc;
     use tokio::sync::Mutex;
 
+    #[allow(clippy::large_enum_variant)]
     enum SandboxState {
         NotStarted,
         Running(Sandbox),