brute 6.1.2 → 6.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e08acbbd71a23c8da28df07f18d191b8df282b81e43faa49934d123274258090
-  data.tar.gz: 418f3ced0cba3f915859741f43f6e64150651c3f00f05ec05825db38376e9033
+  metadata.gz: 6ecf8ae2bc514e6cb9cb6cefb1a06476ab68d13b459446ed0c398effcf334a86
+  data.tar.gz: 612dde127eee6f42a4aeb8f6c1fae039d74aaad8c9ed8481de22c0ec6b6cc2ce
 SHA512:
-  metadata.gz: 117fccb5ea1751db05c53ff3a4d1ceee66de13b4c3384877ed198cd1c384eaad0eb1344280a04486c013fdb0f956a2aab1038e1c7f156bda144de88e52fd200f
-  data.tar.gz: 9a9deb24a12e26280dd77402489140907ff8760a8232616b8288c1586e345c844c090858435d3662a3258089734fb2a16c1fedca75a271abb4aa9b01ca93c60a
+  metadata.gz: fe3ee53f5b02b310397b959f307b5ddf8639ab4cedb6d747264a1b49756e65ef93bee312a609898fe03f896a27775b5adabbb30f9e278c71df9ec68a80547e81
+  data.tar.gz: d0cba57fc257001c634474596f82b1b55517ba9f9410106b34a142df1ab8ee6db29d580016dee6db354e181741dcbad0b6be507e048dc6240193ff1cdbedba4f

data/lib/brute/message_transport/open_router.rb

@@ -13,7 +13,6 @@ module Brute
     #
     #   require "open_router"
     class OpenRouter < MessageTransport
-      EmptyCompletion = Class.new(StandardError)
 
       # What the provider reported about this call — the transport knows its
       # own library's shape, so it knows which detector to ask.
@@ -94,15 +93,6 @@ module Brute
 
           case hash
           in { role: (:system | :user | :assistant | :tool) }
-            # A completion that answered with nothing is not an answer. The
-            # provider was paid for it and something came back -- reasoning,
-            # a refusal, a field this does not know -- and appending it as an
-            # empty assistant message loses the turn quietly: the loop stops
-            # because it is not a tool result, and there is no reply in it.
-            if hash[:role] == :assistant && hash[:content].to_s.strip.empty? && Array(hash[:tool_calls]).empty? && reasoning(hash).nil?
-              raise EmptyCompletion, "the provider answered with no content and no tool calls: #{message.inspect}"
-            end
-
             # Slice away provider extras (refusal, model, ...) that
             # Brute::Message doesn't know. Reasoning is not one of them: it is
             # the model's own thinking and it goes back up with the message.
@@ -193,7 +183,7 @@ describe "brute/message_transport/open_router" do
     out.first.content.should == "hi there"
   end
 
-  it "keeps the model's reasoning, and refuses a completion that answered with nothing at all" do
+  it "keeps the model's reasoning" do
     # Reasoning is the model's own thinking, and it goes back up with the
     # message: a reasoning model that called a tool has to see it on the next
     # pass. Dropping it left an empty assistant message and a lost turn.
@@ -218,22 +208,6 @@ describe "brute/message_transport/open_router" do
       { type: "reasoning.text", text: "step by step", signature: "sig-1" },
     ]
 
-    # A tool call is an answer, even with nothing said alongside it.
-    calling = Struct.new(:choices).new([
-      { "message" => {
-        "role" => "assistant", "content" => nil,
-        "tool_calls" => [{ "id" => "tc1", "type" => "function", "function" => { "name" => "shell", "arguments" => "{}" } }],
-      } },
-    ])
-
-    Brute::MessageTransport::OpenRouter.new(calling).wrap_each.to_a.first.tool_call?.should.be.true
-
-    # Nothing said, nothing thought, nothing called: the provider was paid for
-    # an answer and there is none, which is an error rather than a message.
-    empty = Struct.new(:choices).new([{ "message" => { "role" => "assistant", "content" => "", "refusal" => nil } }])
-
-    lambda { Brute::MessageTransport::OpenRouter.new(empty).wrap_each.to_a }
-      .should.raise(Brute::MessageTransport::OpenRouter::EmptyCompletion)
   end
 
   it "round-trips every detail type under its own payload key" do

data/lib/brute/middleware/006_loop.rb

@@ -50,6 +50,13 @@ module Brute
       # stops when the LLM answers with text only or env[:should_exit] is set
       # (e.g. by MaxIterations).
       #
+      # It also loops on a completion that only thought: a reasoning model
+      # sometimes returns its thinking and stops — no text, no tool call —
+      # having decided what to do without doing it. That is not an answer, and
+      # the turn is not over; the thinking goes back with the conversation,
+      # which is what the provider asks for anyway, and the model does what it
+      # had already worked out.
+      #
       #   use Brute::Middleware::Loop::ToolResult
       #
       class ToolResult < Loop
@@ -58,7 +65,7 @@ module Brute
             next false
           end
 
-          unless env[:messages].last&.role == :tool
+          unless env[:messages].last&.role == :tool || ToolResult.thought_only?(env[:messages].last)
             next false
           end
 
@@ -67,6 +74,14 @@ module Brute
           true
         end
 
+        # Thinking, and nothing done with it.
+        def self.thought_only?(message)
+          message&.role == :assistant &&
+            message.reasoning &&
+            message.content.to_s.strip.empty? &&
+            !message.tool_call?
+        end
+
         def initialize(app)
           super(app, CONDITION)
         end
@@ -136,6 +151,26 @@ describe "brute/middleware/006_loop" do
 
   # --- Loop::BackgroundJobs ---
 
+  it "ToolResult reruns on a completion that only thought, so the turn is not lost" do
+    # A reasoning model sometimes returns the thinking and stops, having
+    # decided what to do without doing it. The turn is not over: the thinking
+    # goes back and the model does what it had worked out.
+    thought = Brute::Message.new(role: :assistant, reasoning: "I should list the events")
+
+    env = { messages: Brute.log.tap { |log| log << thought }, current_iteration: 1 }
+    Brute::Middleware::Loop::ToolResult::CONDITION.call(env).should.be.true
+    env[:current_iteration].should == 2
+
+    # An answer, or a tool call it did make, ends the turn as before.
+    answered = { messages: Brute.log.tap { |log| log.assistant("here they are") }, current_iteration: 1 }
+    Brute::Middleware::Loop::ToolResult::CONDITION.call(answered).should.be.false
+
+    calling = Brute::Message.new(role: :assistant, reasoning: "I should list them",
+      tool_calls: [{ id: "tc1", name: "list_events", arguments: {} }])
+    called = { messages: Brute.log.tap { |log| log << calling }, current_iteration: 1 }
+    Brute::Middleware::Loop::ToolResult::CONDITION.call(called).should.be.false
+  end
+
   it "BackgroundJobs reruns while jobs are running" do
     seen = []
     inner = ->(env) { seen << env[:background_jobs]; env[:background_jobs] = seen.size < 3 }

data/lib/brute/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Brute
-  VERSION = "6.1.2"
+  VERSION = "6.1.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: brute
 version: !ruby/object:Gem::Version
-  version: 6.1.2
+  version: 6.1.3
 platform: ruby
 authors:
 - Brute Contributors