bug_reports_client 0.1.1 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2328f7788a059ba5cedf89c2b43e0aaf56cdac6f5a575b663e2f710ad3a0ef02
-  data.tar.gz: 22357aae7c221c4b41d20cfd56202b0b57679f96a53f11a8a4f9af04d1c81669
+  metadata.gz: cc0b9ca39db5bc33d5bf9fda72ded756fe16e42e877ef5eb93dbd2b833cc6aab
+  data.tar.gz: 71195c18c23223ffa3dd5c3669b47549b50ff36756deb3fc46c90fba039f3f6f
 SHA512:
-  metadata.gz: d60b7fabf51873954b67dd8326a6904b6aa762d72c39c86e18068cdc9c1a83cdf72818c22e866215d57b4241184dde9cb201a39ad62d2e6dfa9d3c91c07f07c4
-  data.tar.gz: 222a8df13cb624d9c674b482fc2b6a5d63ec16d02937f927931cbb78dcd4beb173aa708a34fadca413c744f4d089d68000656ded24d1ebd7628ae6d597ec0df7
+  metadata.gz: 6e125d9c99cfaf74d273509899f1c7e4ec4552b5ebc425552e436b943f47a2ce2d4bd3719aed7fee372fe1fdd98347a3c1c4a5ea76f36ca80c79fe8080d49229
+  data.tar.gz: 91125e1ec8de2e57059605f14ab4a2cc4cf705e2ae5ddc7b596381028c841fce299cc928bd828730b3c8be46f88265e04ad5ced9f6001c586a8ec4d57959600a

data/CHANGELOG.md

@@ -1,6 +1,23 @@
 # Changelog
 
-## 0.1.1 (unreleased)
+## 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
+  resolved, undismissed reports, the host layout shows a single collapsed
+  banner listing every title with one "Dismiss all" button (new
+  `PATCH /dismiss_all` route). A single resolved report keeps its own banner
+  and per-report Dismiss as before. Hosts that copied `_alerts.html.erb` via
+  the views generator will need to re-copy it to pick this up.
+
+## 0.1.1 (24 July 2026)
 
 - The screenshot dropzone's drag-over highlight classes are configurable via
   the `highlight-classes` Stimulus value, so dark-themed hosts can supply

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/app/controllers/bug_reports_client/bug_reports_controller.rb

@@ -90,6 +90,26 @@ module BugReportsClient
       render :new, status: :unprocessable_entity
     end
 
+    # PATCH /dismiss_all
+    # Dismisses every resolved-and-undismissed alert for the current user in
+    # one go - the action behind the collapsed multi-report banner.
+    def dismiss_all
+      scope = BugReport.where(user: bug_reports_current_user).resolved_and_undismissed
+      # Column write, not update!: see dismiss_bug_report
+      scope.update_all(dismissed_at: Time.current, updated_at: Time.current)
+
+      respond_to do |format|
+        format.turbo_stream do
+          streams = [ turbo_stream.remove("bug_report_alerts") ]
+          streams << render_to_string(partial: "bug_reports_client/shared/after_dismiss", formats: [ :turbo_stream ])
+          render turbo_stream: streams
+        end
+        format.html do
+          redirect_to fallback_root_path, notice: t("bug_reports_client.flashes.dismissed_all")
+        end
+      end
+    end
+
     # PATCH /:id
     # Two cases: dismissing a resolved alert (no bug_report params), or
     # editing an open report (re-syncs the remote issue).

data/app/views/bug_reports_client/shared/_alerts.html.erb

@@ -1,7 +1,12 @@
 <%# Dismissable alerts for resolved reports. Rendered in HOST layouts via the
-    bug_report_alerts helper, so routes go through the mount proxy. %>
+    bug_report_alerts helper, so routes go through the mount proxy.
+
+    One resolved report renders as its own banner. Two or more collapse into a
+    single banner listing every title with one "Dismiss all" button, so a
+    batch of fixes does not stack a wall of green boxes on every page. %>
 <div id="bug_report_alerts" class="mb-6">
