deprecool 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 959b5cdb23ec0d5eef80f87185fb7a31a5646cf991cced0030d4d839fb835906
-  data.tar.gz: 87eac39b80c35af2e3bc868ab99b03d534f46f06e0f6ef6945381e5d00e8798b
+  metadata.gz: eca3a279e6790babeed0e659c1093f1b1a9086d9695a637460dda7cfa69d7182
+  data.tar.gz: a9a735cbd4232f568e002c643370c42491bad699a4ee1cfdd8eaae8e8493c723
 SHA512:
-  metadata.gz: d6da4e681409697beeaad714e669ad6854000a66a9086a8bf593ded61016e3741beb5147d616a42d99c3e19b3d51ad886f432d056ba58db2e56a559b0188ae87
-  data.tar.gz: cd41a34a6787fb74f7985e9180893d46a765a19d369b90c09d23738a378ee645bce9d02109fda4e7806195f82bc94f9b0d2ac300ec6c69bf6d0ccb65c9e41e9a
+  metadata.gz: e8ce7b248608aecf4281f836bedc94c5cc6b493a9581145b22d01a9fee87266a304ce389193841dba4c5e1e5e0029af6fb63397b61e7ca877b4bf19e81b974db
+  data.tar.gz: 37d39d356843be3105baff39d2f4021b48f9edbf31816b241e715a43f19d489d0b72f3f5abe04de8e4c23749409158c5b5d05470608f3788b618c907faf6c572

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Deprecool
 
