Merge pull request #7 from santoshxshrestha/fix/time-system-v0.1.9

fix: resolve critical time system bugs in v0.1.9

Author: vimlinuz

CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.1.9] - 2025-09-01
+
+### Fixed
+
+- **Critical Time System Bug**: Fixed incorrect time comparison logic in `tidy` command that was comparing timestamps against duration values instead of proper cutoff times
+- **Time Filtering Logic**: Corrected time filtering in `list -t` and `recover-all -t` commands to show items within the specified time range instead of showing newer items
+- **Consistent Time Handling**: All time-based operations now use consistent comparison logic with proper cutoff time calculations
+- **Borrow Checker Issues**: Resolved Rust ownership issues in tidy function by storing count before moving vectors
+
 ## [0.1.8] - 2025-09-01
 
 ### Added
@@ -166,7 +175,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - MIT License
 - Initial README and project documentation
 
-[Unreleased]: https://github.com/santoshxshrestha/rmxt/compare/v0.1.8...HEAD
+[Unreleased]: https://github.com/santoshxshrestha/rmxt/compare/v0.1.9...HEAD
+[0.1.9]: https://github.com/santoshxshrestha/rmxt/compare/v0.1.8...v0.1.9
 [0.1.8]: https://github.com/santoshxshrestha/rmxt/compare/v0.1.7...v0.1.8
 [0.1.7]: https://github.com/santoshxshrestha/rmxt/compare/v0.1.6...v0.1.7
 [0.1.6]: https://github.com/santoshxshrestha/rmxt/compare/v0.1.5...v0.1.6

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "rmxt"
-version = "0.1.8"
+version = "0.1.9"
 edition = "2024"
 authors = ["Santosh Shrestha <santoshxshrestha@gmail.com> "]
 description = "A replacement for the 'rm' command with a trash feature for safer file deletion."

Cargo.toml

@@ -33,8 +33,9 @@ pub fn list_specific_trash(seconds: i64) -> Result<(), trash::Error> {
     let mut list: Vec<List> = vec![];
     let entries = os_limited::list()?;
     let now = Local::now().timestamp();
+    let cutoff_time = now - seconds;
     for entry in entries {
-        if now - entry.time_deleted < seconds {
+        if entry.time_deleted >= cutoff_time {
             let time_deleted = Local
                 .timestamp_opt(entry.time_deleted, 0)
                 .single()
@@ -84,17 +85,21 @@ pub fn list_trash() {
 
 pub fn tidy_trash(days: i64) -> Result<(), trash::Error> {
     let seconds: i64 = days * 86400;
+    let cutoff_time = Local::now().timestamp() - seconds;
     let content_to_purge = trash::os_limited::list()?
         .into_iter()
-        .filter(|item| item.time_deleted < seconds)
+        .filter(|item| item.time_deleted < cutoff_time)
         .collect::<Vec<TrashItem>>();
 
     if !content_to_purge.is_empty() {
+        let purge_count = content_to_purge.len();
         if let Err(e) = os_limited::purge_all(content_to_purge) {
             eprintln!("{}", format!("Error purging items: {e}").red());
         } else {
-            println!("No items found to purge older than {days} days",);
+            println!("Purged {purge_count} items older than {days} days");
         }
+    } else {
+        println!("No items found to purge older than {days} days");
     }
     Ok(())
 }
@@ -150,8 +155,9 @@ fn main() {
                 return;
             };
             let now = Local::now().timestamp();
+            let cutoff_time = now - seconds;
             for entry in entries {
-                if now - entry.time_deleted < seconds {
+                if entry.time_deleted >= cutoff_time {
                     content_to_recover.push(entry);
                 }
             }