lapsoss 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 34f511accdaa033204fa71e2a498af0a795cc29071afa227c4d59bd224b81bd8
-  data.tar.gz: 2cc9bcd3d87c9cc8ccf026106c5a9414cfd42fbb896de95df5bcf53814007842
+  metadata.gz: 387dbefc113baa7968d4343b8b5135dca057581757056acf740a4234d8fcab4a
+  data.tar.gz: 3d304210fbe0a71c3679be91cd4e867a33966dab2b8f10e441c11f7a3465d9c2
 SHA512:
-  metadata.gz: dc1f028a9f02b5862f458794e26144d571ccb902f963aecdfda2d11e12346b57f46a8ef3fc9ea4c9a5ebf193a8fc4f6ca02988acb7dc518e5e248f20a4621918
-  data.tar.gz: a2bb32697511091f34fe8736261bea39c78e070c31c2ef35dc1b2a20c865844ac1ec0e986729d80c17a31535de1e5bf7c22a030788898c2b7fa35b5d49bee517
+  metadata.gz: a166ab065134b2a6f84d17de91a7e67b64b8131e5a1d2380e4411cc21eb34692e5da56a531c89b04b9a093782b3b9773ad338769e384adb90155bb1ddceeb9cc
+  data.tar.gz: e5f4cf7fcc987d357a7d03b426bd120090c6190cbe37808139b53b3e939d6e27ecd75a72cef6e4d1ddff14bfb9d5c6848ca0b24f3b752b705a58aca9dde64a9c

data/lib/lapsoss/adapters/appsignal_adapter.rb

@@ -1,21 +1,20 @@
 # frozen_string_literal: true
 
-require "json"
-require "socket"
-require_relative "../http_client"
+require 'json'
+require 'socket'
 
 module Lapsoss
   module Adapters
     class AppsignalAdapter < Base
-      PUSH_API_URI = "https://push.appsignal.com"
-      ERRORS_API_URI = "https://appsignal-endpoint.net"
-      JSON_CONTENT_TYPE = "application/json; charset=UTF-8"
+      PUSH_API_URI = 'https://push.appsignal.com'
+      ERRORS_API_URI = 'https://appsignal-endpoint.net'
+      JSON_CONTENT_TYPE = 'application/json; charset=UTF-8'
 
       def initialize(name, settings = {})
-        super(name, settings)
-        @push_api_key = settings[:push_api_key] || ENV["APPSIGNAL_PUSH_API_KEY"]
-        @frontend_api_key = settings[:frontend_api_key] || ENV["APPSIGNAL_FRONTEND_API_KEY"]
-        @app_name = settings[:app_name] || ENV["APPSIGNAL_APP_NAME"]
+        super
+        @push_api_key = settings[:push_api_key] || ENV.fetch('APPSIGNAL_PUSH_API_KEY', nil)
+        @frontend_api_key = settings[:frontend_api_key] || ENV.fetch('APPSIGNAL_FRONTEND_API_KEY', nil)
+        @app_name = settings[:app_name] || ENV.fetch('APPSIGNAL_APP_NAME', nil)
         @environment = Lapsoss.configuration.environment
 
         validate_settings!
@@ -31,7 +30,7 @@ module Lapsoss
         return unless payload
 
         path = "/errors?api_key=#{@frontend_api_key}"
-        headers = { "Content-Type" => JSON_CONTENT_TYPE }
+        headers = { 'Content-Type' => JSON_CONTENT_TYPE }
 
         begin
           @errors_client.post(path, body: JSON.generate(payload), headers: headers)
