ruborg 0.7.1 → 0.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b1659a54e64ed15742467c6e95ea96fd311f0332e2fafdee7e560cec5c2385c
-  data.tar.gz: ec49de1e1231ad2bd189e08aedb70d22ec10f4f322f1b4690c9c3ec41be590bb
+  metadata.gz: 314fb4f24b1b5544d95257f5d3f315099f221ecc92b7920e935066319db72458
+  data.tar.gz: 0ef7e5183f7596e1417b4b12915557d9c177b0b2504f6a143c7ff8aa59f726aa
 SHA512:
-  metadata.gz: '0593d8521ab13110b3fefe00b9a1bc3cd6a8c28ffa70e93dda6e3bfa6b763f0562abd564af25a14c5619b2fe7dabb7ef76ffbe1341c6dcdb4dcac7efbb7be847'
-  data.tar.gz: 9486465c9803962569b5f376556efafa8147d72a294e9a6e3137d29e67c9664b212bed69addc0bab0ad434ddc5f42416e3aa7426087bbf4dd7c9a4ce2c71192f
+  metadata.gz: fb9bcdb3517dc9fd2c1c05d868e8015fd5fe3a55c00e5c277cf7b0257f238535821f8dbcda594dc9bc4b413a8b96d4739d327e1b420f6bad08ac19e724193199
+  data.tar.gz: 6544e4d87969bd59ca7912653ecaebedeca75795f934edb93ad9435a0e48cff5b0c95734aa5bdc70572a38cbed014ea8ac93cff4abee97867a910acba47d5240

data/CHANGELOG.md

@@ -7,6 +7,34 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.7.3] - 2025-10-09
+
+### Changed
+- **Smart Remove-Source for Skipped Files**: Skipped files (unchanged, already backed up) are now deleted when `--remove-source` is used
+  - Previously: Only newly backed-up files were deleted, skipped files remained
+  - Now: Both backed-up AND skipped files are deleted (they're all safely backed up)
+  - Rationale: If a file is skipped because it's already in an archive (verified by hash), it's safe to delete
+  - Makes `--remove-source` behavior consistent: "delete everything that's safely backed up"
+
+### Technical Details
+- Per-file mode verifies files are safely backed up before skipping (path + size + SHA256 hash match)
+- Skipped files are deleted immediately after verification (lib/ruborg/backup.rb:102)
+- Test updated to verify skipped files are deleted (spec/ruborg/per_file_backup_spec.rb:518)
+
+## [0.7.2] - 2025-10-09
+
+### Fixed
+- **Per-File Remove Source Behavior**: Files are now deleted immediately after each successful backup in per-file mode
+  - Previously deleted entire source paths at the end (dangerous - could delete unchanged files)
+  - Now deletes only successfully backed-up files, one at a time
+  - Skipped files (unchanged) are never deleted
+  - Matches the per-file philosophy: individual file handling throughout the backup process
+
+### Added
+- **Test Coverage**: Added 2 new RSpec tests verifying per-file remove-source behavior
+  - Tests immediate file deletion after backup
+  - Tests that skipped files are not deleted
+
 ## [0.7.1] - 2025-10-08
 
 ### Added

data/README.md

@@ -25,7 +25,7 @@ A friendly Ruby frontend for [Borg Backup](https://www.borgbackup.org/). Ruborg
 - 📈 **Summary View** - Quick overview of all repositories and their configurations
 - 🔧 **Custom Borg Path** - Support for custom Borg executable paths per repository
 - 🏠 **Hostname Validation** - NEW! Restrict backups to specific hosts (global or per-repository)
-- ✅ **Well-tested** - Comprehensive test suite with RSpec (286+ examples)
+- ✅ **Well-tested** - Comprehensive test suite with RSpec (288+ examples)
 - 🔒 **Security-focused** - Path validation, safe YAML loading, command injection protection
 
 ## Prerequisites

data/lib/ruborg/backup.rb

@@ -91,12 +91,16 @@ module Ruborg
               stored_hash = stored_info[:hash]
 
               if current_hash == stored_hash
-                # Content truly unchanged
+                # Content truly unchanged - file is already safely backed up
                 puts " - Archive already exists (file unchanged)"
                 @logger&.info(
                   "[#{@repo_name}] Skipped #{file_path} - archive #{archive_name} already exists (file unchanged)"
                 )
                 skipped_count += 1
+
+                # If remove_source is enabled, delete the file (it's already safely backed up)
+                remove_single_file(file_path) if remove_source
+
                 next
               else
                 # Size same but content changed (rare: edited + truncated/padded to same size)
@@ -131,6 +135,9 @@ module Ruborg
         # Log successful action with details
         @logger&.info("[#{@repo_name}] Archived #{file_path} in archive #{archive_name}")
         backed_up_count += 1
+
+        # Remove source file immediately after successful backup in per-file mode
+        remove_single_file(file_path) if remove_source
       end
       # rubocop:enable Metrics/BlockLength
 
@@ -139,9 +146,6 @@ module Ruborg
       else
         puts "✓ Per-file backup completed: #{backed_up_count} file(s) backed up"
       end
-
-      # NOTE: remove_source handled per file after successful backup
-      remove_source_files if remove_source
     end
     # rubocop:enable Metrics/AbcSize, Metrics/MethodLength, Metrics/PerceivedComplexity, Metrics/BlockNesting
 
@@ -333,6 +337,34 @@ module Ruborg
       result
     end
 
+    def remove_single_file(file_path)
+      require "fileutils"
+
+      # Resolve symlinks and validate path
+      begin
+        real_path = File.realpath(file_path)
+      rescue Errno::ENOENT
+        # File doesn't exist (already deleted?), skip
+        @logger&.warn("Source file does not exist, skipping: #{file_path}")
+        return
+      end
+
+      # Security check: ensure file still exists
+      unless File.exist?(real_path)
+        @logger&.warn("Source file no longer exists, skipping: #{real_path}")
+        return
+      end
+
+      # Additional safety: don't delete system files
+      if real_path == "/" || real_path.start_with?("/bin", "/sbin", "/usr", "/etc", "/sys", "/proc")
+        @logger&.error("Refusing to delete system path: #{real_path}")
+        raise BorgError, "Refusing to delete system path: #{real_path}"
+      end
+
+      @logger&.info("Removing file: #{real_path}")
+      FileUtils.rm(real_path)
+    end
+
     def remove_source_files
       require "fileutils"
 

data/lib/ruborg/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Ruborg
-  VERSION = "0.7.1"
+  VERSION = "0.7.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ruborg
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.3
 platform: ruby
 authors:
 - Michail Pantelelis