debride 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d36c868e7a15b6746112225734bc0d747df00d6e
-  data.tar.gz: 84d0559594fe52021133115507afcea6974b3967
+  metadata.gz: bb39a05e306cbbcd061e35997220368ac0cbd964
+  data.tar.gz: 0704f659f6af475a29e97b424eeecb9ef66b1b8a
 SHA512:
-  metadata.gz: 427ce5bf52dd04e4060da5c417fbe90cf918e2e39104b72da41ad3335aac76d359501aa33f4dbdb17b664f55391d2372aeafbb814beaf98d2b018593cae9b3e3
-  data.tar.gz: 19e156808348e4a005febc69774523634cbc2fb3798717a8cfe7bc6516544c1786b858867ef8bc83b7fb9dae30c189e86648c9585cfb73a9c358473219488787
+  metadata.gz: a33032ea66b7b298f0a922978bb3f70b1fdd983bf1147e442fddaf8188be6695b81be4430b32135a4130d1893281d437683e58d8a90b02a533e3909b768edb27
+  data.tar.gz: 0bcaa9a023b64733d5def8e8cec519328b21bb7a68fa8bf1da74aa7284e031ccc9fc2a4ec7692f037af5b0017400665772d88d9395bb3b5c08c28c16d953804e

data/History.rdoc

@@ -1,3 +1,13 @@
+=== 1.2.0 / 2015-03-26
+
+* 1 major enhancement:
+
+  * Added plugin system to allow processing of other file types.
+
+* 1 minor enhancement:
+
+  * Refactored code to allow for plugin system to do multi-phase processing.
+
 === 1.1.0 / 2015-03-18
 
 * 1 minor enhancement:

data/Rakefile

@@ -15,4 +15,10 @@ Hoe.spec "debride" do
   dependency "ruby_parser", "~> 3.6"
 end
 
+task :rails do
+  ENV["GEM_HOME"] = "tmp/isolate/ruby-2.0.0"
+  ENV["GEM_PATH"] = "../../debride-erb/dev/tmp/isolate/ruby-2.0.0"
+  ruby "-Ilib:../../debride-erb/dev/lib bin/debride ~/Work/git/seattlerb.org/{app,lib} --whitelist ~/Work/git/seattlerb.org/whitelist.txt"
+end
+
 # vim: syntax=ruby

data/lib/debride.rb

@@ -9,7 +9,48 @@ require "set"
 # A static code analyzer that points out possible dead methods.
 
 class Debride < MethodBasedSexpProcessor
-  VERSION = "1.1.0" # :nodoc:
+  VERSION = "1.2.0" # :nodoc:
+  PROJECT = "debride"
+
+  def self.expand_dirs_to_files *dirs # TODO: push back up to sexp_processor
+    extensions = self.file_extensions
+
+    dirs.flatten.map { |p|
+      if File.directory? p then
+        Dir[File.join(p, "**", "*.{#{extensions.join(",")}}")]
+      else
+        p
+      end
+    }.flatten.map { |s| s.sub(/^\.\//, "") } # strip "./" from paths
+  end
+
+  def self.load_plugins proj = PROJECT
+    unless defined? @@plugins then
+      @@plugins = []
+
+      task_re = /#{PROJECT}_task/o
+      plugins = Gem.find_files("#{PROJECT}_*.rb").reject { |p| p =~ task_re }
+
+      plugins.each do |plugin|
+        plugin_name = File.basename(plugin, ".rb").sub(/^#{PROJECT}_/o, "")
+        next if @@plugins.include? plugin_name
+        begin
+          load plugin
+          @@plugins << plugin_name
+        rescue LoadError => e
+          warn "error loading #{plugin.inspect}: #{e.message}. skipping..."
+        end
+      end
+    end
+
+    @@plugins
+  rescue
+    # ignore
+  end
+
+  def self.file_extensions
+    %w[rb rake] + load_plugins
+  end
 
   ##
   # Top level runner for bin/debride.
@@ -17,15 +58,38 @@ class Debride < MethodBasedSexpProcessor
   def self.run args
     opt = parse_options args
 
-    callers = Debride.new opt
+    debride = Debride.new opt
+    debride.run expand_dirs_to_files(args)
+    debride
+  end
+
+  def run(*files)
+    files.flatten.each do |file|
+      warn "Processing #{file}" if option[:verbose]
+
+      ext = File.extname(file).sub(/^\./, "")
+      ext = "rb" if ext.nil? || ext.empty?
+      msg = "process_#{ext}"
 
-    expand_dirs_to_files(args).each do |path|
-      warn "processing: #{path}" if opt[:verbose]
-      parser = RubyParser.new
-      callers.process parser.process File.read(path), path
+      unless respond_to? msg then
+        warn "  Unknown file type: #{ext}, defaulting to ruby" if option[:verbose]
+        msg = "process_rb"
+      end
+
+      begin
+        process send(msg, file)
+      rescue RuntimeError, SyntaxError => e
+        warn "  skipping #{file}: #{e.message}"
+      end
     end
+  end
 
-    callers
+  def process_rb file
+    begin
+      RubyParser.new.process(File.binread(file), file, option[:timeout])
+    rescue Timeout::Error
+      warn "TIMEOUT parsing #{file}. Skipping."
+    end
   end
 
   ##
@@ -114,9 +178,11 @@ class Debride < MethodBasedSexpProcessor
 
   def process_call sexp # :nodoc:
     method_name = sexp[2]
-    method_name = :initialize if method_name == :new
 
-    if method_name == :alias_method_chain
+    case method_name
+    when :new then
+      method_name = :initialize
+    when :alias_method_chain
       known[sexp[3]] << klass_name
     end
 

data/test/test_debride.rb

@@ -10,7 +10,7 @@ class TestDebride < Minitest::Test
     end
 
     exp = [["Debride",
-            [:process_call, :process_defn, :process_defs, :report, :run]]]
+            [:process_call, :process_defn, :process_defs, :process_rb, :report]]]
 
     assert_equal exp, debride.missing
   end
@@ -41,7 +41,7 @@ class TestDebride < Minitest::Test
     debride.option[:whitelist] = %w[process_defn]
 
     exp = [["Debride",
-            [:process_call, :process_defs, :report, :run]]]
+            [:process_call, :process_defs, :process_rb, :report]]]
 
     assert_equal exp, debride.missing
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: debride
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Ryan Davis
@@ -29,7 +29,7 @@ cert_chain:
   xJcC6UN6NHMOVMyAXsr2HR0gRRx4ofN1LoP2KhXzSr8UMvQYlwPmE0N5GQv1b5AO
   VpzF30vNaJK6ZT7xlIsIlwmH
   -----END CERTIFICATE-----
-date: 2015-03-18 00:00:00.000000000 Z
+date: 2015-03-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sexp_processor