fix(clippy): derive Default for OnboardingState, collapse nested if

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: pythoninthegrass

crates/mt-tauri/src/agent/setup.rs

@@ -88,11 +88,11 @@ pub async fn pull_model(app: &tauri::AppHandle, model: String) -> Result<PullMod
         }
     }
 
-    if !buffer.trim().is_empty() {
-        if let Some(progress) = parse_pull_progress_line(&buffer) {
-            last_status.clone_from(&progress.status);
-            let _ = app.emit("agent://pull-progress", &progress);
-        }
+    if !buffer.trim().is_empty()
+        && let Some(progress) = parse_pull_progress_line(&buffer)
+    {
+        last_status.clone_from(&progress.status);
+        let _ = app.emit("agent://pull-progress", &progress);
     }
 
     let success = last_status == "success";

crates/mt-tauri/src/agent/types.rs

@@ -169,22 +169,13 @@ pub struct PullProgress {
 }
 
 /// Persisted via tauri-plugin-store under key `"agent_onboarding"`.
-#[derive(Debug, Clone, Serialize, Deserialize)]
+#[derive(Debug, Clone, Default, Serialize, Deserialize)]
 pub struct OnboardingState {
     pub completed: bool,
     #[serde(skip_serializing_if = "Option::is_none")]
     pub model: Option<String>,
 }
 
-impl Default for OnboardingState {
-    fn default() -> Self {
-        Self {
-            completed: false,
-            model: None,
-        }
-    }
-}
-
 #[derive(Debug, Clone, Serialize)]
 pub struct PullModelResult {
     pub success: bool,