chore: cleanup low-severity findings from code review

- Remove no-op test_empty_supplied_paths (asserted literal, not code)
- Restore panic = 'abort' in release profile (CLAUDE.md says it's active)
- Remove commented-out readonly dependency
- Make PROGRESS_THRESHOLD_MILLIS pub const, remove threshold_millis() wrapper

Author: lookbusy1344

Cargo.toml

@@ -6,7 +6,7 @@ edition = "2024"
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
 [profile.release]
-# panic = 'abort'
+panic = 'abort'
 codegen-units = 1
 lto = 'fat'
 strip = true
@@ -21,7 +21,6 @@ hex = "0.4"
 data-encoding = "2.10"
 pico-args = "0.5"
 rayon = "1.11"
-# readonly = "0.2"
 generic-array = "1.3"
 byteorder = { default-features = false, version = "1.5" }
 md-5 = { default-features = false, version = "0.10" }

src/core/worker.rs

@@ -179,7 +179,7 @@ fn hash_with_progress(
     }
 
     if config.debug_mode
-        && elapsed >= Duration::from_millis(ProgressCoordinator::threshold_millis())
+        && elapsed >= Duration::from_millis(crate::progress::PROGRESS_THRESHOLD_MILLIS)
     {
         eprintln!(
             "File '{}' took {:.2}s to hash",

src/progress.rs

@@ -8,7 +8,7 @@ use indicatif::{MultiProgress, ProgressBar, ProgressStyle};
 use std::sync::Arc;
 use std::time::Duration;
 
-const PROGRESS_THRESHOLD_MILLIS: u64 = 200;
+pub const PROGRESS_THRESHOLD_MILLIS: u64 = 200;
 
 /// Coordinator for all progress indication
 ///
@@ -64,14 +64,6 @@ impl ProgressCoordinator {
         pb.enable_steady_tick(Duration::from_millis(350));
         pb
     }
-
-    /// Get the progress threshold in milliseconds
-    ///
-    /// Operations faster than this threshold should not display progress indication
-    /// to avoid visual flashing.
-    pub fn threshold_millis() -> u64 {
-        PROGRESS_THRESHOLD_MILLIS
-    }
 }
 
 impl Default for ProgressCoordinator {

src/unit_tests.rs

@@ -599,22 +599,4 @@ mod glob_tests {
             assert!(!path.ends_with("subdir"));
         }
     }
-
-    #[test]
-    fn test_empty_supplied_paths() {
-        // This would normally read from stdin, but we're testing the config behavior
-        let config = ConfigSettings {
-            debug_mode: false,
-            exclude_fn: false,
-            single_thread: false,
-            case_sensitive: true,
-            no_progress: false,
-            algorithm: HashAlgorithm::SHA3_256,
-            encoding: OutputEncoding::Hex,
-            limit_num: None,
-            supplied_paths: Vec::new(),
-        };
-
-        assert!(config.supplied_paths.is_empty());
-    }
 }