rookout 0.1.34 → 0.1.37

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 27b1869fb9f3a4797e7fec394a45f96ba46e5d2f7724c4557437a7a1939b0772
-  data.tar.gz: ff6c10ba341cfc127c8dd336bda018f4384a4ea280bceceb9220194b8b2aab68
+  metadata.gz: 661b78a375817aaf8b11e7d11baa2a1a55824a4faeb7b07800b2ab77d05fb21a
+  data.tar.gz: d2abdb693639eddc830978c88a426f8d660db1ddcc09359d190eac69e2686de2
 SHA512:
-  metadata.gz: 34066a3301315db678b63868ec8bbc9bee3b5afdc4e1a75c087b5b4caa4f6e32fe95078013f8a133ee4451cef0ae3054d0e0953a55ef0ee095cb95b6f77ba469
-  data.tar.gz: 33a601b3b0950025fc6f38bbf62ccfe19125f90d870543bb51a958543d7d5e5a3f53f99a0b9aba1f0fa2905a945fda2defb92bde35a5de8757c26e283a687b3e
+  metadata.gz: 4cfd877508902044f7a6a003ed6b9f4e87f24452dac85873f4281b30e17445f085f4cc7d46dea0919a6536755f01b81b67d89ce4a21eb063f97f69a51403053b
+  data.tar.gz: 606d7c8765066bd747ab7d3ecf098c39c934f023708d7e6eef7348bded9bb5b5175b101faf548f39838408ff865b383cab876f41c39fae0c0d6c2acd8d33d036

data/lib/rookout/augs/actions/action_run_processor.rb

@@ -5,6 +5,7 @@ module Rookout
     module Actions
       class ActionRunProcessor < Action
         def initialize arguments, processor_factory
+          super()
           @processor = processor_factory.create_processor arguments["operations"]
 
           post_operations = arguments["post_operations"]

data/lib/rookout/augs/aug_factory.rb

@@ -52,9 +52,9 @@ module Rookout
 
         case name
         when "file_line"
-          return Locations::LocationFileLine.new configuration, @output, aug
+          Locations::LocationFileLine.new configuration, @output, aug
         when "exception_handler"
-          return Locations::LocationExceptionHandler.new configuration, @output, aug
+          Locations::LocationExceptionHandler.new configuration, @output, aug
         when "log_handler"
           raise Exceptions::RookUnsupportedLiveLogger
         else

data/lib/rookout/augs/aug_rate_limiter.rb

@@ -75,7 +75,7 @@ module Rookout
         prev_window_usage = @windows[prev_window_key] || 0
 
         # Previous window weight
-        prev_weight = 1 - (now_ns - current_window_key) / @window_size.to_f
+        prev_weight = 1 - ((now_ns - current_window_key) / @window_size.to_f)
 
         # Final weighted usage
         (prev_window_usage * prev_weight) + (@active_count * @active_weight) + current_window_usage
@@ -91,7 +91,7 @@ module Rookout
       end
 
       def cleanup now_ns
-        @windows.reject! { |key, _| key < (now_ns - @window_size * 5) } if @windows.length > 10
+        @windows.reject! { |key, _| key < (now_ns - (@window_size * 5)) } if @windows.length > 10
       end
     end
   end

data/lib/rookout/augs/locations/location_file_line.rb

@@ -21,7 +21,11 @@ module Rookout
           @line_unique = arguments["line_unique"] || false
         end
 
-        attr_reader :filename, :lineno, :file_hash, :line_crc, :line_unique
+        attr_reader :filename
+        attr_reader :lineno
+        attr_reader :file_hash
+        attr_reader :line_crc
+        attr_reader :line_unique
 
         def add_aug trigger_services
           trigger_services.get_service("position").add_aug self

data/lib/rookout/com_ws/agent_com_ws.rb

@@ -68,6 +68,7 @@ module Rookout
         end
 
         @pending_messages.push envelope_wrapper
+        @pending_messages_length += msg_size
       end
 
       def queue_full?
@@ -117,7 +118,7 @@ module Rookout
 
             if @print_on_initial_connection
               @print_on_initial_connection = false
-              STDERR.puts "[Rookout] Successfully connected to controller"
+              $stderr.puts "[Rookout] Successfully connected to controller"
             end
             Logger.instance.debug "WebSocket connected successfully"
             Logger.instance.info "Finished initialization"
@@ -204,9 +205,10 @@ module Rookout
           msg = envelope_wrapper.envelope
           @pending_messages_length -= envelope_wrapper.calculate_size
 
-          if msg.is_a? ExitMessage
+          case msg
+          when ExitMessage
             break if msg.thread == Thread.current
-          elsif msg.is_a? FlushMessage
+          when FlushMessage
             msg.event.set
           else
             begin

data/lib/rookout/com_ws/envelope_wrapper.rb

@@ -8,8 +8,7 @@ class EnvelopeWrapperBase
     timestamp = Google::Protobuf::Timestamp.new
     timestamp.from_time Time.new
     envelope = Com::Rookout::Envelope.new msg: any_message, timestamp: timestamp
-    serialized = Com::Rookout::Envelope.encode envelope
-    serialized
+    Com::Rookout::Envelope.encode envelope
   end
 
   def envelope