@@ -65,7 +64,7 @@ module Lapsoss
       def build_exception_payload(event)
         {
           timestamp: event.timestamp.to_i,
-          namespace: event.context[:namespace] || "backend",
+          namespace: event.context[:namespace] || 'backend',
           error: {
             name: event.exception.class.name,
             message: event.exception.message,
@@ -83,24 +82,24 @@ module Lapsoss
         # Instead of creating fake exceptions, we'll structure the message properly
         # but clearly indicate it's a log message, not an exception
 
-        unless [:error, :fatal, :critical].include?(event.level)
+        unless %i[error fatal critical].include?(event.level)
           # Log when messages are dropped due to level filtering
           Lapsoss.configuration.logger&.debug(
             "[Lapsoss::AppsignalAdapter] Dropping message with level '#{event.level}' - " \
-            "AppSignal only supports :error, :fatal, and :critical levels for messages"
+            'AppSignal only supports :error, :fatal, and :critical levels for messages'
           )
           return nil
         end
 
         {
-          action: event.context[:action] || "log_message",
-          path: event.context[:path] || "/",
+          action: event.context[:action] || 'log_message',
+          path: event.context[:path] || '/',
           exception: {
             # AppSignal requires exception format for messages - this isn't a real exception
             # but rather a way to send structured log messages through their error API
-            name: "LogMessage",  # Clear indication this is a log message
+            name: 'LogMessage', # Clear indication this is a log message
             message: event.message,
-            backtrace: []  # No fake backtrace for log messages
+            backtrace: [] # No fake backtrace for log messages
           },
           tags: stringify_hash(event.context[:tags]),
           params: stringify_hash(event.context[:params]),
@@ -111,9 +110,9 @@ module Lapsoss
 
       def build_environment_context(event)
         {
-          "hostname" => Socket.gethostname,
-          "app_name" => @app_name,
-          "environment" => @environment
+          'hostname' => Socket.gethostname,
+          'app_name' => @app_name,
+          'environment' => @environment
         }.merge(stringify_hash(event.context[:environment] || {}))
       end
 
@@ -123,13 +122,13 @@ module Lapsoss
 
       def validate_settings!
         unless @push_api_key || @frontend_api_key
-          raise ValidationError, "AppSignal API key is required (either push_api_key or frontend_api_key)"
+          raise ValidationError, 'AppSignal API key is required (either push_api_key or frontend_api_key)'
         end
 
-        validate_api_key!(@push_api_key, "AppSignal push API key", format: :uuid) if @push_api_key
-        validate_api_key!(@frontend_api_key, "AppSignal frontend API key", format: :uuid) if @frontend_api_key
-        validate_presence!(@app_name, "AppSignal app name") if @app_name
-        validate_environment!(@environment, "AppSignal environment") if @environment
+        validate_api_key!(@push_api_key, 'AppSignal push API key', format: :uuid) if @push_api_key
+        validate_api_key!(@frontend_api_key, 'AppSignal frontend API key', format: :uuid) if @frontend_api_key
+        validate_presence!(@app_name, 'AppSignal app name') if @app_name
+        validate_environment!(@environment, 'AppSignal environment') if @environment
       end
     end
   end

data/lib/lapsoss/adapters/base.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative "../http_client"
-require_relative "../validators"
+require_relative '../http_client'
+require_relative '../validators'
 
 module Lapsoss
   module Adapters

data/lib/lapsoss/adapters/bugsnag_adapter.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module Lapsoss
+  module Adapters
+    # Bugsnag adapter - backwards compatibility with InsightHub adapter
+    # This allows users to configure with :bugsnag type but uses InsightHub implementation
+    # The InsightHub adapter already checks for BUGSNAG_API_KEY environment variable
+    class BugsnagAdapter < InsightHubAdapter
+      # Inherits all functionality from InsightHubAdapter
+    end
+  end
+end

data/lib/lapsoss/adapters/insight_hub_adapter.rb

@@ -1,25 +1,23 @@
 # frozen_string_literal: true
 
-require "json"
-require "socket"
-require_relative "../http_client"
-require_relative "../backtrace_processor"
+require 'json'
+require 'socket'
 
 module Lapsoss
   module Adapters
     # Adapter for Insight Hub (formerly Bugsnag)
     # Note: The API endpoints still use bugsnag.com domains for backwards compatibility
     class InsightHubAdapter < Base
-      NOTIFY_URI = "https://notify.bugsnag.com"
-      SESSION_URI = "https://sessions.bugsnag.com"
-      JSON_CONTENT_TYPE = "application/json"
+      NOTIFY_URI = 'https://notify.bugsnag.com'
+      SESSION_URI = 'https://sessions.bugsnag.com'
+      JSON_CONTENT_TYPE = 'application/json'
 
       def initialize(name, settings = {})
-        super(name, settings)
-        @api_key = settings[:api_key] || ENV["INSIGHT_HUB_API_KEY"] || ENV["BUGSNAG_API_KEY"]
+        super
+        @api_key = settings[:api_key] || ENV['INSIGHT_HUB_API_KEY'] || ENV.fetch('BUGSNAG_API_KEY', nil)
         @release_stage = settings[:release_stage] || Lapsoss.configuration.environment
         @app_version = settings[:app_version]
-        @app_type = settings[:app_type] || "ruby"
+        @app_type = settings[:app_type] || 'ruby'
 
         validate_settings!
 
@@ -35,14 +33,14 @@ module Lapsoss
         return unless payload
 
         headers = {
-          "Content-Type" => JSON_CONTENT_TYPE,
-          "Bugsnag-Api-Key" => @api_key,
-          "Bugsnag-Payload-Version" => "5.0",
-          "Bugsnag-Sent-At" => Time.now.utc.iso8601
+          'Content-Type' => JSON_CONTENT_TYPE,
+          'Bugsnag-Api-Key' => @api_key,
+          'Bugsnag-Payload-Version' => '5.0',
+          'Bugsnag-Sent-At' => Time.now.utc.iso8601
         }
 
         begin
-          @notify_client.post("/", body: JSON.generate(payload), headers: headers)
+          @notify_client.post('/', body: JSON.generate(payload), headers: headers)
         rescue DeliveryError => e
           # Log the error and potentially notify error handler
           Lapsoss.configuration.logger&.error("[Lapsoss::InsightHubAdapter] Failed to deliver event: #{e.message}")
@@ -72,11 +70,11 @@ module Lapsoss
       def build_payload(event)
         {
           apiKey: @api_key,
-          payloadVersion: "5.0",
+          payloadVersion: '5.0',
           notifier: {
-            name: "Lapsoss",
+            name: 'Lapsoss',
             version: Lapsoss::VERSION,
-            url: "https://github.com/seuros/lapsoss"
+            url: 'https://github.com/seuros/lapsoss'
           },
           events: [build_event(event)]
         }
@@ -98,7 +96,7 @@ module Lapsoss
           severity: map_severity(event.level),
           unhandled: event.context[:unhandled] || false,
           severityReason: {
-            type: event.context[:unhandled] ? "unhandledException" : "handledException"
+            type: event.context[:unhandled] ? 'unhandledException' : 'handledException'
           },
           user: build_user(event),
           context: event.context[:context],
@@ -121,7 +119,7 @@ module Lapsoss
             errorClass: event.exception.class.name,
             message: event.exception.message,
             stacktrace: build_stacktrace(event.exception),
-            type: "ruby"
+            type: 'ruby'
           }]
         }
       end
