debride 1.15.0 → 1.15.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 77baff9dbc4bab278d0e6b6fe16772ecdce26b00c4c7b24469e0680f0d608c41
-  data.tar.gz: 885b6274f63f3d76c6a106543d28be50f177fcc6fa6adad7cf642c0b3ef7f0d8
+  metadata.gz: a2a172fbacd06e56a051cb02614afd1abe8f6d58a89e2fbf316e981b7d984ad2
+  data.tar.gz: ad77390a37d25f0d3af9d7c8b399151ad653321117020b029b15cd1e587b94eb
 SHA512:
-  metadata.gz: 421c0cb4b4aa45ed2e7897254549def50af4f4491a029676bcc7671aa026e8cda45b9774073eef8d3bffa15665f7df0466e1b6a68f236fc0b0c2ee12806461a5
-  data.tar.gz: 2cc4e04095596764513090fffe356835d3af36c192395dbe7753470801e623f826bf72c54371de30bf1d9006d4cf109809cffc1419e887607914623cc8a1bb82
+  metadata.gz: 03610f92787f0dc2e44af469f11582a6a2f4a6463e210b0c79e01c9294c89fe6dd37c2147f6ad1476c6a125a1238efb1c742ac9c29844b2fabad4e27bc60e878
+  data.tar.gz: 6c21f8d4c056aff81158115b7a12880537a96e52d40913ffea8a43ab900eed95f58c2a9c7bd200cf721ed32c981cae7b3c6d1df7f9cfbd6cda8ef4a4c418f314

data/History.rdoc

@@ -1,3 +1,17 @@
+=== 1.15.2 / 2026-04-20
+
+* 3 bug fixes:
+
+  * Fix #process_hash when there's a kwsplat in the middle. (carlos-musetti)
+  * Rescue StandardError in .run to handle wider gamut of errors gracefully.
+  * Add #process override to handle errors as well.
+
+=== 1.15.1 / 2026-01-26
+
+* 1 bug fix:
+
+  * Namespace NotRubyParser to avoid problems with bundler's horrid eager loading. (nateberkopec)
+
 === 1.15.0 / 2026-01-02
 
 * 1 minor enhancement:

data/lib/debride.rb

@@ -11,16 +11,15 @@ require "path_expander"
 require "prism"
 require "prism/translation/ruby_parser"
 
-class NotRubyParser < Prism::Translation::RubyParser # compatibility layer
-end
-
 ##
 # A static code analyzer that points out possible dead methods.
 
 class Debride < MethodBasedSexpProcessor
-  VERSION = "1.15.0" # :nodoc:
+  VERSION = "1.15.2" # :nodoc:
   PROJECT = "debride"
 
+  NotRubyParser = Class.new Prism::Translation::RubyParser # compatibility layer :nodoc:
+
   def self.load_plugins proj = PROJECT
     unless defined? @@plugins then
       @@plugins = []
@@ -83,12 +82,18 @@ class Debride < MethodBasedSexpProcessor
 
       begin
         process send(msg, file)
-      rescue RuntimeError, SyntaxError => e
+      rescue StandardError, SyntaxError => e
         warn "  skipping #{file}: #{e.message}"
       end
     end
   end
 
+  def process exp # :nodoc:
+    super
+  rescue => e # override to capture these but move on
+    warn "  skipping: #{e.message}"
+  end
+
   def process_rb path_or_io
     warn "Processing ruby: #{path_or_io}" if option[:verbose]
 
@@ -486,6 +491,9 @@ class Debride < MethodBasedSexpProcessor
   def process_hash sexp
     _, *pairs = sexp
 
+    # pad kwsplat to survive each_slice 2
+    pairs = pairs.flat_map { |s| s && s.sexp_type == :kwsplat ? [s, nil] : [s] }
+
     pairs.each_slice 2 do |k, v|
       if v.nil? && k.sexp_type == :lit then
         called << k.last
@@ -536,23 +544,24 @@ class Debride < MethodBasedSexpProcessor
   def missing_locations
     focus = option[:focus]
 
-    missing.map { |klass, meths|
-      bad = meths.map { |meth|
-        location =
-          method_locations["#{klass}##{meth}"] ||
-          method_locations["#{klass}::#{meth}"]
+    missing
+      .map { |klass, meths|
+        bad = meths.map { |meth|
+          location =
+            method_locations["#{klass}##{meth}"] ||
+            method_locations["#{klass}::#{meth}"]
 
-        if focus then
-          path = location[/(.+):\d+/, 1]
+          if focus then
+            path = location[/(.+):\d+/, 1]
 
-          next unless File.fnmatch(focus, path)
-        end
+            next unless File.fnmatch(focus, path)
+          end
 
-        [meth, location]
-      }.compact
+          [meth, location]
+        }.compact
 
-      [klass, bad]
-    }
+        [klass, bad]
+      }
       .to_h
       .reject { |k,v| v.empty? }
   end

data/test/test_debride.rb

@@ -425,6 +425,18 @@ class TestDebride < Minitest::Test
     assert_process [["Seattle", [:raining]]], ruby
   end
 
+  def test_hash__kwsplat_bug
+    ruby = <<~RUBY
+      {
+        a:,
+        **b,
+        c:,
+      }
+    RUBY
+
+    assert_process [], ruby
+  end
+
   def test_safe_navigation_operator
     ruby = <<-RUBY.strip
       class Seattle

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: debride
 version: !ruby/object:Gem::Version
