action_reporter 1.5.2 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dcc6906148e31a639c208838e67c4e975bc960184bafa53f22b197f20636259a
-  data.tar.gz: 31887dcc92b62824aa84726f78419587f0e58628835c81c1854f99f80e365a55
+  metadata.gz: 966d8f43c69d090e7c05c60571b9514f70a78dc1f16406c50610c9003e02a9e5
+  data.tar.gz: b04f3e1ecc0a98d68b58169c25df6ad0a3ebbd23bc486b090cf8706b4e6b5b35
 SHA512:
-  metadata.gz: 6a2dd2af3b8f111577787b50091bcb3153903b20190a764ab4ba00552071395b8de6b2f1c7bf9a1affacaa41ce9da2d2c9dce37c9edd641b03023eec42e39b4a
-  data.tar.gz: 34090de9bb4cb9ccbda7540a60f1a947f58dcf724e2b18bfb70f30795e2288a23144c90fcd846decbfa041ddac490fd731a2eb29367cc11d5e984194b087b15e
+  metadata.gz: 6ccd9608cf52f7eb887102e12011b3f889cb7d06c171ddc46aa06c0a7ac69d9330dd7a377b12da3c3b17e71254bd940fe1f69f9f590401fcc3e02c6a29129f9f
+  data.tar.gz: 4f389d9305525cd87a2eb9b0b8f09765867e2195060c88b6fd942199bfcd5f4e118f9b896011b027f62cf55ed6a7965641455226ad479c0dbf32d48c457771b2

data/CHANGELOG.md

@@ -1,77 +1,119 @@
-# 1.5.2
+# CHANGELOG
 
-* Add Audited context support
+## 2.0.0 (2025-11-05)
 
-# 1.5.1
+- Add performance benchmarks to measure CPU, memory, and response-time impact
+  - Benchmarks wall-time overhead for context + notify + reset operations
+  - Tests with 0 to 10 reporters to show scaling behavior
+  - Shows overhead per reporter (~1.5ms per reporter on average)
+  - Helps estimate response-time impact based on number of enabled reporters
+- BREAKING: Complete rewrite of plugin discovery system
+  - New lazy-loaded plugin discovery that auto-discovers reporters from filesystem
+  - Introduced `ActionReporter::PluginDiscovery` module for managing reporter discovery
+  - `AVAILABLE_REPORTERS` constant maintained for backward compatibility but `available_reporters` method is now preferred
+  - Plugin discovery does not block application boot - all discovery is lazy-loaded
+- BREAKING: Thread safety changes - context attributes are now thread-safe
+  - Replace module-level instance variables with thread-local storage using `ActionReporter::Current`
+  - Context attributes (`current_user`, `current_request_uuid`, `current_remote_addr`) are now thread-safe
+  - Prevents data leakage and race conditions in multi-threaded environments
+  - Maintains API compatibility but behavior is now thread-safe
+- Add custom reporter registration support
+  - New `ActionReporter.register_reporter(name, class_name:, require_path:)` method
+  - Allows third-party gems and applications to register custom reporters (e.g., Datadog, New Relic)
+  - Registered reporters automatically appear in `available_reporters`
+  - Supports both file-based and inline-defined reporters
+- Add `ActionReporter.available_reporters` method to get all available reporters (discovered + registered)
+- Add thread-safe plugin discovery with Mutex-based caching
+- Add comprehensive error handling to all reporter methods
+  - Reporter failures no longer break the entire reporting chain
+  - Errors are logged but don't prevent other reporters from executing
+  - Added `ActionReporter.logger` for configurable error logging (defaults to `Rails.logger` if available)
+  - Added `ActionReporter.error_handler` for custom error handling callbacks
+- Fix `reset_context` method to properly reset instance attributes (`@current_user`, `@current_request_uuid`, `@current_remote_addr`)
+- Improve error handling in plugin discovery - gracefully handles missing files, invalid classes, and load errors
+- Add `ActionReporter::PluginDiscovery.reset!` method for testing purposes
+- Add thread safety tests
+- Add error handling tests with fault isolation verification
+- Add comprehensive plugin discovery tests
+- Add custom reporter registration tests
+- Achieve 100% test coverage (297/297 lines)
 
-* Fix Audited current user setter to use `audited_user`
+## 1.5.2 (2024-12-13)
 
-# 1.5.0
+- Add Audited context support
 
