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

Author: githubrobbi

CHANGELOG.md

@@ -23,7 +23,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Cleaned up all TTAPI references from justfile and build scripts
 - Updated justfile header and recipes for UFFS
 
-## [0.2.145] - 2026-01-27
+## [0.2.146] - 2026-01-27
 
 ### Added
 - Baseline CI validation for modernization effort
@@ -46,7 +46,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 - Various MFT parsing edge cases
 
-[Unreleased]: https://github.com/githubrobbi/UltraFastFileSearch/compare/v0.2.145...HEAD
-[0.2.145]: https://github.com/githubrobbi/UltraFastFileSearch/compare/v0.2.114...v0.2.145
+[Unreleased]: https://github.com/githubrobbi/UltraFastFileSearch/compare/v0.2.146...HEAD
+[0.2.146]: https://github.com/githubrobbi/UltraFastFileSearch/compare/v0.2.114...v0.2.146
 [0.2.114]: https://github.com/githubrobbi/UltraFastFileSearch/releases/tag/v0.2.114
 

Cargo.lock

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

Cargo.toml

@@ -130,3 +130,70 @@ let (name, parent_frs, _namespace) = match primary_name {
 
 ### CI Pipeline Status
 
+- Run 5: v0.2.145 - PASSED ✅
+
+---
+
+## Fix 3: Remove Debug Prints from Normal Flow
+
+### What Failed
+
+Debug `eprintln!` statements were polluting stdout/stderr during normal operation:
+```
+[DEBUG] read_all_index: ENTER volume=D
+[DEBUG] read_all_index: INSIDE spawn_blocking volume=D
+[DEBUG] read_all_index: read_mft_index_internal done
+[DEBUG] search_dataframe: before load_or_build_dataframe_cached drive=D
+[DEBUG] search_dataframe: after load_or_build_dataframe_cached
+```
+
+These were left over from debugging and should only appear when tracing is enabled.
+
+### The Fix
+
+Converted `eprintln!("[DEBUG] ...")` to proper `tracing::trace!()` calls:
+- `crates/uffs-mft/src/reader.rs`: 4 debug prints in `read_all_index`
+- `crates/uffs-cli/src/commands.rs`: 2 debug prints in `search_dataframe`
+
+---
+
+## Fix 4: Match C++ Output Format Exactly
+
+### What Was Different
+
+The Rust CLI output didn't match C++ output format exactly:
+1. Missing "Drives?" line AFTER the CSV data (not before)
+2. Missing "MMMmmm that was FAST" message when search completes in <= 1 second
+3. "Finished" message format was slightly different
+
+### C++ Output Format (stdout)
+
+```
+"Path","Name","Path Only",...,"Attributes"
+
+"G:\","","G:\",609893968,...
+...
+"G:\last\file.txt",...
+
+Drives? 	1	G:
+
+MMMmmm that was FAST ... maybe your searchstring was wrong?	*
+Search path. E.g. 'C:/' or 'C:\Prog**'
+```
+
+### C++ Output Format (stderr)
+
+```
+Finished 	in 0 s
+
+```
+
+### The Fix
+
+Modified `crates/uffs-cli/src/commands.rs`:
+1. Added "Drives?" line AFTER the CSV data (to stdout, format: `\nDrives? \t{count}\t{drive_list}\n\n`)
+2. Added "MMMmmm that was FAST" message when elapsed <= 1 second (to stdout)
+3. Updated "Finished" message format to match C++ exactly (to stderr: `\nFinished \tin {secs} s\n`)
+
+### CI Pipeline Status
+

LOG/2026_01_29_22_00_CHANGELOG_HEALING.md

@@ -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.145)
+### Benchmark Results (v0.2.146)
 
 | 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.145** | **18.7 Million** | **~142 seconds** | All disks, fast mode |
+| **UFFS v0.2.146** | **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

