chat_notifier 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 945c1f49cd9931c5e86f1f3597156a1c109255bf90a411075770745c4ed5ce63
-  data.tar.gz: 8becdc536e0b80a656b99cbf9b5a525763c46e02164816331f424798f6be3dc5
+  metadata.gz: 7edc6944f3cf725395443ed30f7aaa41dc8d357d0420c1bf3b6b020e397a01ba
+  data.tar.gz: ce30c767667ce563d983df0db9d688f4a22c72e80a8cd45115a2a2b4cb23caf5
 SHA512:
-  metadata.gz: 0c1f2d3dde65aef1acfdf4a313d0c312a031acac824bff21f64cfbfa86ff951aaea14da04209e49f7063a05bace6657165378afd84858242d8bdee0e55fd9807
-  data.tar.gz: 2547ee1bb2ea7a6532ef1b404812b4f9fea61cb38d0e50e1548399d8dee072e1d1da17b8be484a8a22b074f1f8c476099cdc092425339e386324faf268821b5b
+  metadata.gz: da79615e96a6d4d7e17c20c4bd58bfb3bf817d0bc8f84bcbb40800bbe16f353d93981c575ebed64cee755c149969714c2d39dcc2c09260e370f596e1e2affc65
+  data.tar.gz: 72a2318579a9a3ec0aacd0c352df714dcd8431f39d89d25e299218bc7daee9418077d263155a03d3ebab3fc562ab48dc0c8405b22fbccf314790aa89e0ea5d29

data/CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.4.1] - 2026-07-27
+
+### Fixed
+
+- Status reports sharing a job name fold per run, so any worker's failure fails the job (2e9a81d)
+
 ## [0.4.0] - 2026-07-24
 
 ### Added
@@ -22,9 +28,3 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Malformed NOTIFY_SLACK_THREAD_GROUP_SIZE falls back to the default (b2d5a6d)
 - Replies are skipped when the parent response has no ts, instead of posting unthreaded (b2d5a6d)
 - Notifications no longer fail silently: branch and sha now resolve from CI env or git instead of raising on the app name (8cb8603)
-
-## [0.3.0] - 2026-07-24
-
-### Added
-
-- Threaded Slack failure notifications via a Slack bot token (NOTIFY_SLACK_BOT_TOKEN), grouping failing files into batched thread replies (e453cd0)

data/README.md

@@ -96,9 +96,15 @@ back to its own thread.
 Caveats: verbose mode (`NOTIFIER_VERBOSE`) posts plain messages and bypasses
 episode resolution, and a thread store passed programmatically
 (`ChatNotifier.call(thread_store: ...)`) takes precedence over
-`NOTIFY_THREAD_STORE=none`. A single CI job running both RSpec and Minitest
-posts two status reports under the same job identity and only the earliest per
-run counts in the digest — set `NOTIFY_JOB_NAME` per framework to disambiguate.
+`NOTIFY_THREAD_STORE=none`.
+
+Reporting more than once under one job identity is safe. Parallel runners
+(`parallel_tests`, Knapsack) spawn a process per worker, each with its own
+formatter, and a single job running both RSpec and Minitest posts twice.
+Reports sharing a job name are folded per run — failure counts sum and any
+failure fails the job — so the digest keeps one line per job and a worker that
+passed cannot resolve the episode its sibling opened. Set `NOTIFY_JOB_NAME` if
+you would rather they appear as separate lines.
 
 ### Debug your Slack setup
 

data/lib/chat_notifier/messenger.rb

@@ -144,9 +144,23 @@ module ChatNotifier
       latest_id = grouped.keys.max_by { |id| [id.length, id] }
       # [length, value] orders numeric strings correctly ("9" < "10") and
       # sorts nil run_ids ("") as the oldest run.
-      # conversations.replies returns replies oldest-first, so uniq keeps the
-      # earliest status per job per run; jobs post once per run, so it's fine.
-      grouped.fetch(latest_id, []).uniq { |report| report["job"] }
+      grouped.fetch(latest_id, []).group_by { |report| report["job"] }.map do |job, group|
+        fold(job, group)
+      end
+    end
+
+    # A job identity can report more than once per run: parallel test runners
+    # (parallel_tests, Knapsack) spawn a process per worker, each with its own
+    # formatter, all sharing one job name. Any worker failing fails the job, so
+    # fold the group rather than keeping whichever arrived first — otherwise a
+    # worker that passed silently resolves the episode its sibling just opened.
+    # Commutative, so every writer converges on the same digest.
+    def fold(job, group)
+      {
+        "job" => job,
+        "status" => (group.any? { |report| report["status"] != "passed" }) ? "failed" : "passed",
+        "failures" => group.sum { |report| report["failures"].to_i }
+      }
     end
   end
 end

data/lib/chat_notifier/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ChatNotifier
-  VERSION = "0.4.0"
+  VERSION = "0.4.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chat_notifier
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - Jim Gay