-  version: 1.15.0
+  version: 1.15.2
 platform: ruby
 authors:
 - Ryan Davis
@@ -9,9 +9,9 @@ bindir: bin
 cert_chain:
 - |
   -----BEGIN CERTIFICATE-----
-  MIIDPjCCAiagAwIBAgIBCTANBgkqhkiG9w0BAQsFADBFMRMwEQYDVQQDDApyeWFu
+  MIIDPjCCAiagAwIBAgIBCjANBgkqhkiG9w0BAQsFADBFMRMwEQYDVQQDDApyeWFu
   ZC1ydWJ5MRkwFwYKCZImiZPyLGQBGRYJemVuc3BpZGVyMRMwEQYKCZImiZPyLGQB
-  GRYDY29tMB4XDTI1MDEwNjIzMjcwMVoXDTI2MDEwNjIzMjcwMVowRTETMBEGA1UE
+  GRYDY29tMB4XDTI2MDEwNzAxMDkxNFoXDTI3MDEwNzAxMDkxNFowRTETMBEGA1UE
   AwwKcnlhbmQtcnVieTEZMBcGCgmSJomT8ixkARkWCXplbnNwaWRlcjETMBEGCgmS
   JomT8ixkARkWA2NvbTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBALda
   b9DCgK+627gPJkB6XfjZ1itoOQvpqH1EXScSaba9/S2VF22VYQbXU1xQXL/WzCkx
@@ -21,12 +21,12 @@ cert_chain:
   qhtV7HJxNKuPj/JFH0D2cswvzznE/a5FOYO68g+YCuFi5L8wZuuM8zzdwjrWHqSV
   gBEfoTEGr7Zii72cx+sCAwEAAaM5MDcwCQYDVR0TBAIwADALBgNVHQ8EBAMCBLAw
   HQYDVR0OBBYEFEfFe9md/r/tj/Wmwpy+MI8d9k/hMA0GCSqGSIb3DQEBCwUAA4IB
-  AQAC0WQJcPOWPFwkojhzweilRVjTJ19UiLhiBTw3C1wJO3LVdBkWDmnnhAmKuX4D
-  r7vjQvESlABGIPdutI1Yl7mrHQzTkfLfXvNN6MT0nLChPyIYauT6SZZxubwJrUfA
-  7R0c2CJTIboZ0XaGpLsXqHEF1c29H7TV1QvVuqKAN2mCjh4N82QVn+ZKtys28AwT
-  6GfQX2fqLoi4KSc7xIzHKaNzqxeOICmJofk9w5VZ2rRN6yes8jvFYwz9HR41wdj8
-  bwfinv7Yp5fA6AysuZLhCykyfDuZVRrUp0Vb68YCKsLjJly/Theak+euNTxvHsB+
-  al9oSgPPHICMEX65qvLywitx
+  AQA/X8/PKTTc/IkYQEUL6XWtfK8fAfbuLJzmLcz6f2ZWrtBvPsYvqRuwI1bWUtil
+  2ibJEfIuSIHX6BsUTX18+hlaIussf6EWq/YkE7e2BKmgQE42vSOZMce0kCEAPbx0
+  rQtqonfWTHQ8UbQ7GqCL3gDQ0QDD2B+HUlb4uaCZ2icxqa/eOtxMvHC2tj+h0xKY
+  xL84ipM5kr0bHGf9/MRZJWcw51urueNycBXu1Xh+pEN8qBmYsFRR4SIODLClybAO
+  6cbm2PyPQgKNiqE4H+IQrDVHd9bJs1XgLElk3qoaJBWXc/5fy0J1imYb25UqmiHG
+  snGe1hrppvBRdcyEzvhfIPjI
   -----END CERTIFICATE-----
 date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
@@ -112,15 +112,29 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.5'
+        version: '4.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.5'
-description: Analyze code for potentially uncalled / dead methods, now with auto-removal.
+        version: '4.6'
+description: |-
+  Analyze code for potentially uncalled / dead methods, now with auto-removal.
+
+  == Features/Problems:
+
+  * Static analysis of code. Can be easily hooked up to a CI.
+  * As with all static analysis tools of dynamic languages, can't be 100%.
+  * Whitelisting known good methods by name or regexp.
+  * Use --rails for Rails-specific domain knowledge.
+  * Use debride_rm to brazenly remove all unused methods. BE CAREFUL.
+  * Use `debride_rails_whitelist` to generate an emperical whitelist from logs.
+  * Uses path_expander, so you can use:
+    * dir_arg -- expand a directory automatically
+    * @file_of_args -- persist arguments in a file
+    * -path_to_subtract -- ignore intersecting subsets of files/directories
 email:
 - ryand-ruby@zenspider.com
 executables:
@@ -148,6 +162,7 @@ licenses:
 - MIT
 metadata:
   homepage_uri: https://github.com/seattlerb/debride
+  documentation_uri: http://docs.seattlerb.org/debride
 rdoc_options:
 - "--main"
 - README.rdoc
@@ -166,5 +181,5 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 rubygems_version: 3.7.2
 specification_version: 4
-summary: Analyze code for potentially uncalled / dead methods, now with auto-removal.
+summary: Analyze code for potentially uncalled / dead methods, now with auto-removal
 test_files: []