@@ -405,6 +405,17 @@ pub async fn search(
     // Index path is the default (fast, cached) - DataFrame path is fallback
     let use_index_path = should_use_index_path(mode, index.as_ref(), multi_drives.as_ref());
 
+    // Determine drives to search (needed for C++ compatible "Drives?" output)
+    #[cfg(windows)]
+    let drives_to_search: Vec<char> = if let Some(ref drives) = multi_drives {
+        drives.clone()
+    } else if let Some(drive) = single_drive.or_else(|| filters.parsed.drive()) {
+        vec![drive]
+    } else {
+        // No drive specified - search ALL available NTFS drives
+        uffs_mft::detect_ntfs_drives()
+    };
+
     // Handle raw MFT file input (cross-platform debugging)
     let mut results = if let Some(mft_path) = mft_file.as_ref() {
         info!(path = %mft_path.display(), "📂 Loading from raw MFT file");
@@ -420,20 +431,10 @@ pub async fn search(
         info!("🚀 Using fast cached MftIndex query path");
         #[cfg(windows)]
         {
-            // Determine drives to search
-            let drives_to_search: Vec<char> = if let Some(ref drives) = multi_drives {
-                drives.clone()
-            } else if let Some(drive) = single_drive.or_else(|| filters.parsed.drive()) {
-                vec![drive]
-            } else {
-                // No drive specified - search ALL available NTFS drives
-                let all_drives = uffs_mft::detect_ntfs_drives();
-                if all_drives.is_empty() {
-                    bail!("No NTFS drives found on this system");
-                }
-                info!(drives = ?all_drives, "No drive specified - searching all NTFS drives");
-                all_drives
-            };
+            // Use pre-computed drives_to_search
+            if drives_to_search.is_empty() {
+                bail!("No NTFS drives found on this system");
+            }
 
             if drives_to_search.len() == 1 {
                 // Single drive - use existing function
@@ -482,9 +483,32 @@ pub async fn search(
                     no_bitmap,
                 )
                 .await;
-                // Print timing after streaming completes
+
+                // C++ compatibility: Print "Drives?" and "MMMmmm" to stdout AFTER the data
                 let elapsed = start_time.elapsed();
-                eprintln!("Finished in {} s", elapsed.as_secs());
+                let secs = elapsed.as_secs();
+
+                #[allow(clippy::print_stdout)] // Intentional: C++ compatibility requires stdout
+                if !drives_to_search.is_empty() {
+                    let drive_list: String = drives_to_search
+                        .iter()
+                        .map(|d| format!("{d}:"))
+                        .collect::<Vec<_>>()
+                        .join("|");
+                    // C++ format: "\nDrives? \t{count}\t{drive_list}\n\n"
+                    println!("\nDrives? \t{}\t{drive_list}\n", drives_to_search.len());
+                }
+
+                #[allow(clippy::print_stdout)] // Intentional: C++ compatibility requires stdout
+                if secs <= 1 {
+                    println!(
+                        "MMMmmm that was FAST ... maybe your searchstring was wrong?\t{pattern}\n\
+                         Search path. E.g. 'C:/' or 'C:\\Prog**' "
+                    );
+                }
+
+                // C++ compatibility: "Finished in X s" (to stderr)
+                eprintln!("\nFinished \tin {secs} s\n");
                 return result;
             }
         }
@@ -554,8 +578,34 @@ pub async fn search(
         eprintln!("=== TOTAL: {total_ms} ms ===");
     }
 
+    // C++ compatibility: Print "Drives?" and "MMMmmm" to stdout AFTER the data
+    // These go to stdout (not stderr) to match C++ behavior
     let secs = elapsed.as_secs();
-    eprintln!("Finished in {secs} s");
+
+    #[cfg(windows)]
+    #[allow(clippy::print_stdout)] // Intentional: C++ compatibility requires stdout
+    if !drives_to_search.is_empty() {
+        let drive_list: String = drives_to_search
+            .iter()
+            .map(|d| format!("{d}:"))
+            .collect::<Vec<_>>()
+            .join("|");
+        // C++ format: "\nDrives? \t{count}\t{drive_list}\n\n"
+        println!("\nDrives? \t{}\t{drive_list}\n", drives_to_search.len());
+    }
+
+    // C++ compatibility: "MMMmmm that was FAST" message when elapsed <= 1 second
+    // (to stdout)
+    #[allow(clippy::print_stdout)] // Intentional: C++ compatibility requires stdout
+    if secs <= 1 {
+        println!(
+            "MMMmmm that was FAST ... maybe your searchstring was wrong?\t{pattern}\n\
+             Search path. E.g. 'C:/' or 'C:\\Prog**' "
+        );
+    }
+
+    // C++ compatibility: "Finished in X s" (to stderr)
+    eprintln!("\nFinished \tin {secs} s\n");
 
     info!(count = results.height(), "Search complete");
     Ok(())
@@ -824,9 +874,7 @@ async fn load_and_filter_data(
     if let Some(drive_letter) = effective_drive {
         // Single drive search with proper path resolution
         let t_read = std::time::Instant::now();
-        eprintln!(
-            "[DEBUG] search_dataframe: before load_or_build_dataframe_cached drive={drive_letter}"
-        );
+        tracing::trace!(drive = %drive_letter, "search_dataframe: before load_or_build_dataframe_cached");
 
         // Use cached DataFrame path for performance (Windows only)
         #[cfg(windows)]
@@ -835,7 +883,7 @@ async fn load_and_filter_data(
                 .await
                 .with_context(|| format!("Failed to read MFT for drive {drive_letter}:"))?;
 
-        eprintln!("[DEBUG] search_dataframe: after load_or_build_dataframe_cached");
+        tracing::trace!(drive = %drive_letter, "search_dataframe: after load_or_build_dataframe_cached");
 
         // Non-Windows: read directly (no caching)
         #[cfg(not(windows))]