@@ -23,27 +22,30 @@ end
 
 class Variant2EnvelopeWrapper < EnvelopeWrapperBase
   def initialize agent_id, aug_id, report_id, arguments
+    super()
+
     @aug_report_message = Com::Rookout::AugReportMessage.new agent_id: agent_id,
-                                       aug_id: aug_id,
-                                       report_id: report_id
+                                                             aug_id: aug_id,
+                                                             report_id: report_id
     @serializer = Rookout::Processor::NamespaceSerializer2.new
-    @aug_report_message.arguments2 = @serializer.dump arguments
-  end
-
-  def envelope
-    return @envelope unless @envelope.nil?
+    @aug_report_message.arguments2 = @serializer.dump arguments, true
     @serializer.string_cache.each { |key, value| @aug_report_message.strings_cache[key] = value }
     @envelope = wrap_in_envelope @aug_report_message
     @aug_report_message = nil
   end
 
+  def envelope
+    @envelope
+  end
+
   def calculate_size
-    @envelope.length + @serializer.estimated_pending_bytes
+    @envelope.length
   end
 end
 
 class EnvelopeWrapper < EnvelopeWrapperBase
   def initialize message
+    super()
     @envelope = wrap_in_envelope message
   end
 

data/lib/rookout/com_ws/git.rb

@@ -42,7 +42,7 @@ module Rookout
         link_contents = File.read link_path
 
         if link_contents.start_with? "ref:"
-          next_link = (link_contents.split " ")[1].strip
+          next_link = link_contents.split[1].strip
           follow_sym_links root, next_link
         else
           link_contents.strip

data/lib/rookout/com_ws/information.rb

@@ -21,6 +21,8 @@ module Rookout
           @labels["k8s_namespace"] = k8_namespace
         end
 
+        collect_serverless_labels
+
         @ip_addr = local_ip
 
         @scm_info = create_scm_information
@@ -54,7 +56,8 @@ module Rookout
                                            scm: @scm_info
       end
 
-      attr_accessor :agent_id, :labels
+      attr_accessor :agent_id
+      attr_accessor :labels
 
       private
 
@@ -97,6 +100,47 @@ module Rookout
 
         Com::Rookout::SCMInformation.new origin: user_git_origin, commit: user_git_commit
       end
+
+      def aws_lambda?
+        !ENV["AWS_LAMBDA_FUNCTION_NAME"].nil?
+      end
+
+      def google_cloud_function?
+        !ENV["FUNCTION_TARGET"].nil? && !ENV["FUNCTION_SIGNATURE_TYPE"].nil?
+      end
+
+      def cloud_run_or_fire_base?
+        !ENV["K_SERVICE"].nil? &&
+          !ENV["K_REVISION"].nil? &&
+          !ENV["K_CONFIGURATION"].nil? &&
+          !ENV["PORT"].nil?
+      end
+
+      def azure_function?
+        !ENV["FUNCTIONS_WORKER_RUNTIME"].nil? && !ENV["WEBSITE_SITE_NAME"].nil?
+      end
+
+      def collect_serverless_labels
+        if aws_lambda?
+          @labels["function_name"] = ENV["AWS_LAMBDA_FUNCTION_NAME"]
+          @labels["aws_region"] = ENV["AWS_REGION"]
+
+        elsif google_cloud_function? || cloud_run_or_fire_base?
+          function_name = ENV["FUNCTION_NAME"]
+          if function_name.nil?
+            function_name = ENV["K_SERVICE"]
+          end
+          @labels["function_name"] = function_name
+
+        elsif azure_function?
+          @labels["function_name"] = ENV["WEBSITE_SITE_NAME"]
+          @labels["azure_region"] = ENV["REGION_NAME"]
+        end
+
+        return unless @labels.key? "function_name"
+
+        @labels["rookout_serverless"] = "true"
+      end
     end
   end
 end

data/lib/rookout/com_ws/output.rb

@@ -42,7 +42,8 @@ module Rookout
         @closing = false
       end
 
-      attr_accessor :agent_id, :agent_com
+      attr_accessor :agent_id
+      attr_accessor :agent_com
 
       def close
         @closing = true
@@ -86,13 +87,17 @@ module Rookout
             if arguments.nil? || arguments.call_method("size", "") == 0
               protobuf_arguments = nil
             else
-              protobuf_arguments = Processor::NamespaceSerializer.dump arguments
+              protobuf_arguments = Processor::NamespaceSerializer.dump arguments, true
             end
 
-            envelope_wrapper = EnvelopeWrapper.new(Com::Rookout::AugReportMessage.new(agent_id: @agent_id,
-                                                     aug_id: aug_id,
-                                                     report_id: report_id,
-                                                     arguments: protobuf_arguments))
+            envelope_wrapper = EnvelopeWrapper.new(
+              Com::Rookout::AugReportMessage.new(
+                agent_id: @agent_id,
+                aug_id: aug_id,
+                report_id: report_id,
+                arguments: protobuf_arguments
+              )
+            )
           end
 
           @agent_com.add envelope_wrapper
