bug_reports_client 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ddf7fc7f7b28ffe23e4a4e2965343975fc2d353887f7215e3c7de627259caa34
-  data.tar.gz: f5d7058b59536805b5c3554c7a1b34fc367a4db509fd804489bba639b153fed9
+  metadata.gz: fbb8da2de255d2a5894ce7079e462b6cc34d53fd3d4077560a4cb7ef3985df64
+  data.tar.gz: a01439a1eb84e11e51724dbb6b88456871580c654ede63ca57e261b78ab94c0f
 SHA512:
-  metadata.gz: cc3d27ecbfb780ff73c4d13171d03ce10994c780303b4e50ead54a498b4c51725fe6b38cfb2dcd2d4f818baa875a3ac7286b0618ef082fd9d18bddb2865ef2aa
-  data.tar.gz: 46f0ce019b199b6a86901d4b932905666e28689ba7221e0e5d534a5a98b1a15567bb9353d2978a5f652b4accfdb2de4506391ea1b9a50b80239e14abf966a239
+  metadata.gz: ac6b9dd379ca0530da20787480736f05d415fcac662df5ac58e2d52c61ae6054648bc9e0a15a82523e8c1077a4d4e5c9cd03113ad4acd44be7b0499099c96362
+  data.tar.gz: 780d9c5e1ce19a35eca0c762dc15a0188f7a3be89958f19f540830e7721641b8ec558e0c9ac8ba5850cb1170c3f229d7a473066a41cb66fa5dcb649a8a2d9930

data/CHANGELOG.md

@@ -1,5 +1,25 @@
 # Changelog
 
+## 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
+  theme-appropriate classes instead of the light slate defaults.
+
+- `BUG_REPORT_APP_HOST` takes precedence over the shared `APP_HOST` env var
+  for the engine's public origin. `APP_HOST` is often owned by other host
+  concerns (mailers, OIDC issuers) in formats the engine cannot assume -
+  prefer the dedicated variable or an explicit `config.app_host`.
+
 ## 0.1.0 (23 July 2026)
 
 - Initial release: mountable engine with a schema-driven report form

data/README.md

@@ -55,7 +55,7 @@ Environment variables (or set the equivalents in the initializer):
 | `BUG_REPORT_API_URL` | Base URL of the bug-reports API, e.g. `https://bugs.example.com/api` |
 | `BUG_REPORT_API_KEY` | This app's API token (Bearer auth) |
 | `BUG_REPORT_WEBHOOK_SECRET` | This app's webhook secret (HMAC verification of closure callbacks) |
-| `APP_HOST` | Public HTTPS origin of this app (callback URL + screenshot links) |
+| `BUG_REPORT_APP_HOST` | Public HTTPS origin of this app (callback URL + screenshot links). Falls back to `APP_HOST`; prefer this or `config.app_host` if `APP_HOST` serves other purposes |
 
 On the API side, the app needs an `ApiKey` record and an entry in
 `config/repo_mapping.yml` mapping its `source` name to a GitHub repository.
@@ -208,7 +208,9 @@ Three Stimulus controllers ship with the engine and are pinned into your
 importmap automatically: `bug-reports-client--report-type` toggles the
 bug/feature field groups, `bug-reports-client--screenshot-dropzone` powers
 the drag-and-drop screenshot picker with thumbnail previews (the hidden file
-input stays the submission source, so custom forms can ignore it), and
+input stays the submission source, so custom forms can ignore it; its
+drag-over highlight is themeable via the `highlight-classes` Stimulus
+value), and
 `bug-reports-client--file-limit` is a standalone file-count guard for
 hand-rolled forms. Any standard Stimulus setup that loads controllers from
 the importmap (`eagerLoadControllersFrom` / `lazyLoadControllersFrom`) picks

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/javascript/bug_reports_client/controllers/screenshot_dropzone_controller.js

@@ -17,7 +17,10 @@ export default class extends Controller {
     max: { type: Number, default: 5 },
     maxBytes: { type: Number, default: 10_485_760 },
     limitMessage: String,
-    sizeMessage: String
+    sizeMessage: String,
+    // Classes toggled on the zone while a file is dragged over it. Dark
+    // themes should override via the Stimulus value (space-separated).
+    highlightClasses: { type: String, default: "border-slate-500 bg-slate-100" }
   }
 
   connect() {
@@ -47,11 +50,15 @@ export default class extends Controller {
 
   dragover(event) {
     event.preventDefault()
-    this.zoneTarget.classList.add("border-slate-500", "bg-slate-100")
+    this.zoneTarget.classList.add(...this.highlightClassList())
   }
 
   dragleave() {
-    this.zoneTarget.classList.remove("border-slate-500", "bg-slate-100")
+    this.zoneTarget.classList.remove(...this.highlightClassList())
+  }
+
+  highlightClassList() {
+    return this.highlightClassesValue.split(/\s+/).filter(Boolean)
   }
 
   drop(event) {

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

@@ -54,7 +54,11 @@ module BugReportsClient
       @api_url = ENV.fetch("BUG_REPORT_API_URL", "http://localhost:3002/api")
       @api_key = ENV["BUG_REPORT_API_KEY"]
       @webhook_secret = ENV["BUG_REPORT_WEBHOOK_SECRET"]
-      @app_host = ENV["APP_HOST"]
+      # A dedicated variable takes precedence: APP_HOST is commonly shared
+      # with other host-app concerns (mailers, OIDC issuers) in formats the
+      # engine cannot assume, so hosts can isolate the engine with
+      # BUG_REPORT_APP_HOST - or set config.app_host explicitly.
+      @app_host = ENV["BUG_REPORT_APP_HOST"] || ENV["APP_HOST"]
       @app_name = nil
       @source = nil
       @mount_path = "/bug_reports"

data/lib/bug_reports_client/version.rb

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

data/lib/generators/bug_reports_client/install/templates/initializer.rb

@@ -8,7 +8,8 @@ BugReportsClient.configure do |config|
   config.source = "<%= Rails.application.class.module_parent_name.underscore %>"
 
   # The public HTTPS origin of this app, used for the closure callback URL and
-  # screenshot links in GitHub issues. Defaults to ENV["APP_HOST"].
+  # screenshot links in GitHub issues. Defaults to ENV["BUG_REPORT_APP_HOST"],
+  # then ENV["APP_HOST"] - set it explicitly if APP_HOST serves other purposes.
   # config.app_host = "https://myapp.example.com"
 
   # Who counts as an admin (can see /bug_reports/all).

metadata

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