+## [0.1.3] - 2026-08-24
+ - Added Finder for: "Passing `binds` as a positional argument to `insert` deprecated", see [original commit](https://github.com/rails/rails/commit/2dca5457ab2097626481fdbec233ea56d3fb9ee3)
+ - Added `effort` to the CLI output
+
 ## [0.1.2] - 2026-08-14
 
 - Add Finder for Rails's deprecation of `write_attribute(:id, value)`

data/lib/deprecool/cli.rb

@@ -24,7 +24,7 @@ module Deprecool
         option :gems, type: :array, desc: 'Specify gem name(s) to list which versions have deprecation tracking available'
 
         def call(gems:)
-          CLI.list_finders(Finder.registry)
+          CLI.list_finders(Finder.subclasses)
 
           # TODO: REPL to filter through the available finders?
         end
@@ -116,7 +116,7 @@ module Deprecool
       def text_report(offenses, finders)
         if offenses.empty?
           puts colorize('No deprecations found.', :green)
-          puts "(#{finders.size} finder#{'s' unless finders.size == 1} active)"
+          puts "(#{finders.size} finder(s) active)"
           return
         end
 
@@ -125,10 +125,13 @@ module Deprecool
         offenses.each_value do |offense_array|
           offense = offense_array.first
 
-          puts "\n#{colorize(offense.title, :bold)}\n"
+          puts "\n#{colorize(offense.title, :bold)}"
+          puts
           puts " * #{offense.summary}"
-          puts "  #{colorize('fix:', :green)} #{offense.suggestion}" if offense.suggestion
-          puts "  #{colorize('source:', :green)} #{offense.reference}" if offense.reference
+          puts
+          puts "  #{colorize('effort:', :green).ljust(16)} #{colorize_effort(offense.effort)}"
+          puts "  #{colorize('fix:', :green).ljust(16)} #{offense.suggestion}"   if offense.suggestion
+          puts "  #{colorize('source(s):', :green).ljust(16)} #{offense.reference}" if offense.reference
           puts '  Found At:'
           offense_array.each do |o|
             offense_count   += 1
@@ -158,6 +161,17 @@ module Deprecool
 
         "\e[#{colors.fetch(color, nil)}m#{text}\e[0m"
       end
+
+      def colorize_effort(effort)
+        # the effort method raises if you set anything but these
+        # values so it's safe to assume, it would also be a hassle
+        # to remove duplication here so oh well
+        effort_colors = { low:    :green,
+                          medium: :yellow,
+                          high: :red }
+
+        colorize(effort, effort_colors[effort])
+      end
     end
   end
 end

data/lib/deprecool/finder.rb

@@ -31,17 +31,7 @@ module Deprecool
   class Finder
     include PrismHelpers
 
-    @registry = []
-
     class << self
-      # All finder subclasses, in definition order.
-      attr_reader :registry
-
-      def inherited(subclass)
-        Finder.registry << subclass
-        super
-      end
-
       # defines class methods that set instance variables
       %i[gem title summary suggestion reference].each do |attribute|
         define_method(attribute) do |value = (getter = true)|

data/lib/deprecool/finders/rails/v8.2.0/no_binds_to_insert_update_delete.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+module Deprecool
+  module Finders
+    module Rails
+      module V8_2_0
+        class InsertUpdateDeleteBinds < Deprecool::Finder
+          gem           :rails
+          deprecated_in '8.2.0'
+          removed_in    '9.0.0'
+          title         'Binds being bassed to Insert Update Delete is deprecated'
+          summary       ' Now that `Arel.sql(sql_with_placeholders, *binds)` wraps SQL and its binds' \
+                        'together as an `Arel::Nodes::BoundSqlLiteral`, just like ' \
+                        '`Model.where("... = ?", value)`, the separate positional is no longer needed'
+          suggestion    'Old: `connection.insert("INSERT INTO topics (title) VALUES (?)", nil, nil, ["hello"])` ' \
+                        'New: `connection.insert(Arel.sql("INSERT INTO topics (title) VALUES (?)", "hello"))`'
+          reference     'Original Commit: https://github.com/rails/rails/commit/2dca5457ab2097626481fdbec233ea56d3fb9ee3'
+          effort        :low
+
+          def on_call_node(node)
+            return unless %i[insert update delete].include?(node.name)
+            return unless arguments = unwrap_arguments(node.arguments)
+            return unless arguments.length > 1
+
+            confidence = confidence_from_arguments(arguments)
+            return if confidence == :none
+
+            add_offense(node, confidence: confidence)
+          end
+
+          private
+
+          def confidence_from_arguments(arguments)
+            first = arguments[0]
+
+            # if first argument is string node that starts with "INSERT" "UPDATE" "DELETE" then :high
+            # if first argument is a variable then :low confidence
+            # it it's a Prism::CallNode, and not a variable call, then I don't know what it is here,
+            # maybe it's like 'connection.update(generate_sql(some_arg), ["name"])' and if you are doing that
+            # then good luck?
+            if first.is_a?(Prism::CallNode)
+              return :low if first.variable_call? # can't tell what's stored in the variable, https://docs.ruby-lang.org/en/master/Prism/CallNode.html#method-i-variable_call-3F
+              :none
+            elsif %w[INSERT UPDATE DELETE].include?(first.unescaped[..5].upcase)
+              :high
+            else
+              :none
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/deprecool/finders/ruby/v4.0.0/object_space_id2ref.rb

@@ -27,9 +27,9 @@ module Deprecool
             # I don't think we need much more than this
             # I'm open to be wrong though, maybe lots of people are subclassing
             # ObjectSpace or defining '_id2ref' on custom classes
-            confidence = node.receiver.name == :ObjectSpace ? :high : nil
+            confidence = node.receiver.name == :ObjectSpace ? :high : :none
 
-            return unless confidence
+            return if confidence == :none
 
             add_offense node, confidence: confidence
           end

data/lib/deprecool/finders/ruby/v4.0.0/to_set_arguments.rb

@@ -11,8 +11,8 @@ module Deprecool
           title         'Passing arguments to #to_set is deprecated'
           summary       'Since Ruby 4.0, passing arguments to Set#to_set / Enumerable#to_set ' \
                         'is deprecated and will be removed.'
-          suggestion    'Call #to_set with no arguments. If you were building a ' \
-                        'Set subclass, construct it explicitly instead.'
+          suggestion    'Call #to_set with no arguments. If you want an instance of a subclass of Set' \
+                        'from an enumerable construct it explicitly instead: MySetSubclass.new(enumerable)'
           reference     'https://bugs.ruby-lang.org/issues/21390 https://github.com/ruby/ruby/pull/13489'
           effort        :low
 
@@ -21,7 +21,7 @@ module Deprecool
             return unless node.arguments
 
             confidence = confidence_from_receiver_node(node.receiver)
-            return unless confidence
+            return if confidence == :none
 
             add_offense(node, confidence:)
           end
@@ -35,12 +35,12 @@ module Deprecool
             # a plain `to_set(x)` call to the current scope's own
             # method. This is probably not Enumerable#to_set unless someone has
             # monkeypatched Enumerable and is using to_set with an argument, I guess
-            when nil then nil
+            when nil then :none
             when Prism::ConstantReadNode, Prism::ConstantPathNode
               # These nodes can constants like:
               # MY_DATA.to_set(arg) => likely an array, but can't be sure so :low
               # MyClass.to_set(arg) => has lowercase letters so it's a class method
-              receiver&.name&.match?(/[a-z]/) ? nil : :low
+              receiver&.name&.match?(/[a-z]/) ? :none : :low
             else
               # this branch is when the receiver is some
               # local var, instance variable, method chain, safe navigation, etc.

data/lib/deprecool/registry.rb

@@ -7,7 +7,7 @@ module Deprecool
 
     # All the subclasses of Finder
     def all_finders
-      Finder.registry
+      @all ||= Finder.subclasses
     end
 
     # Gets finders that apply to the given versions.

data/lib/deprecool/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Deprecool
-  VERSION = '0.1.2'
+  VERSION = '0.1.3'
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: deprecool
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Zac Radford
@@ -56,6 +56,7 @@ files:
 - lib/deprecool/cli.rb
 - lib/deprecool/finder.rb
 - lib/deprecool/finders/rails/v7.1.0/serializer_positional_class_argument.rb
+- lib/deprecool/finders/rails/v8.2.0/no_binds_to_insert_update_delete.rb
 - lib/deprecool/finders/rails/v8.2.0/redis_cache_store_default_redis_options.rb
 - lib/deprecool/finders/rails/v8.2.0/to_sql_binds.rb
 - lib/deprecool/finders/rails/v8.2.0/write_attribute_id.rb