feat(lint): run cargo clippy --fix --lib -p auto-commit-rs

Author: gtkacz

src/cache.rs

@@ -220,12 +220,9 @@ pub fn interactive_history() -> Result<()> {
             }
 
             let options: Vec<String> = index.repos.iter().map(|e| e.repo_path.clone()).collect();
-            match Select::new("Select repository:", options).prompt() {
-                Ok(repo_path) => {
-                    let cache = load_repo_cache(&repo_path)?;
-                    show_repo_commits(&cache)?;
-                }
-                Err(_) => {}
+            if let Ok(repo_path) = Select::new("Select repository:", options).prompt() {
+                let cache = load_repo_cache(&repo_path)?;
+                show_repo_commits(&cache)?;
             }
         }
     }

src/cli.rs

@@ -338,10 +338,9 @@ pub fn interactive_config(global: bool) -> Result<()> {
                 show_descriptions = !show_descriptions;
             }
             MenuAction::Search => {
-                if let Some(query) = Text::new("Search:")
+                if let Ok(query) = Text::new("Search:")
                     .with_help_message("Enter text to search for in setting names")
                     .prompt()
-                    .ok()
                 {
                     let query_lower = query.to_lowercase();
                     if !query_lower.is_empty() {

src/preset.rs

@@ -277,7 +277,7 @@ pub fn interactive_presets() -> Result<()> {
             }
             "Manage existing preset..." => {
                 let options: Vec<String> =
-                    file.presets.iter().map(|p| preset_display(p)).collect();
+                    file.presets.iter().map(preset_display).collect();
                 let Ok(choice) = Select::new("Select preset:", options.clone()).prompt() else {
                     continue;
                 };
@@ -516,7 +516,7 @@ pub fn select_and_load_preset(cfg: &mut AppConfig) -> Result<Option<(u32, LlmPre
         println!("  {}", "No presets found.".dimmed());
         return Ok(None);
     }
-    let options: Vec<String> = file.presets.iter().map(|p| preset_display(p)).collect();
+    let options: Vec<String> = file.presets.iter().map(preset_display).collect();
     match Select::new("Select preset to load:", options.clone()).prompt() {
         Ok(choice) => {
             let idx = options.iter().position(|o| o == &choice).unwrap();

src/provider.rs

@@ -192,7 +192,7 @@ pub fn call_llm_with_fallback(
     diff: &str,
 ) -> Result<(String, Option<String>)> {
     match call_llm_inner(cfg, system_prompt, diff) {
-        Ok(msg) => return Ok((msg, None)),
+        Ok(msg) => Ok((msg, None)),
         Err(LlmCallError::TransportError(msg)) => {
             anyhow::bail!("Network error: {msg}");
         }