@@ -123,7 +128,8 @@ module Rookout
           protobuf_arguments = nil
         else
           protobuf_arguments = Processor::NamespaceSerializer.dump(
-            Processor::Namespaces::RubyObjectNamespace.new(arguments)
+            Processor::Namespaces::RubyObjectNamespace.new(arguments),
+            true
           )
         end
 

data/lib/rookout/commit.rb

@@ -1,3 +1,3 @@
 module Rookout
-  COMMIT = "d0e0c4f1b1b08542dfe7a380f52640ea68f223a6".freeze
+  COMMIT = "e56488a23c33cbe4ad104217694c08c994586597".freeze
 end

data/lib/rookout/config.rb

@@ -6,21 +6,26 @@ module Rookout
     # Magic to allow for module variables to be easily accessible
     class << self
       attr_accessor :debug
+
       Rookout::Config.debug = false
 
-      attr_accessor :logger_filename, :logger_log_to_stderr, :logger_log_level
+      attr_accessor :logger_filename
+      attr_accessor :logger_log_to_stderr
+      attr_accessor :logger_log_level
+
       Rookout::Config.logger_filename = "rookout/ruby-rook.log".freeze
       Rookout::Config.logger_log_to_stderr = false
       Rookout::Config.logger_log_level = :info
 
-      attr_accessor :agent_com_configuration_command_thread_name,
-                    :agent_com_max_message_limit,
-                    :agent_com_max_queue_messages_length,
-                    :agent_com_timeout,
-                    :agent_com_ping_interval,
-                    :agent_com_ping_timeout,
-                    :agent_com_flush_timeout,
-                    :agent_com_max_queued_messages
+      attr_accessor :agent_com_configuration_command_thread_name
+      attr_accessor :agent_com_max_message_limit
+      attr_accessor :agent_com_max_queue_messages_length
+      attr_accessor :agent_com_timeout
+      attr_accessor :agent_com_ping_interval
+      attr_accessor :agent_com_ping_timeout
+      attr_accessor :agent_com_flush_timeout
+      attr_accessor :agent_com_max_queued_messages
+
       Rookout::Config.agent_com_max_message_limit = 1024 * 1024
       Rookout::Config.agent_com_max_queue_messages_length = 15 * 1024 * 1024
       Rookout::Config.agent_com_timeout = 3
@@ -29,47 +34,60 @@ module Rookout
       Rookout::Config.agent_com_flush_timeout = 3
       Rookout::Config.agent_com_max_queued_messages = 100
 
-      attr_accessor :backoff_factor, :backoff_reset_time, :backoff_max_time
+      attr_accessor :backoff_factor
+      attr_accessor :backoff_reset_time
+      attr_accessor :backoff_max_time
+
       Rookout::Config.backoff_factor = 0.2
       Rookout::Config.backoff_max_time = 60
       Rookout::Config.backoff_reset_time = 3 * 60.0
 
-      attr_accessor :output_max_status_updates,
-                    :output_max_aug_messages,
-                    :output_max_log_items,
-                    :output_bucket_refresh_rate
+      attr_accessor :output_max_status_updates
+      attr_accessor :output_max_aug_messages
+      attr_accessor :output_max_log_items
+      attr_accessor :output_bucket_refresh_rate
+
       Rookout::Config.output_max_status_updates = 200
       Rookout::Config.output_max_aug_messages = 100
       Rookout::Config.output_max_log_items = 200
       Rookout::Config.output_bucket_refresh_rate = 10
 
       attr_accessor :instrumentation_max_aug_time
+
       Rookout::Config.instrumentation_max_aug_time = 400
 
-      attr_accessor :user_git_commit, :user_git_origin
+      attr_accessor :user_git_commit
+      attr_accessor :user_git_origin
+
       Rookout::Config.user_git_commit = nil
       Rookout::Config.user_git_origin = nil
 
-      attr_accessor :rookout_version, :rookout_commit
+      attr_accessor :rookout_version
+      attr_accessor :rookout_commit
+
       Rookout::Config.rookout_version = Rookout::VERSION
       Rookout::Config.rookout_commit = Rookout::COMMIT
 
       attr_accessor :protobuf_version2
+
       Rookout::Config.protobuf_version2 = false
 
       attr_accessor :true_values
+
       Rookout::Config.true_values = ["y", "Y", "yes", "Yes", "YES", "true", "True", "TRUE", "1"]
 
 
       def update_config configuration
         is_config_proto2 = configuration["RUBY_PROTOBUF_VERSION_2"]
         return if is_config_proto2.nil?
-        is_env_proto2 = ENV["PROTOBUF_VERSION_2"]
+        is_env_proto2 = ENV["ROOKOUT_Protobuf_Version2"]
         if is_env_proto2.nil?
-          @protobuf_version2 = true if @true_values.includes? is_config_proto2
-        elsif @true_values.includes? is_env_proto2
+          @protobuf_version2 = true if @true_values.include? is_config_proto2
+        elsif @true_values.include? is_env_proto2
+          Logger.instance.debug "ROOKOUT_Protobuf_Version2 env is preset: #{is_env_proto2}"
           @protobuf_version2 = true
         end
+        Logger.instance.info "Updating ROOKOUT_Protobuf_Version2 value to: #{@protobuf_version2}"
       end
     end
   end

data/lib/rookout/exceptions.rb