-* BREAKING: Rename `audited_user` to `current_user`
-* Add `current_request_uuid` and `current_remote_addr` getters and setters
-* Memoize `current_user`, `current_request_uuid`, and `current_remote_addr`
+## 1.5.1 (2024-10-29)
 
-# 1.4.1
+- Fix Audited current user setter to use `audited_user`
 
-* Add paper_trail support
+## 1.5.0 (2024-10-29)
 
-# 1.4.0
+- BREAKING: Rename `audited_user` to `current_user`
+- Add `current_request_uuid` and `current_remote_addr` getters and setters
+- Memoize `current_user`, `current_request_uuid`, and `current_remote_addr`
 
-* Set minimum ruby version requirement to 2.5.0
+## 1.4.1 (2024-02-02)
 
-# 1.3.1
+- Add paper_trail support
 
-* Update gem configuration
+## 1.4.0 (2023-08-30)
 
-# 1.3.0
+- Set minimum ruby version requirement to 2.5.0
 
-* Update ruby version to 3.2.2
-* Update dependencies to latest
+## 1.3.2 (2023-08-29)
 
-# 1.2.0
+## 1.3.1 (2023-08-29)
 
-* Major fixes for class resolvers
-* Implemented ActionReporter::Error class
-* Improved test coverage
+- Update gem configuration
 
-# 1.1.1
+## 1.3.0 (2023-08-15)
 
-* Update check-in logic
+- Update ruby version to 3.2.2
+- Update dependencies to latest
 
-# 1.1.0
+## 1.2.0 (2023-04-20)
 
-* Add reporter check-in method
+- Major fixes for class resolvers
+- Implemented ActionReporter::Error class
+- Improved test coverage
 
-# 1.0.7
+## 1.1.1 (2023-04-18)
 
-* Moving Sentry context under `context` key
+- Update check-in logic
 
-# 1.0.6
+## 1.1.0 (2023-04-18)
 
-* Possible fix for Sentry context setting
+- Add reporter check-in method
 
-# 1.0.5
+## 1.0.7 (2023-04-18)
 
-* Fix Sentry reporting and context setting
+- Moving Sentry context under `context` key
 
-# 1.0.4
+## 1.0.6 (2023-04-18)
 
-* Move `transform_context` to individual reporter classes
+- Possible fix for Sentry context setting
 
-# 1.0.3
+## 1.0.5 (2023-04-18)
 
-* Fix scoutapm notice method
+- Fix Sentry reporting and context setting
 
-# 1.0.2
+## 1.0.4 (2023-04-18)
 
-* Add ruby version support for versions lower than 3.2.0
+- Move `transform_context` to individual reporter classes
 
-# 1.0.1
+## 1.0.3 (2023-04-18)
 
-* Fix scoutapm reset_context method
-* Update README notes
+- Fix scoutapm notice method
 
-# 1.0.0
+## 1.0.2 (2023-04-18)
 
-* Initial version
+- Add ruby version support for versions lower than 3.2.0
+
+## 1.0.1 (2023-04-18)
+
+- Fix scoutapm reset_context method
+- Update README notes
+
+## 1.0.0 (2023-04-18)
+
+- Initial version

data/README.md

