claude_agent 0.7.13 → 0.7.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6e5c5f3de29ca5a58accdb4c6dcc0937ed867fd87579c4909c42ae4d7c82e8d7
-  data.tar.gz: 9209ef4868fc7d6267c8f215c6e46222d69cf672a8223c4277db35131510c466
+  metadata.gz: 6b6188550282e29001c6bf2996842b5c11cd013de37aeeded995c982cef67399
+  data.tar.gz: 33e7d2e6b000d5f085093f236fd531c136e3711be170c3af4c11715fbdedda77
 SHA512:
-  metadata.gz: c78cf5cc5e99c4b97130950763ce04192cbe10dccc4def0f245d9dae298123a1a56e54a75ea079f23f1d9da1415ff9758f7b2a75943ba34027a602fff12cdf5d
-  data.tar.gz: 7c3146bd079c24011f2bf95dbd1e12c6a0a81a0dad1634221827138516467df9f344e54813d2281f76bc1e6bd0ff97dd04b0ea81b854222b992f467e541d6c4d
+  metadata.gz: b5ec2de8caa5ef32681e671645dd6d6540d58f86221c3d99186cc7ab7e9a076b892fe41c16e2472b94399c296439b5ceba72f573eb0b218b8b2ff507661d162f
+  data.tar.gz: c00a1312f295f1f15949cac076155750b2259f5412cfb0cbe6acffa8bcb789927887bb58ebacdc94fd92fa492851f985b975a14bb31abe68b78e23ce4fda8c5d

data/CHANGELOG.md

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.7.14] - 2026-03-14
+
+### Fixed
+- Replace busy-wait polling (`Queue#pop(true)` + `sleep 0.01`) in `ControlProtocol::Messaging#each_message` with blocking `Queue#pop` and `:done` sentinel, eliminating CPU waste and up-to-10ms per-message latency
+- Fix `Conversation#partition_kwargs` misrouting `on_elicitation` (and any future `on_*` Options attributes) as event callbacks instead of forwarding to Options; now uses explicit allowlist derived from `EventHandler::EVENTS`
+- Remove global `ENV["CLAUDE_CODE_ENTRYPOINT"]` mutation from `Client#connect` and `ClaudeAgent.query`; the subprocess already receives this via `Options#to_env`
+- Log stderr callback errors instead of silently swallowing them in `Transport::Subprocess`
+
 ## [0.7.13] - 2026-03-14
 
 ### Added

data/lib/claude_agent/client.rb

@@ -77,8 +77,6 @@ module ClaudeAgent
     def connect(prompt: nil)
       raise CLIConnectionError, "Already connected" if @connected
 
-      ENV["CLAUDE_CODE_ENTRYPOINT"] = "sdk-rb-client"
-
       logger.info("client") { "Connecting" }
       @protocol = ControlProtocol.new(transport: @transport, options: @options)
       @protocol.permission_queue = @permission_queue

data/lib/claude_agent/control_protocol/lifecycle.rb

@@ -72,7 +72,8 @@ module ClaudeAgent
           @condition.broadcast
         end
 
-        # Terminate the transport
+        # Unblock the consumer and terminate the transport
+        @message_queue.push(:done)
         @transport.terminate if @transport.respond_to?(:terminate)
       end
 
@@ -107,6 +108,8 @@ module ClaudeAgent
       rescue AbortError
         logger.debug("protocol") { "Reader thread exiting: abort signal" }
         @running = false
+      ensure
+        @message_queue.push(:done)
       end
     end
   end

data/lib/claude_agent/control_protocol/messaging.rb

@@ -26,19 +26,16 @@ module ClaudeAgent
       def each_message
         return enum_for(:each_message) unless block_given?
 
-        while @running || !@message_queue.empty?
-          # Check abort signal
+        loop do
           @abort_signal&.check!
 
+          raw = @message_queue.pop  # blocks until data available
+          break if raw == :done     # sentinel from reader_loop
+
           begin