@@ -11,9 +11,6 @@ module Rookout
     end
 
     class RookInterfaceException < ToolException
-      def initialize message
-        super message
-      end
     end
 
     class RookVersionNotSupported < ToolException
@@ -92,8 +89,8 @@ module Rookout
     class RookMessageSizeExceeded < ToolException
       def initialize message_size, max_message_size
         super "Message size of #{message_size} exceeds max size limit of #{max_message_size}. " \
-                  "Change the depth of collection or change the default by setting ROOKOUT_MAX_MESSAGE_SIZE " \
-                  "as environment variable or system property",
+              "Change the depth of collection or change the default by setting ROOKOUT_MAX_MESSAGE_SIZE " \
+              "as environment variable or system property",
               { "message_size" => message_size, "max_message_size" => max_message_size }
       end
     end
@@ -101,7 +98,7 @@ module Rookout
     class RookRuleRateLimited < ToolException
       def initialize
         super "Breakpoint was disabled due to rate-limiting. " \
-                   "For more information: https://docs.rookout.com/docs/breakpoints-tasks.html#rate-limiting"
+              "For more information: https://docs.rookout.com/docs/breakpoints-tasks.html#rate-limiting"
       end
     end
 
@@ -139,9 +136,6 @@ module Rookout
     end
 
     class RookInvalidOptions < ToolException
-      def initialize description
-        super description
-      end
     end
 
     class RookInvalidLabel < ToolException

data/lib/rookout/interface.rb

@@ -14,7 +14,7 @@ module Rookout
 
     def print_debug_messages
       puts "[Rookout] Running in debug mode"
-      puts "[Rookout] Rookout SDK for ruby: " + Config.rookout_version
+      puts "[Rookout] Rookout SDK for ruby: #{Config.rookout_version}"
     end
 
     def start options = {}
@@ -36,16 +36,16 @@ module Rookout
         rook.connect(**@start_options)
       rescue LoadError
         raise if throw_errors
-        STDERR.puts "[Rookout] Failed to load Rookout. Please make sure to force build native extensions by setting" \
-          "'bundle config force_ruby_platform true'"
+        $stderr.puts "[Rookout] Failed to load Rookout. Please make sure to force build native extensions by setting" \
+                     "'bundle config force_ruby_platform true'"
       rescue RookMissingToken, RookInvalidToken, RookInvalidOptions, RookVersionNotSupported, RookBadProtobuf => e
         raise if throw_errors
-        STDERR.puts "[Rookout] Failed to start Rookout: #{e.message}"
+        $stderr.puts "[Rookout] Failed to start Rookout: #{e.message}"
       rescue RookCommunicationException => e
         raise if throw_errors
-        Logger.instance.warning "[Rookout] " + e.message
+        Logger.instance.warning "[Rookout] #{e.message}"
       rescue Exception => e
-        STDERR.puts e.full_message if Config.debug
+        $stderr.puts e.full_message if Config.debug
         raise if throw_errors
       end
     end
@@ -82,12 +82,13 @@ module Rookout
       # Only test alpine
       return unless File.exist? "/etc/alpine-release"
 
+      # Make sure protobuf is built from source
       protobuf = Gem::Specification.find_by_path "google/protobuf"
-      STDERR.puts RookBadProtobuf.new.message if protobuf.nil?
+      $stderr.puts RookBadProtobuf.new.message if protobuf.nil?
       return unless protobuf.platform != "ruby"
 
       error = RookBadProtobufPlatform.new protobuf.platform
-      STDERR.puts error.message
+      $stderr.puts error.message
     end
 
     def configure_globals options

data/lib/rookout/logger.rb

@@ -66,7 +66,7 @@ module Rookout
 
         arguments.each do |argument|
           if argument.is_a? Exception
-            @formatted_message += "\n" + argument.message + "\n" + argument.backtrace.join("\n\t")
+            @formatted_message += "\n#{argument.message}\n#{argument.backtrace.join "\n\t"}"
           end
         end
 
@@ -74,7 +74,13 @@ module Rookout
         set_caller
       end
 
-      attr_reader :level, :time, :filename, :lineno, :message, :formatted_message, :arguments
+      attr_reader :level
+      attr_reader :time
+      attr_reader :filename
+      attr_reader :lineno
+      attr_reader :message
+      attr_reader :formatted_message
+      attr_reader :arguments
 
       def format
         "#{@time} #{Process.pid}:#{Thread.current.name}-" \
@@ -137,19 +143,19 @@ module Rookout
         file = nil
 
         if Config.debug
-          STDERR.puts "[Rookout] Failed to open log file: #{log_file_path}"
-          STDERR.puts e.backtrace
+          $stderr.puts "[Rookout] Failed to open log file: #{log_file_path}"
+          $stderr.puts e.backtrace
         end
       end
 
-      ->(record) { file.write record.format + "\n" if file }
+      ->(record) { file.write "#{record.format}\n" if file }
     end
 
     def calculate_log_file_path
       return Config.logger_filename if absolute_path? Config.logger_filename
 
       if RUBY_PLATFORM.include? "darwin"