@@ -131,10 +129,10 @@ module Lapsoss
           exceptions: [{
             # Insight Hub (Bugsnag) requires exception format for messages - this isn't a real exception
             # but rather a way to send structured log messages through their error API
-            errorClass: "LogMessage",  # Clear indication this is a log message
+            errorClass: 'LogMessage', # Clear indication this is a log message
             message: event.message,
             stacktrace: [],  # No fake backtrace for log messages
-            type: "log"      # Mark as log type, not ruby exception
+            type: 'log'      # Mark as log type, not ruby exception
           }]
         }
       end
@@ -146,14 +144,13 @@ module Lapsoss
         @backtrace_processor.format_frames(frames, :bugsnag)
       end
 
-
       def build_breadcrumbs(event)
         breadcrumbs = event.context[:breadcrumbs] || []
         breadcrumbs.map do |crumb|
           {
             timestamp: crumb[:timestamp]&.iso8601 || Time.now.utc.iso8601,
             name: crumb[:message] || crumb[:name],
-            type: crumb[:type] || "manual",
+            type: crumb[:type] || 'manual',
             metaData: crumb[:data] || {}
           }
         end
@@ -172,18 +169,18 @@ module Lapsoss
 
       def map_severity(level)
         case level
-        when :debug, :info then "info"
-        when :warning, :warn then "warning"
-        when :error, :fatal, :critical then "error"
-        else "error"
+        when :debug, :info then 'info'
+        when :warning, :warn then 'warning'
+        when :error, :fatal, :critical then 'error'
+        else 'error'
         end
       end
 
       def validate_settings!
-        validate_presence!(@api_key, "Insight Hub API key")
-        validate_api_key!(@api_key, "Insight Hub API key", format: :alphanumeric) if @api_key
-        validate_environment!(@app_version, "Insight Hub app version") if @app_version
-        validate_type!(@release_stage, [String, Symbol], "Insight Hub release stage") if @release_stage
+        validate_presence!(@api_key, 'Insight Hub API key')
+        validate_api_key!(@api_key, 'Insight Hub API key', format: :alphanumeric) if @api_key
+        validate_environment!(@app_version, 'Insight Hub app version') if @app_version
+        validate_type!(@release_stage, [String, Symbol], 'Insight Hub release stage') if @release_stage
       end
     end
   end

data/lib/lapsoss/adapters/logger_adapter.rb

@@ -1,13 +1,13 @@
 # frozen_string_literal: true
 