-            raw = @message_queue.pop(true)
             message = @parser.parse(raw)
             yield message
-          rescue ThreadError
-            # Queue empty, wait a bit
-            sleep 0.01
           rescue AbortError
-            # Re-raise abort errors
             raise
           rescue => e
             logger.warn("protocol") { "Message parse error: #{e.message}" }

data/lib/claude_agent/control_protocol.rb

@@ -25,6 +25,28 @@ module ClaudeAgent
   #   protocol.start
   #   protocol.each_message { |msg| process(msg) }
   #
+  # State ownership across modules:
+  #
+  # Owned by Lifecycle:
+  #   @running, @reader_thread
+  #
+  # Owned by RequestHandling:
+  #   @hook_callbacks
+  #
+  # Owned by Primitives:
+  #   write_message, read helpers (stateless)
+  #
+  # Owned by Commands:
+  #   interrupt, rewind (stateless, uses @transport)
+  #
+  # Shared (initialized here, used by multiple modules):
+  #   @transport, @options, @parser, @server_info
+  #   @request_counter, @pending_requests, @pending_results (Primitives + RequestHandling)
+  #   @mutex, @condition (Primitives + Lifecycle + RequestHandling)
+  #   @message_queue (Lifecycle + Messaging)
+  #   @abort_signal (Lifecycle + Messaging)
+  #   @permission_queue (RequestHandling, set externally by Client)
+  #
   class ControlProtocol
     DEFAULT_TIMEOUT = 60
     REQUEST_ID_PREFIX = "req"
@@ -78,6 +100,7 @@ module ClaudeAgent
 
       # Abort signal from options
       @abort_signal = options&.abort_signal
+      @abort_signal&.on_abort { @message_queue.push(:done) }
     end
 
     private

data/lib/claude_agent/conversation.rb

@@ -193,6 +193,13 @@ module ClaudeAgent
 
     private
 
+    # Callback keys derived from EventHandler events + aliases + on_permission
+    CALLBACK_KEYS = (
+      EventHandler::EVENTS.map { |e| :"on_#{e}" } +
+      CALLBACK_ALIASES.keys +
+      [ :on_permission ]
+    ).freeze
+
     def partition_kwargs(kwargs)
       callbacks = {}
       conversation_kwargs = {}
@@ -201,7 +208,7 @@ module ClaudeAgent
       kwargs.each do |key, value|
         if CONVERSATION_KEYS.include?(key)
           conversation_kwargs[key] = value
-        elsif key.to_s.start_with?("on_")
+        elsif CALLBACK_KEYS.include?(key)
           callbacks[key] = value
         else
           options_kwargs[key] = value

data/lib/claude_agent/query.rb

@@ -41,8 +41,6 @@ module ClaudeAgent
       transport ||= Transport::Subprocess.new(options: options)
 
       Enumerator.new do |yielder|
-        # Set entrypoint environment variable
-        ENV["CLAUDE_CODE_ENTRYPOINT"] = "sdk-rb"
         query_logger = options.effective_logger
         query_logger.info("query") { "Starting query" }
         query_start = Process.clock_gettime(Process::CLOCK_MONOTONIC)

data/lib/claude_agent/transport/subprocess.rb

@@ -387,8 +387,8 @@ module ClaudeAgent
           @stderr.each_line do |line|
             # Call callback if provided, otherwise just drain
             @options.stderr_callback&.call(line.chomp)
-          rescue
-            # Ignore callback errors
+          rescue => e
+            logger.debug("transport") { "stderr callback error: #{e.message}" }
           end
         rescue IOError
           # Stream closed, exit thread

data/lib/claude_agent/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ClaudeAgent
-  VERSION = "0.7.13"
+  VERSION = "0.7.14"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: claude_agent
 version: !ruby/object:Gem::Version
-  version: 0.7.13
+  version: 0.7.14
 platform: ruby
 authors:
 - Thomas Carr