@@ -2,89 +2,163 @@
 
 [![Gem Version](https://badge.fury.io/rb/action_reporter.svg)](https://badge.fury.io/rb/action_reporter) [![Test Status](https://github.com/amkisko/action_reporter.rb/actions/workflows/test.yml/badge.svg)](https://github.com/amkisko/action_reporter.rb/actions/workflows/test.yml) [![codecov](https://codecov.io/gh/amkisko/action_reporter.rb/graph/badge.svg?token=JCV2A7NWTE)](https://codecov.io/gh/amkisko/action_reporter.rb)
 
-[![Codacy Badge](https://app.codacy.com/project/badge/Grade/f4bef9a52eac43a5a0f6d8c1b58cc6af)](https://app.codacy.com/gh/amkisko/action_reporter.rb/dashboard?utm_source=gh&utm_medium=referral&utm_content=&utm_campaign=Badge_grade) [![Codacy Badge](https://app.codacy.com/project/badge/Coverage/f4bef9a52eac43a5a0f6d8c1b58cc6af)](https://app.codacy.com/gh/amkisko/action_reporter.rb/dashboard?utm_source=gh&utm_medium=referral&utm_content=&utm_campaign=Badge_coverage)
-
 Ruby wrapper for multiple reporting services.
 
-Supported services:
-- Rails logger
-- Audited
-- PaperTrail
-- Sentry
-- Honeybadger
-- scoutapm
+Supported services: Rails logger, gem audited, gem PaperTrail, Sentry, Honeybadger, scoutapm.
 
 Sponsored by [Kisko Labs](https://www.kiskolabs.com).
 
-## Install
+<a href="https://www.kiskolabs.com">
+  <img src="kisko.svg" width="200" alt="Sponsored by Kisko Labs" />
+</a>
 
-Using Bundler:
-```sh
-bundle add action_reporter
-```
 
-Using RubyGems:
-```sh
-gem install action_reporter
-```
+## Installation
 
-## Gemfile
+Add to your Gemfile:
 
 ```ruby
-gem 'action_reporter'
+gem "action_reporter"
 ```
 
+Run `bundle install` or `gem install action_reporter`.
+
+
 ## Usage
 
-Put this in your `config/initializers/action_reporter.rb` file:
+Create `config/initializers/action_reporter.rb`:
 
 ```ruby
 ActionReporter.enabled_reporters = [
-  ActionReporter::RailsReporter.new,
+  (ActionReporter::RailsReporter.new if Rails.env.development?),
   # ActionReporter::AuditedReporter.new,
   # ActionReporter::PaperTrailReporter.new,
   # ActionReporter::SentryReporter.new,
   # ActionReporter::HoneybadgerReporter.new,
   # ActionReporter::ScoutApmReporter.new
-]
+].compact
 ```
 
-Then you can use it in your code:
+Set context and report errors:
 
 ```ruby
 ActionReporter.current_user = current_user
-ActionReporter.current_request_uuid = request.env['action_dispatch.request_id']
-ActionReporter.current_remote_addr = request.env['action_dispatch.remote_ip']
+ActionReporter.current_request_uuid = request.env["action_dispatch.request_id"]
+ActionReporter.current_remote_addr = request.remote_ip
+
 ActionReporter.context(entry_id: entry.id)
-ActionReporter.notify('Something went wrong', context: { record: record })
+ActionReporter.notify("Something went wrong", context: { record: record })
+ActionReporter.reset_context
+```
+
+## Transaction support
+
+ActionReporter supports transaction tracking with automatic context preservation:
+
+```ruby
+# Attribute-style setters
+ActionReporter.transaction_id = "txn-123"
+ActionReporter.transaction_name = "GET /api/users"
+
+# Block-based (preserves previous values)
+ActionReporter.transaction(name: "GET /api/users", id: "txn-123") do
+  # Your code here
+end
 ```
 
-## Hook debugger to notify method
+## Custom Reporters
 
-Apply patch on initializer level or before running the main code:
+Create custom reporters by inheriting from `ActionReporter::Base`:
 
 ```ruby
 module ActionReporter
-  class RailsReporter < Base
+  class CustomReporter < Base
     def notify(error, context: {})
-      super
-      binding.pry
+      new_context = transform_context(context)
+      # Send to your service
+    end
+
+    def context(args)
+      new_context = transform_context(args)
+      # Set context in your service
     end
   end
 end
+
+ActionReporter.enabled_reporters = [ActionReporter::CustomReporter.new]
 ```
 
+See `doc/CUSTOM_REPORTERS.md` for detailed documentation.
+
+## Advanced Integration
+
+ActionReporter can be extended with custom methods and integrated with `ActiveSupport::CurrentAttributes` for automatic context propagation:
+
+```ruby
+module ActionReporter
+  def self.set_transaction_id(transaction_id)
+    context(transaction_id: transaction_id)
+    Sentry.set_tags(transactionId: transaction_id) if defined?(Sentry)
+  end
+end
+
+class Current < ActiveSupport::CurrentAttributes
+  attribute :user, :reporter_transaction_id
+
+  def user=(user)
+    super
+    ActionReporter.current_user = user
+  end
+
+  def reporter_transaction_id=(transaction_id)
+    super
+    ActionReporter.transaction_id = transaction_id
+  end
+end
+```
+
+## API
+
+- `ActionReporter.enabled_reporters = [...]` - Configure enabled reporters
+- `ActionReporter.current_user = user` - Set current user (thread-safe)
+- `ActionReporter.current_request_uuid = uuid` - Set request UUID
+- `ActionReporter.current_remote_addr = addr` - Set remote address
+- `ActionReporter.context(**args)` - Set context for all reporters
+- `ActionReporter.notify(error, context: {})` - Report errors/messages
+- `ActionReporter.reset_context` - Reset all context
+- `ActionReporter.transaction_id = id` - Set transaction ID
+- `ActionReporter.transaction_name = name` - Set transaction name
+- `ActionReporter.transaction(name:, id:, **context, &block)` - Block-based transaction with context preservation
+- `ActionReporter.check_in(identifier)` - Heartbeat/check-in
+- `ActionReporter.logger = logger` - Configure error logger
+- `ActionReporter.error_handler = proc` - Configure error handler callback
+
+
+## Development
+
+```bash
+bundle install
+bundle exec appraisal install
+bundle exec rspec
+bin/appraisals
+bundle exec standardrb --fix
+```
+
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/amkisko/action_reporter.rb
 
 Contribution policy:
+- New features are not necessarily added to the gem
+- Pull request should have test coverage for affected parts
+- Pull request should have changelog entry
+
+Review policy:
 - It might take up to 2 calendar weeks to review and merge critical fixes
 - It might take up to 6 calendar months to review and merge pull request
 - It might take up to 1 calendar year to review an issue
-- New integrations and third-party features are not nessessarily added to the gem
-- Pull request should have test coverage for affected parts
-- Pull request should have changelog entry
+
 
 ## Publishing
 
@@ -94,6 +168,7 @@ gem build action_reporter.gemspec
 gem push action_reporter-*.gem
 ```
 
+
 ## License
 
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/action_reporter.gemspec

@@ -1,6 +1,8 @@
+require_relative "lib/action_reporter/version"
+
 Gem::Specification.new do |gem|
   gem.name = "action_reporter"
-  gem.version = File.read(File.expand_path("../lib/action_reporter.rb", __FILE__)).match(/VERSION\s*=\s*'(.*?)'/)[1]
+  gem.version = ActionReporter::VERSION
 
   gem.license = "MIT"
 
@@ -9,7 +11,7 @@ Gem::Specification.new do |gem|
   repository_url = "https://github.com/amkisko/action_reporter.rb"
 
   gem.authors = ["Andrei Makarov"]
-  gem.email = ["andrei@kiskolabs.com"]
+  gem.email = ["contact@kiskolabs.com"]
   gem.homepage = repository_url
   gem.description = "Ruby wrapper for multiple reporting services"
   gem.summary = "See description"
@@ -37,10 +39,14 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency "scout_apm", "~> 5"
   gem.add_development_dependency "paper_trail", "~> 15"
 
-  gem.add_development_dependency "rspec", "~> 3"
+  gem.add_development_dependency "bundler", "~> 2"
+  gem.add_development_dependency "rspec", "~> 3.12"
   gem.add_development_dependency "rspec_junit_formatter", "~> 0.6"
   gem.add_development_dependency "webmock", "~> 3"
   gem.add_development_dependency "pry", "~> 0.14"
   gem.add_development_dependency "simplecov", "~> 0.21"
-  gem.add_development_dependency "simplecov-cobertura", "~> 2"
+  gem.add_development_dependency "simplecov-cobertura", "~> 3"
+  gem.add_development_dependency "standard", "~> 1.0"
+  gem.add_development_dependency "appraisal", "~> 2.4"
+  gem.add_development_dependency "rbs", "~> 3.0"
 end

data/lib/action_reporter/base.rb

@@ -1,4 +1,4 @@
-require_relative 'error'
+require_relative "error"
 
 module ActionReporter
   class Base
@@ -7,13 +7,15 @@ module ActionReporter
       define_method(method_name) do
         raise ActionReporter::Error.new("#{class_name} is not defined") unless Object.const_defined?(class_name)
 
-        @@class_cache ||= {}
-        @@class_cache[class_name] ||= begin
+        # Use instance variable instead of class variable for thread safety
+        # Each class gets its own cache, avoiding cross-class contamination
+        @class_cache ||= {}
+        @class_cache[class_name] ||= begin
           if gem_spec
             gem_name, version = gem_spec.scan(/([^(\s]+)\s*(?:\(([^)]+)\))?/).first
             latest_spec = Gem.loaded_specs[gem_name]
             version_satisfied = latest_spec && Gem::Requirement.new(version).satisfied_by?(latest_spec.version)
-            raise ActionReporter::Error.new("#{gem_spec} is not loaded") if !version_satisfied
+            raise ActionReporter::ConfigurationError.new("#{gem_spec} is not loaded") if !version_satisfied
           end
           Object.const_get(class_name)
         end
@@ -36,7 +38,7 @@ module ActionReporter
       elsif identifier.respond_to?(:to_s)
         identifier.to_s
       else
-        raise ArgumentError.new("Unknown check-in identifier: #{identifier.inspect}")
+        raise ActionReporter::Error.new("Unknown check-in identifier: #{identifier.inspect}")
       end
     end
 
@@ -48,5 +50,11 @@ module ActionReporter
 
     def reset_context
     end
+
+    def transaction_id=(transaction_id)
+    end
+
+    def transaction_name=(transaction_name)
+    end
   end
 end

data/lib/action_reporter/current.rb

@@ -0,0 +1,59 @@
+require_relative "error"
+
+module ActionReporter
+  # Thread-safe storage for ActionReporter context attributes
+  # Uses Thread.current for maximum compatibility and thread safety
+  # Note: ActiveSupport::CurrentAttributes is request-scoped (not thread-scoped),
+  # so we use Thread.current for proper thread isolation
+  class Current
+    class << self
+      def current_user
+        Thread.current[:action_reporter_current_user]
+      end
+
+      def current_user=(user)
+        Thread.current[:action_reporter_current_user] = user
+      end
+
+      def current_request_uuid
+        Thread.current[:action_reporter_current_request_uuid]
+      end
+
+      def current_request_uuid=(uuid)
+        Thread.current[:action_reporter_current_request_uuid] = uuid
+      end
+
+      def current_remote_addr
+        Thread.current[:action_reporter_current_remote_addr]
+      end
+
+      def current_remote_addr=(addr)
+        Thread.current[:action_reporter_current_remote_addr] = addr
+      end
+
+      def transaction_id
+        Thread.current[:action_reporter_transaction_id]
+      end
+
+      def transaction_id=(transaction_id)
+        Thread.current[:action_reporter_transaction_id] = transaction_id
+      end
+
+      def transaction_name
+        Thread.current[:action_reporter_transaction_name]
+      end
+
+      def transaction_name=(transaction_name)
+        Thread.current[:action_reporter_transaction_name] = transaction_name
+      end
+
+      def reset
+        Thread.current[:action_reporter_current_user] = nil
+        Thread.current[:action_reporter_current_request_uuid] = nil
+        Thread.current[:action_reporter_current_remote_addr] = nil
+        Thread.current[:action_reporter_transaction_id] = nil
+        Thread.current[:action_reporter_transaction_name] = nil
+      end
+    end
+  end
+end

data/lib/action_reporter/error.rb

@@ -1,3 +1,9 @@
 module ActionReporter
   class Error < StandardError; end
+
+  # Raised when there's a configuration issue (e.g., missing gem, invalid setup)
+  class ConfigurationError < Error; end
+
+  # Raised when a reporter encounters an error during operation
+  class ReporterError < Error; end
 end

data/lib/action_reporter/plugin_discovery.rb

@@ -0,0 +1,143 @@
+require_relative "base"
+
+module ActionReporter
+  # Plugin discovery mechanism for auto-discovering reporters
+  # This is lazy-loaded to avoid blocking application boot
+  module PluginDiscovery
+    class << self
+      # Initialize class instance variables
+      @registered_reporters = {}
+      @discovered_reporters = nil
+      @discovery_lock = Mutex.new
+
+      # Accessors for class instance variables
+      def registered_reporters
+        @registered_reporters ||= {}
+      end
+
+      attr_reader :discovered_reporters
+
+      attr_writer :discovered_reporters
+
+      def discovery_lock
+        @discovery_lock ||= Mutex.new
+      end
+
+      # Register a reporter manually (useful for custom reporters)
+      # @param name [Symbol] Reporter name
+      # @param class_name [String] Fully qualified class name
+      # @param require_path [String] Path to require (e.g., "action_reporter/custom_reporter")
+      def register(name, class_name:, require_path:)
+        registered_reporters[name] = {
+          class_name: class_name,
+          require_path: require_path
+        }
+      end
+
+      # Discover reporters from the filesystem (lazy-loaded, cached)
+      # Only discovers files matching *_reporter.rb pattern in action_reporter/ directory
+      # @return [Array<Class>] Array of reporter classes
+      def discover
+        return discovered_reporters if discovered_reporters
+
+        discovery_lock.synchronize do
+          return discovered_reporters if discovered_reporters
+
+          self.discovered_reporters = []
+          # __dir__ is lib/action_reporter/, so we're already in the right directory
+          base_path = __dir__
+
+          # Discover built-in reporters
+          Dir.glob(File.join(base_path, "*_reporter.rb")).each do |file|
+            reporter_class = discover_reporter_from_file(file)
+            discovered_reporters << reporter_class if reporter_class
+          rescue => e
+            # Silently skip files that can't be loaded (non-blocking)
+            warn "ActionReporter: Failed to discover reporter from #{file}: #{e.message}" if logger
+          end
+
+          self.discovered_reporters = discovered_reporters.freeze
+        end
+      end
+
+      # Get all available reporters (discovered + registered)
+      # @return [Array<Class>] Array of reporter classes
+      def available_reporters
+        reporters = discover.dup
+
+        # Add registered reporters (lazy-loaded)
+        registered_reporters.each_value do |config|
+          reporter_class = load_registered_reporter(config)
+          reporters << reporter_class if reporter_class
+        rescue => e
+          # Silently skip registered reporters that can't be loaded (non-blocking)
+          warn "ActionReporter: Failed to load registered reporter #{config[:class_name]}: #{e.message}" if logger
+        end
+
+        reporters.uniq.freeze
+      end
+
+      # Reset discovery cache (useful for testing)
+      def reset!
+        discovery_lock.synchronize do
+          self.discovered_reporters = nil
+        end
+      end
+
+      private
+
+      def discover_reporter_from_file(file_path)
+        base_name = File.basename(file_path, ".rb")
+
+        # Convert snake_case to PascalCase
+        # e.g., "scout_apm_reporter" -> "ScoutApmReporter"
+        # e.g., "rails_reporter" -> "RailsReporter"
+        class_name = base_name
+          .split("_")
+          .map(&:capitalize)
+          .join
+
+        full_class_name = "ActionReporter::#{class_name}"
+
+        # Check if class is already defined (files are pre-required at boot)
+        return nil unless Object.const_defined?(full_class_name)
+
+        klass = Object.const_get(full_class_name)
+        return nil unless klass < Base
+
+        klass
+      end
+
+      def load_registered_reporter(config)
+        # Check if class is already defined (e.g., in tests or already loaded)
+        return nil unless Object.const_defined?(config[:class_name])
+
+        klass = Object.const_get(config[:class_name])
+        return nil unless klass < Base
+
+        # Only require if class is not already available and require_path is provided
+        # This allows classes defined inline (e.g., in tests) to work without requiring files
+        if config[:require_path] && !required?(config[:require_path])
+          begin
+            require config[:require_path]
+          rescue LoadError => e
+            # If file doesn't exist but class is already defined, that's okay
+            # (e.g., class defined inline in tests or already loaded)
+            warn "ActionReporter: Could not require #{config[:require_path]}: #{e.message}" if logger
+            # Continue - class might already be defined
+          end
+        end
+
+        klass
+      end
+
+      def required?(path)
+        $LOADED_FEATURES.any? { |feature| feature.include?(path) }
+      end
+
+      def logger
+        ActionReporter.logger
+      end
+    end
+  end
+end

data/lib/action_reporter/sentry_reporter.rb

@@ -25,5 +25,15 @@ module ActionReporter
     def current_user=(user)
       sentry_class.set_user(user_global_id: user&.to_global_id&.to_s)
     end
+
+    def transaction_id=(transaction_id)
+      sentry_class.set_tags(transactionId: transaction_id)
+    end
+
+    def transaction_name=(transaction_name)
+      sentry_class.configure_scope do |scope|
+        scope.set_transaction_name(transaction_name)
+      end
+    end
   end
 end

data/lib/action_reporter/version.rb

@@ -0,0 +1,3 @@
+module ActionReporter
+  VERSION = "2.0.0"
+end

data/lib/action_reporter.rb

@@ -1,17 +1,23 @@
-require 'action_reporter/utils'
-require 'action_reporter/base'
-require 'action_reporter/rails_reporter'
-require 'action_reporter/honeybadger_reporter'
-require 'action_reporter/sentry_reporter'
-require 'action_reporter/scout_apm_reporter'
-require 'action_reporter/audited_reporter'
-require 'action_reporter/paper_trail_reporter'
+require "action_reporter/version"
+require "action_reporter/utils"
+require "action_reporter/base"
+require "action_reporter/current"
+require "action_reporter/plugin_discovery"
+
+# Core reporters are still required for backward compatibility
+# But discovery mechanism allows for lazy loading and custom reporters
+require "action_reporter/rails_reporter"
+require "action_reporter/honeybadger_reporter"
+require "action_reporter/sentry_reporter"
+require "action_reporter/scout_apm_reporter"
+require "action_reporter/audited_reporter"
+require "action_reporter/paper_trail_reporter"
 
 module ActionReporter
   module_function
 
-  VERSION = '1.5.2'.freeze
-
+  # Legacy hardcoded list (maintained for backward compatibility)
+  # Use `available_reporters` for auto-discovered reporters
   AVAILABLE_REPORTERS = [
     ActionReporter::RailsReporter,
     ActionReporter::HoneybadgerReporter,
@@ -21,76 +27,163 @@ module ActionReporter
     ActionReporter::PaperTrailReporter
   ].freeze
 
+  # Get available reporters (auto-discovered + registered)
+  # This is lazy-loaded and does not block application boot
+  # @return [Array<Class>] Array of reporter classes
+  def available_reporters
+    PluginDiscovery.available_reporters
+  end
+
+  # Register a custom reporter
+  # This allows third-party gems or custom code to register reporters
+  # @param name [Symbol] Reporter name
+  # @param class_name [String] Fully qualified class name
+  # @param require_path [String] Path to require
+  # @example
+  #   ActionReporter.register_reporter(:custom, class_name: "MyApp::CustomReporter", require_path: "my_app/custom_reporter")
+  def register_reporter(name, class_name:, require_path:)
+    PluginDiscovery.register(name, class_name: class_name, require_path: require_path)
+  end
+
   @enabled_reporters = []
+  @logger = nil
+  @error_handler = nil
 
   def enabled_reporters=(reporters)
-    @enabled_reporters = reporters
+    @enabled_reporters = reporters || []
   end
 
   def enabled_reporters
-    @enabled_reporters
+    @enabled_reporters || []
+  end
+
+  def logger
+    @logger || ((defined?(Rails) && Rails.respond_to?(:logger)) ? Rails.logger : nil)
+  end
+
+  def logger=(logger)
+    @logger = logger
+  end
+
+  def error_handler
+    @error_handler
+  end
+
+  def error_handler=(handler)
+    @error_handler = handler
+  end
+
+  def handle_reporter_error(reporter, error, method_name)
+    error_message = "ActionReporter: #{reporter.class}##{method_name} failed: #{error.class} - #{error.message}"
+
+    if logger
+      logger.error(error_message)
+      logger.debug(error.backtrace.join("\n")) if error.backtrace
+    end
+
+    if error_handler&.respond_to?(:call)
+      error_handler.call(error, reporter, method_name)
+    end
+  rescue => e
+    # If error handling itself fails, log to stderr as last resort
+    warn "ActionReporter: Error handler failed: #{e.message}"
   end
 
   def notify(error, context: {})
     enabled_reporters.each do |reporter|
       next unless reporter.respond_to?(:notify)
 
-      reporter.notify(error, context: context)
+      begin
+        reporter.notify(error, context: context)
+      rescue => e
+        handle_reporter_error(reporter, e, "notify")
+      end
     end
   end
 
   def context(args)
+    raise ArgumentError, "context must be a Hash" unless args.is_a?(Hash)
+
     enabled_reporters.each do |reporter|
       next unless reporter.respond_to?(:context)
 
-      reporter.context(args)
+      begin
+        reporter.context(args)
+      rescue => e
+        handle_reporter_error(reporter, e, "context")
+      end
     end
   end
 
   def reset_context
+    Current.current_user = nil
+    Current.current_request_uuid = nil
+    Current.current_remote_addr = nil
+    Current.transaction_id = nil
+    Current.transaction_name = nil
+
     enabled_reporters.each do |reporter|
       next unless reporter.respond_to?(:reset_context)
 
-      reporter.reset_context
+      begin
+        reporter.reset_context
+      rescue => e
+        handle_reporter_error(reporter, e, "reset_context")
+      end
     end
+
+    # Reset Current attributes if supported
+    Current.reset if Current.respond_to?(:reset)
   end
 
   def current_user
-    @current_user
+    Current.current_user
   end
 
   def current_user=(user)
-    @current_user = user
+    Current.current_user = user
     enabled_reporters.each do |reporter|
       next unless reporter.respond_to?(:current_user=)
 
-      reporter.current_user = user
+      begin
+        reporter.current_user = user
+      rescue => e
+        handle_reporter_error(reporter, e, "current_user=")
+      end
     end
   end
 
   def current_request_uuid
-    @current_request_uuid
+    Current.current_request_uuid
   end
 
   def current_request_uuid=(request_uuid)
-    @current_request_uuid = request_uuid
+    Current.current_request_uuid = request_uuid
     enabled_reporters.each do |reporter|
       next unless reporter.respond_to?(:current_request_uuid=)
 
-      reporter.current_request_uuid = request_uuid
+      begin
+        reporter.current_request_uuid = request_uuid
+      rescue => e
+        handle_reporter_error(reporter, e, "current_request_uuid=")
+      end
     end
   end
 
   def current_remote_addr
-    @current_remote_addr
+    Current.current_remote_addr
   end
 
   def current_remote_addr=(remote_addr)
-    @current_remote_addr = remote_addr
+    Current.current_remote_addr = remote_addr
     enabled_reporters.each do |reporter|
       next unless reporter.respond_to?(:current_remote_addr=)
 
-      reporter.current_remote_addr = remote_addr
+      begin
+        reporter.current_remote_addr = remote_addr
+      rescue => e
+        handle_reporter_error(reporter, e, "current_remote_addr=")
+      end
     end
   end
 
@@ -98,7 +191,68 @@ module ActionReporter
     enabled_reporters.each do |reporter|
       next unless reporter.respond_to?(:check_in)
 
-      reporter.check_in(identifier)
+      begin
+        reporter.check_in(identifier)
+      rescue => e
+        handle_reporter_error(reporter, e, "check_in")
+      end
+    end
+  end
+
+  def transaction_id
+    Current.transaction_id
+  end
+
+  def transaction_id=(transaction_id)
+    Current.transaction_id = transaction_id
+    context(transaction_id: transaction_id)
+
+    enabled_reporters.each do |reporter|
+      next unless reporter.respond_to?(:transaction_id=)
+
+      begin
+        reporter.transaction_id = transaction_id
+      rescue => e
+        handle_reporter_error(reporter, e, "transaction_id=")
+      end
+    end
+  end
+
+  def transaction_name
+    Current.transaction_name
+  end
+
+  def transaction_name=(transaction_name)
+    Current.transaction_name = transaction_name
+    context(transaction_name: transaction_name)
+
+    enabled_reporters.each do |reporter|
+      next unless reporter.respond_to?(:transaction_name=)
+
+      begin
+        reporter.transaction_name = transaction_name
+      rescue => e
+        handle_reporter_error(reporter, e, "transaction_name=")
+      end
+    end
+  end
+
+  def transaction(name: nil, id: nil, **context_options, &block)
+    raise ArgumentError, "transaction requires a block" unless block_given?
+
+    previous_name = transaction_name
+    previous_id = transaction_id
+
+    begin
+      self.transaction_name = name if name
+      self.transaction_id = id if id
+
+      context(context_options) if context_options.any?
+
+      block.call
+    ensure
+      self.transaction_name = previous_name if name
+      self.transaction_id = previous_id if id
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: action_reporter
 version: !ruby/object:Gem::Version
-  version: 1.5.2
+  version: 2.0.0
 platform: ruby
 authors:
 - Andrei Makarov
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-13 00:00:00.000000000 Z
+date: 2025-11-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -94,20 +94,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '15'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '3.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '3.12'
 - !ruby/object:Gem::Dependency
   name: rspec_junit_formatter
   requirement: !ruby/object:Gem::Requirement
@@ -170,17 +184,59 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2'
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2'
+        version: '3'
+- !ruby/object:Gem::Dependency
+  name: standard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: appraisal
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
+- !ruby/object:Gem::Dependency
+  name: rbs
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
 description: Ruby wrapper for multiple reporting services
 email:
-- andrei@kiskolabs.com
+- contact@kiskolabs.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -192,13 +248,16 @@ files:
 - lib/action_reporter.rb
 - lib/action_reporter/audited_reporter.rb
 - lib/action_reporter/base.rb
+- lib/action_reporter/current.rb
 - lib/action_reporter/error.rb
 - lib/action_reporter/honeybadger_reporter.rb
 - lib/action_reporter/paper_trail_reporter.rb
+- lib/action_reporter/plugin_discovery.rb
 - lib/action_reporter/rails_reporter.rb
 - lib/action_reporter/scout_apm_reporter.rb
 - lib/action_reporter/sentry_reporter.rb
 - lib/action_reporter/utils.rb
+- lib/action_reporter/version.rb
 homepage: https://github.com/amkisko/action_reporter.rb
 licenses:
 - MIT