-  <% resolved_bug_reports.each do |report| %>
+  <% if resolved_bug_reports.size == 1 %>
+    <% report = resolved_bug_reports.first %>
     <div id="bug_report_<%= report.id %>" class="flex items-center justify-between p-3 mb-2 border border-green-200 rounded-lg bg-green-50">
       <div class="flex items-center gap-2">
         <svg class="flex-shrink-0 w-4 h-4 text-green-500" fill="none" stroke="currentColor" viewBox="0 0 24 24">
@@ -18,5 +23,32 @@
         <%= t("bug_reports_client.alerts.dismiss") %>
       <% end %>
     </div>
+  <% else %>
+    <div id="bug_report_alerts_collapsed" class="flex items-start justify-between gap-4 p-3 mb-2 border border-green-200 rounded-lg bg-green-50">
+      <div class="flex items-start gap-2 min-w-0">
+        <svg class="flex-shrink-0 w-4 h-4 mt-0.5 text-green-500" fill="none" stroke="currentColor" viewBox="0 0 24 24">
+          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M9 12l2 2 4-4m6 2a9 9 0 11-18 0 9 9 0 0118 0z"/>
+        </svg>
+        <div class="text-sm text-green-800 min-w-0">
+          <p class="font-medium">
+            <%= t("bug_reports_client.alerts.resolved_many", count: resolved_bug_reports.size) %>
+          </p>
+          <ul class="mt-1 space-y-0.5 list-disc list-inside text-green-700">
+            <% resolved_bug_reports.each do |report| %>
+              <li id="bug_report_<%= report.id %>" class="truncate">
+                <span class="text-green-600"><%= report.type_noun.capitalize %>:</span>
+                <span class="font-medium"><%= report.title %></span>
+              </li>
+            <% end %>
+          </ul>
+        </div>
+      </div>
+      <%= button_to bug_reports_client.dismiss_all_bug_reports_path,
+          method: :patch,
+          class: "text-xs text-green-600 hover:text-green-800 font-medium flex-shrink-0 cursor-pointer whitespace-nowrap",
+          data: { turbo_stream: true } do %>
+        <%= t("bug_reports_client.alerts.dismiss_all") %>
+      <% end %>
+    </div>
   <% end %>
 </div>

data/config/locales/en.yml

@@ -163,11 +163,14 @@ en:
       submit_failed: "Unable to submit your report. Please try again later."
       only_open_editable: "Only open %{type_noun}s can be updated."
       dismissed: "%{type_noun} notification dismissed."
+      dismissed_all: "All resolved alerts dismissed."
       not_permitted: "You do not have permission for this action."
 
     alerts:
       resolved_html: "Your %{type_noun} <span class=\"font-medium\">\"%{title}\"</span> has been resolved."
+      resolved_many: "%{count} of your reports have been resolved:"
       dismiss: "Dismiss"
+      dismiss_all: "Dismiss all"
 
     filters:
       type_label: "Type"

data/config/routes.rb

@@ -2,11 +2,14 @@
 #   GET  /bug_reports          - my reports
 #   GET  /bug_reports/new      - the report form
 #   GET  /bug_reports/all      - every report (admins, via config.admin_check)
+#   PATCH /bug_reports/dismiss_all - dismiss every resolved alert at once
 #   POST /bug_reports/webhook  - signed closure callbacks from the API
 BugReportsClient::Engine.routes.draw do
   resources :bug_reports, path: "", only: %i[index new create edit update] do
     collection do
       get :all
+      # Dismiss every resolved alert at once (the collapsed multi-report banner)
+      patch :dismiss_all
     end
   end
 

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/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.1"
+  VERSION = "0.1.3"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bug_reports_client
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.3
 platform: ruby
 authors:
 - PSA Squash Tour
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-07-24 00:00:00.000000000 Z
+date: 2026-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails