opera 0.3.3 → 0.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 66df118360280b866b0d104645f8f837d6046536167999f14fb4fdd97bb082a8
-  data.tar.gz: cfe3b8d4d05f8e28900bf0339a70a5bb648ad26ee92c2c449eccb0ecee492887
+  metadata.gz: 5d44889b440322513ca1bb949f4d57d317141196e76bb56a67447b200d8dac47
+  data.tar.gz: fdd2f513da963271101a5f9f5bcf9f3d4d8c65ea9a15d743c804cc97b764e501
 SHA512:
-  metadata.gz: b5bae55e20de7a7759b600050874bba4b14612635bb0616941317898ba0c537828131e24a145626b08c6d8d783ed38059dae8c55d59895dd705174aa385bfaae
-  data.tar.gz: 2eb1c3540d2a855a9e71046499ef38f7fc9477d4acc76f92a7f0f3148daadb95cb8d4166a1e7d18a151f9a0107e42a4db50e965b232ef3228f0c61fb2dfa1962
+  metadata.gz: d744526b9a983f73f2f5560f62938794bbf10b8dab9e2564c9916ec921aa9509489ff6c6611256502085c6f7b9f5c2a631e90e31f2cebc89bf5a3df2bd7e7ce8
+  data.tar.gz: 3b7a366e473983a416fcb4b22ed01791d1f89b5cc06fb9daf1f6af244df158e8891de9def6233654611217787cb16417d55d5a3b117ac47b8f2dcd7a60faf360

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Opera Changelog
 
+### 0.3.4 - February 19, 2025
+
+- Add support for `benchmark` label
+- Add support for instrumentation
+
 ### 0.3.3 - January 15, 2025
 
 - Fix issue with handling exceptions in nested operations

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    opera (0.3.3)
+    opera (0.3.4)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -37,7 +37,10 @@ Simply initialise the configuration and chose what method you want to use to rep
 Opera::Operation::Config.configure do |config|
   config.transaction_class = ActiveRecord::Base
   config.transaction_method = :transaction
-  config.transaction_options = { requires_new: true }
+  config.transaction_options = { requires_new: true, level: :step } # or level: :operation - default
+  config.instrumentation_class = Datadog::Tracing
+  config.instrumentation_method = :trace
+  config.instrumentation_options = { service: :operation }
   config.mode = :development # Can be set to production too
   config.reporter = defined?(Rollbar) ? Rollbar : Rails.logger
 end
@@ -52,7 +55,6 @@ Once opera gem is in your project you can start to build Operations
 
 ```ruby
 class A < Opera::Operation::Base
-
   configure do |config|
     config.transaction_class = Profile
     config.reporter = Rails.logger
@@ -96,18 +98,44 @@ Start developing your business logic, services and interactions as Opera::Operat
 When using Opera::Operation inside an engine add the following
 configuration to your spec_helper.rb or rails_helper.rb:
 
-```
+```ruby
 Opera::Operation::Config.configure do |config|
   config.transaction_class = ActiveRecord::Base
 end
 ```
 
 Without this extra configuration you will receive:
-```
+```ruby
 NoMethodError:
   undefined method `transaction' for nil:NilClass
 ```
 
+### Instrumentation
+
+When you want to easily instrument your operations you can add this to the opera config:
+
+```ruby
+Rails.application.configure do
+  config.x.instrumentation_class = Datadog::Tracing
+  config.x.instrumentation_method = :trace
+  config.x.instrumentation_options = { service: :opera }
+end
+```
+
+You can also instrument individual operations by adding this to the operation config:
+
+```ruby
+class A < Opera::Operation::Base
+  configure do |config|
+    config.instrumentation_class = Datadog::Tracing
+    config.instrumentation_method = :trace
+    config.instrumentation_options = { service: :opera, level: :step }
+  end
+
+  # steps
+end
+```
+
 ### Debugging
 
 When you want to easily debug exceptions you can add this
@@ -674,10 +702,12 @@ class Profile::Create < Opera::Operation::Base
 
   validate :profile_schema
 
-  step :create
-  step :update
+  benchmark :fast_section do
+    step :create
+    step :update
+  end
 
-  benchmark do
+  benchmark :slow_section do
     step :send_email
     step :output
   end
@@ -717,7 +747,7 @@ Profile::Create.call(params: {
 }, dependencies: {
   current_account: Account.find(1)
 })
