lite-command 3.1.3 → 3.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 39377d9ecb58d9d61ae0c99305d9db08672b7f97eb368194fc209b419cec1d0a
-  data.tar.gz: 5ec36c8fcb1d68e2a6bd3ae0866f687a9c04469d502dcba0bc7fe092a9b88351
+  metadata.gz: d88e18025d91c08352d81be6288346138b4c07f8c6c1b7e437c1309f4c4ed330
+  data.tar.gz: 1011eca8e0ca7854dfd9b488d8f91a467d15ff2dccc60f5a36eb433ef3761fe7
 SHA512:
-  metadata.gz: 521be42ef9eab72ca99c2cd8dfba53ca3156538daf1e7715aff5123aefc6a70d7c4b439962306b8d1d22b87897bed1f54167353f98bea32d83ebf5eaaceffefe
-  data.tar.gz: 6c33d9b29e925c30b2e5abe06088fcfb0bccbd9cedbd241d76f398a89cfb7a679bf2a59b86ae766ccec69d94496640613d39995e9eca35ac3938d165a8e96eed
+  metadata.gz: 42eb9126777ee23506f1d98c3313e6de9c03da120b6711a399ad99051c88faac769080e4f41891c3a2658381aa4ad93eaf191c4ec00e446c8507f5f5ca8108fe
+  data.tar.gz: f314544be97beead7f55e6e8ee3f164e9a80013368aaa1380f632e0a702dd3fecb555a06d9e100990454aff6249a01a35c688a28792aa9b1e7351e624e38b542

data/CHANGELOG.md

@@ -6,6 +6,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [3.1.4] - 2024-10-27
+### Added
+- Add exception data to results hash
+### Changed
+- Relocated some results methods to a runtimes internal module
+- Assign correct fault even when child command is hard called
+- Renamed `fault_exception` to `command_exception`
+
 ## [3.1.3] - 2024-10-26
 ### Added
 - Added passing `original_exception` to fault `!` calls

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    lite-command (3.1.3)
+    lite-command (3.1.4)
       activemodel
       ostruct
 

data/lib/lite/command/base.rb

@@ -11,10 +11,11 @@ module Lite
 
         base.include ActiveModel::Validations
 
+        base.include Internals::Runtimes
         base.include Internals::Attributes
+        base.include Internals::Faults
         base.include Internals::Calls
         base.include Internals::Executions
-        base.include Internals::Faults
         base.include Internals::Results
 
         if Lite::Command.configuration.raise_dynamic_faults # rubocop:disable Style/GuardClause

data/lib/lite/command/fault_streamer.rb

@@ -15,7 +15,7 @@ module Lite
         if object.respond_to?(:reason)
           object.reason
         elsif object.is_a?(StandardError)
-          "[#{object.class.name}] #{object.message}".chomp(".")
+          Utils.pretty_exception(object)
         else
           object
         end
@@ -35,7 +35,7 @@ module Lite
         Utils.try(object, :thrown_by) || command.caused_by
       end
 
-      def fault_exception
+      def command_exception
         return if command.success?
 
         Fault.build(

data/lib/lite/command/internals/calls.rb

@@ -75,7 +75,7 @@ module Lite
           str.nil? || str == reason
         end
 
-        def fault(object, s, m, oe) # rubocop:disable Naming/MethodParameterName
+        def fault(object, s, m, exception: nil) # rubocop:disable Naming/MethodParameterName
           return if s == SUCCESS || status != SUCCESS
 
           @status   = s
@@ -87,8 +87,8 @@ module Lite
           @caused_by ||= fault_streamer.caused_by
           @thrown_by ||= fault_streamer.thrown_by
 
-          @fault_exception    ||= fault_streamer.fault_exception
-          @original_exception ||= oe || fault_exception
+          @command_exception  ||= fault_streamer.command_exception
+          @original_exception ||= exception || command_exception
         end
 
         FAULTS.each do |f|
@@ -96,9 +96,9 @@ module Lite
           define_method(:"#{f}!") do |object, metadata: nil, original_exception: nil|
             return unless success?
 
-            fault(object, f, metadata, original_exception)
+            fault(object, f, metadata, exception: original_exception)
 
-            raise(fault_exception)
+            raise(command_exception)
           end
         end
 

data/lib/lite/command/internals/executions.rb

@@ -75,7 +75,7 @@ module Lite
           around_execution { call }
           Utils.try(self, :on_success)
         rescue StandardError => e
-          fault(e, ERROR, metadata, e)
+          fault(e, Utils.try(e, :type) || ERROR, metadata, exception: e)
           after_execution
           Utils.try(self, :"on_#{status}", e)
         ensure
@@ -86,7 +86,7 @@ module Lite
           around_execution { call }
           Utils.try(self, :on_success)
         rescue StandardError => e
-          fault(e, ERROR, metadata, e)
+          fault(e, Utils.try(e, :type) || ERROR, metadata, exception: e)
           after_execution
           Utils.try(self, :"on_#{status}", e)
           raise(e)

data/lib/lite/command/internals/faults.rb

@@ -6,7 +6,7 @@ module Lite
       module Faults
 
         def self.included(base)
-          base.class_eval { attr_reader :fault_exception, :original_exception }
+          base.class_eval { attr_reader :command_exception, :original_exception }
         end
 
         def caused_by
@@ -34,7 +34,7 @@ module Lite
         end
 
         def raise!(original: false)
-          exception = (fault_exception unless original) || original_exception
+          exception = (command_exception unless original) || original_exception
           return if exception.nil?
 
           raise(exception)

data/lib/lite/command/internals/results.rb

@@ -1,20 +1,10 @@
 # frozen_string_literal: true
 
-require "securerandom" unless defined?(SecureRandom)
-
 module Lite
   module Command
     module Internals
       module Results
 
-        def index
-          @index ||= context.index ||= 0
-        end
-
-        def cmd_id
-          @cmd_id ||= context.cmd_id ||= SecureRandom.uuid
-        end
-
         def outcome
           return state if pending? || thrown?
 
@@ -36,7 +26,9 @@ module Lite
             reason:,
             metadata:,
             caused_by: caused_by&.index,
+            caused_exception: Utils.pretty_exception(caused_by&.original_exception),
             thrown_by: thrown_by&.index,
+            thrown_exception: Utils.pretty_exception(thrown_by&.command_exception),
             runtime:
           }.compact
         end