-require "logger"
+require 'logger'
 
 module Lapsoss
   module Adapters
     class LoggerAdapter < Base
       def initialize(name, settings = {})
         @logger = settings[:logger] || Logger.new($stdout)
-        super(name, settings)
+        super
       end
 
       def capabilities
@@ -28,12 +28,12 @@ module Lapsoss
         end
 
         # Log breadcrumbs if present in the event context
-        if event.context[:breadcrumbs]&.any?
-          event.context[:breadcrumbs].each do |breadcrumb|
-            breadcrumb_msg = "[BREADCRUMB] [#{breadcrumb[:type].upcase}] #{breadcrumb[:message]}"
-            breadcrumb_msg += " | #{breadcrumb[:metadata].inspect}" unless breadcrumb[:metadata].empty?
-            @logger.debug(breadcrumb_msg)
-          end
+        return unless event.context[:breadcrumbs]&.any?
+
+        event.context[:breadcrumbs].each do |breadcrumb|
+          breadcrumb_msg = "[BREADCRUMB] [#{breadcrumb[:type].upcase}] #{breadcrumb[:message]}"
+          breadcrumb_msg += " | #{breadcrumb[:metadata].inspect}" unless breadcrumb[:metadata].empty?
+          @logger.debug(breadcrumb_msg)
         end
       end
 

data/lib/lapsoss/adapters/rollbar_adapter.rb

@@ -1,22 +1,20 @@
 # frozen_string_literal: true
 
-require "json"
-require "socket"
-require "securerandom"
-require_relative "../http_client"
-require_relative "../backtrace_processor"
+require 'json'
+require 'socket'
+require 'securerandom'
 
 module Lapsoss
   module Adapters
     class RollbarAdapter < Base
-      API_URI = "https://api.rollbar.com"
-      API_VERSION = "1"
-      JSON_CONTENT_TYPE = "application/json"
+      API_URI = 'https://api.rollbar.com'
+      API_VERSION = '1'
+      JSON_CONTENT_TYPE = 'application/json'
 
       def initialize(name, settings = {})
-        super(name, settings)
-        @access_token = settings[:access_token] || ENV["ROLLBAR_ACCESS_TOKEN"]
-        @environment = settings[:environment] || Lapsoss.configuration.environment || "development"
+        super
+        @access_token = settings[:access_token] || ENV.fetch('ROLLBAR_ACCESS_TOKEN', nil)
+        @environment = settings[:environment] || Lapsoss.configuration.environment || 'development'
 
         validate_settings!
 
@@ -32,8 +30,8 @@ module Lapsoss
 
         path = "/api/#{API_VERSION}/item/"
         headers = {
-          "Content-Type" => JSON_CONTENT_TYPE,
-          "X-Rollbar-Access-Token" => @access_token
+          'Content-Type' => JSON_CONTENT_TYPE,
+          'X-Rollbar-Access-Token' => @access_token
         }
 
         begin
@@ -67,8 +65,8 @@ module Lapsoss
           body: build_body(event),
           level: map_level(event.level),
           timestamp: event.timestamp.to_i,
-          platform: "ruby",
-          language: "ruby",
+          platform: 'ruby',
+          language: 'ruby',
           framework: detect_framework,
           context: event.context[:context],
           request: event.context[:request],
@@ -77,7 +75,7 @@ module Lapsoss
           title: event.message || (event.exception&.message if event.type == :exception),
           uuid: SecureRandom.uuid,
           notifier: {
-            name: "lapsoss",
+            name: 'lapsoss',
             version: Lapsoss::VERSION
           }
         }
@@ -132,25 +130,26 @@ module Lapsoss
 
       def map_level(level)
         case level
-        when :debug then "debug"
-        when :info then "info"
-        when :warning, :warn then "warning"
-        when :error then "error"
-        when :fatal, :critical then "critical"
-        else "error"
+        when :debug then 'debug'
+        when :info then 'info'
+        when :warning, :warn then 'warning'
+        when :error then 'error'
+        when :fatal, :critical then 'critical'
+        else 'error'
         end
       end
 
       def detect_framework
-        return "rails" if defined?(Rails)
-        return "sinatra" if defined?(Sinatra)
-        "ruby"
+        return 'rails' if defined?(Rails)
+        return 'sinatra' if defined?(Sinatra)
+
+        'ruby'
       end
 
       def validate_settings!
