fix(package-manager): add thread synchronization to prevent race conditions

- Add global mutex lock using OnceLock for file system operations
- Protect critical sections in download_package_manager function
- Add test-specific lock for temp directory creation
- Fix "Directory not empty" errors during concurrent test execution

The synchronization ensures concurrent calls won't interfere when:
- Checking if files exist
- Removing old directories
- Renaming temporary directories
- Creating shim files

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

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

Author: fengmk2

crates/vite_package_manager/src/package_manager.rs

@@ -4,11 +4,12 @@ use std::{
     fs::{self, File},
     io::{BufReader, Seek, SeekFrom},
     path::Path,
+    sync::OnceLock,
 };
 
 use semver::{Version, VersionReq};
 use serde::{Deserialize, Serialize};
-use tokio::fs::remove_dir_all;
+use tokio::{fs::remove_dir_all, sync::Mutex};
 use vite_error::Error;
 use vite_path::{AbsolutePath, AbsolutePathBuf, RelativePathBuf};
 use vite_str::Str;
@@ -19,6 +20,13 @@ use crate::{
     shim,
 };
 
+// Global lock for file system operations to prevent race conditions
+static FS_OPERATION_LOCK: OnceLock<Mutex<()>> = OnceLock::new();
+
+fn get_fs_lock() -> &'static Mutex<()> {
+    FS_OPERATION_LOCK.get_or_init(|| Mutex::new(()))
+}
+
 #[derive(Serialize, Deserialize, Clone, Default)]
 #[serde(rename_all = "camelCase")]
 struct PackageJson {
@@ -440,14 +448,21 @@ async fn download_package_manager(
         return Ok(install_dir);
     }
 
-    // rename $target_dir_tmp to $target_dir
-    tracing::debug!("Rename {:?} to {:?}", target_dir_tmp, target_dir);
-    remove_dir_all_force(&target_dir).await?;
-    tokio::fs::rename(&target_dir_tmp, &target_dir).await?;
+    {
+        // Acquire lock for critical file system operations
+        let _lock = get_fs_lock().lock().await;
+
+        // rename $target_dir_tmp to $target_dir
+        tracing::debug!("Rename {:?} to {:?}", target_dir_tmp, target_dir);
+        remove_dir_all_force(&target_dir).await?;
+        tokio::fs::rename(&target_dir_tmp, &target_dir).await?;
 
-    // create shim file
-    tracing::debug!("Create shim files for {}", bin_name);
-    create_shim_files(package_manager_type, &bin_prefix).await?;
+        // create shim file
+        tracing::debug!("Create shim files for {}", bin_name);
+        create_shim_files(package_manager_type, &bin_prefix).await?;
+
+        // Lock is automatically released when _lock goes out of scope
+    }
 
     Ok(install_dir)
 }
@@ -555,13 +570,21 @@ fn format_path_env(bin_prefix: impl AsRef<Path>) -> String {
 
 #[cfg(test)]
 mod tests {
-    use std::fs;
+    use std::{fs, sync::Mutex as StdMutex};
 
     use tempfile::{TempDir, tempdir};
 
     use super::*;
 
+    // Test-specific lock for temp directory creation
+    static TEST_TEMP_DIR_LOCK: OnceLock<StdMutex<()>> = OnceLock::new();
+
+    fn get_test_lock() -> &'static StdMutex<()> {
+        TEST_TEMP_DIR_LOCK.get_or_init(|| StdMutex::new(()))
+    }
+
     fn create_temp_dir() -> TempDir {
+        let _lock = get_test_lock().lock().unwrap();
         tempdir().expect("Failed to create temp directory")
     }