lapsoss 0.2.0 → 0.3.1

data/lib/lapsoss/backtrace_processor.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'active_support/cache'
-require 'active_support/core_ext/numeric/time'
-require_relative 'backtrace_frame'
+require "set"
+require "active_support/cache"
+require "active_support/core_ext/numeric/time"
 
 module Lapsoss
   class BacktraceProcessor
@@ -42,10 +42,10 @@ module Lapsoss
                         exclude_patterns: config.backtrace_exclude_patterns,
                         strip_load_path: config.backtrace_strip_load_path
                       }
-                    else
+      else
                       # Hash passed
                       config
-                    end
+      end
 
       @config = DEFAULT_CONFIG.merge(config_hash)
       @file_cache = ActiveSupport::Cache::MemoryStore.new(
@@ -83,10 +83,13 @@ module Lapsoss
 
       frames = process_backtrace(exception.backtrace)
 
-      # Add exception-specific context
-      frames.each_with_index do |frame, index|
-        frame.define_singleton_method(:crash_frame?) { index.zero? }
-        frame.define_singleton_method(:exception_class) { exception.class.name }
+      # Wrap frames with exception-specific context
+      frames = frames.map.with_index do |frame, index|
+        ExceptionBacktraceFrame.new(
+          frame,
+          exception_class: exception.class.name,
+          is_crash_frame: index.zero?
+        )
       end
 
       # Follow exception causes if requested
@@ -179,7 +182,7 @@ module Lapsoss
         file_cache: {
           # ActiveSupport::Cache::MemoryStore doesn't expose detailed stats
           # but we can provide basic info
-          type: 'ActiveSupport::Cache::MemoryStore',
+          type: "ActiveSupport::Cache::MemoryStore",
           configured_size: @file_cache.options[:size]
         },
         config: @config,
@@ -207,8 +210,8 @@ module Lapsoss
       return nil if line_index.negative? || line_index >= lines.length
 
       # Calculate context range
-      start_line = [0, line_index - context_lines].max
-      end_line = [lines.length - 1, line_index + context_lines].min
+      start_line = [ 0, line_index - context_lines ].max
+      end_line = [ lines.length - 1, line_index + context_lines ].min
 
       {
         pre_context: lines[start_line...line_index],
@@ -234,7 +237,7 @@ module Lapsoss
     def parse_frames(backtrace)
       load_paths = determine_load_paths
       frames = backtrace.map do |line|
-        BacktraceFrame.new(
+        BacktraceFrameFactory.from_raw_line(
           line,
           in_app_patterns: @config[:in_app_patterns],
           exclude_patterns: @config[:exclude_patterns],
@@ -248,7 +251,7 @@ module Lapsoss
 
     def filter_frames(frames)
       # Remove excluded frames
-      frames = frames.reject(&:excluded?)
+      frames = frames.reject { |frame| frame.excluded?(@config[:exclude_patterns]) }
 
       # Filter gems if configured
       unless @config[:include_gems_in_context]
@@ -284,9 +287,9 @@ module Lapsoss
       # Get tail frames (original cause)
       tail_frames = if tail_count.positive?
                       frames.last(tail_count)
-                    else
+      else
                       []
-                    end
+      end
 
       head_frames + tail_frames
     end
@@ -296,8 +299,9 @@ module Lapsoss
       context_frames = frames.select(&:app_frame?)
       context_frames += frames.select(&:library_frame?).first(3)
 
-      context_frames.each do |frame|
-        frame.add_code_context(self, @config[:context_lines])
+      context_frames.each_with_index do |frame, _index|
+        updated_frame = frame.add_code_context(self, @config[:context_lines])
+        frames[frames.index(frame)] = updated_frame if updated_frame
       end
     end
 
@@ -305,7 +309,7 @@ module Lapsoss
       seen = Set.new
       frames.select do |frame|
         # Create a key based on filename, line, and method
-        key = [frame.filename, frame.line_number, frame.function].join(':')
+        key = [ frame.filename, frame.line_number, frame.function ].join(":")
 
         if seen.include?(key)
           false
@@ -325,16 +329,16 @@ module Lapsoss
       # Add common Rails paths if in Rails
       if defined?(Rails) && Rails.respond_to?(:root) && Rails.root
         paths << Rails.root.to_s
-        paths << Rails.root.join('app').to_s
-        paths << Rails.root.join('lib').to_s
-        paths << Rails.root.join('config').to_s
+        paths << Rails.root.join("app").to_s
+        paths << Rails.root.join("lib").to_s
+        paths << Rails.root.join("config").to_s
       end
 
       # Add current working directory
       paths << Dir.pwd
 
       # Add gem paths
-      paths.concat(Gem.path.map { |p| File.join(p, 'gems') }) if defined?(Gem)
+      paths.concat(Gem.path.map { |p| File.join(p, "gems") }) if defined?(Gem)
 
       # Sort by length (longest first) for better matching
       paths.uniq.sort_by(&:length).reverse

data/lib/lapsoss/client.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require 'concurrent'
-require_relative 'scope'
-require_relative 'current'
+require "concurrent"
 
 module Lapsoss
   class Client
@@ -47,7 +45,7 @@ module Lapsoss
       original_scope = current_scope
 
       # Create a merged scope with the new context
-      merged_scope = MergedScope.new([context], original_scope)
+      merged_scope = MergedScope.new([ context ], original_scope)
       Current.scope = merged_scope
 
       yield(merged_scope)

data/lib/lapsoss/configuration.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'validators'
-require 'active_support/configurable'
+require "active_support/configurable"
 
 module Lapsoss
   class Configuration
@@ -15,7 +14,7 @@ module Lapsoss
                   :backtrace_context_lines, :backtrace_in_app_patterns, :backtrace_exclude_patterns,
                   :backtrace_strip_load_path, :backtrace_max_frames, :backtrace_enable_code_context,
                   :enable_pipeline, :pipeline_builder, :sampling_strategy,
-                  :skip_rails_cache_errors, :force_sync_http
+                  :skip_rails_cache_errors, :force_sync_http, :capture_request_context
     attr_reader :fingerprint_callback, :environment, :before_send, :sample_rate, :error_handler, :transport_timeout,
                 :transport_max_retries, :transport_initial_backoff, :transport_max_backoff, :transport_backoff_multiplier, :transport_ssl_verify, :default_context, :adapter_configs
 
@@ -64,6 +63,8 @@ module Lapsoss
       @skip_rails_cache_errors = true
       # HTTP client settings
       @force_sync_http = false
+      # Capture request context in middleware
+      @capture_request_context = true
     end
 
     # Register a named adapter configuration
@@ -172,7 +173,6 @@ module Lapsoss
 
     # Pipeline configuration
     def configure_pipeline
-      require_relative 'pipeline'
       @pipeline_builder = PipelineBuilder.new
       yield(@pipeline_builder) if block_given?
       @pipeline_builder
@@ -184,8 +184,6 @@ module Lapsoss
 
     # Sampling configuration
     def configure_sampling(strategy = nil, &block)
-      require_relative 'sampling'
-
       if strategy
         @sampling_strategy = strategy
       elsif block_given?
@@ -194,8 +192,6 @@ module Lapsoss
     end
 
     def create_sampling_strategy
-      require_relative 'sampling'
-
       case @sampling_strategy
       when Symbol
         case @sampling_strategy
@@ -219,80 +215,80 @@ module Lapsoss
 
     # Validation and setter overrides
     def sample_rate=(value)
-      validate_sample_rate!(value, 'sample_rate') if value
+      validate_sample_rate!(value, "sample_rate") if value
       @sample_rate = value
     end
 
     def before_send=(value)
-      validate_callable!(value, 'before_send')
+      validate_callable!(value, "before_send")
       @before_send = value
     end
 
     def error_handler=(value)
-      validate_callable!(value, 'error_handler')
+      validate_callable!(value, "error_handler")
       @error_handler = value
     end
 
     def environment=(value)
-      validate_environment!(value, 'environment') if value
+      validate_environment!(value, "environment") if value
       @environment = value&.to_s
     end
 
     def transport_timeout=(value)
-      validate_timeout!(value, 'transport_timeout') if value
+      validate_timeout!(value, "transport_timeout") if value
       @transport_timeout = value
     end
 
     def transport_max_retries=(value)
-      validate_retries!(value, 'transport_max_retries') if value
+      validate_retries!(value, "transport_max_retries") if value
       @transport_max_retries = value
     end
 
     def transport_initial_backoff=(value)
-      validate_timeout!(value, 'transport_initial_backoff') if value
+      validate_timeout!(value, "transport_initial_backoff") if value
       @transport_initial_backoff = value
     end
 
     def transport_max_backoff=(value)
-      validate_timeout!(value, 'transport_max_backoff') if value
+      validate_timeout!(value, "transport_max_backoff") if value
       @transport_max_backoff = value
     end
 
     def transport_backoff_multiplier=(value)
       if value
-        validate_type!(value, [Numeric], 'transport_backoff_multiplier')
-        validate_numeric_range!(value, 1.0..10.0, 'transport_backoff_multiplier')
+        validate_type!(value, [ Numeric ], "transport_backoff_multiplier")
+        validate_numeric_range!(value, 1.0..10.0, "transport_backoff_multiplier")
       end
       @transport_backoff_multiplier = value
     end
 
     def transport_ssl_verify=(value)
-      validate_boolean!(value, 'transport_ssl_verify') if value
+      validate_boolean!(value, "transport_ssl_verify") if value
       @transport_ssl_verify = value
     end
 
     def fingerprint_callback=(value)
-      validate_callable!(value, 'fingerprint_callback')
+      validate_callable!(value, "fingerprint_callback")
       @fingerprint_callback = value
     end
 
     # Configuration validation
     def validate!
-      validate_sample_rate!(@sample_rate, 'sample_rate') if @sample_rate
-      validate_callable!(@before_send, 'before_send')
-      validate_callable!(@error_handler, 'error_handler')
-      validate_callable!(@fingerprint_callback, 'fingerprint_callback')
-      validate_environment!(@environment, 'environment') if @environment
+      validate_sample_rate!(@sample_rate, "sample_rate") if @sample_rate
+      validate_callable!(@before_send, "before_send")
+      validate_callable!(@error_handler, "error_handler")
+      validate_callable!(@fingerprint_callback, "fingerprint_callback")
+      validate_environment!(@environment, "environment") if @environment
 
       # Validate transport settings
-      validate_timeout!(@transport_timeout, 'transport_timeout')
-      validate_retries!(@transport_max_retries, 'transport_max_retries')
-      validate_timeout!(@transport_initial_backoff, 'transport_initial_backoff')
-      validate_timeout!(@transport_max_backoff, 'transport_max_backoff')
+      validate_timeout!(@transport_timeout, "transport_timeout")
+      validate_retries!(@transport_max_retries, "transport_max_retries")
+      validate_timeout!(@transport_initial_backoff, "transport_initial_backoff")
+      validate_timeout!(@transport_max_backoff, "transport_max_backoff")
 
       if @transport_backoff_multiplier
-        validate_type!(@transport_backoff_multiplier, [Numeric], 'transport_backoff_multiplier')
-        validate_numeric_range!(@transport_backoff_multiplier, 1.0..10.0, 'transport_backoff_multiplier')
+        validate_type!(@transport_backoff_multiplier, [ Numeric ], "transport_backoff_multiplier")
+        validate_numeric_range!(@transport_backoff_multiplier, 1.0..10.0, "transport_backoff_multiplier")
       end
 
       # Validate that initial backoff is less than max backoff
@@ -311,7 +307,7 @@ module Lapsoss
 
     def validate_adapter_config!(name, config)
       validate_presence!(config[:type], "adapter type for '#{name}'")
-      validate_type!(config[:settings], [Hash], "adapter settings for '#{name}'")
+      validate_type!(config[:settings], [ Hash ], "adapter settings for '#{name}'")
     end
   end
 end

data/lib/lapsoss/current.rb

@@ -1,9 +1,17 @@
 # frozen_string_literal: true
 
-require 'active_support/all'
+require "active_support/all"
 
 module Lapsoss
   class Current < ActiveSupport::CurrentAttributes
-    attribute :scope
+    attribute :scope, default: -> { Scope.new }
+
+    def self.with_clean_scope
+      previous_scope = scope
+      self.scope = Scope.new
+      yield
+    ensure
+      self.scope = previous_scope
+    end
   end
 end

data/lib/lapsoss/event.rb

@@ -1,9 +1,5 @@
 # frozen_string_literal: true
 
-require_relative 'scrubber'
-require_relative 'fingerprinter'
-require_relative 'backtrace_processor'
-
 module Lapsoss
   class Event
     attr_accessor :type, :timestamp, :level, :message, :exception, :context, :environment, :fingerprint,
@@ -12,7 +8,7 @@ module Lapsoss
     def initialize(type:, level: :info, **attributes)
       @type = type
       @level = level
-      @timestamp = Time.zone.now
+      @timestamp = Utils.current_time
       @context = {}
       @environment = Lapsoss.configuration.environment
 
@@ -23,6 +19,9 @@ module Lapsoss
       # Process backtrace if we have an exception
       @backtrace_frames = process_backtrace if @exception
 
+      # Set message from exception if not provided
+      @message ||= @exception.message if @exception
+
       # Generate fingerprint after all attributes are set (unless explicitly set to nil or a value)
       @fingerprint = generate_fingerprint if @fingerprint.nil? && !attributes.key?(:fingerprint)
     end
@@ -104,5 +103,29 @@ module Lapsoss
 
       @backtrace_frames.map(&:to_h)
     end
+
+    public
+
+    def exception_type
+      exception&.class&.name
+    end
+
+    def backtrace
+      if @backtrace_frames
+        @backtrace_frames.map(&:raw_line)
+      elsif exception
+        exception.backtrace
+      else
+        []
+      end
+    end
+
+    def request_context
+      # Request context is stored in extra by the middleware
+      # Check both string and symbol keys for compatibility
+      return nil unless context[:extra]
+
+      context[:extra]["request"] || context[:extra][:request]
+    end
   end
 end

data/lib/lapsoss/exception_backtrace_frame.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+module Lapsoss
+  # Wrapper for BacktraceFrame that adds exception-specific metadata
+  class ExceptionBacktraceFrame
+    attr_reader :frame, :exception_class, :is_crash_frame
+
+    def initialize(frame, exception_class: nil, is_crash_frame: false)
+      @frame = frame
+      @exception_class = exception_class
+      @is_crash_frame = is_crash_frame
+    end
+
+    def crash_frame?
+      @is_crash_frame
+    end
+
+    # Delegate all other methods to the wrapped frame
+    def method_missing(method, *, &)
+      if @frame.respond_to?(method)
+        @frame.send(method, *, &)
+      else
+        super
+      end
+    end
+
+    def respond_to_missing?(method, include_private = false)
+      @frame.respond_to?(method, include_private) || super
+    end
+
+    # Override to_h to include exception metadata
+    def to_h
+      @frame.to_h.merge(
+        exception_class: @exception_class,
+        crash_frame: @is_crash_frame
+      ).compact
+    end
+  end
+end

data/lib/lapsoss/exclusion_configuration.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module Lapsoss
+  # Configuration helper for exclusions
+  module ExclusionConfiguration
+    def self.configure_exclusions(config, preset: nil, **custom_config)
+      exclusion_config = if preset
+                           case preset
+                           when Array
+                             ExclusionPresets.combined(preset)
+                           else
+                             ExclusionPresets.send(preset)
+                           end
+      else
+                           {}
+      end
+
+      # Merge custom configuration
+      exclusion_config.merge!(custom_config)
+
+      # Create exclusion filter
+      exclusion_filter = ExclusionFilter.new(exclusion_config)
+
+      # Add to configuration
+      config.exclusion_filter = exclusion_filter
+
+      exclusion_filter
+    end
+  end
+end