dead_code_detector 0.0.10 → 0.0.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 60644539c351c9c6fba75e96f9a0e0b1509d9d2fb5b392613f5ee396d01fc170
-  data.tar.gz: b56f91fc57521d3405db882b51b2051972132e8af36663e7eb468a75e0633c00
+  metadata.gz: b1e25ae55d24dbc0c957f819cbe68bb4db71d71fba805505b3a72db1d8446426
+  data.tar.gz: 46ab5c442af8cfbe750e7fd596fd0f4816d417ccfb424f28f19add022f0a1f53
 SHA512:
-  metadata.gz: b18b4fe4c42e4fd445faf30afd2a74110d819b3518f23f7d7ad3803ca6f72bd58edaf4d02abef4991d1f2a171b0a808e15a3edb4a7cd4ac3521e70e9ef8bc343
-  data.tar.gz: 8ddce2288950a82afe6674151918e2c130eb331c37f0b6e32485343b85686a36b1381f44e098f8e2611f7b008b8e925bc2c5dce3f16b64a6100777db77cb3fe0
+  metadata.gz: 0ba9ee6774898c90413e24aacccc82bb40be0a6a1af54018f2c1ba0f99a6bb8df02b4c7f568745feaeef88dcad17fb5af42fb3a30bfbdcf09575d1ca92f286f1
+  data.tar.gz: 274a69dba37db8a83851bc4021ae9d5da98252901ae0e990466d66bf0888af1f89fc21c7ffcd80c44db1e7e5a2585b3e7e104e425a1f0093f48715b88d29028b

data/Gemfile.lock

@@ -1,14 +1,14 @@
 PATH
   remote: .
   specs:
-    dead_code_detector (0.0.10)
+    dead_code_detector (0.0.12)
 
 GEM
   remote: https://rubygems.org/
   specs:
     byebug (11.0.1)
     diff-lcs (1.3)
-    rake (10.5.0)
+    rake (13.0.1)
     rspec (3.7.0)
       rspec-core (~> 3.7.0)
       rspec-expectations (~> 3.7.0)
@@ -30,7 +30,7 @@ DEPENDENCIES
   bundler (~> 1.17)
   byebug
   dead_code_detector!
-  rake (~> 10.0)
+  rake (~> 13.0)
   rspec (~> 3.0)
 
 BUNDLED WITH

data/dead_code_detector.gemspec

@@ -33,7 +33,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_development_dependency "bundler", "~> 1.17"
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "rake", "~> 13.0"
   spec.add_development_dependency "rspec", "~> 3.0"
   spec.add_development_dependency "byebug"
 end

data/lib/dead_code_detector/base_method_wrapper.rb

@@ -33,7 +33,9 @@ module DeadCodeDetector
 
     def refresh_cache
       clear_cache
-      DeadCodeDetector.config.storage.add(self.class.record_key(klass.name), default_methods)
+      if default_methods.any?
+        DeadCodeDetector.config.storage.add(self.class.record_key(klass.name), default_methods)
+      end
     end
 
     private

data/lib/dead_code_detector/class_method_wrapper.rb

@@ -24,7 +24,7 @@ module DeadCodeDetector
 
     def wrap_method(original_method)
       original_class = klass
-      klass.define_singleton_method(original_method.name) do |*args, &block|
+      klass.define_singleton_method(original_method.name) do |*args, **kwargs, &block|
         begin
           DeadCodeDetector::ClassMethodWrapper.unwrap_method(original_class, original_method)
         rescue StandardError => e
@@ -39,7 +39,7 @@ module DeadCodeDetector
         # is calling the method
         unbound_method = original_method.unbind
         method_bound_to_caller = unbound_method.bind(self)
-        method_bound_to_caller.call(*args, &block)
+        method_bound_to_caller.call(*args, **kwargs, &block)
       end
     end
 
@@ -53,12 +53,17 @@ module DeadCodeDetector
       return true if DeadCodeDetector.config.ignore_paths.nil?
       source_location = klass.method(method_name).source_location&.first
       return false if source_location.nil?
+      return false if source_location == "(eval)"
       source_location !~ DeadCodeDetector.config.ignore_paths
     end
 
     def owned_method?(method_name)
       original_method = klass.method(method_name)
-      klass.singleton_class <= original_method.owner && !(klass.superclass.singleton_class <= original_method.owner)
+      if klass.respond_to?(:superclass)
+        klass.singleton_class <= original_method.owner && !(klass.superclass.singleton_class <= original_method.owner)
+      else
+        klass.singleton_class <= original_method.owner
+      end
     end
 
   end

data/lib/dead_code_detector/configuration.rb

@@ -1,7 +1,7 @@
 module DeadCodeDetector
   class Configuration
 
