fix: resolve all clippy warnings, formatting, and Mistral endpoint

- Fix Mistral provider base URL (missing /v1 prefix caused 404s)
- Resolve 55 clippy warnings across 28 warning types
- Apply cargo fmt to 44 formatting violations
- Remove unused imports (process_message, MultiObserver, VerboseObserver,
  ChatResponse, ToolCall, Path, TempDir)
- Replace format!+push_str with write! macro
- Fix unchecked Duration subtraction, redundant closures, clamp patterns
- Declare missing feature flags (sandbox-landlock, sandbox-bubblewrap,
  browser-native) in Cargo.toml
- Derive Default where manual impls were redundant
- Add separators to long numeric literals (115200 → 115_200)
- Restructure unreachable code in arduino_flash platform branches

All 1,500 tests pass. Zero clippy warnings. Clean formatting.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Cargo.toml

@@ -139,6 +139,11 @@ landlock = ["sandbox-landlock"]
 probe = ["dep:probe-rs"]
 # rag-pdf = PDF ingestion for datasheet RAG
 rag-pdf = ["dep:pdf-extract"]
+# sandbox backends (optional, platform-specific)
+sandbox-landlock = []
+sandbox-bubblewrap = []
+# native browser backend (optional, adds WebDriver dependency)
+browser-native = []
 
 [profile.release]
 opt-level = "z"      # Optimize for size

src/agent/mod.rs

@@ -7,7 +7,7 @@ pub mod prompt;
 
 #[allow(unused_imports)]
 pub use agent::{Agent, AgentBuilder};