-#<Opera::Operation::Result:0x007ff414a01238 @errors={}, @exceptions={}, @information={:real=>1.800013706088066e-05, :total=>0.0}, @executions=[:profile_schema, :create, :update, :send_email, :output], @output={:model=>#<Profile id: 30, user_id: nil, linkedin_uid: nil, picture: nil, headline: nil, summary: nil, first_name: "foo", last_name: "bar", created_at: "2020-08-19 10:46:00", updated_at: "2020-08-18 10:46:00", agree_to_terms_and_conditions: nil, registration_status: "", account_id: 1, start_date: nil, supervisor_id: nil, picture_processing: false, statistics: {}, data: {}, notification_timestamps: {}, suggestions: {}, notification_settings: {}, contact_information: []>}>
+#<Opera::Operation::Result:0x007ff414a01238 @errors={}, @exceptions={}, @information={fast_section: {:real=>0.300013706088066e-05, :total=>0.0}, slow_section: {:real=>1.800013706088066e-05, :total=>0.0}}, @executions=[:profile_schema, :create, :update, :send_email, :output], @output={:model=>#<Profile id: 30, user_id: nil, linkedin_uid: nil, picture: nil, headline: nil, summary: nil, first_name: "foo", last_name: "bar", created_at: "2020-08-19 10:46:00", updated_at: "2020-08-18 10:46:00", agree_to_terms_and_conditions: nil, registration_status: "", account_id: 1, start_date: nil, supervisor_id: nil, picture_processing: false, statistics: {}, data: {}, notification_timestamps: {}, suggestions: {}, notification_settings: {}, contact_information: []>}>
 ```
 
 ### Success

data/lib/opera/operation/base.rb

@@ -33,7 +33,9 @@ module Opera
         def call(args = {})
           operation = new(params: args.fetch(:params, {}), dependencies: args.fetch(:dependencies, {}))
           executor = Executor.new(operation)
-          executor.evaluate_instructions(instructions)
+          Instrumentation.new(config).instrument(name: self.name, level: :operation) do
+            executor.evaluate_instructions(instructions)
+          end
           executor.result
         end
 

data/lib/opera/operation/builder.rb

@@ -35,6 +35,7 @@ module Opera
             instructions << if !blk.nil?
                               {
                                 kind: instruction,
+                                label: method,
                                 instructions: InnerBuilder.new(&blk).instructions
                               }
                             else

data/lib/opera/operation/config.rb

@@ -6,12 +6,18 @@ module Opera
       DEVELOPMENT_MODE = :development
       PRODUCTION_MODE = :production
 
-      attr_accessor :transaction_class, :transaction_method, :transaction_options, :reporter, :mode
+      attr_accessor :transaction_class, :transaction_method, :transaction_options, 
+                    :instrumentation_class, :instrumentation_method, :instrumentation_options, :mode, :reporter
 
       def initialize
         @transaction_class = self.class.transaction_class
         @transaction_method = self.class.transaction_method || :transaction
         @transaction_options = self.class.transaction_options
+
+        @instrumentation_class = self.class.instrumentation_class
+        @instrumentation_method = self.class.instrumentation_method || :instrument
+        @instrumentation_options = self.class.instrumentation_options || {}
+
         @mode = self.class.mode || DEVELOPMENT_MODE
         @reporter = custom_reporter || self.class.reporter
 
@@ -35,7 +41,8 @@ module Opera
       end
 
       class << self
-        attr_accessor :transaction_class, :transaction_method, :transaction_options, :reporter, :mode
+        attr_accessor :transaction_class, :transaction_method, :transaction_options, 
+                      :instrumentation_class, :instrumentation_method, :instrumentation_options, :mode, :reporter
 
         def configure
           yield self

data/lib/opera/operation/instructions/executors/benchmark.rb

@@ -7,10 +7,17 @@ module Opera
         class Benchmark < Executor
           def call(instruction)
             benchmark = ::Benchmark.measure do
+              instruction[:kind] = :step
               super
             end
 
-            result.add_information(real: benchmark.real, total: benchmark.total)
+            result.add_information(benchmark_key(instruction) => { real: benchmark.real, total: benchmark.total })
+          end
+
+          private
+
+          def benchmark_key(instruction)
+            instruction[:method] || instruction[:label] || instruction[:instructions].map { |e| e[:method] }.join('-').to_sym
           end
         end
       end

data/lib/opera/operation/instructions/executors/step.rb

@@ -8,8 +8,10 @@ module Opera
           def call(instruction)
             method = instruction[:method]
 
-            operation.result.add_execution(method) unless production_mode?
-            operation.send(method)
+            Instrumentation.new(config).instrument(name: "##{method}", level: :step) do
+              operation.result.add_execution(method) unless production_mode?
+              operation.send(method)
+            end
           rescue StandardError => exception
             reporter&.error(exception)
             operation.result.add_exception(method, "#{exception.message}, for #{operation.inspect}", classname: operation.class.name)

data/lib/opera/operation/instrumentation.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module Opera
+  module Operation
+    class Instrumentation
+      attr_reader :config
+
+      def initialize(config)
+        @config = config
+      end
+
+      def instrument(name:, level: :operation)
+        return yield if !instrumentation_enabled?
+        return yield if level == :step && instrumentation_level != :step
+
+        instrumentation_class.send(instrumentation_method, name, **instrumentation_options.except(:level)) do
+          yield
+        end
+      end
+
+      private
+
+      def instrumentation_options
+        config.instrumentation_options
+      end
+
+      def instrumentation_method
+        config.instrumentation_method
+      end
+
+      def instrumentation_class
+        config.instrumentation_class
+      end
+
+      def instrumentation_enabled?
+        !!config.instrumentation_class
+      end
+
+      def instrumentation_level
+        instrumentation_options[:level] || :operation
+      end
+    end
+  end
+end

data/lib/opera/operation.rb

@@ -4,6 +4,7 @@ require 'opera/operation/attributes_dsl'
 require 'opera/operation/builder'
 require 'opera/operation/base'
 require 'opera/operation/executor'
+require 'opera/operation/instrumentation'
 require 'opera/operation/result'
 require 'opera/operation/config'
 require 'opera/operation/instructions/executors/success'

data/lib/opera/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Opera
-  VERSION = '0.3.3'
+  VERSION = '0.3.4'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: opera
 version: !ruby/object:Gem::Version
-  version: 0.3.3
+  version: 0.3.4
 platform: ruby
 authors:
 - ProFinda Development Team
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-01-15 00:00:00.000000000 Z
+date: 2025-02-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dry-validation
@@ -91,6 +91,7 @@ files:
 - lib/opera/operation/instructions/executors/success.rb
 - lib/opera/operation/instructions/executors/transaction.rb
 - lib/opera/operation/instructions/executors/validate.rb
+- lib/opera/operation/instrumentation.rb
 - lib/opera/operation/result.rb
 - lib/opera/version.rb
 - opera.gemspec