cmdx 1.18.0 → 1.19.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 80c55fa1f758dc69cebff57fa5dd770b4bd7692f4a37768c320b6813d28cc9fb
-  data.tar.gz: babf14b98faa436de370d039cc2b6c796f2d0c43d9e408137ccdd1f666f4abe3
+  metadata.gz: ead1d181ed1eac565ea3287eeaaca3216e5d24a40f4071000539584435fc1327
+  data.tar.gz: ba0273c00286e10621f06fc5b0ba7be760afa99dd8d3fbfc6fc7da1c3c11891f
 SHA512:
-  metadata.gz: cbcb58370126bc18aec53218930b11a7c6a3194b4b1e1c6247ae2d6e693ef16f84cc4dbf90a7dbd816e900f421e625cac3ca227aca7ce4898fa711c1451230e0
-  data.tar.gz: de88ce41eacbf4f843803f84055d46dcfb7774aa5532396c820112a5ca3d80d68deb9dbc296f8b782d990122ed6ccc082bbc582f93d3714373c2705c55a936fe
+  metadata.gz: 30e8c09228e765f2ac8e9b4f8db854da21a13e3a2d7e67b9ad61ecad66d09dd7f8ca9327ac9a8ef8777e78d56d5844a06f52e55d282cc1d78d7ddb9141b9acc2
+  data.tar.gz: 922de5ad8342195dc94bf8137e7023612db4df09b3979581c0e77aecf599cae971004e64b9ab71261ea6795feb8ca8b7fe3fe4fc3f37a570ed50157c404aff33

data/CHANGELOG.md

@@ -6,6 +6,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 
 ## [UNRELEASED]
 
+### Changed
+- Add attribute `source` fallback to `:context` when no task is given
+- Improve falsy attribute derived Hash value lookup
+- Freeze chain results
+- Fix missing fault cause no method error issue
+- Add context respond_to? with setter methods
+- Fix validator `allow_nil` inverted logic
+- Array coercion JSON parse error no returns CoercionError
+- Boolean coercions now return `false` for `nil` and `""`
+- Coerce anaglous date, datetime, and time class checks to rely on `to_date`, `to_time`, `to_datetime` methods
+
 ## [1.18.0] - 2025-03-09
 
 ### Changed

data/lib/cmdx/attribute.rb

@@ -208,7 +208,8 @@ module CMDx
       !!@required
     end
 
-    # Determines the source of the attribute value.
+    # Determines the source of the attribute value. Returns :context
+    # as a safe fallback when task is not yet set (e.g., schema introspection).
     #
     # @return [Symbol] The source identifier for the attribute value
     #
@@ -217,13 +218,15 @@ module CMDx
     #
     # @rbs () -> untyped
     def source
-      @source ||= parent&.method_name || begin
+      return @source if defined?(@source)
+
+      @source = parent&.method_name || begin
         value = options[:source]
 
         if value.is_a?(Proc)
-          task.instance_eval(&value)
+          task ? task.instance_eval(&value) : :context
         elsif value.respond_to?(:call)
-          value.call(task)
+          task ? value.call(task) : :context
         else
           value || :context
         end
@@ -239,7 +242,9 @@ module CMDx
     #
     # @rbs () -> Symbol
     def method_name
-      @method_name ||= options[:as] || begin
+      return @method_name if defined?(@method_name)
+
+      @method_name = options[:as] || begin
         prefix = AFFIX.call(options[:prefix]) { "#{source}_" }
         suffix = AFFIX.call(options[:suffix]) { "_#{source}" }
 

data/lib/cmdx/attribute_value.rb

@@ -166,7 +166,12 @@ module CMDx
       derived_value =
         case source_value
         when Context then source_value[name]
-        when Hash then source_value[name.to_s] || source_value[name.to_sym]
+        when Hash
+          if source_value.key?(name.to_s)
+            source_value[name.to_s]
+          elsif source_value.key?(name.to_sym)
+            source_value[name.to_sym]
+          end
         when Symbol then source_value.send(name)
         when Proc then task.instance_exec(name, &source_value)
         else

data/lib/cmdx/chain.rb

@@ -136,6 +136,20 @@ module CMDx
       !!@dry_run
     end
 
+    # Freezes the chain and its internal results to prevent modifications.
+    #
+    # @return [Chain] the frozen chain
+    #
+    # @example
+    #   chain.freeze
+    #   chain.results << result # => raises FrozenError
+    #
+    # @rbs () -> self
+    def freeze
+      results.freeze
+      super
+    end
+
     # Converts the chain to a hash representation.
     #
     # @option return [String] :id The chain identifier

data/lib/cmdx/coercions/array.rb

@@ -18,7 +18,7 @@ module CMDx
       #
       # @return [Array] The converted array value
       #
-      # @raise [JSON::ParserError] If the string value contains invalid JSON
+      # @raise [CoercionError] If the value cannot be converted to an array
       #
       # @example Convert a JSON-like string to an array
       #   Array.call("[1, 2, 3]") # => [1, 2, 3]
@@ -26,6 +26,8 @@ module CMDx
       #   Array.call("hello")     # => ["hello"]
       #   Array.call(42)          # => [42]
       #   Array.call(nil)         # => []
+      # @example Handle invalid JSON-like strings
+      #   Array.call("[not json") # => raises CoercionError
       #
       # @rbs (untyped value, ?Hash[Symbol, untyped] options) -> Array[untyped]
       def call(value, options = {})
@@ -37,6 +39,9 @@ module CMDx
         else
           Array(value)
         end
+      rescue JSON::ParserError
+        type = Locale.t("cmdx.types.array")
+        raise CoercionError, Locale.t("cmdx.coercions.into_an", type:)
       end
 
     end

data/lib/cmdx/coercions/boolean.rb