-    attr_accessor :redis, :classes_to_monitor, :error_handler, :allowed, :cache_expiry, :ignore_paths
+    attr_accessor :redis, :classes_to_monitor, :error_handler, :allowed, :cache_expiry, :ignore_paths, :max_seconds_to_enable
 
     STORAGE_BACKENDS = {
       memory: Storage::MemoryBackend,
@@ -12,6 +12,7 @@ module DeadCodeDetector
       @allowed = true
       @classes_to_monitor = []
       @cache_expiry = 60 * 60 * 24 * 14
+      @max_seconds_to_enable = 1
     end
 
     def storage=(backend_type)

data/lib/dead_code_detector/initializer.rb

@@ -3,6 +3,8 @@ module DeadCodeDetector
 
     class << self
 
+      attr_accessor :fully_enabled, :last_enabled_class
+
       def refresh_caches
         DeadCodeDetector.config.classes_to_monitor.each do |klass|
           refresh_cache_for(klass)
@@ -10,7 +12,8 @@ module DeadCodeDetector
       end
 
       def refresh_cache_for(klass)
-        @enabled = false
+        self.fully_enabled = false
+        self.last_enabled_class = nil
         classes = [klass, *descendants_of(klass)]
         classes.each do |class_to_enable|
           cache_methods_for(class_to_enable)
@@ -34,13 +37,18 @@ module DeadCodeDetector
       end
 
       def enable_for_cached_classes!
-        return if @enabled
+        return if fully_enabled
         return unless allowed?
-        @enabled = true
-        cached_classes.each do |class_name|
+        classes = cached_classes.sort.to_a
+        starting_index = (classes.index(last_enabled_class) || -1) + 1
+        start_time = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+        classes[starting_index..-1].each do |class_name|
           klass = Object.const_get(class_name) rescue nil
           enable(klass) if klass
+          self.last_enabled_class = class_name
+          return if Process.clock_gettime(Process::CLOCK_MONOTONIC) - start_time > DeadCodeDetector.config.max_seconds_to_enable
         end
+        self.fully_enabled = true
       end
 
       def allowed?
@@ -57,7 +65,9 @@ module DeadCodeDetector
 
       private
       def descendants_of(parent_class)
-        ObjectSpace.each_object(parent_class.singleton_class).select { |klass| klass < parent_class }
+        ObjectSpace.each_object(parent_class.singleton_class).select do |klass|
+          klass < parent_class && !klass.name.nil?
+        end
       end
 
       def cache_methods_for(klass)

data/lib/dead_code_detector/instance_method_wrapper.rb

@@ -24,7 +24,7 @@ module DeadCodeDetector
 
     def wrap_method(original_method)
       original_class = klass
-      klass.send(:define_method, original_method.name) do |*args, &block|
+      klass.send(:define_method, original_method.name) do |*args, **kwargs, &block|
         begin
           DeadCodeDetector::InstanceMethodWrapper.unwrap_method(original_class, original_method)
         rescue StandardError => e
@@ -33,7 +33,7 @@ module DeadCodeDetector
           end
         end
         method_bound_to_caller = original_method.bind(self)
-        method_bound_to_caller.call(*args, &block)
+        method_bound_to_caller.call(*args, **kwargs, &block)
       end
     end
 
@@ -47,12 +47,17 @@ module DeadCodeDetector
       return true if DeadCodeDetector.config.ignore_paths.nil?
       source_location = klass.instance_method(method_name).source_location&.first
       return false if source_location.nil?
+      return false if source_location == "(eval)"
       source_location !~ DeadCodeDetector.config.ignore_paths
     end
 
     def owned_method?(method_name)
       original_method = klass.instance_method(method_name)
-      klass <= original_method.owner && !(klass.superclass <= original_method.owner)
+      if klass.respond_to?(:superclass)
+        klass <= original_method.owner && !(klass.superclass <= original_method.owner)
+      else
+        klass <= original_method.owner
+      end
     end
 
   end

data/lib/dead_code_detector/version.rb

@@ -1,3 +1,3 @@
 module DeadCodeDetector
-  VERSION = "0.0.10"
+  VERSION = "0.0.12"
 end

data/overlord.yml

@@ -0,0 +1 @@
+tier: 4

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dead_code_detector
 version: !ruby/object:Gem::Version
-  version: 0.0.10
+  version: 0.0.12
 platform: ruby
 authors:
 - Kyle d'Oliveira
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2019-05-15 00:00:00.000000000 Z
+date: 2023-09-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -96,13 +96,14 @@ files:
 - lib/dead_code_detector/storage/memory_backend.rb
 - lib/dead_code_detector/storage/redis_backend.rb
 - lib/dead_code_detector/version.rb
+- overlord.yml
 homepage: https://github.com/clio/dead_code_detector
 licenses:
 - MIT
 metadata:
   homepage_uri: https://github.com/clio/dead_code_detector
   source_code_uri: https://github.com/clio/dead_code_detector
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -117,8 +118,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.33
+signing_key:
 specification_version: 4
 summary: Help find unused code in production.
 test_files: []