debride 1.15.1 → 1.15.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a141179f7eaccb00427a5f08718c4feb5f4cc71f088c85c9fa5ca9a2a9d11320
-  data.tar.gz: 90cf53e502fb9174beaba59f09f45c1c316c2c0179f5c35509523af2c0c49cb8
+  metadata.gz: a2a172fbacd06e56a051cb02614afd1abe8f6d58a89e2fbf316e981b7d984ad2
+  data.tar.gz: ad77390a37d25f0d3af9d7c8b399151ad653321117020b029b15cd1e587b94eb
 SHA512:
-  metadata.gz: e716a26676e9a6e3eef63dff026138910c7bd49ff4e89c0741f6be172cfe1e470a01944db3912555efc0e34a981fd6903164ee6a139795ed78e507bfa4911d7d
-  data.tar.gz: b4d27a4b57292cfd04ee94849cc42fbb281a9446ee850f5c3e049fdf4a6cc6fde04eeca89092dbf08147f52e5ba1dbf644145a62ea12eb9a95bd6557f2cc15ad
+  metadata.gz: 03610f92787f0dc2e44af469f11582a6a2f4a6463e210b0c79e01c9294c89fe6dd37c2147f6ad1476c6a125a1238efb1c742ac9c29844b2fabad4e27bc60e878
+  data.tar.gz: 6c21f8d4c056aff81158115b7a12880537a96e52d40913ffea8a43ab900eed95f58c2a9c7bd200cf721ed32c981cae7b3c6d1df7f9cfbd6cda8ef4a4c418f314

data/History.rdoc

@@ -1,3 +1,11 @@
+=== 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:

data/lib/debride.rb

@@ -15,7 +15,7 @@ require "prism/translation/ruby_parser"
 # A static code analyzer that points out possible dead methods.
 
 class Debride < MethodBasedSexpProcessor
-  VERSION = "1.15.1" # :nodoc:
+  VERSION = "1.15.2" # :nodoc:
   PROJECT = "debride"
 
   NotRubyParser = Class.new Prism::Translation::RubyParser # compatibility layer :nodoc:
@@ -82,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]
 
@@ -485,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

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.1
+  version: 1.15.2
 platform: ruby
 authors:
 - Ryan Davis
@@ -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: []

metadata.gz.sig

@@ -1,2 +1,2 @@
-9�óyO'Z�T�X@wm�'(ՏJ|-6;Ҝ��1c�Ƞ����菾E�ARG�f�4��^d+*����qc	 0C��j�Wa��<�_R49�����[�k��<V��xȴ���4e�`e�ܡue�͊�{P	N?˝F���Y%�4�	2$���Rw����Z~1�-�k/����W�]���/�ڵw;���i��u,eϗ}
-�ݎ^�o��w�۠p4L��H��C�l����$�fm��h�lK[O��
+.����B'$���4��v8������цX��/�s@�S��h������ȑ�>ow�/�5��������r����o�_�oq@e��;p��vֲ�l��9�识`Ke�}-�$�TV�I#O�.#����P5�������v���
+��pVv�@�����a	T�y�Y{���[�j�%��X�K��/S��=�2����?����(z@&;ng�h�thUI�DIr�y	E��Zr�V�+ߗ[�n�w�l�D