opentrace 0.15.1 → 0.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e5c67d0aecd2be7ec0624bd339bda99a46d8c2de8d0a04009d6778f94cefee3d
-  data.tar.gz: 0b54c97978becc24a2339e1f615de4f715388fc9247dc88099880dc7e1532a43
+  metadata.gz: e9eff1e2688a1239f8ebafd7cd59487b4ffab4619107c7fda9db580f9de04bed
+  data.tar.gz: 6a4151670c6f6b803e7fff47fe2e7621ab17c52a7576d2d43c0fd416e337ccc6
 SHA512:
-  metadata.gz: 60bc99ef6e4e950851358a9dbf3b2b8342209f81ff24d58eb248c07362657d4c040c1f60e6e8fdb3cb56d625cac4f8b9a4641e953241a1af52195f27232790f6
-  data.tar.gz: d655b2237f513c8b3fe8a12e936086937fc80b1f50d5bb3766b6e5ef3ed1a137d90bf9798546aa99413633dba9e9588c87f2b99d4e54e226c0842fea720f309e
+  metadata.gz: 1931c39a5b24f9aea0eeb81aacb77945a4c0f2386f13c3862172119e59c90eb13fab43e7af42981c6c8626f660a6a548bd040d6f1d087262716045e971d81ae0
+  data.tar.gz: ef27898fabf3fe17bc1aaf98cd4003b4008740685124398c5eeb80ddb47c737284ee36e99c516103d9306d00f2c18ea016d2346b93380452dfa56b15f6c6b01c

data/lib/opentrace/error_subscriber.rb

@@ -0,0 +1,84 @@
+# frozen_string_literal: true
+
+module OpenTrace
+  # Rails 7.0+ Error Reporter subscriber.
+  #
+  # Captures ALL exceptions reported via Rails.error (including those
+  # rescued by rescue_from in controllers). This fills the gap where
+  # process_action.action_controller only includes exception data for
+  # unhandled exceptions — rescued exceptions show status 500 but no
+  # exception_class, backtrace, or error_fingerprint.
+  #
+  # Registered automatically in the Railtie when Rails.error is available.
+  class ErrorSubscriber
+    SEVERITY_MAP = {
+      error: "ERROR",
+      warning: "WARN",
+      info: "INFO"
+    }.freeze
+
+    def report(error, handled:, severity:, context: {}, source: nil)
+      return unless OpenTrace.enabled?
+
+      # Skip if already inside an OpenTrace logging call (re-entrance guard)
+      return if Fiber[:opentrace_logging]
+
+      level = SEVERITY_MAP[severity] || "ERROR"
+
+      meta = {}
+      meta[:exception_class] = error.class.name
+      meta[:exception_message] = error.message&.slice(0, 500)
+      meta[:handled] = handled
+      meta[:error_source] = source if source
+
+      if error.backtrace
+        cleaned = clean_backtrace(error.backtrace)
+        meta[:backtrace] = cleaned.first(15)
+        meta[:error_fingerprint] = OpenTrace.send(:compute_error_fingerprint, error.class.name, cleaned)
+      end
+
+      # Capture exception cause chain
+      if error.cause
+        meta[:exception_causes] = OpenTrace.send(:build_cause_chain, error.cause, depth: 0)
+      end
+
+      # Include request params from context if available
+      if context[:params].is_a?(Hash)
+        params = context[:params].except("controller", "action")
+        meta[:params] = truncate_hash(params, 2048) unless params.empty?
+      end
+
+      # Include controller/action context
+      meta[:controller] = context[:controller] if context[:controller]
+      meta[:action] = context[:action] if context[:action]
+
+      # Merge any additional context from the reporter
+      context.each do |k, v|
+        next if %i[params controller action].include?(k)
+        meta[k] = v
+      end
+
+      OpenTrace.log(level, "#{error.class}: #{error.message&.slice(0, 200)}", meta)
+    rescue StandardError
+      # Never raise to the host app
+    end
+
+    private
+
+    def clean_backtrace(backtrace)
+      if defined?(::Rails) && ::Rails.respond_to?(:backtrace_cleaner)
+        ::Rails.backtrace_cleaner.clean(backtrace)
+      else
+        backtrace.reject { |line| line.include?("/gems/") }
+      end
+    end
+
+    def truncate_hash(hash, max_bytes)
+      json = JSON.generate(hash)
+      return hash if json.bytesize <= max_bytes
+      { _truncated: true, _size: json.bytesize }
+    rescue StandardError
+      { _truncated: true }
+    end
+  end
+end

data/lib/opentrace/rails.rb

@@ -15,6 +15,13 @@ if defined?(::Rails::Railtie)
       config.after_initialize do |app|
         next unless OpenTrace.enabled?
 
+        # Rails Error Reporter subscriber (Rails 7.0+) — captures ALL exceptions
+        # including those rescued by rescue_from in controllers.
+        if defined?(Rails.error) && Rails.error.respond_to?(:subscribe)
+          require_relative "error_subscriber"
+          Rails.error.subscribe(OpenTrace::ErrorSubscriber.new)
+        end
+
         # Automatic log trace injection (opt-in) — wraps Rails logger formatter
         if OpenTrace.config.log_trace_injection
           require_relative "trace_formatter"
@@ -207,6 +214,13 @@ if defined?(::Rails::Railtie)
             extract_request_headers(payload, extra)
           end
 
+          # Always include params on error responses (status >= 500)
+          # so the AI agent can diagnose what input caused the failure.
+          if !extra&.key?(:params) && payload[:status].to_i >= 500
+            extra ||= {}
+            extract_params(payload, extra)
+          end
+
           # SQL counters for non-collector path (only present for sampled requests)
           sql_count = Fiber[:opentrace_sql_count]
           if sql_count && !Fiber[:opentrace_collector]

data/lib/opentrace/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OpenTrace
-  VERSION = "0.15.1"
+  VERSION = "0.16.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: opentrace
 version: !ruby/object:Gem::Version
-  version: 0.15.1
+  version: 0.16.0
 platform: ruby
 authors:
 - OpenTrace
@@ -38,6 +38,7 @@ files:
 - lib/opentrace/circuit_breaker.rb
 - lib/opentrace/client.rb
 - lib/opentrace/config.rb
+- lib/opentrace/error_subscriber.rb
 - lib/opentrace/http_tracker.rb
 - lib/opentrace/local_vars.rb
 - lib/opentrace/log_forwarder.rb