Clean fspy paths at strip_path_prefix site, simplify downstream code

Move path_clean::PathClean normalization into the strip_path_prefix
callback so all fspy-reported paths are clean from the start. This
removes the need for separate cleaning in the negative glob filter
and in PostRunFingerprint::create.

https://claude.ai/code/session_01PR9yhnScRoVoHUcviV47u5

Author: claude

crates/vite_task/src/session/execute/fingerprint.rs

@@ -84,21 +84,9 @@ impl PostRunFingerprint {
         let inferred_inputs = inferred_path_reads
             .par_iter()
             .map(|(relative_path, path_read)| {
-                // Clean the absolute path to normalize `..` from fspy-tracked paths
-                // (e.g., `packages/sub-pkg/../shared/dist/output.js`).
-                let cleaned_abs =
-                    path_clean::PathClean::clean(base_dir.join(relative_path).as_path());
-
-                // Derive a cleaned workspace-relative key so stored paths are normalized
-                let clean_key = cleaned_abs
-                    .strip_prefix(base_dir.as_path())
-                    .ok()
-                    .and_then(|p| RelativePathBuf::new(p).ok())
-                    .unwrap_or_else(|| relative_path.clone());
-
-                let full_path = Arc::<AbsolutePath>::from(base_dir.join(&clean_key));
+                let full_path = Arc::<AbsolutePath>::from(base_dir.join(relative_path));
                 let fingerprint = fingerprint_path(&full_path, *path_read)?;
-                Ok((clean_key, fingerprint))
+                Ok((relative_path.clone(), fingerprint))
             })
             .collect::<anyhow::Result<HashMap<_, _>>>()?;
 

crates/vite_task/src/session/execute/spawn.rs

@@ -190,14 +190,17 @@ pub async fn spawn_with_tracking(
     let path_writes = &mut path_accesses.path_writes;
 
     for access in termination.path_accesses.iter() {
+        // Clean the path to normalize `..` components since fspy may report
+        // paths like `packages/sub-pkg/../shared/dist/output.js`.
         let relative_path = access.path.strip_path_prefix(workspace_root, |strip_result| {
             let Ok(stripped_path) = strip_result else {
                 return None;
             };
             // On Windows, paths are possible to be still absolute after stripping the workspace root.
             // For example: c:\workspace\subdir\c:\workspace\subdir
             // Just ignore those accesses.
-            RelativePathBuf::new(stripped_path).ok()
+            let cleaned = path_clean::PathClean::clean(stripped_path);
+            RelativePathBuf::new(cleaned).ok()
         });
 
         let Some(relative_path) = relative_path else {
@@ -211,15 +214,8 @@ pub async fn spawn_with_tracking(
         }
 
         // Filter against resolved negative globs (both are workspace-root-relative).
-        // Clean the relative path to normalize `..` components since fspy may report
-        // paths like `packages/sub-pkg/../shared/dist/output.js`.
-        if !resolved_negatives.is_empty() {
-            let cleaned = path_clean::PathClean::clean(relative_path.as_path());
-            if let Some(cleaned_str) = cleaned.to_str()
-                && resolved_negatives.iter().any(|neg| neg.is_match(cleaned_str))
-            {
-                continue;
-            }
+        if resolved_negatives.iter().any(|neg| neg.is_match(relative_path.as_str())) {
+            continue;
         }
 
         if access.mode.contains(AccessMode::READ) {