refactor: inline the one-use redrive helpers

`active_scope_for` wrapped one interpolation and `batch_size` wrapped
one calculation, each with a single caller.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: cardmagic

lib/solid_objects/redrive_manager.rb

@@ -11,7 +11,7 @@ class RedriveManager
     # @rbs (scope: DeadLetterScope, filters: Hash[String, untyped], authorization_context: untyped) -> RedriveTask
     def start(scope:, filters:, authorization_context:)
       scope.authorize!(:redrive, authorization_context:)
-      active_scope = active_scope_for(kind: scope.kind, filters:)
+      active_scope = "#{scope.kind}:#{Digest::SHA256.hexdigest(filters.to_json)}"
       running = Redrive.find_by(active_scope:)
       return task_for(running) if running
 
@@ -114,11 +114,6 @@ def remaining_for(record)
       [ matching, limit - record.moved ].min
     end
 
-    # @rbs (kind: String, filters: Hash[String, untyped]) -> String
-    def active_scope_for(kind:, filters:)
-      "#{kind}:#{Digest::SHA256.hexdigest(filters.to_json)}"
-    end
-
     # @rbs (Symbol, authorization_context: untyped, ?resource_id: String?) -> void
     def authorize!(action, authorization_context:, resource_id: nil)
       authorized = SolidObjects.configuration.authorize_administration.call(

lib/solid_objects/redrive_runner.rb

@@ -32,7 +32,9 @@ def advance(record)
     # @rbs (Redrive) -> Integer
     def move_batch(record)
       scope = DeadLetterScope.for_kind(record.kind)
-      size = batch_size(record)
+      configured = SolidObjects.configuration.redrive_batch_size
+      limit = record.move_limit
+      size = limit ? [ configured, limit - record.moved ].min : configured
       return 0 unless size.positive?
 
       identifiers = scope
@@ -47,15 +49,6 @@ def move_batch(record)
       revived
     end
 
-    # @rbs (Redrive) -> Integer
-    def batch_size(record)
-      configured = SolidObjects.configuration.redrive_batch_size
-      limit = record.move_limit
-      return configured unless limit
-
-      [ configured, limit - record.moved ].min
-    end
-
     # @rbs (Redrive) -> void
     def finish(record)
       manager = SolidObjects.redrives

sig/generated/lib/solid_objects/redrive_manager.rbs

@@ -37,9 +37,6 @@ module SolidObjects
     # @rbs (Redrive) -> Integer
     def remaining_for: (Redrive) -> Integer
 
-    # @rbs (kind: String, filters: Hash[String, untyped]) -> String
-    def active_scope_for: (kind: String, filters: Hash[String, untyped]) -> String
-
     # @rbs (Symbol, authorization_context: untyped, ?resource_id: String?) -> void
     def authorize!: (Symbol, authorization_context: untyped, ?resource_id: String?) -> void
   end

sig/generated/lib/solid_objects/redrive_runner.rbs

@@ -16,9 +16,6 @@ module SolidObjects
     # @rbs (Redrive) -> Integer
     def move_batch: (Redrive) -> Integer
 
-    # @rbs (Redrive) -> Integer
-    def batch_size: (Redrive) -> Integer
-
     # @rbs (Redrive) -> void
     def finish: (Redrive) -> void
   end