bug_reports_client 0.1.2 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fbb8da2de255d2a5894ce7079e462b6cc34d53fd3d4077560a4cb7ef3985df64
-  data.tar.gz: a01439a1eb84e11e51724dbb6b88456871580c654ede63ca57e261b78ab94c0f
+  metadata.gz: af1649eb6a9f0c2f134c12ff56daa1b518596abb811df41b3a09646cf11cf723
+  data.tar.gz: 350efaea07db0ad969590657a5469cef5d31cb5fabb5f54f0d4b7e5d2301c381
 SHA512:
-  metadata.gz: ac6b9dd379ca0530da20787480736f05d415fcac662df5ac58e2d52c61ae6054648bc9e0a15a82523e8c1077a4d4e5c9cd03113ad4acd44be7b0499099c96362
-  data.tar.gz: 780d9c5e1ce19a35eca0c762dc15a0188f7a3be89958f19f540830e7721641b8ec558e0c9ac8ba5850cb1170c3f229d7a473066a41cb66fa5dcb649a8a2d9930
+  metadata.gz: 420ca42a1fac80cbb2f9cc426c5ba5d9759dcdeb41d45c3affad1396cd922fb93c3402ec43edec37f2be1a5e9b324af8ba1d2c8bf8b21976b9360ff706690256
+  data.tar.gz: a76dea291d1ae17e671b06d05f1717866a5270347b585b39bd571624bc7fe8e9771fcef26ce946eee93e24ce02ac55238fc395a8a16b63646c7f121ee876a7d2

data/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Changelog
 
+## 0.1.4 (16 October 2026)
+
+- The engine's helpers and the error-context `before_action` are now added
+  to `ActionController::Base` in `config.to_prepare`, not in an
+  `on_load(:action_controller_base)` hook. A gem that loads
+  `ActionController::Base` while Bundler is still requiring gems ran the
+  hook before engine helpers could autoload, and the host failed to boot
+  with `NameError: uninitialized constant BugReportsClient::BugReportsHelper`.
+  `omniauth-rails_csrf_protection` 2.x does this on Rails 8.1. Hosts need no
+  changes.
+
+## 0.1.3 (7 October 2026)
+
+- Unhandled errors raised by `bin/rails runner` scripts (Rails.error source
+  `"application.runner.railties"`) are no longer reported. They have no
+  application frame, so they all shared one fingerprint and merged into one
+  misleading issue. Requests and jobs are still captured. The skipped sources
+  are configurable via the new `config.ignored_error_sources` setting.
+
 ## 0.1.2 (16 August 2026)
 
 - Resolved-report alerts no longer stack. When a user has two or more

data/README.md

@@ -224,6 +224,7 @@ Opt in per host and unhandled 500s become GitHub issues, deduplicated:
 config.error_reporting_enabled = true
 config.error_ignore = []          # extra exception class names to skip
 config.error_throttle_period = 300 # seconds between posts per fingerprint
+config.ignored_error_sources = [ "application.runner.railties" ] # the default
 ```
 
 The engine subscribes to the Rails error reporter, so anything that would
@@ -236,6 +237,15 @@ slowed down. The API keeps one open issue per fingerprint, counting repeat
 occurrences; when a closed error recurs, a fresh issue is filed (a
 regression deserves new attention).
 
+Errors are also skipped when the `source:` Rails passes to the error
+reporter is in `config.ignored_error_sources`. By default that list holds
+`"application.runner.railties"`, the source Rails uses for unhandled errors
+in `bin/rails runner` scripts: the person running the script already sees the
+error in their terminal, and with no application frame in the backtrace
+every such error would share one fingerprint and merge into one misleading
+issue. Requests and jobs are still captured. Add other sources to skip them
+too, or set the list to `[]` to capture runner errors as well.
+
 Captured errors are also attributed to the signed-in user who hit them
 (stored locally in `bug_report_error_events`, pruned after 30 days). When
 that user opens the report form within 24 hours, it asks "did this happen

data/lib/bug_reports_client/configuration.rb

@@ -44,6 +44,10 @@ module BugReportsClient
     # limits how often one fingerprint is posted from this app.
     attr_accessor :error_reporting_enabled, :error_ignore, :error_throttle_period
 
+    # Rails.error `source:` values never reported. Defaults to `bin/rails runner`
+    # script errors, which the operator sees in the terminal and which lack an app frame.
+    attr_accessor :ignored_error_sources
+
     # Optional overrides for the form definition and GitHub issue template.
     # When nil, the engine looks for config/bug_report_form.yml and
     # config/bug_report_issue.md in the host app, then falls back to its own
@@ -82,6 +86,7 @@ module BugReportsClient
       @error_reporting_enabled = false
       @error_ignore = []
       @error_throttle_period = 300
+      @ignored_error_sources = [ "application.runner.railties" ]
     end
 
     # The URL the API calls back when a report's GitHub issue is closed.

data/lib/bug_reports_client/engine.rb

@@ -29,13 +29,16 @@ module BugReportsClient
     # Make engine helpers (bug_report_alerts and the badge helpers) available
     # in host views, so layouts can render the resolved-report alerts with a
     # single line.
-    initializer "bug_reports_client.helpers" do
-      ActiveSupport.on_load(:action_controller_base) do
-        helper BugReportsClient::Engine.helpers
-        # Tag requests with the current user so captured 500s can be
-        # attributed to whoever hit them (see ErrorContext).
-        include BugReportsClient::ErrorContext
-      end
+    #
+    # Keep this in to_prepare, not on_load(:action_controller_base): a gem that
+    # loads ActionController::Base during Bundler.require fires on_load hooks
+    # before engine helpers can autoload, and the host fails to boot with
+    # NameError on BugReportsHelper.
+    config.to_prepare do
+      ActionController::Base.helper BugReportsClient::Engine.helpers
+      # Tag requests with the current user so captured 500s can be
+      # attributed to whoever hit them (see ErrorContext).
+      ActionController::Base.include BugReportsClient::ErrorContext
     end
 
     # Automatic 500 capture: subscribe to the Rails error reporter. The

data/lib/bug_reports_client/error_reporter.rb

@@ -7,6 +7,8 @@ module BugReportsClient
   #
   #   - Only unhandled errors are captured, and only ones Rails would answer
   #     with a 5xx (RecordNotFound and friends map to 4xx and are skipped).
+  #   - Errors from sources in config.ignored_error_sources are skipped; by
+  #     default that is `bin/rails runner` scripts.
   #   - Errors are fingerprinted by exception class + the top application
   #     backtrace frame (line numbers stripped, so deploys that shift code
   #     around keep the same fingerprint).
@@ -23,6 +25,7 @@ module BugReportsClient
     def report(error, handled:, severity: nil, context: {}, source: nil)
       return if handled
       return unless BugReportsClient.config.error_reporting_enabled
+      return if BugReportsClient.config.ignored_error_sources.include?(source)
       return if ignored?(error)
 
       fingerprint = self.class.fingerprint_for(error)

data/lib/bug_reports_client/version.rb

@@ -1,3 +1,3 @@
 module BugReportsClient
-  VERSION = "0.1.2"
+  VERSION = "0.1.4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bug_reports_client
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.4
 platform: ruby
 authors:
 - PSA Squash Tour
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-08-16 00:00:00.000000000 Z
+date: 2026-11-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails