Rename cop and general for deprecated statuses

Author: tuxagon

changelog/add_unprocessable_content_status_cop.md

@@ -0,0 +1 @@
+* [#1520](https://github.com/rubocop/rubocop-rails/pull/1520): New `Rails/DeprecatedHttpStatusNames` cop. ([@tuxagon][])

changelog/new_deprecated_http_status_names_cop.md

@@ -394,6 +394,14 @@ Rails/DeprecatedActiveModelErrorsMethods:
   VersionAdded: '2.14'
   VersionChanged: '2.18'
 
+Rails/DeprecatedHttpStatusNames:
+  Description: 'Use the new HTTP status names instead of deprecated ones.'
+  Enabled: pending
+  Severity: warning
+  VersionAdded: '<<next>>'
+  Include:
+    - '**/app/controllers/**/*.rb'
+
 Rails/DotSeparatedKeys:
   Description: 'Enforces the use of dot-separated keys instead of `:scope` options in `I18n` translation methods.'
   StyleGuide: 'https://rails.rubystyle.guide/#dot-separated-keys'
@@ -1232,14 +1240,6 @@ Rails/UnknownEnv:
     - test
     - production
 
-Rails/UnprocessableContentStatus:
-  Description: 'Use `:unprocessable_content` instead of `:unprocessable_entity`.'
-  Enabled: pending
-  Severity: warning
-  VersionAdded: '<<next>>'
-  Include:
-    - '**/app/controllers/**/*.rb'
-
 Rails/UnusedIgnoredColumns:
   Description: 'Remove a column that does not exist from `ignored_columns`.'
   Enabled: false

config/default.yml

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Rails
+      # Enforces the use of the current HTTP status names instead of deprecated ones.
+      #
+      # @example
+      #   # bad
+      #   render json: { error: "Invalid data" }, status: :unprocessable_entity
+      #   head :payload_too_large
+      #
+      #   # good
+      #   render json: { error: "Invalid data" }, status: :unprocessable_content
+      #   head :content_too_large
+      #
+      class DeprecatedHttpStatusNames < Base
+        extend AutoCorrector
+
+        requires_gem 'rack', '>= 3.1.0'
+
+        RESTRICT_ON_SEND = %i[render redirect_to head assert_response assert_redirected_to].freeze
+
+        DEPRECATED_STATUSES = {
+          unprocessable_entity: :unprocessable_content,
+          payload_too_large: :content_too_large
+        }.freeze
+
+        MSG = 'Use `:%<preferred>s` instead of `:%<deprecated>s`. The `:%<deprecated>s` status is deprecated.'
+
+        def_node_matcher :status_method_call, <<~PATTERN
+          {
+            (send nil? {:render :redirect_to} _ $hash)
+            (send nil? {:render :redirect_to} $hash)
+            (send nil? {:head :assert_response} $_ ...)
+            (send nil? :assert_redirected_to _ $hash ...)
+            (send nil? :assert_redirected_to $hash ...)
+          }
+        PATTERN
+
+        def_node_matcher :status_hash_value, <<~PATTERN
+          (hash <(pair (sym :status) $_) ...>)
+        PATTERN
+
+        def on_send(node)
+          status_method_call(node) do |status_node|
+            if status_node.hash_type?
+              # Handle hash arguments like { status: :unprocessable_entity }
+              status_hash_value(status_node) do |status_value|
+                find_deprecated_status_names(status_value)
+              end
+            else
+              # Handle positional arguments like head :unprocessable_entity
+              find_deprecated_status_names(status_node)
+            end
+          end
+        end
+
+        private
+
+        def find_deprecated_status_names(node)
+          if node.sym_type? && DEPRECATED_STATUSES.key?(node.value)
+            deprecated_status = node.value
+            preferred_status = DEPRECATED_STATUSES[deprecated_status]
+
+            message = format(MSG, deprecated: deprecated_status, preferred: preferred_status)
+
+            add_offense(node, message: message) do |corrector|
+              corrector.replace(node, ":#{preferred_status}")
+            end
+          elsif node.respond_to?(:children)
+            # Recursively search child nodes (handles ternary expressions, etc.)
+            node.children.each do |child|
+              find_deprecated_status_names(child) if child.is_a?(Parser::AST::Node)
+            end
+          end
+        end
+      end
+    end
+  end
+end

lib/rubocop/cop/rails/deprecated_http_status_names.rb

@@ -41,6 +41,7 @@
 require_relative 'rails/delegate'
 require_relative 'rails/delegate_allow_blank'
 require_relative 'rails/deprecated_active_model_errors_methods'
+require_relative 'rails/deprecated_http_status_names'
 require_relative 'rails/dot_separated_keys'
 require_relative 'rails/duplicate_association'
 require_relative 'rails/duplicate_scope'
@@ -136,7 +137,6 @@
 require_relative 'rails/uniq_before_pluck'
 require_relative 'rails/unique_validation_without_index'
 require_relative 'rails/unknown_env'
-require_relative 'rails/unprocessable_content_status'
 require_relative 'rails/unused_ignored_columns'
 require_relative 'rails/unused_render_content'
 require_relative 'rails/validation'