@@ -34,14 +34,18 @@ module CMDx
       #   Boolean.call("false")  # => false
       #   Boolean.call("no")     # => false
       #   Boolean.call("0")      # => false
+      #   Boolean.call(nil)      # => false
+      #   Boolean.call("")       # => false
       # @example Handle case-insensitive input
       #   Boolean.call("TRUE")   # => true
       #   Boolean.call("False")  # => false
+      # @example Handle edge cases
+      #   Boolean.call("abc")    # => raises CoercionError
       #
       # @rbs (untyped value, ?Hash[Symbol, untyped] options) -> bool
       def call(value, options = {})
         case value.to_s
-        when FALSEY then false
+        when FALSEY, "" then false
         when TRUTHY then true
         else
           type = Locale.t("cmdx.types.boolean")

data/lib/cmdx/coercions/date.rb

@@ -11,11 +11,6 @@ module CMDx
 
       extend self
 
-      # Types that are already date-like and don't need conversion
-      #
-      # @rbs ANALOG_TYPES: Array[String]
-      ANALOG_TYPES = %w[Date DateTime Time].freeze
-
       # Converts a value to a Date object
       #
       # @param value [Object] The value to convert to a Date
@@ -38,7 +33,7 @@ module CMDx
       #
       # @rbs (untyped value, ?Hash[Symbol, untyped] options) -> Date
       def call(value, options = {})
-        return value if ANALOG_TYPES.include?(value.class.name)
+        return value.to_date if value.respond_to?(:to_date)
         return ::Date.strptime(value, options[:strptime]) if options[:strptime]
 
         ::Date.parse(value)

data/lib/cmdx/coercions/date_time.rb

@@ -11,11 +11,6 @@ module CMDx
 
       extend self
 
-      # Types that are already date-time-like and don't need conversion
-      #
-      # @rbs ANALOG_TYPES: Array[String]
-      ANALOG_TYPES = %w[Date DateTime Time].freeze
-
       # Converts a value to a DateTime
       #
       # @param value [Object] The value to convert to DateTime
@@ -38,7 +33,7 @@ module CMDx
       #
       # @rbs (untyped value, ?Hash[Symbol, untyped] options) -> DateTime
       def call(value, options = {})
-        return value if ANALOG_TYPES.include?(value.class.name)
+        return value.to_datetime if value.respond_to?(:to_datetime)
         return ::DateTime.strptime(value, options[:strptime]) if options[:strptime]
 
         ::DateTime.parse(value)

data/lib/cmdx/coercions/integer.rb

@@ -30,10 +30,9 @@ module CMDx
       #   Integer.call(3.14)      # => 3
       #   Integer.call(0.0)       # => 0
       # @example Handle edge cases
-      #   Integer.call("")        # => 0
-      #   Integer.call(nil)       # => 0
-      #   Integer.call(false)     # => 0
-      #   Integer.call(true)      # => 1
+      #   Integer.call("")        # => raises CoercionError
+      #   Integer.call(nil)       # => raises CoercionError
+      #   Integer.call("abc")     # => raises CoercionError
       #
       # @rbs (untyped value, ?Hash[Symbol, untyped] options) -> Integer
       def call(value, options = {})

data/lib/cmdx/coercions/time.rb

@@ -11,11 +11,6 @@ module CMDx
 
       extend self
 
-      # Types that are already time-like and don't need conversion
-      #
-      # @rbs ANALOG_TYPES: Array[String]
-      ANALOG_TYPES = %w[DateTime Time].freeze
-
       # Converts a value to a Time object
       #
       # @param value [Object] The value to convert to a Time object
@@ -40,7 +35,6 @@ module CMDx
       #
       # @rbs (untyped value, ?Hash[Symbol, untyped] options) -> Time
       def call(value, options = {})
-        return value if ANALOG_TYPES.include?(value.class.name)
         return value.to_time if value.respond_to?(:to_time)
         return ::Time.strptime(value, options[:strptime]) if options[:strptime]
 

data/lib/cmdx/executor.rb

@@ -169,7 +169,7 @@ module CMDx
           jitter.to_f * current_retry
         end
 
-      sleep(jitter) if jitter.positive?
+      sleep(jitter) if Float(jitter).positive?
 
       true
     end
@@ -291,8 +291,8 @@ module CMDx
     def log_backtrace!
       return unless result.failed?
 
-      exception = result.caused_failure.cause
-      return if exception.is_a?(Fault)
+      exception = result.caused_failure&.cause
+      return if exception.nil? || exception.is_a?(Fault)
 
       task.logger.error do
         "[#{exception.class}] #{exception.message}\n" <<

data/lib/cmdx/task.rb

@@ -95,7 +95,9 @@ module CMDx
       @id = Identifier.generate
       @context = Context.build(context)
       @result = Result.new(self)
-      @chain = Chain.build(@result, dry_run: @context.delete(:dry_run))
+
+      dry_run = @context.delete(:dry_run)
+      @chain = Chain.build(@result, dry_run:)
     end
 
     class << self

data/lib/cmdx/validator_registry.rb

@@ -102,7 +102,7 @@ module CMDx
       match =
         if options.is_a?(Hash)
           case options
-          in allow_nil: then allow_nil && value.nil?
+          in allow_nil: then !(allow_nil && value.nil?)
           else Utils::Condition.evaluate(task, options, value)
           end
         else

data/lib/cmdx/version.rb

@@ -5,6 +5,6 @@ module CMDx
   # @return [String] the version of the CMDx gem
   #
   # @rbs return: String
-  VERSION = "1.18.0"
+  VERSION = "1.19.0"
 
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cmdx
 version: !ruby/object:Gem::Version
-  version: 1.18.0
+  version: 1.19.0
 platform: ruby
 authors:
 - Juan Gomez