Fix race condition by removing directory deletion before rename

Remove the remove_dir_all_force call before rename to avoid race condition where:
1. Thread A removes target_dir
2. Thread B removes target_dir (ignored due to NotFound)
3. Thread B renames tmp to target_dir
4. Thread A tries to rename and gets AlreadyExists

Now we just attempt the rename directly, which is more atomic. If it fails with AlreadyExists,
we know another thread succeeded and we handle it gracefully by validating/fixing the installation.

Co-authored-by: fengmk2 <156269+fengmk2@users.noreply.github.com>

Author: Copilot

crates/vite_install/src/package_manager.rs

@@ -381,11 +381,11 @@ async fn download_package_manager(
     }
 
     // rename $target_dir_tmp to $target_dir
+    // We don't remove target_dir first to avoid race conditions where another thread
+    // might be in the middle of creating it. Instead, we rely on the rename operation
+    // to fail if the directory already exists, which we handle gracefully.
     tracing::debug!("Rename {:?} to {:?}", target_dir_tmp, target_dir);
-    remove_dir_all_force(&target_dir).await?;
 
-    // Handle potential race condition where another concurrent process may have
-    // already created the target directory
     match tokio::fs::rename(&target_dir_tmp, &target_dir).await {
         Ok(()) => {
             // create shim file