service_actor 3.7.0 → 3.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5654d6b33f83eaccbc860a7072e0f3f3fcf694e4e9ba5fdb4224ab6bd180f860
-  data.tar.gz: 755220b315eccaf564f6eeb85c234a4da1deeb24ea3e71740f7bba3cb5029135
+  metadata.gz: ca3921af29250e8b2b68925ede79f0ad223d87bfa1be362e8e50cc178999658f
+  data.tar.gz: 86f770c478e718d0c1c20f0809ade458d1687cabeac24b35cfa39d35430ea35d
 SHA512:
-  metadata.gz: 753d97cc97069242febc4212ca78c3f1149083dd928578e581c52073f556d40a6f275d08d28a28b97e1fd04841c05e78e3be3b122c2005cbd2775b7257f93b91
-  data.tar.gz: 511d62768af23e9b84aa8ae6d268901f5227caf54956cd6261c1835d518823413c43dae16d278ecf63d45737df4373dbb755b5d1e779afd4c90d7bec78848bc4
+  metadata.gz: 29cfec59fba54ea9ea8ddd4a250588513ad3132d5ecef79f6d6f43a58c5fa6aded756bc34cd6e698f233e4d584072f7f7e7342e8e3ee71cb1027463334450c7b
+  data.tar.gz: 73b264aa00a02f23513da5b3b61b5a4a2b9a5afc72bb627d929f0a27c44a5cebf01b561d7e62c17fdb5293886f1bf65a6af0b69d0bd68674c2cad19eaef92ab3

data/README.md

@@ -303,7 +303,7 @@ class BuildGreeting < Actor
   input :adjective, default: "wonderful"
   input :length_of_time, default: -> { ["day", "week", "month"].sample }
   input :article,
-        default: -> context { context.adjective =~ /^aeiou/ ? 'an' : 'a' }
+        default: -> context { context.adjective.match?(/^aeiou/) ? "an" : "a" }
 
   output :greeting
 
@@ -475,7 +475,7 @@ end
             is: [TrueClass, FalseClass],
             message: (lambda do |input_key:, expected_type:, given_type:, **|
               "Wrong type \"#{given_type}\" for \"#{input_key}\". " \
-              "Expected: \"#{expected_type}\""
+                "Expected: \"#{expected_type}\""
             end)
           }
   end

data/lib/service_actor/arguments_validator.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module ServiceActor::ArgumentsValidator
+  module_function
+
+  def validate_origin_name(name, origin:)
+    return unless ServiceActor::Result.instance_methods.include?(name.to_sym)
+
+    raise ArgumentError,
+          "#{origin} `#{name}` overrides `ServiceActor::Result` instance method"
+  end
+
+  def validate_error_class(value)
+    return if value.is_a?(Class) && value <= Exception
+
+    raise ArgumentError, "Expected #{value} to be a subclass of Exception"
+  end
+end

data/lib/service_actor/attributable.rb

@@ -7,8 +7,10 @@
 #     output :name
 #   end
 module ServiceActor::Attributable
-  def self.included(base)
-    base.extend(ClassMethods)
+  class << self
+    def included(base)
+      base.extend(ClassMethods)
+    end
   end
 
   module ClassMethods
@@ -20,14 +22,18 @@ module ServiceActor::Attributable
     end
 
     def input(name, **arguments)
+      ServiceActor::ArgumentsValidator.validate_origin_name(
+        name, origin: :input
+      )
+
       inputs[name] = arguments
 
       define_method(name) do
         result[name]
       end
 
-      # For avoid method redefined warning messages.
-      alias_method name, name if method_defined?(name)
+      # To avoid method redefined warning messages.
+      alias_method(name, name) if method_defined?(name)
 
       protected name
     end
@@ -37,17 +43,21 @@ module ServiceActor::Attributable
     end
 
     def output(name, **arguments)
+      ServiceActor::ArgumentsValidator.validate_origin_name(
+        name, origin: :output
+      )
+
       outputs[name] = arguments
 
       define_method(name) do
         result[name]
       end
+      protected name
 
-      define_method("#{name}=") do |value|
+      define_method(:"#{name}=") do |value|
         result[name] = value
       end
-
-      protected name, "#{name}="
+      protected :"#{name}="
     end
 
     def outputs

data/lib/service_actor/base.rb

@@ -3,16 +3,18 @@
 require "service_actor/support/loader"
 
 module ServiceActor::Base