-        File.join ENV["HOME"], Config.logger_filename
+        File.join Dir.home, Config.logger_filename
       elsif RUBY_PLATFORM.match?(/cygwin|mswin|mingw|bccwin|wince|emx/)
         File.join ENV["USERPROFILE"], Config.logger_filename
       else
@@ -158,7 +164,7 @@ module Rookout
     end
 
     def new_stderr_handler
-      ->(record) { STDERR.puts record.format }
+      ->(record) { $stderr.puts record.format }
     end
 
     def new_remote_handler

data/lib/rookout/processor/namespace_serializer.rb

@@ -7,7 +7,7 @@ module Rookout
 
       module_function
 
-      def dump namespace, log_errors = true
+      def dump namespace, log_errors
         namespace.dump log_errors
       rescue StandardError => e
         message = "Failed to serialize namespace"

data/lib/rookout/processor/namespace_serializer2.rb

@@ -14,7 +14,8 @@ module Rookout
         @estimated_pending_bytes = 0
       end
 
-      attr_reader :string_cache, :estimated_pending_bytes
+      attr_reader :string_cache
+      attr_reader :estimated_pending_bytes
 
       def get_string_index_in_cache str
         if @string_cache.key? str
@@ -35,13 +36,16 @@ module Rookout
         variant.variant_type_max_depth = (type << 1) | 1
       end
 
-      def dump namespace, log_errors = true
-        if namespace.is_a? Namespaces::RubyObjectNamespace
+      def dump namespace, log_errors
+        case namespace
+        when Namespaces::RubyObjectNamespace
           dump_ruby_object_namespace namespace, log_errors
-        elsif namespace.is_a? Namespaces::ContainerNamespace
+        when Namespaces::ContainerNamespace
           dump_container_namespace namespace, log_errors
-        elsif namespace.is_a? Namespaces::TracebackNamespace
+        when Namespaces::TracebackNamespace
           dump_traceback_namespace namespace
+        else
+          raise NotImplementedError
         end
       rescue StandardError => e
         message = "Failed to serialize namespace"
@@ -80,6 +84,7 @@ module Rookout
       # rubocop:disable Metrics/AbcSize
       # rubocop:disable Metrics/PerceivedComplexity
       # rubocop:disable Metrics/CyclomaticComplexity
+      # rubocop:disable Style/ClassEqualityComparison
       def unsafe_dump_object obj, current_depth, config, log_object_errors
         variant = Com::Rookout::Variant2.new original_type_index_in_cache: get_string_index_in_cache(obj.class.to_s)
 
@@ -105,6 +110,7 @@ module Rookout
 
         variant
       end
+      # rubocop:enable Style/ClassEqualityComparison
       # rubocop:enable Metrics/AbcSize
       # rubocop:enable Metrics/PerceivedComplexity
       # rubocop:enable Metrics/CyclomaticComplexity
@@ -120,20 +126,21 @@ module Rookout
       INT64_MAX = (1 << 63) - 1
 
       def dump_numeric obj, variant, config
-        if obj == true
+        case obj
+        when true
           dump_variant_type variant, Com::Rookout::Variant::Type::VARIANT_LONG
           variant.long_value = 1
-        elsif obj == false
+        when false
           dump_variant_type variant, Com::Rookout::Variant::Type::VARIANT_LONG
           variant.long_value = 0
-        elsif obj.is_a? Integer
+        when Integer
           dump_integer obj, variant
-        elsif obj.is_a? Float
+        when Float
           dump_variant_type variant, Com::Rookout::Variant::Type::VARIANT_DOUBLE
           variant.double_value = obj.to_f
-        elsif obj.is_a? BigDecimal
+        when BigDecimal
           dump_string obj.to_s, variant, config # TODO: NS: This might cut the decimal value, is that ok?
-        elsif obj.is_a? Complex
+        when Complex
           dump_variant_type variant, Com::Rookout::Variant::Type::VARIANT_COMPLEX
           variant.complex_value = Com::Rookout::Variant::Complex.new real: obj.real.to_f,
                                                                      imaginary: obj.imaginary.to_f

data/lib/rookout/processor/namespaces/container_namespace.rb

@@ -11,6 +11,7 @@ module Rookout
 
       class ContainerNamespace < Namespace
         def initialize hash = {}
+          super()
           @hash = hash
         end
 

data/lib/rookout/processor/namespaces/frame_namespace.rb

@@ -8,6 +8,7 @@ module Rookout
 
       class FrameNamespace < Namespace
         def initialize binding
+          super()
           @binding = binding
         end
 
@@ -19,14 +20,12 @@ module Rookout
 
         def call_method name, args
           case name
-          when "filename"
+          when "filename", "module"
             RubyObjectNamespace.new @binding.source_location[0]
           when "line"
             RubyObjectNamespace.new @binding.source_location[1]
           when "function"
             RubyObjectNamespace.new @binding.eval("__method__").to_s
-          when "module"
-            RubyObjectNamespace.new @binding.source_location[0]
           when "locals"
             locals args
           when "dump"

data/lib/rookout/processor/namespaces/namespace.rb

@@ -3,6 +3,7 @@ module Rookout
     module Namespaces
       class Namespace
         require_relative "../../exceptions"
+
         def call_method name, _args
           raise Exceptions::RookMethodNotFound.new(self.class.to_s, name)
         end

