chore: Fix cargo clippy warnings

Author: unhappychoice

src/extractor/parser.rs

@@ -38,17 +38,12 @@ impl CodeExtractor {
             .build();
 
         let mut total_files_to_process = 0;
-        let mut scanned_count = 0;
 
         for entry in walker_count {
             let entry =
                 entry.map_err(|e| GitTypeError::ExtractionFailed(format!("Walk error: {}", e)))?;
             let path = entry.path();
 
-            scanned_count += 1;
-
-            if scanned_count % 200 == 0 {}
-
             if !path.is_file() {
                 continue;
             }

src/game/models/loading_steps/cloning_step.rs

@@ -49,24 +49,20 @@ impl Step for CloningStep {
 
     fn execute(&self, context: &mut ExecutionContext) -> Result<StepResult> {
         if let Some(repo_spec) = context.repo_spec {
-            match RepoManager::parse_repo_url(repo_spec)? {
-                repo_info => {
-                    let repo_display = format!(
-                        "Repository: {}/{} ({})",
-                        repo_info.owner, repo_info.name, repo_info.origin
-                    );
+            let repo_info = RepoManager::parse_repo_url(repo_spec)?;
+            let repo_display = format!(
+                "Repository: {}/{} ({})",
+                repo_info.owner, repo_info.name, repo_info.origin
+            );
 
-                    // Set repo info in LoadingScreen
-                    if let Some(screen) = context.loading_screen {
-                        let _ = screen.set_repo_info(repo_display);
-                    }
-
-                    // Clone repository
-                    let repo_path =
-                        RepoManager::clone_or_update_repo(&repo_info, context.loading_screen)?;
-                    Ok(StepResult::RepoPath(repo_path))
-                }
+            // Set repo info in LoadingScreen
+            if let Some(screen) = context.loading_screen {
+                let _ = screen.set_repo_info(repo_display);
             }
+
+            // Clone repository
+            let repo_path = RepoManager::clone_or_update_repo(&repo_info, context.loading_screen)?;
+            Ok(StepResult::RepoPath(repo_path))
         } else {
             // Skip this step if no remote repo specified
             Ok(StepResult::Skipped)

src/game/models/loading_steps/step_manager.rs

@@ -10,6 +10,12 @@ pub struct StepManager {
     steps: Vec<Box<dyn Step>>,
 }
 
+impl Default for StepManager {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 impl StepManager {
     pub fn new() -> Self {
         Self {

src/game/screens/loading_screen.rs

@@ -65,7 +65,6 @@ pub struct StepInfo {
 
 pub struct LoadingScreen {
     state: LoadingScreenState,
-    step_manager: Arc<StepManager>,
     render_handle: Option<thread::JoinHandle<Result<()>>>,
 }
 
@@ -105,7 +104,6 @@ impl LoadingScreen {
 
         Ok(Self {
             state,
-            step_manager,
             render_handle: None,
         })
     }
@@ -289,19 +287,11 @@ impl LoadingScreen {
         // Create vertical centering layout
         let vertical_layout = Layout::default()
             .direction(Direction::Vertical)
-            .constraints(if has_repo_info {
-                vec![
-                    Constraint::Length(vertical_margin),
-                    Constraint::Length(content_height),
-                    Constraint::Min(0),
-                ]
-            } else {
-                vec![
-                    Constraint::Length(vertical_margin),
-                    Constraint::Length(content_height),
-                    Constraint::Min(0),
-                ]
-            })
+            .constraints(vec![
+                Constraint::Length(vertical_margin),
+                Constraint::Length(content_height),
+                Constraint::Min(0),
+            ])
             .split(size);
 
         // Main content layout

src/repo_manager.rs

@@ -270,7 +270,7 @@ impl RepoManager {
 
         // Phase 1: Git garbage collection
         let _output = Command::new("git")
-            .args(&["gc", "--aggressive", "--prune=now"])
+            .args(["gc", "--aggressive", "--prune=now"])
             .current_dir(repo_path)
             .output();