gem-sweep 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 755da66034352c0d2af35834d2bc6806754d77244430fc5b85b3498c99c388d6
-  data.tar.gz: 1cead26dfdb7a79af3d3313ce38bea1dd8087f64e46716c20f4e095f62085457
+  metadata.gz: e5c68c560cc9e65be251fe942759ebd0cffcaa4eac563404e0e64d16c1e43684
+  data.tar.gz: 97cd1b17370e0f5d54ba9964db4c219475367c2fd864e89e12cb17e3480f3eba
 SHA512:
-  metadata.gz: 6366bd855ec3a33a0e26df16d338dc4225f0cc446ba202dbb76902c17864b85c2bf77f6baacca17856b5e82c90d967c1e74cc93cbb8f578cd0ec757c896949bb
-  data.tar.gz: 3994c438bd7c77bbbf1175d9e334edd03a62e61e835ffcf6989713a4e1a0463f2d80ace9d877cafea32757e1f754bd98e0cc64c2a218aa4eb18df5bab94a287d
+  metadata.gz: b7712862259767144e657591e907ae08859c0db1b25a161b45280eb4e05e43d7b1745c3c9ddbdd547812215854d1ae2b6cec7d00b564fe7b6d2b88b823e9eea4
+  data.tar.gz: 92c659ecf2aebbfa1f56b9461ed24366710cf338c2a72fa0cf578c14d06a269bd85d3c176fc8764af13507f42e0defb1b8fc67fd59c96be1cf0894b50f3c3189

data/lib/rubygems/commands/sweep_command.rb

@@ -3,7 +3,15 @@ require "pathname"
 require "rubygems/command"
 
 module GemSweep
-  def self.clean(spec, aggressive: false)
+  def self.clean(spec, aggressive: false, dryrun: false)
+    targets = collect_cleanup_targets(spec, aggressive: aggressive)
+    remove_targets(targets, dryrun: dryrun)
+  end
+
+  def self.collect_cleanup_targets(spec, aggressive: false)
+    targets = []
+
+    # Collect extension files
     (spec.full_require_paths - [spec.extension_dir]).each do |path|
       begin
         Dir.glob(File.join(path, "**/*")).each do |file|
@@ -11,30 +19,45 @@ module GemSweep
           next if file.sub(/#{spec.full_gem_path}\//, "") =~ /(\d+\.\d+\/)/
 
           if Pathname(file).extname == ".#{RbConfig::CONFIG["DLEXT"]}"
-            File.delete(file)
-            puts "Delete #{file}"
+            targets << file
           end
         end
       rescue Errno::EPERM
       end
     end
 
+    # Collect development directories if aggressive mode
     if aggressive
-      clean_test_directories(spec)
+      # Collect test/spec/features directories
+      %w[test spec features].each do |dir_name|
+        dir_path = File.join(spec.full_gem_path, dir_name)
+        targets << dir_path if Dir.exist?(dir_path)
+      end
+
+      # Collect tmp directories recursively
+      begin
+        Dir.glob(File.join(spec.full_gem_path, "**/tmp")).each do |tmp_dir|
+          targets << tmp_dir if Dir.exist?(tmp_dir)
+        end
+      rescue Errno::EPERM
+      end
     end
+
+    targets
   end
 
-  def self.clean_test_directories(spec)
-    %w[test spec features].each do |dir_name|
-      test_dir = File.join(spec.full_gem_path, dir_name)
-      if Dir.exist?(test_dir)
-        begin
+  def self.remove_targets(targets, dryrun: false)
+    targets.each do |path|
+      begin
+        if dryrun
+          puts "Would delete #{path}#{Dir.exist?(path) ? '/' : ''}"
+        else
           require "fileutils"
-          FileUtils.rm_rf(test_dir)
-          puts "Delete #{test_dir}/"
-        rescue Errno::EPERM
-          puts "Permission denied: #{test_dir}/"
+          FileUtils.rm_rf(path)
+          puts "Delete #{path}#{Dir.exist?(path) ? '/' : ''}"
         end
+      rescue Errno::EPERM
+        puts "Permission denied: #{path}#{Dir.exist?(path) ? '/' : ''}"
       end
     end
   end
@@ -43,18 +66,28 @@ end
 class Gem::Commands::SweepCommand < Gem::Command
   def initialize
     super "sweep", "Clean up unnecessary extension files"
-    
+
     add_option("-a", "--aggressive", "Also remove test, spec, and features directories") do |value, options|
       options[:aggressive] = true
     end
+
+    add_option("-n", "--dryrun", "Show what would be deleted without actually deleting") do |value, options|
+      options[:dryrun] = true
+    end
   end
 
   def execute
     aggressive = options[:aggressive]
-    
+    dryrun = options[:dryrun]
+
+    if dryrun
+      puts "Dry run mode: showing what would be deleted"
+      puts
+    end
+
     Gem::Specification.each do |spec|
       if aggressive || !spec.extensions.empty?
-        GemSweep.clean(spec, aggressive: aggressive)
+        GemSweep.clean(spec, aggressive: aggressive, dryrun: dryrun)
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gem-sweep
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Hiroshi SHIBATA