fix: prevent duplicate LoadingScreen initialization causing repository clone conflicts

- Remove duplicate init() call from run_main_loop to prevent concurrent repository processing
- Keep init() call only in set_current_screen for proper screen initialization
- Add repository cloning log for better debugging visibility
- Clean up unused variables and maintain original LoadingScreen instance creation logic

🤖 Generated with [Claude Code](https://claude.ai/code)

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

Author: unhappychoice

src/game/screen_manager.rs

@@ -376,6 +376,7 @@ impl ScreenManager {
                 _ => {}
             }
 
+            log::info!("Initializing screen: {:?}", self.current_screen_type);
             new_screen.init()?;
         }
 
@@ -613,10 +614,6 @@ impl ScreenManager {
         // Initialize current screen and force initial render
         Self::with_instance(|screen_manager| -> Result<()> {
             let mut manager = screen_manager.borrow_mut();
-            let current_screen_type = manager.current_screen_type.clone();
-            if let Some(current_screen) = manager.screens.get_mut(&current_screen_type) {
-                current_screen.init()?;
-            }
             manager.render_current_screen()
         })?;
 
@@ -642,36 +639,6 @@ impl ScreenManager {
         Ok(())
     }
 
-    pub fn run(&mut self) -> Result<()> {
-        // Initialize terminal for standalone usage (not used by run_global)
-        self.initialize_terminal()?;
-
-        if let Some(current_screen) = self.screens.get_mut(&self.current_screen_type) {
-            current_screen.init()?;
-        }
-
-        // Force initial render to display the screen
-        self.render_current_screen()?;
-
-        // Main game loop - continue until exit is requested
-        loop {
-            self.update_and_render()?;
-            self.handle_input()?;
-
-            // Exit the main loop when exit is requested
-            if self.exit_requested {
-                break;
-            }
-        }
-
-        if let Some(current_screen) = self.screens.get_mut(&self.current_screen_type) {
-            current_screen.cleanup()?;
-        }
-
-        self.cleanup_terminal()?;
-        Ok(())
-    }
-
     fn update_and_render(&mut self) -> Result<()> {
         if let Some(screen) = self.screens.get_mut(&self.current_screen_type) {
             let strategy = screen.get_update_strategy();

src/game/screens/loading_screen.rs

@@ -345,19 +345,13 @@ impl Screen for LoadingScreen {
         if let Some((repo_spec, repo_path, extraction_options)) =
             GameData::get_processing_parameters()
         {
-            log::info!(
-                "LoadingScreen::init() called with repo_spec={:?}, repo_path={:?}",
-                repo_spec,
-                repo_path
-            );
-            log::info!("Starting background processing");
             self.start_background_processing(
                 repo_spec.as_deref(),
                 repo_path.as_ref(),
                 extraction_options,
             )?;
         } else {
-            log::warn!("No processing parameters available in GameData - background processing not started");
+            log::error!("No processing parameters found in GameData");
         }
 
         self.show_initial()

src/repository_manager.rs

@@ -118,6 +118,8 @@ impl RepositoryManager {
         repo_info: &RepoInfo,
         loading_screen: Option<&crate::game::screens::loading_screen::LoadingScreen>,
     ) -> Result<PathBuf> {
+        log::info!("Cloning repository: {}/{}", repo_info.owner, repo_info.name);
+
         let local_path = Self::get_local_repo_path(repo_info)?;
 
         if local_path.exists() {