diffbroker 0.12.0 → 0.12.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b3251c8cf1dc34ca4cfda98157ac33e2cdd3181d2b7464b6732a73b6731de43
-  data.tar.gz: cf1c80fb6862c480b58442d6432ef38d05e7817a44f5813642b402d2b1450a7e
+  metadata.gz: 93366baf36d2624668bda7070960557ef337a5699c80be427798638386adcdda
+  data.tar.gz: 0013377ed45686039a4f4eeeae00c995844fdea592a885dfae774d6c1ba9812c
 SHA512:
-  metadata.gz: 4561d11449c3387cd2f97b44bb60b2f4bf0f3259ae5d5bc97fb7c13764c00c68a60f787fec09f45a876dccb1c861edd4d7f8acf24448fec0d2f16b3878e0e336
-  data.tar.gz: 79116f0dff8028b473d6c526606cca711c091e1b71bdf7fdc304482d41e2ca9d171e3d0aa01b1396c0c4e22870abf1300b8aa2360ed2a66ccbf566ea6e8cfab5
+  metadata.gz: 78525e73d46c72bc944a63181b1e86bf774d12d7273d81010a1024b0ea17012f601ff208d3bbbd7d34b3992dd8ddfbeff1d0471abaadf691f047e9411fe5f54f
+  data.tar.gz: 3e74c23e1d5091235d00482aa790af5a9b40af3263ff7caf9b5c03e8f9cccee2f1e0c65a5b66ddd15a91353ebfc589e0bf768db9210a2d481a7c1426dc998c7b

data/lib/diffbroker/daemon.rb

@@ -149,6 +149,7 @@ module Diffbroker
       end
     ensure
       if @owner
+        @task_runner.interrupt_all! if @task_runner.respond_to?(:interrupt_all!)
         @relay&.stop
         @lifecycle&.clear_pid
         @state&.clear(pid: Process.pid) unless @rejected # a rejected daemon leaves its reason for `status`
@@ -220,7 +221,9 @@ module Diffbroker
         rescan(client)
         @usage_at = nil
         client.perform("hello", "version" => VERSION, "platform" => Diffbroker.platform,
-                                "service" => @lifecycle ? @lifecycle.service_installed? : false, "agent" => @agent)
+                                "service" => @lifecycle ? @lifecycle.service_installed? : false, "agent" => @agent,
+                                # What this daemon is running: a run of this device that is missing here was lost with a restart.
+                                "running" => @running_lock.synchronize { @running.to_a })
         @out.puts "Connected. #{@repositories.size} repositories reported."
       end
       client.on_tick do

data/lib/diffbroker/task_runner.rb

@@ -89,6 +89,16 @@ module Diffbroker
       finished(id, "failed", 0, "agent_failed", e.message)
     end
 
+    # The daemon is ending (stop, restart, upgrade). Its agents end with it, so none keeps working on a session the next companion
+    # continues, and nothing is reported as failed: the server hands the lost runs back when the next companion says hello.
+    def interrupt_all!
+      handles = @lock.synchronize do
+        @interrupted = true
+        @handles.values
+      end
+      handles.each { signal("TERM", _1.pid) }
+    end
+
     # Spec §4.2 step 9: SIGINT to the agent's process group, SIGTERM after the grace period if it is still running.
     def cancel(task_id)
       # A cancel that lands between a verified run's two processes has no handle; remember it so the verifier never starts.
@@ -288,6 +298,8 @@ module Diffbroker
     end
 
     def finished(id, status, cost_usd, error_code, last_output)
+      return if @lock.synchronize { @interrupted }
+
       last = last_output.to_s.lines.last.to_s.strip
       @report.call("finished", { "task_id" => id, "status" => status, "cost_usd" => cost_usd, "error_code" => error_code,
                                  "last_output" => last[0, MAX_OUTPUT] })

data/lib/diffbroker/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Diffbroker
-  VERSION = "0.12.0"
+  VERSION = "0.12.1"
 end

data/plugin/.claude-plugin/plugin.json

@@ -1,5 +1,5 @@
 {
   "name": "diffbroker",
-  "version": "0.12.0",
+  "version": "0.12.1",
   "description": "Skills for working on Diff Broker agent tasks: the diffbroker MCP tools, write rules and report_task."
 }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: diffbroker
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.12.1
 platform: ruby
 authors:
 - Yuri Sidorov (@newstler)