action_reporter 3.1.0 → 3.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ee447f96cc8b2c64df7f7f824b49cd7b5fbbbb591496b507c1a352883468701
-  data.tar.gz: 777dd9085576844c30d5a0a81ce57933072be54653c9c2326c041c7de61d796f
+  metadata.gz: 06c235516e05cc644caaa30e28f8b670fe711f20bc2f849d457ca17b4a7f8625
+  data.tar.gz: e82f1fac79ab8df40168305ede20edb759a57fce079f45bb4bd1e30f114eb352
 SHA512:
-  metadata.gz: 52bf9687ec703a12869bef96419ae1873b36d178d1e5cab06d5683dcd048ef1b49066efe0b7f2c1128d64b8bc762eafc89c1890767b900d49dcbc8c35f6e67e1
-  data.tar.gz: 94debcd931e0307f881721f94a13d493f65f4e2fc5de61d11ce6ad8a0536e6a2325646f4241613ef80be3a7036fd8608854a7ec22a727cff1890f8aebf890307
+  metadata.gz: a62dcee03f8dc0b21806ed98ea68fc32f2444e4dedf4caace3ebb5ae7bdfdde1356f2ecc8b2b5aca586a1b161ee2b0283c6d00b142e1b91a39da538b6f91fd64
+  data.tar.gz: 58121c6340d635e26d950b3091d7727eb4a66c3eee48c0a5d19d665b60b935d06e0d50de46cc1664b47612ca685d6fa1af4d9dea02ffb8872979897c82251f74

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # CHANGELOG
 
+## 3.2.1 (2026-07-13)
+
+- Fix check-in identifier errors to raise `ActionReporter::Error` instead of stack overflow when an identifier cannot be stringified
+
+## 3.2.0 (2026-07-13)
+
+- Improve context propagation across reporters
+  - Merge updates into existing reporter context instead of replacing prior keys on each call
+  - Remove context keys when an update passes `nil` for that key
+  - Apply shared merge semantics to `ActiveVersionReporter`, `AuditedReporter`, `HoneybadgerReporter`, `ScoutApmReporter`, and `SentryReporter`
+- Improve `SentryReporter` reset to clear the reporter context bucket from the Sentry scope
+- Remove unused `gem_spec` parameter from reporter `class_accessor` declarations (version checks were dropped in 2.0.1)
+- Improve plugin discovery warnings for reporter files and registrations that fail to load
+
 ## 3.1.0 (2026-03-27)
 
 - Add `ActionReporter.user_id_resolver` for configuring how to resolve user IDs

data/action_reporter.gemspec

@@ -31,7 +31,9 @@ Gem::Specification.new do |gem|
   gem.required_ruby_version = ">= 2.5.0"
   gem.require_path = "lib"
 
-  gem.add_runtime_dependency "rails", "> 5"
+  gem.add_runtime_dependency "activesupport", ">= 6.0"
+
+  gem.add_development_dependency "rails", "> 5"
 
   gem.add_development_dependency "audited", "~> 5"
   gem.add_development_dependency "honeybadger", "~> 5"
@@ -40,13 +42,13 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency "paper_trail", "~> 15"
   gem.add_development_dependency "active_version", "~> 1"
 
-  gem.add_development_dependency "bundler", "~> 2"
+  gem.add_development_dependency "bundler", ">= 2"
   gem.add_development_dependency "rspec", "~> 3"
+  gem.add_development_dependency "polyrun", ">= 2.2.0"
+  gem.add_development_dependency "prosopite", "~> 2.0"
   gem.add_development_dependency "rspec_junit_formatter", "~> 0.6"
   gem.add_development_dependency "webmock", "~> 3"
   gem.add_development_dependency "pry", "~> 0.15"
-  gem.add_development_dependency "simplecov", "~> 0.22"
-  gem.add_development_dependency "simplecov-cobertura", "~> 3"
   gem.add_development_dependency "standard", "~> 1.52"
   gem.add_development_dependency "standard-custom", "~> 1.0"
   gem.add_development_dependency "standard-performance", "~> 1.8"

data/bin/polyrun

