diffbroker 0.12.0 → 0.12.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b3251c8cf1dc34ca4cfda98157ac33e2cdd3181d2b7464b6732a73b6731de43
-  data.tar.gz: cf1c80fb6862c480b58442d6432ef38d05e7817a44f5813642b402d2b1450a7e
+  metadata.gz: 843632a85c52d68a7b94d6a1f7dd1229e7353961a5611a7ed9c1cb05d7f11542
+  data.tar.gz: 7a0c380a109114c7feeb13b203fc45a05e95f3da9773288e0b77e52d6df12a58
 SHA512:
-  metadata.gz: 4561d11449c3387cd2f97b44bb60b2f4bf0f3259ae5d5bc97fb7c13764c00c68a60f787fec09f45a876dccb1c861edd4d7f8acf24448fec0d2f16b3878e0e336
-  data.tar.gz: 79116f0dff8028b473d6c526606cca711c091e1b71bdf7fdc304482d41e2ca9d171e3d0aa01b1396c0c4e22870abf1300b8aa2360ed2a66ccbf566ea6e8cfab5
+  metadata.gz: 1d432acb33edef5d5f4d7699445a4e1a53f119499ecc94ea70e10001eb7c894358efe5d74984495d660230924510f89ff9656e9252afe11e73bb96d11b4d54db
+  data.tar.gz: 60b4a66bd37677fcf6f39ea798d7b7c0fee21cd04167cd102d7958af51cfa1f2f6ccbba05070825eba84239c2209da07c3c91ef6870d4373be4d46d6e906d33a

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
@@ -364,9 +367,8 @@ module Diffbroker
       return unless @running_lock.synchronize { !@upgrading && @running.add?(id) } # the upgrade's restart would kill it
 
       @state&.update(busy_tasks: @running_lock.synchronize { @running.size })
-      repositories = @repositories
       @spawner.call do
-        @task_runner.run(message, repositories:)
+        @task_runner.run(message, repositories: known_repositories)
       rescue StandardError => e
         @out.puts "Task #{id} crashed: #{e.class}: #{e.message}"
       ensure
@@ -377,6 +379,14 @@ module Diffbroker
       end
     end
 
+    # The server hands over pending tasks while the connection is still being set up, before the first scan: scan now rather
+    # than fail the task with "no checkout".
+    def known_repositories
+      @repositories = @scanner.scan if @scanned_at.nil?
+      @scanned_at ||= @clock.call
+      @repositories
+    end
+
     # Helpers an agent left in worktrees nobody works in (rubocop --server and the like). Only while no task runs here.
     def sweep_strays
       return unless @reaper && @running_lock.synchronize { @running.empty? }

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.2"
 end

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

@@ -1,5 +1,5 @@
 {
   "name": "diffbroker",
-  "version": "0.12.0",
+  "version": "0.12.2",
   "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.2
 platform: ruby
 authors:
 - Yuri Sidorov (@newstler)