diffbroker 0.12.1 → 0.12.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 93366baf36d2624668bda7070960557ef337a5699c80be427798638386adcdda
-  data.tar.gz: 0013377ed45686039a4f4eeeae00c995844fdea592a885dfae774d6c1ba9812c
+  metadata.gz: 5bdc96c23de6492a2ea0660c7f897c8c04efc4ea3993cf96e9b4c6bf57385644
+  data.tar.gz: b1159b4577c725eb8f9a92277ccdfd84fcdf9cf655d09dbe384d70d6cc00bc75
 SHA512:
-  metadata.gz: 78525e73d46c72bc944a63181b1e86bf774d12d7273d81010a1024b0ea17012f601ff208d3bbbd7d34b3992dd8ddfbeff1d0471abaadf691f047e9411fe5f54f
-  data.tar.gz: 3e74c23e1d5091235d00482aa790af5a9b40af3263ff7caf9b5c03e8f9cccee2f1e0c65a5b66ddd15a91353ebfc589e0bf768db9210a2d481a7c1426dc998c7b
+  metadata.gz: 07e7a17ef7f21c5350e7796d58b75b5255d51a7fcac5ca7f2036e86f82915f4bfaa96a2058572667279e3ffe50d85953ad224977bfbf5da50fc88eec150aa6d5
+  data.tar.gz: 7457ee92b3fbad1376448ae6be16e700a74ca4e7427a5f58aea9e713a4b09da88d4424f85201b6c95be8f1ba5f84e558f976fc570b8658a7110572e9750b0cee

data/lib/diffbroker/agents/claude.rb

@@ -30,11 +30,16 @@ module Diffbroker
       def resumes? = true
 
       def autonomous_command(task, mcp_config:)
-        args = task["read_only"] ? READ_ONLY_ARGS : @config.claude_args
+        args = task["read_only"] ? READ_ONLY_ARGS : with_diffbroker_tools(@config.claude_args)
         [ @config.claude_command, "-p", task.fetch("prompt"), *resume(task), "--output-format", "stream-json", "--verbose", *args,
           "--mcp-config", mcp_config, *strict(task), "--plugin-dir", @plugin_dir ]
       end
 
+      # In `auto` mode a classifier approves each tool call and refused Diff Broker's own report_task, so a run that fixed CI or
+      # posted comments ended with "no report". The tools of this server are allowed up front, as in the read-only runs: the
+      # task token limits what each may do. A user who sets --allowedTools themselves keeps full control of it.
+      def with_diffbroker_tools(args) = args.include?("--allowedTools") ? args : [ *args, "--allowedTools", "mcp__diffbroker__*" ]
+
       # The permissions above are passed again: a resumed session keeps its conversation, not its sandbox.
       def resume(task) = task["resume_session_id"] ? [ "--resume", task["resume_session_id"] ] : []
 

data/lib/diffbroker/daemon.rb

@@ -367,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
@@ -380,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/version.rb

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

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

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