chore: development v0.2.94 - comprehensive testing complete [auto-commit]

Author: githubrobbi

Cargo.lock

@@ -39,7 +39,7 @@ exclude = [
 # Workspace Package Metadata (inherited by all crates)
 # ─────────────────────────────────────────────────────────────────────────────
 [workspace.package]
-version = "0.2.93"
+version = "0.2.94"
 edition = "2024"
 rust-version = "1.85"
 license = "MPL-2.0 OR LicenseRef-UFFS-Commercial"

Cargo.toml

@@ -21,7 +21,7 @@ Traditional file search tools (including `os.walk`, `FindFirstFile`, etc.) work
 
 **UFFS reads the MFT directly** - once - and queries it in memory using Polars DataFrames. This is like reading the entire phonebook once instead of looking up each name individually.
 
-### Benchmark Results (v0.2.93)
+### Benchmark Results (v0.2.94)
 
 | Drive Type | Records | Time | Throughput |
 |------------|---------|------|------------|
@@ -33,7 +33,7 @@ Traditional file search tools (including `os.walk`, `FindFirstFile`, etc.) work
 
 | Comparison | Records | Time | Notes |
 |------------|---------|------|-------|
-| **UFFS v0.2.93** | **18.7 Million** | **~142 seconds** | All disks, fast mode |
+| **UFFS v0.2.94** | **18.7 Million** | **~142 seconds** | All disks, fast mode |
 | UFFS v0.1.30 | 18.7 Million | ~315 seconds | Baseline |
 | Everything | 19 Million | 178 seconds | All disks |
 | WizFile | 6.5 Million | 299 seconds | Single HDD |

README.md

@@ -428,13 +428,21 @@ impl SearchResult {
     /// stream).
     #[must_use]
     pub fn from_record(record: &FileRecord, index: &MftIndex) -> Self {
+        let is_directory = record.is_directory();
+        // C++ parity: directories have empty name, files have actual name
+        let name = if is_directory {
+            String::new()
+        } else {
+            index.record_name(record).to_owned()
+        };
+
         Self {
-            name: index.record_name(record).to_owned(),
+            name,
             path: None, // Path resolution is expensive, done on demand
             size: record.first_stream.size.length,
             frs: record.frs,
             parent_frs: u64::from(record.first_name.parent_frs),
-            is_directory: record.is_directory(),
+            is_directory,
             stream_name: String::new(),
             name_index: 0,
             stream_index: 0,
@@ -455,14 +463,22 @@ impl SearchResult {
         let stream_info = index
             .get_stream_at(record, stream_idx)
             .unwrap_or(&record.first_stream);
+        let is_directory = record.is_directory();
+
+        // C++ parity: directories have empty name, files have actual name
+        let name = if is_directory {
+            String::new()
+        } else {
+            index.get_name(&name_info.name).to_owned()
+        };
 
         Self {
-            name: index.get_name(&name_info.name).to_owned(),
+            name,
             path: None,
             size: stream_info.size.length,
             frs: record.frs,
             parent_frs: u64::from(name_info.parent_frs),
-            is_directory: record.is_directory(),
+            is_directory,
             stream_name: index.stream_name(stream_info).to_owned(),
             name_index: name_idx,
             stream_index: stream_idx,
@@ -802,13 +818,14 @@ impl<'a> IndexQuery<'a> {
 
                 (0..name_count).flat_map(move |name_idx| {
                     let inner_cached_path = outer_cached_path.clone();
+                    let is_dir = record.is_directory();
                     (0..stream_count).map(move |stream_idx| {
                         let mut result =
                             SearchResult::from_expanded(record, index, name_idx, stream_idx);
                         if resolve_paths {
                             if let Some(stream) = index.get_stream_at(record, stream_idx) {
                                 // Use cached path for primary name (idx 0), build for hard links
-                                let base_path = if name_idx == 0 {
+                                let mut base_path = if name_idx == 0 {
                                     inner_cached_path
                                         .clone()
                                         .unwrap_or_else(|| index.build_path(record.frs))
@@ -819,6 +836,10 @@ impl<'a> IndexQuery<'a> {
                                 // Append stream name for ADS
                                 let stream_name = index.stream_name(stream);
                                 let path = if stream_name.is_empty() {
+                                    // Add trailing backslash for directories (C++ parity)
+                                    if is_dir && !base_path.ends_with('\\') {
+                                        base_path.push('\\');
+                                    }
                                     base_path
                                 } else {
                                     format!("{base_path}:{stream_name}")

crates/uffs-core/src/index_search.rs

@@ -270,8 +270,8 @@ impl FastPathResolver {
             }
         }
 
-        // Build final path
-        path_buf.push(self.volume);
+        // Build final path (uppercase drive letter for C++ parity)
+        path_buf.push(self.volume.to_ascii_uppercase());
         path_buf.push_str(":\\");
 
         // Append components in reverse order
@@ -618,9 +618,13 @@ impl PathResolver {
             }
         }
 
-        // Build path from components (reverse order)
+        // Build path from components (reverse order, uppercase drive letter)
         components.reverse();
-        let path = format!("{}:\\{}", self.volume, components.join("\\"));
+        let path = format!(
+            "{}:\\{}",
+            self.volume.to_ascii_uppercase(),
+            components.join("\\")
+        );
 
         // Cache the result
         self.cache.insert(frs, path.clone());

crates/uffs-core/src/path_resolver.rs

@@ -1053,12 +1053,12 @@ impl MftIndex {
             }
         }
 
-        // Reverse and join with volume prefix
+        // Reverse and join with volume prefix (backslash for C++ parity)
         components.reverse();
         format!(
-            "{}:/{}",
-            self.volume.to_ascii_lowercase(),
-            components.join("/")
+            "{}:\\{}",
+            self.volume.to_ascii_uppercase(),
+            components.join("\\")
         )
     }
 
@@ -1115,12 +1115,12 @@ impl MftIndex {
             current_frs = u64::from(parent_frs);
         }
 
-        // Reverse and join
+        // Reverse and join (backslash for C++ parity)
         components.reverse();
         format!(
-            "{}:/{}",
-            self.volume.to_ascii_lowercase(),
-            components.join("/")
+            "{}:\\{}",
+            self.volume.to_ascii_uppercase(),
+            components.join("\\")
         )
     }
 }
@@ -1254,14 +1254,14 @@ impl PathResolver {
 
         // Build path with single allocation
         let mut path = String::with_capacity(total_len);
-        path.push(self.volume.to_ascii_lowercase());
+        path.push(self.volume.to_ascii_uppercase());
         path.push(':');
 
         for &chain_idx in chain.iter().rev() {
             if let Some(record) = index.records.get(chain_idx) {
                 let name = index.record_name(record);
                 if !name.is_empty() && name != "." {
-                    path.push('/');
+                    path.push('\\');
                     path.push_str(name);
                 }
             }
@@ -1287,7 +1287,7 @@ impl PathResolver {
         let parent_path = if let Some(pidx) = index.frs_to_idx_opt(parent_frs) {
             self.materialize_path(index, pidx)
         } else if parent_frs == ROOT_FRS {
-            format!("{}:", self.volume.to_ascii_lowercase())
+            format!("{}:", self.volume.to_ascii_uppercase())
         } else {
             return String::new();
         };
@@ -1298,7 +1298,7 @@ impl PathResolver {
         } else {
             let mut path = String::with_capacity(parent_path.len() + 1 + name.len());
             path.push_str(&parent_path);
-            path.push('/');
+            path.push('\\');
             path.push_str(name);
             path
         }