data/lib/rookout/processor/namespaces/noop_namespace.rb

@@ -7,10 +7,6 @@ module Rookout
       require_relative "../../protobuf/variant_pb"
 
       class NoopNamespace < Namespace
-        def initialize
-          super
-        end
-
         def call_method _name, _args
           Rookout::Processor::Namespaces::RubyObjectNamespace.new nil
         end

data/lib/rookout/processor/namespaces/ruby_object_namespace.rb

@@ -17,7 +17,10 @@ module Rookout
           @max_string = max_string
         end
 
-        attr_reader :max_depth, :max_width, :max_collection_depth, :max_string
+        attr_reader :max_depth
+        attr_reader :max_width
+        attr_reader :max_collection_depth
+        attr_reader :max_string
       end
 
       OBJECT_DUMP_CONFIG_STRICT = ObjectDumpConfig.new 2, 10, 2, 128
@@ -37,12 +40,14 @@ module Rookout
       class RubyObjectNamespace < Namespace
         include RubyObjectSerializer
 
-        def initialize obj, dump_config = OBJECT_DUMP_CONFIG_DEFAULT
+        def initialize obj, dump_config = nil
+          super()
           @obj = obj
-          @dump_config = dump_config
+          @dump_config = dump_config || OBJECT_DUMP_CONFIG_DEFAULT
         end
 
-        attr_reader :obj, :dump_config
+        attr_reader :obj
+        attr_reader :dump_config
 
         def tailor_limits!
           if @obj.is_a? String
@@ -62,18 +67,11 @@ module Rookout
         end
 
         def read_key key
-          if @obj.is_a?(Array) || @obj.is_a?(String)
-            key_int = key.to_i
-            return RubyObjectNamespace.new @obj[key_int] if key_int >= 0 && key_int < @obj.length
-            raise Exceptions::RookKeyNotFound, key
-
-          elsif @obj.is_a? Hash
-            return RubyObjectNamespace.new @obj[key] if @obj.key? key
-            return RubyObjectNamespace.new @obj[key.to_sym] if @obj.key? key.to_s.to_sym
-
-            @obj.each { |it, value| return RubyObjectNamespace.new value if it.to_s == key }
-
-            raise Exceptions::RookKeyNotFound, key
+          case @obj
+          when Array, String
+            read_key_as_int key
+          when Hash
+            read_key_from_hash key
           else
             raise Exceptions::RookInvalidObjectForAccess.new(@obj.class.to_s, "ReadKey")
           end
@@ -94,6 +92,23 @@ module Rookout
           dump_raw_object @obj, 0, @dump_config, log_object_errors
         end
       end
+
+      private
+
+      def read_key_as_int key
+        key_int = key.to_i
+        return RubyObjectNamespace.new @obj[key_int] if key_int >= 0 && key_int < @obj.length
+        raise Exceptions::RookKeyNotFound, key
+      end
+
+      def read_key_from_hash key
+        return RubyObjectNamespace.new @obj[key] if @obj.key? key
+        return RubyObjectNamespace.new @obj[key.to_sym] if @obj.key? key.to_s.to_sym
+
+        @obj.each { |it, value| return RubyObjectNamespace.new value if it.to_s == key }
+
+        raise Exceptions::RookKeyNotFound, key
+      end
     end
   end
 end

data/lib/rookout/processor/namespaces/ruby_object_serializer.rb

@@ -24,6 +24,7 @@ module Rookout
 
         # rubocop:disable Metrics/PerceivedComplexity
         # rubocop:disable Metrics/CyclomaticComplexity
+        # rubocop:disable Style/ClassEqualityComparison
         def unsafe_dump_object obj, current_depth, config, log_object_errors
           variant = create_base_variant obj, current_depth, config, log_object_errors
 
@@ -49,7 +50,7 @@ module Rookout
 
           variant
         end
-
+        # rubocop:enable Style/ClassEqualityComparison
         # rubocop:enable Metrics/CyclomaticComplexity
         # rubocop:enable Metrics/PerceivedComplexity
 
@@ -83,23 +84,24 @@ module Rookout
         INT64_MAX = (1 << 63) - 1
 
         def dump_numeric obj, variant
-          if obj == true
+          case obj
+          when true
             variant.variant_type = :VARIANT_INT
             variant.int_value = 1
-          elsif obj == false
+          when false
             variant.variant_type = :VARIANT_INT
             variant.int_value = 0
-          elsif obj.is_a? Integer
+          when Integer
             dump_integer obj, variant
-          elsif obj.is_a? Float
+          when Float
             variant.variant_type = :VARIANT_DOUBLE
             variant.double_value = obj.to_f
-          elsif obj.is_a? BigDecimal
+          when BigDecimal
             serialized_decimal = obj.to_s
             variant.variant_type = :VARIANT_STRING
             variant.string_value = Com::Rookout::Variant::String.new value: serialized_decimal,
                                                                      original_size: serialized_decimal.length
-          elsif obj.is_a? Complex
+          when Complex
             variant.variant_type = :VARIANT_COMPLEX
             variant.complex_value = Com::Rookout::Variant::Complex.new real: obj.real.to_f,
                                                                        imaginary: obj.imaginary.to_f

