fix(cortex-common): fix Windows test compatibility for cwd_guard, dirs, and path_utils

- Use dunce::canonicalize() in cwd_guard tests to avoid UNC path prefix
  on Windows (\\?\C:\...) which differs from env::current_dir() format
- Use std::env::temp_dir() for cross-platform temp directory in dirs
  test_env_override instead of hardcoded Unix path /tmp
- Add cfg_attr ignore for test_ensure_parent_dir_root_path on Windows
  since Unix root path /file.txt is not applicable on Windows

Author: factorydroid

Cargo.lock

@@ -39,3 +39,4 @@ async = ["tokio"]
 [dev-dependencies]
 tempfile = { workspace = true }
 serial_test = { workspace = true }
+dunce = { workspace = true }

cortex-common/Cargo.toml

@@ -314,13 +314,12 @@ mod tests {
     fn test_cwd_guard_restores() {
         let original = env::current_dir().unwrap();
         let temp_dir = TempDir::new().unwrap();
+        // Use dunce::canonicalize on Windows to avoid UNC paths (\\?\C:\...)
+        let expected_temp_path = dunce::canonicalize(temp_dir.path()).unwrap();
 
         {
             let _guard = CwdGuard::new(temp_dir.path()).unwrap();
-            assert_eq!(
-                env::current_dir().unwrap(),
-                temp_dir.path().canonicalize().unwrap()
-            );
+            assert_eq!(env::current_dir().unwrap(), expected_temp_path);
         }
 
         // Should be back to original after guard dropped
@@ -348,7 +347,8 @@ mod tests {
     fn test_cwd_guard_disabled() {
         let original = env::current_dir().unwrap();
         let temp_dir = TempDir::new().unwrap();
-        let temp_path = temp_dir.path().canonicalize().unwrap();
+        // Use dunce::canonicalize on Windows to avoid UNC paths (\\?\C:\...)
+        let temp_path = dunce::canonicalize(temp_dir.path()).unwrap();
 
         {
             let mut guard = CwdGuard::new(temp_dir.path()).unwrap();
@@ -382,7 +382,8 @@ mod tests {
     fn test_in_directory() {
         let original = env::current_dir().unwrap();
         let temp_dir = TempDir::new().unwrap();
-        let temp_path = temp_dir.path().canonicalize().unwrap();
+        // Use dunce::canonicalize on Windows to avoid UNC paths (\\?\C:\...)
+        let temp_path = dunce::canonicalize(temp_dir.path()).unwrap();
 
         let result = in_directory(temp_dir.path(), || env::current_dir().unwrap()).unwrap();
 

cortex-common/src/cwd_guard.rs

@@ -218,12 +218,14 @@ mod tests {
 
     #[test]
     fn test_env_override() {
+        // Use a cross-platform temp directory path for the test
+        let test_path = std::env::temp_dir().join("test-cortex");
         // SAFETY: This test runs in isolation and only modifies test-specific env vars
         unsafe {
-            std::env::set_var("CORTEX_HOME", "/tmp/test-cortex");
+            std::env::set_var("CORTEX_HOME", &test_path);
         }
         let dirs = AppDirs::new().unwrap();
-        assert_eq!(dirs.config_dir, PathBuf::from("/tmp/test-cortex"));
+        assert_eq!(dirs.config_dir, test_path);
         unsafe {
             std::env::remove_var("CORTEX_HOME");
         }

cortex-common/src/dirs.rs

@@ -514,6 +514,7 @@ mod tests {
     }
 
     #[test]
+    #[cfg_attr(windows, ignore = "Unix root path not applicable on Windows")]
     fn test_ensure_parent_dir_root_path() {
         let path = Path::new("/file.txt");