-  def self.included(base)
-    # Essential mechanics
-    base.include(ServiceActor::Core)
-    base.include(ServiceActor::Configurable)
-    base.include(ServiceActor::Attributable)
-    base.include(ServiceActor::Playable)
+  class << self
+    def included(base)
+      # Essential mechanics
+      base.include(ServiceActor::Core)
+      base.include(ServiceActor::Configurable)
+      base.include(ServiceActor::Attributable)
+      base.include(ServiceActor::Playable)
 
-    # Extra concerns
-    base.include(ServiceActor::Checkable)
-    base.include(ServiceActor::Defaultable)
-    base.include(ServiceActor::Failable)
+      # Extra concerns
+      base.include(ServiceActor::Checkable)
+      base.include(ServiceActor::Defaultable)
+      base.include(ServiceActor::Failable)
+    end
   end
 end

data/lib/service_actor/checkable.rb

@@ -1,11 +1,21 @@
 # frozen_string_literal: true
 
 module ServiceActor::Checkable
-  def self.included(base)
-    base.prepend(PrependedMethods)
+  class << self
+    def included(base)
+      base.prepend(PrependedMethods)
+    end
   end
 
   module PrependedMethods
+    CHECK_CLASSES = [
+      ServiceActor::Checks::TypeCheck,
+      ServiceActor::Checks::MustCheck,
+      ServiceActor::Checks::InclusionCheck,
+      ServiceActor::Checks::NilCheck,
+    ].freeze
+    private_constant :CHECK_CLASSES
+
     def _call
       self.service_actor_argument_errors = []
 
@@ -20,7 +30,8 @@ module ServiceActor::Checkable
 
     # rubocop:disable Metrics/MethodLength
     def service_actor_checks_for(origin)