data/lib/rookout/processor/namespaces/ruby_utils_namespace.rb

@@ -9,10 +9,6 @@ module Rookout
       require_relative "ruby_object_namespace"
 
       class RubyUtilsNamespace < Namespace
-        def initialize
-          super
-        end
-
         def call_method name, args
           case name
           when "exception"

data/lib/rookout/processor/namespaces/stack_namespace.rb

@@ -7,6 +7,7 @@ module Rookout
 
       class StackNamespace < Namespace
         def initialize backtrace, offset = 0
+          super()
           @backtrace = backtrace
           @offset = offset
         end

data/lib/rookout/processor/namespaces/traceback_namespace.rb

@@ -8,12 +8,15 @@ module Rookout
 
       class TracebackNamespace < Namespace
         def initialize backtrace, offset, depth
+          super()
           @backtrace = backtrace
           @offset = offset
           @depth = depth
         end
 
-        attr_reader :backtrace, :offset, :depth
+        attr_reader :backtrace
+        attr_reader :offset
+        attr_reader :depth
 
         def read_key key
           FrameNamespace.new @backtrace[key + @offset]

data/lib/rookout/processor/operations/set_operation.rb

@@ -11,6 +11,8 @@ module Rookout
 
       class SetOperation < Operation
         def initialize configuration, factory
+          super()
+
           @paths = []
 
           configuration["paths"].each do |key, value|

data/lib/rookout/processor/paths/arithmetic_path.rb

@@ -13,6 +13,8 @@ module Rookout
 
       class ArithmeticPath < Path
         def initialize path
+          super()
+
           if path.is_a? Hash
             path = path["path"] || path[:path]
           end
@@ -20,7 +22,7 @@ module Rookout
           @raw_path = path
 
           if path.start_with?("NOT(") && path.end_with?(")")
-            @path = path["NOT(".length..-2]
+            @path = path[("NOT(".length)..-2]
             @negation = true
           else
             @path = path

data/lib/rookout/processor/paths/canopy/actions.rb

@@ -79,6 +79,8 @@ module Rookout
           end
 
           # rubocop:disable Metrics/AbcSize
+          # rubocop:disable Metrics/CyclomaticComplexity
+          # rubocop:disable Metrics/PerceivedComplexity
           def make_comp_exp _input, _start, _finish, elements
             # We can assume the following: atom ( opt_ atom )*
             # the first (which must be) will be simple atom
@@ -108,7 +110,7 @@ module Rookout
                 flat_elements.each_with_index do |e, index|
                   if e.is_a?(Opt) && e.level == level
                     result = e.execute_operation flat_elements[index - 1], flat_elements[index + 1]
-                    flat_elements = flat_elements[0...index - 1] + [result] + flat_elements[index + 2..-1]
+                    flat_elements = flat_elements[0...index - 1] + [result] + flat_elements[index + 2..]
                     restart_scan = true
                     break
                   end
@@ -122,6 +124,8 @@ module Rookout
 
             flat_elements[0]
           end
+          # rubocop:enable Metrics/PerceivedComplexity
+          # rubocop:enable Metrics/CyclomaticComplexity
           # rubocop:enable Metrics/AbcSize
 
           def make_opt _input, _start, _finish, elements

data/lib/rookout/processor/paths/canopy/consts.rb

@@ -50,9 +50,10 @@ module Rookout
             # If it's a dict, scan the keys, compensating for additional abstraction
             if b.is_a? Hash
               b.each_key do |key|
-                if key.is_a? ObjectMarker
+                case key
+                when ObjectMarker
                   return true if key.obj == a
-                elsif key == a
+                when a
                   return true
                 end
               end
@@ -62,9 +63,10 @@ module Rookout
             # If it's an array, scan the value, compensating for additional abstraction
             if b.is_a? Array
               b.each do |value|
-                if value.is_a? ObjectMarker
+                case value
+                when ObjectMarker
                   return true if value.obj == a
-                elsif value == a
+                when a
                   return true
                 end
               end

data/lib/rookout/processor/paths/canopy/markers.rb

@@ -7,7 +7,11 @@ module Rookout
         require_relative "../../../exceptions"
         require_relative "../../namespaces/container_namespace"
 
-        class Marker; end
+        # rubocop:disable Lint/EmptyClass
+        class Marker
+          # Empty base class
+        end
+        # rubocop:enable Lint/EmptyClass
 
         class Operation < Marker
           def read _namespace, _create
@@ -21,6 +25,7 @@ module Rookout
 
         class FunctionOperation < Operation
           def initialize name, args
+            super()
             @name = name
             @args = args
           end
@@ -32,6 +37,7 @@ module Rookout
 
         class AttributeOperation < Operation
           def initialize name
+            super()
             @name = name
           end
 
@@ -49,6 +55,7 @@ module Rookout
 
         class LookupOperation < Operation
           def initialize text
+            super()
             if (text.start_with?("'") && text.end_with?("'")) || (text.start_with?("\"") && text.end_with?("\""))
               @key = text[1...-1]
             else
@@ -63,11 +70,13 @@ module Rookout
 
         class ObjectMarker < Marker
           def initialize text, obj
+            super()
             @text = text
             @obj = obj
           end
 
