refactor: remove redundant try_exists guard, add debug log

- Remove redundant try_exists check before remove_or_rename_to_old
  since it already handles NotFound
- Log the original remove_file error at debug level before falling
  back to rename

Author: fengmk2

crates/vite_global_cli/src/commands/env/global_install.rs

@@ -409,9 +409,7 @@ async fn create_package_shim(
         let shim_path = bin_dir.join(format!("{}.exe", bin_name));
 
         // Delete before overwrite; falls back to rename if the exe is locked.
-        if tokio::fs::try_exists(&shim_path).await.unwrap_or(false) {
-            super::setup::remove_or_rename_to_old(&shim_path).await;
-        }
+        super::setup::remove_or_rename_to_old(&shim_path).await;
 
         // Copy the trampoline binary as <bin_name>.exe.
         // The trampoline detects the tool name from its own filename and sets

crates/vite_global_cli/src/commands/env/setup.rs

@@ -362,9 +362,10 @@ pub(crate) async fn remove_or_rename_to_old(path: &vite_path::AbsolutePath) {
     match tokio::fs::remove_file(path).await {
         Ok(()) => return,
         Err(e) if e.kind() == std::io::ErrorKind::NotFound => return,
-        Err(_) => {}
+        Err(e) => {
+            tracing::debug!("remove_file failed ({}), attempting rename", e);
+        }
     }
-    // File exists but is locked (e.g., running process) — rename instead.
     rename_to_old(path).await;
 }