-      self.class.public_send("#{origin}s").each do |input_key, input_options|
+      check_classes = CHECK_CLASSES.select { _1.applicable_to_origin?(origin) }
+      self.class.public_send(:"#{origin}s").each do |input_key, input_options|
         input_options.each do |check_name, check_conditions|
           check_classes.each do |check_class|
             argument_errors = check_class.check(
@@ -48,15 +59,5 @@ module ServiceActor::Checkable
       raise self.class.argument_error_class,
             service_actor_argument_errors.first
     end
-
-    def check_classes
-      [
-        ServiceActor::Checks::TypeCheck,
-        ServiceActor::Checks::MustCheck,
-        ServiceActor::Checks::InclusionCheck,
-        ServiceActor::Checks::NilCheck,
-        ServiceActor::Checks::DefaultCheck
-      ]
-    end
   end
 end

data/lib/service_actor/checks/base.rb

@@ -1,6 +1,12 @@
 # frozen_string_literal: true
 
 class ServiceActor::Checks::Base
+  class << self
+    def applicable_to_origin?(_origin)
+      true
+    end
+  end
+
   def initialize
     @argument_errors = []
   end

data/lib/service_actor/checks/inclusion_check.rb

@@ -22,21 +22,23 @@ class ServiceActor::Checks::InclusionCheck < ServiceActor::Checks::Base
   DEFAULT_MESSAGE = lambda do |input_key:, actor:, inclusion_in:, value:|
     "The \"#{input_key}\" input must be included " \
       "in #{inclusion_in.inspect} on \"#{actor}\" " \
-        "instead of #{value.inspect}"
+      "instead of #{value.inspect}"
   end
 
   private_constant :DEFAULT_MESSAGE
 
-  def self.check(check_name:, input_key:, actor:, conditions:, result:, **) # rubocop:disable Metrics/ParameterLists
-    # DEPRECATED: `in` is deprecated in favor of `inclusion`.
-    return unless %i[inclusion in].include?(check_name)
+  class << self
+    def check(check_name:, input_key:, actor:, conditions:, result:, **)
+      # DEPRECATED: `in` is deprecated in favor of `inclusion`.
+      return unless %i[inclusion in].include?(check_name)
 
-    new(
-      input_key: input_key,
-      actor: actor,
-      inclusion: conditions,
-      value: result[input_key],
-    ).check
+      new(
+        input_key: input_key,
+        actor: actor,
+        inclusion: conditions,
+        value: result[input_key],
+      ).check
+    end
   end
 
   def initialize(input_key:, actor:, inclusion:, value:)
@@ -67,6 +69,7 @@ class ServiceActor::Checks::InclusionCheck < ServiceActor::Checks::Base
 
   def define_inclusion_and_message
     if @inclusion.is_a?(Hash)
+      @inclusion[:message] ||= DEFAULT_MESSAGE
       @inclusion.values_at(:in, :message)
     else
       [@inclusion, DEFAULT_MESSAGE]

data/lib/service_actor/checks/must_check.rb

@@ -33,15 +33,17 @@ class ServiceActor::Checks::MustCheck < ServiceActor::Checks::Base
 
   private_constant :DEFAULT_MESSAGE
 
-  def self.check(check_name:, input_key:, actor:, conditions:, result:, **) # rubocop:disable Metrics/ParameterLists
-    return unless check_name == :must
+  class << self
+    def check(check_name:, input_key:, actor:, conditions:, result:, **)
+      return unless check_name == :must
 
-    new(
-      input_key: input_key,
-      actor: actor,
-      nested_checks: conditions,
-      value: result[input_key],
-    ).check
+      new(
+        input_key: input_key,
+        actor: actor,
+        nested_checks: conditions,
+        value: result[input_key],
+      ).check
+    end
   end
 
   def initialize(input_key:, actor:, nested_checks:, value:)
@@ -86,6 +88,7 @@ class ServiceActor::Checks::MustCheck < ServiceActor::Checks::Base
 
   def define_check_and_message_from(nested_check_conditions)
     if nested_check_conditions.is_a?(Hash)
+      nested_check_conditions[:message] ||= DEFAULT_MESSAGE
       nested_check_conditions.values_at(:is, :message)
     else
       [nested_check_conditions, DEFAULT_MESSAGE]

data/lib/service_actor/checks/nil_check.rb

@@ -35,23 +35,25 @@ class ServiceActor::Checks::NilCheck < ServiceActor::Checks::Base
 
   private_constant :DEFAULT_MESSAGE
 
-  def self.check( # rubocop:disable Metrics/ParameterLists
-    origin:,
-    input_key:,
-    input_options:,
-    actor:,
-    conditions:,
-    result:,
-    **
-  ) # do
-    new(
-      origin: origin,
-      input_key: input_key,
-      input_options: input_options,
-      actor: actor,
-      allow_nil: conditions,
-      value: result[input_key],
-    ).check
+  class << self
+    def check(
+      origin:,
+      input_key:,
+      input_options:,
+      actor:,
+      conditions:,
+      result:,
+      **
+    ) # do
+      new(
+        origin: origin,
+        input_key: input_key,
+        input_options: input_options,
+        actor: actor,
+        allow_nil: conditions,
+        value: result[input_key],
+      ).check
+    end
   end
 
   def initialize( # rubocop:disable Metrics/ParameterLists
@@ -92,6 +94,7 @@ class ServiceActor::Checks::NilCheck < ServiceActor::Checks::Base
 
   def define_allow_nil_and_message_from(allow_nil)
     if allow_nil.is_a?(Hash)
+      allow_nil[:message] ||= DEFAULT_MESSAGE
       allow_nil.values_at(:is, :message)
     else
       [allow_nil, DEFAULT_MESSAGE]

data/lib/service_actor/checks/type_check.rb

@@ -34,24 +34,26 @@ class ServiceActor::Checks::TypeCheck < ServiceActor::Checks::Base
 
   private_constant :DEFAULT_MESSAGE
 
-  def self.check( # rubocop:disable Metrics/ParameterLists
-    check_name:,
-    origin:,
-    input_key:,
-    actor:,
-    conditions:,
-    result:,
-    **
-  ) # do
-    return unless check_name == :type
-
-    new(
-      origin: origin,
-      input_key: input_key,
-      actor: actor,
-      type_definition: conditions,
-      given_type: result[input_key],
-    ).check
+  class << self
+    def check(
+      check_name:,
+      origin:,
+      input_key:,
+      actor:,
+      conditions:,
+      result:,
+      **
+    ) # do
+      return unless check_name == :type
+
+      new(
+        origin: origin,
+        input_key: input_key,
+        actor: actor,
+        type_definition: conditions,
+        given_type: result[input_key],
+      ).check
+    end
   end
 
   def initialize(origin:, input_key:, actor:, type_definition:, given_type:)
@@ -86,6 +88,8 @@ class ServiceActor::Checks::TypeCheck < ServiceActor::Checks::Base
 
   def define_types_and_message
     if @type_definition.is_a?(Hash)
+      @type_definition[:message] ||= DEFAULT_MESSAGE
+
       @type_definition, message =
         @type_definition.values_at(:is, :message)
     else

data/lib/service_actor/configurable.rb

@@ -1,8 +1,10 @@
 # frozen_string_literal: true
 
 module ServiceActor::Configurable
-  def self.included(base)
-    base.extend(ClassMethods)
+  class << self
+    def included(base)
+      base.extend(ClassMethods)
+    end
   end
 
   module ClassMethods
@@ -15,6 +17,18 @@ module ServiceActor::Configurable
       child.failure_class = failure_class || ServiceActor::Failure
     end
 
-    attr_accessor :argument_error_class, :failure_class
+    def argument_error_class=(value)
+      ServiceActor::ArgumentsValidator.validate_error_class(value)
+
+      @argument_error_class = value
+    end
+
+    def failure_class=(value)
+      ServiceActor::ArgumentsValidator.validate_error_class(value)
+
+      @failure_class = value
+    end
+
+    attr_reader :argument_error_class, :failure_class
   end
 end

data/lib/service_actor/core.rb

@@ -1,8 +1,10 @@
 # frozen_string_literal: true
 
 module ServiceActor::Core
-  def self.included(base)
-    base.extend(ClassMethods)
+  class << self
+    def included(base)
+      base.extend(ClassMethods)
+    end
   end
 
   module ClassMethods

data/lib/service_actor/defaultable.rb

@@ -26,8 +26,10 @@
 #           }
 #   end
 module ServiceActor::Defaultable
-  def self.included(base)
-    base.prepend(PrependedMethods)
+  class << self
+    def included(base)
+      base.prepend(PrependedMethods)
+    end
   end
 
   module PrependedMethods
@@ -43,13 +45,19 @@ module ServiceActor::Defaultable
 
         default = input[:default]
 
-        if default.is_a?(Hash)
+        if default.is_a?(Hash) && default[:is]
           default_for_advanced_mode_with(result, key, default)
         else
           default_for_normal_mode_with(result, key, default)
         end
       end
 
+      self.class.outputs.each do |key, options|
+        unless result.key?(key)
+          result.send(:"#{key}=", options[:default])
+        end
+      end
+
       super
     end
 

data/lib/service_actor/failable.rb

@@ -7,9 +7,11 @@
 #     fail_on ServiceActor::ArgumentError
 #   end
 module ServiceActor::Failable
-  def self.included(base)
-    base.extend(ClassMethods)
-    base.prepend(PrependedMethods)
+  class << self
+    def included(base)
+      base.extend(ClassMethods)
+      base.prepend(PrependedMethods)
+    end
   end
 
   module ClassMethods
@@ -20,6 +22,10 @@ module ServiceActor::Failable
     end
 
     def fail_on(*exceptions)
+      exceptions.each do |exception|
+        ServiceActor::ArgumentsValidator.validate_error_class(exception)
+      end
+
       fail_ons.push(*exceptions)
     end
 

data/lib/service_actor/playable.rb

@@ -9,9 +9,11 @@
 #          SendWelcomeEmail
 #   end
 module ServiceActor::Playable
-  def self.included(base)
-    base.extend(ClassMethods)
-    base.prepend(PrependedMethods)
+  class << self
+    def included(base)
+      base.extend(ClassMethods)
+      base.prepend(PrependedMethods)
+    end
   end
 
   module ClassMethods
@@ -20,6 +22,12 @@ module ServiceActor::Playable
     end
 
     def alias_input(**options)
+      options.each_key do |new|
+        ServiceActor::ArgumentsValidator.validate_origin_name(
+          new, origin: :alias
+        )
+      end
+
       lambda do |actor|
         options.each do |new, original|
           define_alias_input(actor, new, original)
@@ -39,17 +47,8 @@ module ServiceActor::Playable
 
     private
 
-    def define_alias_input(actor, new, original)
-      actor[new] = actor[original]
-      actor.instance_exec do
-        [original, new].each do |method|
-          singleton_class.send(:undef_method, "#{method}=")
-          define_singleton_method("#{method}=") do |v|
-            actor[original] = v
-            actor[new] = v
-          end
-        end
-      end
+    def define_alias_input(actor, new_input, original_input)
+      actor[new_input] = actor.delete!(original_input)
     end
   end
 
@@ -113,7 +112,7 @@ module ServiceActor::Playable
       return unless actor.is_a?(Class)
       return unless actor.ancestors.map(&:name).include?("Interactor")
 
-      result.merge!(actor.call(result).to_h)
+      result.merge!(actor.call(result.to_h).to_h)
     end
   end
 end

data/lib/service_actor/result.rb

@@ -1,30 +1,57 @@
 # frozen_string_literal: true
 
-require "ostruct"
-
 # Represents the context of an actor, holding the data from both its inputs
 # and outputs.
-class ServiceActor::Result < OpenStruct
-  def self.to_result(data)
-    return data if data.is_a?(self)
+class ServiceActor::Result < BasicObject
+  class << self
+    def to_result(data)
+      return data if data.is_a?(self)
+
+      new(data.to_h)
+    end
+  end
+
+  %i[
+    class
+    is_a?
+    kind_of?
+    send
+    public_send
+    tap
+    then
+    block_given?
+    object_id
+    instance_variables
+  ].each do |method_name|
+    define_method(method_name, ::Kernel.instance_method(method_name))
+  end
 
-    new(data.to_h)
+  alias_method :yield_self, :then
+
+  def initialize(data = {})
+    @data = data.to_h
+  end
+
+  def to_h
+    data
   end
 
   def inspect
     "<#{self.class.name} #{to_h}>"
   end
 
+  alias_method :pretty_print, :inspect
+
   def fail!(failure_class = nil, result = {})
-    if failure_class.nil? || failure_class.is_a?(Hash)
+    if failure_class.nil? || failure_class.is_a?(::Hash)
       result = failure_class.to_h
-      failure_class = ServiceActor::Failure
+      failure_class = ::ServiceActor::Failure
     end
 
-    merge!(result)
-    merge!(failure?: true)
+    data.merge!(result)
+    data[:failure] = true
 
-    raise failure_class, self
+    ::Kernel.raise failure_class, self
   end
 
   def success?
@@ -32,13 +59,15 @@ class ServiceActor::Result < OpenStruct
   end
 
   def failure?
-    self[:failure?] || false
+    data[:failure] || false
+  end
+
+  def error
+    data[:error] || nil
   end
 
   def merge!(result)
-    result.each_pair do |key, value|
-      self[key] = value
-    end
+    data.merge!(result)
 
     self
   end
@@ -48,34 +77,63 @@ class ServiceActor::Result < OpenStruct
   end
 
   def [](name)
-    to_h[name]
+    data[name]
   end
 
-  # Defined here to override the method on `Object`.
-  def display
-    to_h.fetch(:display)
+  def []=(key, value)
+    data[key] = value
   end
 
-  private
+  def delete!(key)
+    data.delete(key)
+  end
+
+  def respond_to?(method_name, include_private = false)
+    self.class.instance_methods.include?(method_name) ||
+      respond_to_missing?(method_name, include_private)
+  end
+
+  def deconstruct_keys(keys)
+    deconstructed_keys = to_h.merge(success: success?, failure: failure?)
 
-  def respond_to_missing?(method_name, include_private = false)
-    method_name.to_s.end_with?("?") || super
+    keys ? deconstructed_keys.slice(*keys) : deconstructed_keys
   end
 
-  def method_missing(symbol, *args)
-    attribute = symbol.to_s.chomp("?")
+  private
+
+  attr_reader :data
+
+  def respond_to_missing?(method_name, _include_private = false)
+    return true if method_name.end_with?("=")
+    if method_name.end_with?("?") &&
+        data.key?(method_name.to_s.chomp("?").to_sym)
+      return true
+    end
 
-    if symbol.to_s.end_with?("?") && respond_to?(attribute)
-      define_singleton_method symbol do
-        value = send(attribute.to_sym)
+    return true if data.key?(method_name)
 
-        # Same as ActiveSupport’s #present?
-        value.respond_to?(:empty?) ? !value.empty? : !!value
-      end
+    false
+  end
 
-      return send(symbol)
+  def method_missing(method_name, *args) # rubocop:disable Metrics/AbcSize
+    if method_name.end_with?("?") &&
+        data.key?(key = method_name.to_s.chomp("?").to_sym)
+      value = data[key]
+      value.respond_to?(:empty?) ? !value.empty? : !!value
+    elsif method_name.end_with?("=")
+      data[method_name.to_s.chomp("=").to_sym] = args.first
+    elsif data.key?(method_name)
+      data[method_name]
+    else
+      warn_on_undefined_method_invocation(method_name)
     end
+  end
 
-    super symbol, *args
+  def warn_on_undefined_method_invocation(message)
+    ::Kernel.warn(
+      "DEPRECATED: Invoking undefined methods on `ServiceActor::Result` will " \
+        "lead to runtime errors in the next major release of Actor. " \
+        "Invoked method: `#{message}`",
+    )
   end
 end

data/lib/service_actor/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ServiceActor
-  VERSION = "3.7.0"
+  VERSION = "3.8.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: service_actor
 version: !ruby/object:Gem::Version
-  version: 3.7.0
+  version: 3.8.1
 platform: ruby
 authors:
 - Sunny Ripert
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-05-29 00:00:00.000000000 Z
+date: 2024-03-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: zeitwerk
@@ -16,146 +16,182 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.12'
 - !ruby/object:Gem::Dependency
   name: rubocop-lts
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: '18.2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '18.2'
+- !ruby/object:Gem::Dependency
+  name: standard-rubocop-lts
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 12.0.1
+        version: 1.0.10
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: 1.0.10
+- !ruby/object:Gem::Dependency
+  name: rubocop-gradual
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 12.0.1
+        version: '0.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.3'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.1'
 - !ruby/object:Gem::Dependency
   name: code-scanning-rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.6'
 - !ruby/object:Gem::Dependency
   name: interactor
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.0'
 description: Service objects for your application logic
 email:
 - sunny@sunfox.org
@@ -169,11 +205,11 @@ files:
 - README.md
 - lib/service_actor.rb
 - lib/service_actor/argument_error.rb
+- lib/service_actor/arguments_validator.rb
 - lib/service_actor/attributable.rb
 - lib/service_actor/base.rb
 - lib/service_actor/checkable.rb
 - lib/service_actor/checks/base.rb
-- lib/service_actor/checks/default_check.rb
 - lib/service_actor/checks/inclusion_check.rb
 - lib/service_actor/checks/must_check.rb
 - lib/service_actor/checks/nil_check.rb
@@ -204,7 +240,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.4'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="

data/lib/service_actor/checks/default_check.rb

@@ -1,90 +0,0 @@
-# frozen_string_literal: true
-
-# Adds the `default:` option to inputs. Accepts regular values and lambdas.
-# If no default is set and the value has not been given, raises an error.
-#
-# Example:
-#
-#   class MultiplyThing < Actor
-#     input :counter, default: 1
-#     input :multiplier, default: -> { rand(1..10) }
-#   end
-#
-#   class MultiplyThing < Actor
-#     input :counter,
-#           default: {
-#             is: 1,
-#             message: "Counter is required"
-#           }
-#
-#     input :multiplier,
-#           default: {
-#             is: -> { rand(1..10) },
-#             message: (lambda do |input_key:, actor:|
-#               "Input \"#{input_key}\" is required"
-#             end)
-#           }
-#   end
-class ServiceActor::Checks::DefaultCheck < ServiceActor::Checks::Base
-  def self.check(result:, input_key:, input_options:, actor:, **)
-    new(
-      result: result,
-      input_key: input_key,
-      input_options: input_options,
-      actor: actor,
-    ).check
-  end
-
-  def initialize(result:, input_key:, input_options:, actor:)
-    super()
-
-    @result = result
-    @input_key = input_key
-    @input_options = input_options
-    @actor = actor
-  end
-
-  def check # rubocop:disable Metrics/MethodLength
-    return if @result.key?(@input_key)
-
-    unless @input_options.key?(:default)
-      return add_argument_error(
-        "The \"#{@input_key}\" input on \"#{@actor}\" is missing",
-      )
-    end
-
-    default = @input_options[:default]
-
-    if default.is_a?(Hash)
-      default_for_advanced_mode_with(default)
-    else
-      default_for_normal_mode_with(default)
-    end
-
-    nil
-  end
-
-  private
-
-  def default_for_normal_mode_with(default)
-    default = default.call if default.is_a?(Proc)
-    @result[@input_key] = default
-  end
-
-  def default_for_advanced_mode_with(content)
-    default, message = content.values_at(:is, :message)
-
-    unless default
-      return add_argument_error(
-        message,
-        input_key: @input_key,
-        actor: self.class,
-      )
-    end
-
-    default = default.call if default.is_a?(Proc)
-    @result[@input_key] = default
-
-    message.call(@input_key, self.class)
-  end
-end