-        validate_presence!(@access_token, "Rollbar access token")
-        validate_api_key!(@access_token, "Rollbar access token", format: :alphanumeric) if @access_token
-        validate_environment!(@environment, "Rollbar environment") if @environment
+        validate_presence!(@access_token, 'Rollbar access token')
+        validate_api_key!(@access_token, 'Rollbar access token', format: :alphanumeric) if @access_token
+        validate_environment!(@environment, 'Rollbar environment') if @environment
       end
     end
   end

data/lib/lapsoss/adapters/sentry_adapter.rb

@@ -1,19 +1,17 @@
 # frozen_string_literal: true
 
-require "securerandom"
-require_relative "../http_client"
-require_relative "../backtrace_processor"
+require 'securerandom'
 
 module Lapsoss
   module Adapters
     class SentryAdapter < Base
       PROTOCOL_VERSION = 7
-      CONTENT_TYPE = "application/x-sentry-envelope"
+      CONTENT_TYPE = 'application/x-sentry-envelope'
       GZIP_THRESHOLD = 1024 * 30 # 30KB
-      USER_AGENT = "lapsoss/#{Lapsoss::VERSION}"
+      USER_AGENT = "lapsoss/#{Lapsoss::VERSION}".freeze
 
       def initialize(name, settings = {})
-        super(name, settings)
+        super
         validate_settings!
         return unless settings[:dsn]
 
@@ -62,11 +60,11 @@ module Lapsoss
         header = {
           event_id: event.context[:event_id] || SecureRandom.uuid,
           sent_at: Time.now.iso8601,
-          sdk: { name: "lapsoss", version: Lapsoss::VERSION }
+          sdk: { name: 'lapsoss', version: Lapsoss::VERSION }
         }
 
-        item_type = event.type == :transaction ? "transaction" : "event"
-        item_header = { type: item_type, content_type: "application/json" }
+        item_type = event.type == :transaction ? 'transaction' : 'event'
+        item_header = { type: item_type, content_type: 'application/json' }
         item_payload = build_event_payload(event)
 
         [header, item_header, item_payload]
@@ -90,7 +88,7 @@ module Lapsoss
 
       def build_event_payload(event)
         {
-          platform: "ruby",
+          platform: 'ruby',
           level: map_level(event.level),
           timestamp: event.timestamp.to_f,
           environment: @settings[:environment],
@@ -123,9 +121,9 @@ module Lapsoss
 
       def build_headers(compressed)
         {
-          "Content-Type" => CONTENT_TYPE,
-          "X-Sentry-Auth" => auth_header,
-          "Content-Encoding" => ("gzip" if compressed)
+          'Content-Type' => CONTENT_TYPE,
+          'X-Sentry-Auth' => auth_header,
+          'Content-Encoding' => ('gzip' if compressed)
         }.compact
       end
 
@@ -144,16 +142,14 @@ module Lapsoss
         # - Custom service: https://public_key@custom.com/api/v1/errors -> /api/v1/errors
 
         # Extract project ID for auth header (usually the last path segment)
-        path_parts = uri.path.split("/").reject(&:empty?)
-        project_id = path_parts.last || "unknown"
+        path_parts = uri.path.split('/').reject(&:empty?)
+        project_id = path_parts.last || 'unknown'
 
         # Use the DSN path directly - this is what the service expects
         api_path = uri.path
 
         # For standard Sentry DSNs (just /project_id), build the envelope path
-        if path_parts.length == 1 && project_id.match?(/^\d+$/)
-          api_path = "/api/#{project_id}/envelope/"
-        end
+        api_path = "/api/#{project_id}/envelope/" if path_parts.length == 1 && project_id.match?(/^\d+$/)
 
         {
           public_key: uri.user,
@@ -176,21 +172,19 @@ module Lapsoss
 
       def map_level(level)
         case level
-        when :debug then "debug"
-        when :info then "info"
-        when :warn, :warning then "warning"
-        when :error then "error"
-        when :fatal then "fatal"
-        else "info"
+        when :debug then 'debug'
+        when :info then 'info'
+        when :warn, :warning then 'warning'
+        when :error then 'error'
+        when :fatal then 'fatal'
+        else 'info'
         end
       end
 
       def validate_settings!
-        if @settings[:dsn]
-          validate_dsn!(@settings[:dsn], "Sentry DSN")
-        else
-          raise ValidationError, "Sentry DSN is required"
-        end
+        raise ValidationError, 'Sentry DSN is required' unless @settings[:dsn]
+
+        validate_dsn!(@settings[:dsn], 'Sentry DSN')
       end
     end
   end

data/lib/lapsoss/backtrace_frame.rb

@@ -53,12 +53,12 @@ module Lapsoss
       '/System/Library/Frameworks'
     ].freeze
 
-    attr_reader :filename, :line_number, :method_name, :in_app, :raw_line
-    attr_reader :function, :module_name, :code_context, :block_info
+    attr_reader :filename, :line_number, :method_name, :in_app, :raw_line, :function, :module_name, :code_context,
+                :block_info
 
     # Backward compatibility aliases
-    alias_method :lineno, :line_number
-    alias_method :raw, :raw_line
+    alias lineno line_number
+    alias raw raw_line
 
     def initialize(raw_line, in_app_patterns: [], exclude_patterns: [], load_paths: [])
       @raw_line = raw_line.to_s.strip
@@ -123,7 +123,7 @@ module Lapsoss
       # Try to make path relative to load paths
       @load_paths.each do |load_path|
         if @filename.start_with?(load_path)
-          relative = @filename.sub(/^#{Regexp.escape(load_path)}\/?/, '')
+          relative = @filename.sub(%r{^#{Regexp.escape(load_path)}/?}, '')
           return relative unless relative.empty?
         end
       end
@@ -134,23 +134,21 @@ module Lapsoss
     private
 
     def parse_backtrace_line
-      BACKTRACE_PATTERNS.each_with_index do |pattern, pattern_index|
+      BACKTRACE_PATTERNS.each_with_index do |pattern, _pattern_index|
         match = @raw_line.match(pattern)
         next unless match
 
         @filename = match[:filename]
         # Handle malformed line numbers - convert invalid numbers to 0
-        if match.names.include?('line') && match[:line]
-          @line_number = match[:line].match?(/^\d+$/) ? match[:line].to_i : 0
-        else
-          @line_number = nil
-        end
+        @line_number = if match.names.include?('line') && match[:line]
+                         match[:line].match?(/^\d+$/) ? match[:line].to_i : 0
+                       end
         @method_name = match.names.include?('method') ? match[:method] : nil
         @native_gem = match.names.include?('native_gem') ? match[:native_gem] : nil
         @block_level = match.names.include?('block_level') ? match[:block_level] : nil
 
         # Set default method name for lines without methods (top-level execution)
-        @method_name = "<main>" if @method_name.nil?
+        @method_name = '<main>' if @method_name.nil?
 
         process_method_info
         return
@@ -159,7 +157,7 @@ module Lapsoss
       # Fallback: treat entire line as filename if no pattern matches
       @filename = @raw_line
       @line_number = nil
-      @method_name = "<main>"
+      @method_name = '<main>'
     end
 
     def process_method_info
@@ -199,9 +197,7 @@ module Lapsoss
       }
 
       # Extract the method that contains the block
-      if @method_name.match(/block (?:\([^)]+\)\s+)?in (.+)/)
-        @block_info[:in_method] = $1
-      end
+      @block_info[:in_method] = ::Regexp.last_match(1) if @method_name =~ /block (?:\([^)]+\)\s+)?in (.+)/
     end
 
     def determine_app_status
@@ -223,7 +219,7 @@ module Lapsoss
       end
 
       # Default heuristics: check for library indicators
-      @in_app = !LIBRARY_INDICATORS.any? { |indicator| @filename.include?(indicator) }
+      @in_app = LIBRARY_INDICATORS.none? { |indicator| @filename.include?(indicator) }
 
       # Special cases
       @in_app = false if @native_gem # Native extensions are not app code
@@ -234,25 +230,19 @@ module Lapsoss
       return unless @filename
 
       # Expand relative paths
-      if @filename.start_with?('./')
-        @filename = File.expand_path(@filename)
-      end
+      @filename = File.expand_path(@filename) if @filename.start_with?('./')
 
       # Handle Windows paths on Unix systems (for cross-platform stack traces)
-      if @filename.include?('\\') && !@filename.include?('/')
-        @filename = @filename.tr('\\', '/')
-      end
+      @filename = @filename.tr('\\', '/') if @filename.include?('\\') && @filename.exclude?('/')
 
       # Strip load paths to make traces more readable
-      if @load_paths.any?
-        original = @filename
-        @filename = relative_filename
+      return unless @load_paths.any?
 
-        # Keep absolute path if relative didn't work well
-        if @filename.empty? || @filename == '.'
-          @filename = original
-        end
-      end
+      original = @filename
+      @filename = relative_filename
+
+      # Keep absolute path if relative didn't work well
+      @filename = original if @filename.empty? || @filename == '.'
     end
   end
 end