-pub use loop_::{process_message, run};
+pub use loop_::run;
 
 #[cfg(test)]
 mod tests {
@@ -18,7 +18,6 @@ mod tests {
     #[test]
     fn run_function_is_reexported() {
         assert_reexport_exists(run);
-        assert_reexport_exists(process_message);
         assert_reexport_exists(loop_::run);
         assert_reexport_exists(loop_::process_message);
     }

src/gateway/mod.rs

@@ -810,7 +810,9 @@ mod tests {
                 .requests
                 .lock()
                 .unwrap_or_else(std::sync::PoisonError::into_inner);
-            guard.1 = Instant::now() - Duration::from_secs(RATE_LIMITER_SWEEP_INTERVAL_SECS + 1);
+            guard.1 = Instant::now()
+                .checked_sub(Duration::from_secs(RATE_LIMITER_SWEEP_INTERVAL_SECS + 1))
+                .unwrap();
             // Clear timestamps for ip-2 and ip-3 to simulate stale entries
             guard.0.get_mut("ip-2").unwrap().clear();
             guard.0.get_mut("ip-3").unwrap().clear();

src/memory/backend.rs

@@ -7,6 +7,7 @@ pub enum MemoryBackendKind {
     Unknown,
 }
 
+#[allow(clippy::struct_excessive_bools)]
 #[derive(Debug, Clone, Copy, Eq, PartialEq)]
 pub struct MemoryBackendProfile {
     pub key: &'static str,

src/memory/lucid.rs

@@ -74,6 +74,7 @@ impl LucidMemory {
     }
 
     #[cfg(test)]
+    #[allow(clippy::too_many_arguments)]
     fn with_options(
         workspace_dir: &Path,
         local: SqliteMemory,

src/memory/response_cache.rs

@@ -166,7 +166,7 @@ impl ResponseCache {
             |row| row.get(0),
         )?;
 
-        #[allow(clippy::cast_sign_loss)]
+        #[allow(clippy::cast_sign_loss, clippy::cast_possible_truncation)]
         Ok((count as usize, hits as u64, tokens_saved as u64))
     }
 

src/observability/mod.rs

@@ -6,11 +6,9 @@ pub mod traits;
 pub mod verbose;
 
 pub use self::log::LogObserver;
-pub use self::multi::MultiObserver;
 pub use noop::NoopObserver;
 pub use otel::OtelObserver;
 pub use traits::{Observer, ObserverEvent};
-pub use verbose::VerboseObserver;
 
 use crate::config::ObservabilityConfig;
 

src/onboard/wizard.rs

@@ -2271,14 +2271,11 @@ fn setup_memory() -> Result<MemoryConfig> {
     let backend = backend_key_from_choice(choice);
     let profile = memory_backend_profile(backend);
 
-    let auto_save = if !profile.auto_save_default {
+    let auto_save = profile.auto_save_default
-        false
+        && Confirm::new()
-    } else {
-        Confirm::new()
             .with_prompt("  Auto-save conversations to memory?")
             .default(true)
-            .interact()?
+            .interact()?;
-    };
 
     println!(
         "  {} Memory: {} (auto-save: {})",

src/peripherals/arduino_flash.rs

@@ -38,6 +38,10 @@ pub fn ensure_arduino_cli() -> Result<()> {
             anyhow::bail!("brew install arduino-cli failed. Install manually: https://arduino.github.io/arduino-cli/");
         }
         println!("arduino-cli installed.");
+        if !arduino_cli_available() {
+            anyhow::bail!("arduino-cli still not found after install. Ensure it's in PATH.");
+        }
+        return Ok(());
     }
 
     #[cfg(target_os = "linux")]
@@ -54,11 +58,6 @@ pub fn ensure_arduino_cli() -> Result<()> {
         println!("arduino-cli not found. Install it: https://arduino.github.io/arduino-cli/");
         anyhow::bail!("arduino-cli not installed.");
     }
-
-    if !arduino_cli_available() {
-        anyhow::bail!("arduino-cli still not found after install. Ensure it's in PATH.");
-    }
-    Ok(())
 }
 
 /// Ensure arduino:avr core is installed.

src/peripherals/serial.rs

@@ -112,6 +112,7 @@ pub struct SerialPeripheral {
 
 impl SerialPeripheral {
     /// Create and connect to a serial peripheral.
+    #[allow(clippy::unused_async)]
     pub async fn connect(config: &PeripheralBoardConfig) -> anyhow::Result<Self> {
         let path = config
             .path

src/providers/mod.rs

@@ -269,7 +269,7 @@ pub fn create_provider_with_url(
             "Groq", "https://api.groq.com/openai", key, AuthStyle::Bearer,
         ))),
         "mistral" => Ok(Box::new(OpenAiCompatibleProvider::new(
-            "Mistral", "https://api.mistral.ai", key, AuthStyle::Bearer,
+            "Mistral", "https://api.mistral.ai/v1", key, AuthStyle::Bearer,
         ))),
         "xai" | "grok" => Ok(Box::new(OpenAiCompatibleProvider::new(
             "xAI", "https://api.x.ai", key, AuthStyle::Bearer,

src/security/pairing.rs

@@ -184,7 +184,7 @@ fn generate_token() -> String {
     use rand::RngCore;
     let mut bytes = [0u8; 32];
     rand::thread_rng().fill_bytes(&mut bytes);
-    format!("zc_{}", hex::encode(&bytes))
+    format!("zc_{}", hex::encode(bytes))
 }
 
 /// SHA-256 hash a bearer token for storage. Returns lowercase hex.

src/tools/hardware_board_info.rs

@@ -124,10 +124,11 @@ impl Tool for HardwareBoardInfoTool {
                     });
                 }
                 Err(e) => {
-                    output.push_str(&format!(
+                    use std::fmt::Write;
-                        "probe-rs attach failed: {}. Using static info.\n\n",
+                    let _ = write!(
-                        e
+                        output,
-                    ));
+                        "probe-rs attach failed: {e}. Using static info.\n\n"
+                    );
                 }
             }
         }
@@ -135,13 +136,15 @@ impl Tool for HardwareBoardInfoTool {
         if let Some(info) = self.static_info_for_board(board) {
             output.push_str(&info);
             if let Some(mem) = memory_map_static(board) {
-                output.push_str(&format!("\n\n**Memory map:**\n{}", mem));
+                use std::fmt::Write;
+                let _ = write!(output, "\n\n**Memory map:**\n{mem}");
             }
         } else {
-            output.push_str(&format!(
+            use std::fmt::Write;
-                "Board '{}' configured. No static info available.",
+            let _ = write!(
-                board
+                output,
-            ));
+                "Board '{board}' configured. No static info available."
+            );
         }
 
         Ok(ToolResult {

src/tools/hardware_memory_map.rs

@@ -122,14 +122,16 @@ impl Tool for HardwareMemoryMapTool {
 
         if !probe_ok {
             if let Some(map) = self.static_map_for_board(board) {
-                output.push_str(&format!("**{}** (from datasheet):\n{}", board, map));
+                use std::fmt::Write;
+                let _ = write!(output, "**{board}** (from datasheet):\n{map}");
             } else {
+                use std::fmt::Write;
                 let known: Vec<&str> = MEMORY_MAPS.iter().map(|(b, _)| *b).collect();
-                output.push_str(&format!(
+                let _ = write!(
-                    "No memory map for board '{}'. Known boards: {}",
+                    output,
-                    board,
+                    "No memory map for board '{board}'. Known boards: {}",
                     known.join(", ")
-                ));
+                );
             }
         }