-          attr_reader :text, :obj
+          attr_reader :text
+          attr_reader :obj
 
           def to_s
             @obj.to_s
@@ -136,6 +145,8 @@ module Rookout
 
         class Opt < Marker
           def initialize opt
+            super()
+
             @opt = opt
             @level = nil
 
@@ -160,9 +171,8 @@ module Rookout
               return
             end
 
-            unless first_obj.nil?
-              raise Exceptions::RookNonPrimitiveObjectType, first_obj.class.to_s unless
-                PRIMITIVES.include? first_obj.class
+            if !first_obj.nil? && !(PRIMITIVES.include? first_obj.class)
+              raise Exceptions::RookNonPrimitiveObjectType, first_obj.class.to_s
             end
 
             return if second_obj.nil?

data/lib/rookout/processor/processor_factory.rb

@@ -5,8 +5,6 @@ module Rookout
     require_relative "paths/arithmetic_path"
 
     class ProcessorFactory
-      def initialize; end
-
       def create_operation configuration
         Operations::SetOperation.new configuration, self
       end

data/lib/rookout/processor/rook_error.rb

@@ -24,7 +24,9 @@ module Rookout
         end
       end
 
-      attr_reader :message, :exception, :parameters
+      attr_reader :message
+      attr_reader :exception
+      attr_reader :parameters
 
       def dumps
         parameters = NamespaceSerializer.dump Namespaces::RubyObjectNamespace.new(@parameters), false

data/lib/rookout/rookout_singleton.rb

@@ -40,7 +40,7 @@ module Rookout
       @services_started = false
     end
 
-    def connect token: nil, host: nil, port: nil, proxy: nil, labels: [], async_start: false, fork: false,
+    def connect token: nil, host: nil, port: nil, proxy: nil, labels: {}, async_start: false, fork: false,
                 throw_errors: false
       raise Exceptions::RookInterfaceException, "Multiple connection attempts not supported!" unless @agent_com.nil?
 
@@ -82,9 +82,10 @@ module Rookout
     private
 
     def check_version_supported
-      raise Exceptions:: RookVersionNotSupported.new("platform", RUBY_ENGINE) unless RUBY_ENGINE == "ruby"
+      raise Exceptions::RookVersionNotSupported.new("platform", RUBY_ENGINE) unless RUBY_ENGINE == "ruby"
       raise Exceptions::RookVersionNotSupported.new("version", RUBY_VERSION) unless
-          RUBY_VERSION.start_with?("2.7") || RUBY_VERSION.start_with?("2.6")
+          RUBY_VERSION.start_with?("2.7") || RUBY_VERSION.start_with?("2.6") ||
+          RUBY_VERSION.start_with?("3.0") || RUBY_VERSION.start_with?("3.1")
     end
   end
 end

data/lib/rookout/services/position.rb

@@ -11,7 +11,8 @@ module Rookout
         @method = method
       end
 
-      attr_reader :lineno, :method
+      attr_reader :lineno
+      attr_reader :method
     end
 
     class PositionResolver
@@ -54,7 +55,7 @@ module Rookout
       end
 
       def clear_augs
-        @augs.values.each do |aug_id|
+        @augs.each_value do |aug_id|
           remove_aug aug_id
         end
 

data/lib/rookout/services/tracer.rb

@@ -26,7 +26,7 @@ module Rookout
 
           # We add and remove a dummy trace point to re-align the tracing mechanism as a result of some bug in adding
           # a breakpoint
-          dummy_trace_point = TracePoint.new(:line) {}
+          dummy_trace_point = TracePoint.new(:line) {} # Dummy
           dummy_trace_point.enable target: position.method, target_line: position.lineno
           dummy_trace_point.disable
 
@@ -53,7 +53,7 @@ module Rookout
       end
 
       def clear_augs
-        @augs.values.each do |aug_id|
+        @augs.each_value do |aug_id|
           remove_aug aug_id
         end
 

data/lib/rookout/utils.rb

@@ -15,6 +15,6 @@ module Utils
   def time_to_nanoseconds time_obj
     secs = time_obj.to_i
     nsecs = time_obj.nsec
-    (10**9) * secs + nsecs
+    ((10**9) * secs) + nsecs
   end
 end

data/lib/rookout/version.rb

@@ -1,3 +1,3 @@
 module Rookout
-  VERSION = "0.1.34".freeze
+  VERSION = "0.1.37".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rookout
 version: !ruby/object:Gem::Version
-  version: 0.1.34
+  version: 0.1.37
 platform: ruby
 authors:
 - Liran Haimovitch
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-05-15 00:00:00.000000000 Z
+date: 2022-07-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: binding_of_caller
@@ -86,56 +86,70 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.24.0
+        version: 1.26.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.24.0
+        version: 1.26.1
+- !ruby/object:Gem::Dependency
+  name: async
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.0.0
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.14'
+        version: 5.16.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.14'
+        version: 5.16.2
 - !ruby/object:Gem::Dependency
   name: minitest-autotest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.1.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.1.1
 - !ruby/object:Gem::Dependency
   name: minitest-focus
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: 1.3.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: 1.3.1
 - !ruby/object:Gem::Dependency
   name: minitest-rg
   requirement: !ruby/object:Gem::Requirement