@@ -0,0 +1,108 @@
+#!/usr/bin/env bash
+# Delegates to `bundle exec polyrun` with repo config.
+# - No args → same as `polyrun` with no subcommand (auto parallel RSpec from polyrun.yml / spec/**/*_spec.rb).
+# - Path-like argv (spec files, globs, dirs) → polyrun implicit parallel (path-only argv).
+# - Named subcommands (plan, config, parallel-rspec, ci-shard-rspec, …) → pass through.
+# - Otherwise (RSpec flags first, e.g. --format) → `start --workers N -- bundle exec rspec "$@"`.
+# Globals: -c/--config, -v/--verbose, -h/--help, --help (same as bundle exec polyrun).
+# Workers: POLYRUN_WORKERS (default 5, max 10, min 1) — read by polyrun run-shards/start.
+set -eo pipefail
+ROOT="$(cd "$(dirname "$0")/.." && pwd)"
+cd "$ROOT"
+
+W="${POLYRUN_WORKERS:-5}"
+case "$W" in ''|*[!0-9]*) W=5 ;; esac
+if [ "$W" -gt 10 ]; then W=10; fi
+if [ "$W" -lt 1 ]; then W=1; fi
+
+CONFIG="${POLYRUN_CONFIG:-polyrun.yml}"
+polyrun_prefix=()
+have_config=0
+
+while [ $# -gt 0 ]; do
+  case "$1" in
+    -c|--config)
+      polyrun_prefix+=("$1" "$2")
+      CONFIG="$2"
+      have_config=1
+      shift 2
+      ;;
+    -v|--verbose)
+      polyrun_prefix+=("$1")
+      shift
+      ;;
+    -h|--help)
+      if [ "$have_config" -eq 0 ]; then
+        exec bundle exec polyrun "${polyrun_prefix[@]}" -c "$CONFIG" -h
+      else
+        exec bundle exec polyrun "${polyrun_prefix[@]}" -h
+      fi
+      ;;
+    *)
+      break
+      ;;
+  esac
+done
+
+if [ "${1:-}" = "--version" ]; then
+  if [ "$have_config" -eq 0 ]; then
+    exec bundle exec polyrun "${polyrun_prefix[@]}" -c "$CONFIG" version
+  else
+    exec bundle exec polyrun "${polyrun_prefix[@]}" version
+  fi
+fi
+
+is_polyrun_command() {
+  case "$1" in
+    plan|prepare|merge-coverage|report-coverage|report-junit|report-timing|env|config|merge-timing|\
+    db:setup-template|db:setup-shard|db:clone-shards|run-shards|parallel-rspec|start|build-paths|\
+    ci-shard-run|ci-shard-rspec|init|queue|quick|help|version)
+      return 0 ;;
+    *)
+      return 1 ;;
+  esac
+}
+
+# True if argv should use polyrun implicit path parallel (any path-like token).
+implicit_path_in_argv() {
+  for a in "$@"; do
+    case "$a" in
+      -*) [ "$a" = "-" ] || continue ;;
+    esac
+    case "$a" in
+      *.rb|spec/*|test/*|./|/*|*[\*\?[]*) return 0 ;;
+    esac
+    [ -e "$a" ] && return 0
+  done
+  return 1
+}
+
+run_polyrun() {
+  if [ "$have_config" -eq 0 ]; then
+    exec bundle exec polyrun "${polyrun_prefix[@]}" -c "$CONFIG" "$@"
+  else
+    exec bundle exec polyrun "${polyrun_prefix[@]}" "$@"
+  fi
+}
+
+exec_parallel_rspec() {
+  if [ "$have_config" -eq 0 ]; then
+    exec bundle exec polyrun "${polyrun_prefix[@]}" -c "$CONFIG" start --workers "$W" -- bundle exec rspec "$@"
+  else
+    exec bundle exec polyrun "${polyrun_prefix[@]}" start --workers "$W" -- bundle exec rspec "$@"
+  fi
+}
+
+if [ $# -eq 0 ]; then
+  run_polyrun
+fi
+
+if is_polyrun_command "$1"; then
+  run_polyrun "$@"
+fi
+
+if implicit_path_in_argv "$@"; then
+  run_polyrun "$@"
+fi
+
+exec_parallel_rspec "$@"

data/lib/action_reporter/active_version_reporter.rb

@@ -1,12 +1,14 @@
 module ActionReporter
   class ActiveVersionReporter < Base
-    class_accessor "ActiveVersion", gem_spec: "active_version (~> 1)"
+    class_accessor "ActiveVersion"
 
     def notify(*)
     end
 
     def context(args)
-      ActiveVersion.context = ActiveVersion.context.merge(args) if ActiveVersion.respond_to?(:context=)
+      return unless ActiveVersion.respond_to?(:context=)
+
+      ActiveVersion.context = merge_context_updates(ActiveVersion.context, args)
     end
 
     def reset_context

data/lib/action_reporter/audited_reporter.rb

@@ -1,12 +1,14 @@
 module ActionReporter
   class AuditedReporter < Base
-    class_accessor "Audited", gem_spec: "audited (~> 5)"
+    class_accessor "Audited"
 
     def notify(*)
     end
 
     def context(args)
-      Audited.context = Audited.context.merge(args) if Audited.respond_to?(:context=)
+      return unless Audited.respond_to?(:context=)
+
+      Audited.context = merge_context_updates(Audited.context, args)
     end
 
     def reset_context

data/lib/action_reporter/base.rb

@@ -2,7 +2,7 @@ require_relative "error"
 
 module ActionReporter
   class Base
-    def self.class_accessor(class_name, gem_spec: nil)
+    def self.class_accessor(class_name)
       method_name = class_name.gsub("::", "_").downcase + "_class"
       define_method(method_name) do
         raise ActionReporter::Error.new("#{class_name} is not defined") unless Object.const_defined?(class_name)
@@ -24,13 +24,23 @@ module ActionReporter
       end
     end
 
+    def merge_context_updates(current, updates)
+      transformed = transform_context(updates)
+      keys_to_remove = transformed.each_with_object([]) do |(key, value), removals|
+        removals << key if value.nil?
+      end
+      current.merge(transformed).except(*keys_to_remove)
+    end
+
     def resolve_check_in_id(identifier)
       if identifier.respond_to?(:reporter_check_in)
         identifier.reporter_check_in
       elsif identifier.respond_to?(:to_s)
         identifier.to_s
       else
-        raise ActionReporter::Error.new("Unknown check-in identifier: #{identifier.inspect}")
+        raise ActionReporter::Error.new(
+          "Unknown check-in identifier (#{identifier.class}, object_id=#{identifier.object_id})"
+        )
       end
     end
 

data/lib/action_reporter/current.rb

@@ -1,3 +1,4 @@
+require "active_support/isolated_execution_state"
 require_relative "error"
 
 module ActionReporter

data/lib/action_reporter/honeybadger_reporter.rb

@@ -1,6 +1,6 @@
 module ActionReporter
   class HoneybadgerReporter < Base
-    class_accessor "Honeybadger", gem_spec: "honeybadger (~> 5)"
+    class_accessor "Honeybadger"
 
     def notify(error, context: {})
       new_context = transform_context(context)
@@ -8,8 +8,10 @@ module ActionReporter
     end
 
     def context(args)
-      new_context = transform_context(args)
-      honeybadger_class.context(new_context)
+      current = honeybadger_class.get_context || {}
+      merged = merge_context_updates(current, args)
+      honeybadger_class.context.clear!
+      honeybadger_class.context(merged) unless merged.empty?
     end
 
     def reset_context

data/lib/action_reporter/paper_trail_reporter.rb

@@ -1,6 +1,6 @@
 module ActionReporter
   class PaperTrailReporter < Base
-    class_accessor "PaperTrail", gem_spec: "paper_trail (~> 15)"
+    class_accessor "PaperTrail"
 
     def notify(*)
     end

data/lib/action_reporter/plugin_discovery.rb

@@ -55,7 +55,7 @@ module ActionReporter
             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
+            warn "ActionReporter: Failed to discover reporter from #{file}: #{e.message}"
           end
 
           self.discovered_reporters = discovered_reporters.freeze
@@ -74,7 +74,7 @@ module ActionReporter
           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
+          warn "ActionReporter: Failed to load registered reporter #{config[:class_name]}: #{e.message}"
         end
 
         reporters.uniq.freeze
@@ -125,7 +125,7 @@ module ActionReporter
           begin
             require config[:require_path]
           rescue LoadError => e
-            warn "ActionReporter: Could not require #{config[:require_path]}: #{e.message}" if logger
+            warn "ActionReporter: Could not require #{config[:require_path]}: #{e.message}"
           end
         end
 

data/lib/action_reporter/scout_apm_reporter.rb

@@ -1,7 +1,7 @@
 module ActionReporter
   class ScoutApmReporter < Base
-    class_accessor "ScoutApm::Error", gem_spec: "scout_apm (~> 5)"
-    class_accessor "ScoutApm::Context", gem_spec: "scout_apm (~> 5)"
+    class_accessor "ScoutApm::Error"
+    class_accessor "ScoutApm::Context"
 
     def notify(error, context: {})
       self.context(context)
@@ -9,8 +9,14 @@ module ActionReporter
     end
 
     def context(args)
-      new_context = transform_context(args)
-      scoutapm_context_class.add(new_context)
+      context_object = scoutapm_context_class.current
+      extra = context_object.instance_variable_get(:@extra).dup
+      # ScoutApm::Context.add ignores nil values and has no per-key delete API; replace @extra directly.
+      context_object.instance_variable_set(:@extra, merge_context_updates(extra, args))
+    end
+
+    def reset_context
+      scoutapm_context_class.clear! if scoutapm_context_class.respond_to?(:clear!)
     end
 
     def current_remote_addr=(remote_addr)

data/lib/action_reporter/sentry_reporter.rb

@@ -1,6 +1,6 @@
 module ActionReporter
   class SentryReporter < Base
-    class_accessor "Sentry", gem_spec: "sentry-ruby (~> 5)"
+    class_accessor "Sentry"
 
     def notify(error, context: {})
       sentry_class.with_scope do |temp_scope|
@@ -15,11 +15,11 @@ module ActionReporter
     end
 
     def context(args)
-      sentry_class.get_current_scope.set_context("context", transform_context(args))
+      apply_reporter_context(merge_context_updates(current_reporter_context, args))
     end
 
     def reset_context
-      sentry_class.get_current_scope.set_context("context", {})
+      apply_reporter_context({})
     end
 
     def current_user=(user)
@@ -39,8 +39,20 @@ module ActionReporter
 
     private
 
-    def blank_user_id?(id)
-      id.nil? || (id.respond_to?(:empty?) && id.empty?)
+    def current_reporter_context
+      sentry_class.get_current_scope.contexts["context"] || {}
+    end
+
+    # sentry-ruby merges via set_context and cannot remove keys; replace the bucket via protected setter.
+    def apply_reporter_context(context_hash)
+      scope = sentry_class.get_current_scope
+      contexts = scope.contexts.dup
+      if context_hash.empty?
+        contexts.delete("context")
+      else
+        contexts["context"] = context_hash
+      end
+      scope.send(:contexts=, contexts)
     end
   end
 end

data/lib/action_reporter/version.rb

@@ -1,3 +1,3 @@
 module ActionReporter
-  VERSION = "3.1.0"
+  VERSION = "3.2.1"
 end

data/lib/action_reporter.rb

@@ -37,6 +37,7 @@ module ActionReporter
   @logger = nil
   @error_handler = nil
   @user_id_resolver = nil
+  @no_reporters_warned = false
 
   def enabled_reporters=(reporters)
     @enabled_reporters = reporters || []
@@ -91,6 +92,8 @@ module ActionReporter
   end
 
   def notify(error, context: {})
+    warn_if_no_reporters(:notify)
+
     enabled_reporters.each do |reporter|
       next unless reporter.respond_to?(:notify)
 
@@ -105,6 +108,8 @@ module ActionReporter
   def context(args)
     raise ArgumentError, "context must be a Hash" unless args.is_a?(Hash)
 
+    warn_if_no_reporters(:context)
+
     enabled_reporters.each do |reporter|
       next unless reporter.respond_to?(:context)
 
@@ -254,6 +259,18 @@ module ActionReporter
     ensure
       self.transaction_name = previous_name if name
       self.transaction_id = previous_id if id
+      if context_options.any?
+        context(context_options.transform_values { nil })
+      end
     end
   end
+
+  def warn_if_no_reporters(method_name)
+    return if @no_reporters_warned
+    return unless enabled_reporters.empty?
+
+    @no_reporters_warned = true
+    warn "[action_reporter] #{method_name} called but enabled_reporters is empty"
+  end
+  private :warn_if_no_reporters
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: action_reporter
 version: !ruby/object:Gem::Version
-  version: 3.1.0
+  version: 3.2.1
 platform: ruby
 authors:
 - Andrei Makarov
@@ -9,6 +9,20 @@ bindir: bin
 cert_chain: []
 date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
@@ -16,7 +30,7 @@ dependencies:
     - - ">"
       - !ruby/object:Gem::Version
         version: '5'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -111,14 +125,14 @@ dependencies:
   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
@@ -136,75 +150,75 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3'
 - !ruby/object:Gem::Dependency
-  name: rspec_junit_formatter
+  name: polyrun
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: 2.2.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: 2.2.0
 - !ruby/object:Gem::Dependency
-  name: webmock
+  name: prosopite
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: rspec_junit_formatter
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.15'
+        version: '0.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.15'
+        version: '0.6'
 - !ruby/object:Gem::Dependency
-  name: simplecov
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.22'
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.22'
+        version: '3'
 - !ruby/object:Gem::Dependency
-  name: simplecov-cobertura
+  name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '0.15'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '0.15'
 - !ruby/object:Gem::Dependency
   name: standard
   requirement: !ruby/object:Gem::Requirement
@@ -356,6 +370,7 @@ files:
 - LICENSE.md
 - README.md
 - action_reporter.gemspec
+- bin/polyrun
 - lib/action_reporter.rb
 - lib/action_reporter/active_version_reporter.rb
 - lib/action_reporter/audited_reporter.rb
@@ -393,7 +408,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.6
 specification_version: 4
 summary: See description
 test_files: []