@@ -44,26 +36,6 @@ module Lite
 
         private
 
-        def assign_execution_cmd_id
-          @cmd_id = context.cmd_id ||= cmd_id
-        end
-
-        def increment_execution_index
-          @index = context.index = index.next
-        end
-
-        def start_monotonic_time
-          @start_monotonic_time ||= Utils.monotonic_time
-        end
-
-        def stop_monotonic_time
-          @stop_monotonic_time ||= Utils.monotonic_time
-        end
-
-        def runtime
-          stop_monotonic_time - start_monotonic_time
-        end
-
         def append_execution_result
           results.push(self).sort_by!(&:index)
         end

data/lib/lite/command/internals/runtimes.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+require "securerandom" unless defined?(SecureRandom)
+
+module Lite
+  module Command
+    module Internals
+      module Runtimes
+
+        def index
+          @index ||= context.index ||= 0
+        end
+
+        def cmd_id
+          @cmd_id ||= context.cmd_id ||= SecureRandom.uuid
+        end
+
+        private
+
+        def assign_execution_cmd_id
+          @cmd_id = context.cmd_id ||= cmd_id
+        end
+
+        def increment_execution_index
+          @index = context.index = index.next
+        end
+
+        def start_monotonic_time
+          @start_monotonic_time ||= Utils.monotonic_time
+        end
+
+        def stop_monotonic_time
+          @stop_monotonic_time ||= Utils.monotonic_time
+        end
+
+        def runtime
+          stop_monotonic_time - start_monotonic_time
+        end
+
+      end
+    end
+  end
+end

data/lib/lite/command/utils.rb

@@ -10,6 +10,12 @@ module Lite
         Process.clock_gettime(Process::CLOCK_MONOTONIC)
       end
 
+      def pretty_exception(exception)
+        return if exception.nil?
+
+        "[#{exception.class.name}] #{exception.message}".chomp(".")
+      end
+
       def descendant_of?(object, other)
         object_class = object.respond_to?(:new) ? object : object.class
         other_class = other.respond_to?(:new) ? other : other.class

data/lib/lite/command/version.rb

@@ -3,7 +3,7 @@
 module Lite
   module Command
 
-    VERSION = "3.1.3"
+    VERSION = "3.1.4"
 
   end
 end

data/lib/lite/command.rb

@@ -11,10 +11,11 @@ require "lite/command/utils"
 require "lite/command/context"
 require "lite/command/fault"
 require "lite/command/fault_streamer"
+require "lite/command/internals/runtimes"
 require "lite/command/internals/attributes"
+require "lite/command/internals/faults"
 require "lite/command/internals/calls"
 require "lite/command/internals/executions"
-require "lite/command/internals/faults"
 require "lite/command/internals/results"
 require "lite/command/base"
 require "lite/command/step"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lite-command
 version: !ruby/object:Gem::Version
-  version: 3.1.3
+  version: 3.1.4
 platform: ruby
 authors:
 - Juan Gomez
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-10-26 00:00:00.000000000 Z
+date: 2024-10-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -202,6 +202,7 @@ files:
 - lib/lite/command/internals/executions.rb
 - lib/lite/command/internals/faults.rb
 - lib/lite/command/internals/results.rb
+- lib/lite/command/internals/runtimes.rb
 - lib/lite/command/sequence.rb
 - lib/lite/command/step.rb
 - lib/lite/command/utils.rb