gemlings 0.3.1 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fe9ef70286e116937fc896cdc9d23b3ac6ea88b5d071c5055355664784fe9ecc
-  data.tar.gz: 724554f2afc3d3d71862673ea44388ab89c70568ad609b4de32a0d4fe3e3d59b
+  metadata.gz: 9e1ddaa59fb419c9b79901c5bc307303a2fe7b8f9d2860918d2cda34a95b917f
+  data.tar.gz: c136329db04947954e039bdd139081572aaae1c66a08c3196eb63fc36d453448
 SHA512:
-  metadata.gz: 0d2f461b85343d4617626aece17873b5d2d1c60c5cf79303abac0b9276f0764b77cdf21bed385139dfa7fc094f5fe9402ad408cbbae7059ea12507bdefce15b9
-  data.tar.gz: f3289f4a29b87a9ab296d47f2a441e0699e5702839d8bdc33111d8efc36c4072417d12d16638efba7d0d32675a5fac1ce56b2f6dc5c0ba18906b2dd04a906c9a
+  metadata.gz: d45d3793c999be51f362b868ef95039da9dadd7e52a1fb8199508a14553b0a07974146a99c652adc357a7581d997febb1639415eb3fb3bf9dbea8e97c49c868e
+  data.tar.gz: 6a0daa95cf7d74009dca1857b0f44036d663efb23cb3e8187ec0d5888b4463640634b9e8edb8f8bd493d955b770232e6c665b253986cbe72996801d906490324

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.3.2
+
+Bug fixes and Ollama improvements.
+
+- **Fix Anthropic tool_result format** -- ToolCallingAgent now emits structured `tool_result` blocks instead of plain text observations, fixing 400 errors on multi-step runs with Anthropic models (thanks @parolkar)
+- **Fix trailing whitespace in messages** -- Strip trailing whitespace from message content to avoid Anthropic API rejections
+- **Fix Ollama connectivity** -- Default `ollama_api_base` to `http://localhost:11434/v1` so Ollama works out of the box without setting `OLLAMA_HOST`
+- **Fix MCP transport leak** -- Close MCP transport on error paths (no tools found, tool name not found)
+- **Fix `planning_interval: 0`** -- Guard against `ZeroDivisionError` when planning interval is zero
+
 ## 0.3.0
 
 JRuby 10 support, interactive UI, and CI.

data/lib/gemlings/memory.rb

@@ -102,7 +102,18 @@ module Gemlings
           messages << assistant_msg if assistant_msg
 
           if step.observation
-            messages << { role: "user", content: "Observation: #{step.observation}" }
+            if step.tool_calls&.any?
+              # Some LLM APIs (e.g. Anthropic) require that every tool_use block in an
+              # assistant message is immediately followed by a tool_result block in the
+              # next user message — a plain text "Observation:" message is rejected.
+              # Build a structured tool_result entry for each tool call in this step.
+              tool_results = step.tool_calls.map do |tc|
+                { type: "tool_result", tool_use_id: tc.id, content: step.observation }
+              end
+              messages << { role: "user", content: tool_results }
+            else
+              messages << { role: "user", content: "Observation: #{step.observation}" }
+            end
           elsif step.error
             messages << {
               role: "user",
@@ -113,7 +124,12 @@ module Gemlings
         end
       end
 
-      messages.each { |m| m[:content] = sanitize_utf8(m[:content]) if m[:content] }
+      # Some LLM APIs (e.g. Anthropic) reject messages whose string content ends with
+      # trailing whitespace. Strip it from every message to avoid API errors.
+      messages.each do |m|
+        m[:content] = sanitize_utf8(m[:content]) if m[:content].is_a?(String)
+        m[:content] = m[:content].rstrip if m[:content].is_a?(String)
+      end
     end
 
     def last_step

data/lib/gemlings/models/ruby_llm_adapter.rb

@@ -44,7 +44,8 @@ module Gemlings
           config.gemini_api_key = ENV["GEMINI_API_KEY"] if ENV["GEMINI_API_KEY"]
           config.deepseek_api_key = ENV["DEEPSEEK_API_KEY"] if ENV["DEEPSEEK_API_KEY"]
           config.openrouter_api_key = ENV["OPENROUTER_API_KEY"] if ENV["OPENROUTER_API_KEY"]
-          config.ollama_api_base = ENV["OLLAMA_HOST"] if ENV["OLLAMA_HOST"]
+          ollama_base = ENV["OLLAMA_HOST"] || "http://localhost:11434"
+          config.ollama_api_base = "#{ollama_base.chomp("/")}/v1"
         end
 
         @configured = true
@@ -58,7 +59,7 @@ module Gemlings
 
       def load_messages(chat, messages)
         messages.each do |msg|
-          role = msg[:role]
+          role    = msg[:role]
           content = msg[:content]
 
           case role
@@ -71,11 +72,30 @@ module Gemlings
             end
             chat.add_message(attrs)
           else
-            chat.add_message(role: role.to_sym, content: content || "")
+            # A user message whose content is an array of tool_result hashes is the
+            # structured response to one or more tool_use calls in the preceding
+            # assistant message. ruby_llm expects these to be added as separate
+            # messages with role: :tool and a tool_call_id, not as plain user text.
+            if tool_result_content?(content)
+              content.each do |tr|
+                chat.add_message(
+                  role: :tool,
+                  content: (tr[:content] || tr["content"]).to_s,
+                  tool_call_id: tr[:tool_use_id] || tr["tool_use_id"]
+                )
+              end
+            else
+              chat.add_message(role: role.to_sym, content: content || "")
+            end
           end
         end
       end
 
+      def tool_result_content?(content)
+        content.is_a?(Array) &&
+          content.all? { |c| c.is_a?(Hash) && (c[:type] == "tool_result" || c["type"] == "tool_result") }
+      end
+
       def load_tools(chat, tool_schemas)
         tool_schemas.each { |schema| chat.with_tool(build_stub_tool(schema)) }
       end

data/lib/gemlings/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Gemlings
-  VERSION = "0.3.1"
+  VERSION = "0.3.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gemlings
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.2
 platform: ruby
 authors:
 - Chris Hasiński