claude_swarm 1.0.9 → 1.0.10

data/lib/swarm_sdk/agent/{chat → chat_helpers}/context_tracker.rb

@@ -2,7 +2,7 @@
 
 module SwarmSDK
   module Agent
-    class Chat < RubyLLM::Chat
+    module ChatHelpers
       # Manages context tracking, delegation tracking, and logging callbacks
       #
       # Responsibilities:
@@ -63,65 +63,19 @@ module SwarmSDK
 
         # Extract agent name from delegation tool name
         #
-        # Converts "DelegateTaskTo[AgentName]" to "agent_name"
-        # Example: "DelegateTaskToWorker" -> "worker"
+        # Converts "#{Tools::Delegate::TOOL_NAME_PREFIX}[AgentName]" to "agent_name"
+        # Example: "WorkWithWorker" -> "worker"
         #
         # @param tool_name [String] Delegation tool name
         # @return [String] Agent name
         def extract_delegate_agent_name(tool_name)
-          # Remove "DelegateTaskTo" prefix and lowercase first letter
-          agent_name = tool_name.to_s.sub(/^DelegateTaskTo/, "")
+          # Remove tool name prefix and lowercase first letter
+          agent_name = tool_name.to_s.sub(/^#{Tools::Delegate::TOOL_NAME_PREFIX}/, "")
           # Convert from PascalCase to lowercase (e.g., "Worker" -> "worker", "BackendDev" -> "backendDev")
           agent_name[0] = agent_name[0].downcase unless agent_name.empty?
           agent_name
         end
 
-        # Check if context usage has crossed warning thresholds and emit warnings
-        #
-        # This should be called after each LLM response to check if we've crossed
-        # any warning thresholds (80%, 90%, etc.)
-        #
-        # @return [void]
-        def check_context_warnings
-          current_percentage = @chat.context_usage_percentage
-
-          Context::CONTEXT_WARNING_THRESHOLDS.each do |threshold|
-            # Only warn once per threshold
-            next if @agent_context.warning_threshold_hit?(threshold)
-            next if current_percentage < threshold
-
-            # Mark threshold as hit and emit warning
-            @agent_context.hit_warning_threshold?(threshold)
-
-            # Emit context_threshold_hit event for snapshot reconstruction
-            LogStream.emit(
-              type: "context_threshold_hit",
-              agent: @agent_context.name,
-              threshold: threshold,
-              current_usage_percentage: current_percentage.round(2),
-            )
-
-            # Trigger automatic compression at 60% threshold
-            if threshold == Context::COMPRESSION_THRESHOLD
-              trigger_automatic_compression
-            end
-
-            # Emit legacy context_limit_warning for backwards compatibility
-            LogStream.emit(
-              type: "context_limit_warning",
-              agent: @agent_context.name,
-              model: @chat.model.id,
-              threshold: "#{threshold}%",
-              current_usage: "#{current_percentage}%",
-              tokens_used: @chat.cumulative_total_tokens,
-              tokens_remaining: @chat.tokens_remaining,
-              context_limit: @chat.context_limit,
-              metadata: @agent_context.metadata,
-              compression_triggered: threshold == Context::COMPRESSION_THRESHOLD,
-            )
-          end
-        end
-
         private
 
         # Extract usage information from an assistant message
@@ -187,6 +141,10 @@ module SwarmSDK
                 # Final response (finish_reason: "stop") - fire agent_stop
                 trigger_agent_stop(message, tool_executions: @tool_executions)
               end
+
+              # Check context warnings after each assistant message
+              # Uses unified implementation in HookIntegration
+              @chat.check_context_warnings if @chat.respond_to?(:check_context_warnings)
             when :tool
               # Handle delegation tracking and logging (technical plumbing)
               if @agent_context.delegation?(call_id: message.tool_call_id)
@@ -305,43 +263,6 @@ module SwarmSDK
           )
         end
       end
-
-      # Trigger automatic message compression
-      #
-      # Called when context usage crosses 60% threshold. Compresses old tool
-      # results to save context window space while preserving accuracy.
-      #
-      # @return [void]
-      def trigger_automatic_compression
-        return unless @chat.respond_to?(:context_manager)
-
-        # Calculate tokens before compression
-        tokens_before = @chat.cumulative_total_tokens
-
-        # Get compressed messages from ContextManager
-        compressed = @chat.context_manager.auto_compress_on_threshold(@chat.messages, keep_recent: 10)
-
-        # Count how many messages were actually compressed
-        messages_compressed = compressed.count do |msg|
-          msg.content.to_s.include?("[truncated for context management]")
-        end
-
-        # Replace messages array with compressed version
-        @chat.messages.clear
-        compressed.each { |msg| @chat.messages << msg }
-
-        # Log compression event
-        LogStream.emit(
-          type: "context_compression",
-          agent: @agent_context.name,
-          total_messages: @chat.messages.size,
-          messages_compressed: messages_compressed,
-          tokens_before: tokens_before,
-          current_usage: "#{@chat.context_usage_percentage}%",
-          compression_strategy: "progressive_tool_result_compression",
-          keep_recent: 10,
-        ) if LogStream.enabled?
-      end
     end
   end
 end

data/lib/swarm_sdk/agent/chat_helpers/event_emitter.rb

@@ -0,0 +1,204 @@
+# frozen_string_literal: true
+
+require "monitor"
+
+module SwarmSDK
+  module Agent
+    module ChatHelpers
+      # Minimal event emitter that mirrors RubyLLM::Chat's callback pattern
+      #
+      # Provides multi-subscriber support for events like tool_call, tool_result,
+      # new_message, end_message. This is thread-safe and supports unsubscription.
+      module EventEmitter
+        # Represents an active subscription to a callback event.
+        # Returned by {#subscribe} and can be used to unsubscribe later.
+        class Subscription
+          attr_reader :tag
+
+          def initialize(callback_list, callback, monitor:, tag: nil)
+            @callback_list = callback_list
+            @callback = callback
+            @monitor = monitor
+            @tag = tag
+            @active = true
+          end
+
+          # Removes this subscription from the callback list.
+          # @return [Boolean] true if successfully unsubscribed, false if already inactive
+          def unsubscribe # rubocop:disable Naming/PredicateMethod
+            @monitor.synchronize do
+              return false unless @active
+
+              @callback_list.delete(@callback)
+              @active = false
+            end
+            true
+          end
+
+          # Checks if this subscription is still active.
+          # @return [Boolean] true if still subscribed
+          def active?
+            @monitor.synchronize do
+              @active && @callback_list.include?(@callback)
+            end
+          end
+
+          def inspect
+            "#<#{self.class.name} tag=#{@tag.inspect} active=#{active?}>"
+          end
+        end
+
+        # Initialize the event emitter system
+        #
+        # Sets up @callbacks hash and @callback_monitor for thread safety.
+        # Must be called in Chat#initialize.
+        #
+        # @return [void]
+        def initialize_event_emitter
+          @callbacks = {
+            new_message: [],
+            end_message: [],
+            tool_call: [],
+            tool_result: [],
+          }
+          @callback_monitor = Monitor.new
+        end
+
+        # Subscribes to an event with the given block.
+        # Returns a {Subscription} that can be used to unsubscribe.
+        #
+        # @param event [Symbol] The event to subscribe to
+        # @param tag [String, nil] Optional tag for debugging/identification
+        # @yield The block to call when the event fires
+        # @return [Subscription] An object that can be used to unsubscribe
+        # @raise [ArgumentError] if event is not recognized
+        def subscribe(event, tag: nil, &block)
+          @callback_monitor.synchronize do
+            unless @callbacks.key?(event)
+              raise ArgumentError, "Unknown event: #{event}. Valid events: #{@callbacks.keys.join(", ")}"
+            end
+
+            @callbacks[event] << block
+            Subscription.new(@callbacks[event], block, monitor: @callback_monitor, tag: tag)
+          end
+        end
+
+        # Subscribes to an event that automatically unsubscribes after firing once.
+        #
+        # @param event [Symbol] The event to subscribe to
+        # @param tag [String, nil] Optional tag for debugging/identification
+        # @yield The block to call when the event fires (once)
+        # @return [Subscription] An object that can be used to unsubscribe before it fires
+        def once(event, tag: nil, &block)
+          subscription = nil
+          wrapper = lambda do |*args|
+            subscription&.unsubscribe
+            block.call(*args)
+          end
+          subscription = subscribe(event, tag: tag, &wrapper)
+        end
+
+        # Registers a callback for when a new message starts being generated.
+        # Multiple callbacks can be registered and all will fire in registration order.
+        #
+        # @yield Block called when a new message starts
+        # @return [self] for chaining
+        def on_new_message(&block)
+          subscribe(:new_message, &block)
+          self
+        end
+
+        # Registers a callback for when a message is complete.
+        # Multiple callbacks can be registered and all will fire in registration order.
+        #
+        # @yield [Message] Block called with the completed message
+        # @return [self] for chaining
+        def on_end_message(&block)
+          subscribe(:end_message, &block)
+          self
+        end
+
+        # Registers a callback for when a tool is called.
+        # Multiple callbacks can be registered and all will fire in registration order.
+        #
+        # @yield [ToolCall] Block called with the tool call object
+        # @return [self] for chaining
+        def on_tool_call(&block)
+          subscribe(:tool_call, &block)
+          self
+        end
+
+        # Registers a callback for when a tool returns a result.
+        # Multiple callbacks can be registered and all will fire in registration order.
+        #
+        # @yield [Object] Block called with the tool result
+        # @return [self] for chaining
+        def on_tool_result(&block)
+          subscribe(:tool_result, &block)
+          self
+        end
+
+        # Clears all callbacks for the specified event, or all events if none specified.
+        #
+        # @param event [Symbol, nil] The event to clear callbacks for, or nil for all events
+        # @return [self] for chaining
+        def clear_callbacks(event = nil)
+          @callback_monitor.synchronize do
+            if event
+              @callbacks[event]&.clear
+            else
+              @callbacks.each_value(&:clear)
+            end
+          end
+          self
+        end
+
+        # Returns the number of callbacks registered for the specified event.
+        #
+        # @param event [Symbol, nil] The event to count callbacks for, or nil for all events
+        # @return [Integer, Hash] Count for specific event, or hash of counts for all events
+        def callback_count(event = nil)
+          @callback_monitor.synchronize do
+            if event
+              @callbacks[event]&.size || 0
+            else
+              @callbacks.transform_values(&:size)
+            end
+          end
+        end
+
+        private
+
+        # Emits an event to all registered subscribers.
+        # Callbacks are executed in registration order (FIFO).
+        # Errors in callbacks are isolated - one failing callback doesn't prevent others from running.
+        #
+        # @param event [Symbol] The event to emit
+        # @param args [Array] Arguments to pass to each callback
+        # @return [void]
+        def emit(event, *args)
+          # Snapshot callbacks under lock (fast operation)
+          callbacks = @callback_monitor.synchronize { @callbacks[event]&.dup || [] }
+
+          # Execute callbacks outside lock (safe, non-blocking)
+          callbacks.each do |callback|
+            callback.call(*args)
+          rescue StandardError => e
+            handle_callback_error(event, callback, e)
+          end
+        end
+
+        # Hook for custom error handling when a callback raises an exception.
+        # Override this method in Chat to customize error behavior.
+        #
+        # @param event [Symbol] The event that was being emitted
+        # @param callback [Proc] The callback that raised the error
+        # @param error [StandardError] The error that was raised
+        # @return [void]
+        def handle_callback_error(event, _callback, error)
+          warn("[SwarmSDK] Callback error in #{event}: #{error.class} - #{error.message}")
+        end
+      end
+    end
+  end
+end

data/lib/swarm_sdk/agent/{chat → chat_helpers}/hook_integration.rb

@@ -2,7 +2,7 @@
 
 module SwarmSDK
   module Agent
-    class Chat < RubyLLM::Chat
+    module ChatHelpers
       # Integrates SwarmSDK's hook system with Agent::Chat
       #
       # Responsibilities:
@@ -71,43 +71,25 @@ module SwarmSDK
           @hook_agent_hooks[event].sort_by! { |cb| -cb.priority }
         end
 
-        # Override ask to trigger user_prompt hooks
+        # NOTE: The ask() method override has been removed.
         #
-        # This wraps the Agent::Chat#ask implementation to inject hooks AFTER
-        # system reminders are handled.
+        # In the new wrapper-based architecture, Agent::Chat#ask handles:
+        # 1. System reminder injection
+        # 2. User prompt hooks via trigger_user_prompt
+        # 3. Global semaphore acquisition
+        # 4. Delegation to RubyLLM::Chat
         #
-        # @param prompt [String] User prompt
-        # @param options [Hash] Additional options (may include source: "user" or "delegation")
-        # @return [RubyLLM::Message] LLM response
-        def ask(prompt, **options)
-          # Extract source for hook tracking (not passed to RubyLLM)
-          source = options.delete(:source) || "user"
-
-          # Trigger user_prompt hook before sending to LLM (can halt or modify prompt)
-          if @hook_executor
-            hook_result = trigger_user_prompt(prompt, source: source)
-
-            # Check if hook halted execution
-            if hook_result[:halted]
-              # Return a halted message instead of calling LLM
-              return RubyLLM::Message.new(
-                role: :assistant,
-                content: hook_result[:halt_message],
-                model_id: model.id,
-              )
-            end
-
-            # Use modified prompt if hook provided one (stdout injection)
-            prompt = hook_result[:modified_prompt] if hook_result[:modified_prompt]
-          end
-
-          # Call original ask implementation (Agent::Chat handles system reminders)
-          super(prompt, **options)
-        end
+        # The hook integration is now done directly in Agent::Chat#ask rather than
+        # through module override, since there's no inheritance chain to call super on.
 
         # Override check_context_warnings to trigger our hook system
         #
         # This wraps the default context warning behavior to also trigger hooks.
+        # Unified implementation that:
+        # 1. Emits context_threshold_hit events (for snapshot reconstruction)
+        # 2. Optionally triggers automatic compression at 60% (if no custom handler)
+        # 3. Emits context_limit_warning events (backward compatibility)
+        # 4. Triggers user-defined context_warning hooks
         def check_context_warnings
           return unless respond_to?(:context_usage_percentage)
 
@@ -121,20 +103,40 @@ module SwarmSDK
             # Mark threshold as hit
             @agent_context.hit_warning_threshold?(threshold)
 
-            # Emit existing log event (for backward compatibility)
+            # Emit context_threshold_hit event (for snapshot reconstruction) - CRITICAL
+            LogStream.emit(
+              type: "context_threshold_hit",
+              agent: @agent_context.name,
+              threshold: threshold,
+              current_usage_percentage: current_percentage.round(2),
+            )
+
+            # Check if user has defined custom handler for context_warning
+            # Custom handlers take responsibility for managing context at this threshold
+            has_custom_handler = (@hook_agent_hooks[:context_warning] || []).any?
+
+            # Trigger automatic compression at 60% ONLY if no custom handler
+            compression_triggered = false
+            if threshold == Context::COMPRESSION_THRESHOLD && !has_custom_handler
+              compressed_count = apply_automatic_compression
+              compression_triggered = compressed_count > 0
+            end
+
+            # Emit legacy context_limit_warning for backwards compatibility
             LogStream.emit(
               type: "context_limit_warning",
               agent: @agent_context.name,
-              model: model.id,
+              model: model_id,
               threshold: "#{threshold}%",
               current_usage: "#{current_percentage}%",
               tokens_used: cumulative_total_tokens,
               tokens_remaining: tokens_remaining,
               context_limit: context_limit,
               metadata: @agent_context.metadata,
+              compression_triggered: compression_triggered,
             )
 
-            # Trigger hook system
+            # Trigger hook system (user-defined handlers)
             trigger_context_warning(threshold, current_percentage) if @hook_executor
           end
         end
@@ -220,6 +222,45 @@ module SwarmSDK
 
         private
 
+        # Apply automatic message compression when context threshold is hit
+        #
+        # Called when context usage crosses 60% threshold and no custom handler exists.
+        # Compresses old tool results to save context window space while preserving accuracy.
+        #
+        # @return [Integer] Number of messages compressed (0 if compression not applied)
+        def apply_automatic_compression
+          return 0 unless respond_to?(:context_manager) && respond_to?(:messages)
+
+          # Calculate tokens before compression
+          tokens_before = cumulative_total_tokens
+
+          # Get compressed messages from ContextManager
+          compressed = context_manager.auto_compress_on_threshold(messages, keep_recent: 10)
+
+          # Count how many messages were actually compressed
+          messages_compressed = compressed.count do |msg|
+            msg.content.to_s.include?("[truncated for context management]")
+          end
+
+          # Replace messages using proper abstraction
+          replace_messages(compressed)
+
+          # Log compression event
+          LogStream.emit(
+            type: "context_compression",
+            agent: @agent_context.name,
+            total_messages: message_count,
+            messages_compressed: messages_compressed,
+            tokens_before: tokens_before,
+            current_usage: "#{context_usage_percentage}%",
+            compression_strategy: "progressive_tool_result_compression",
+            keep_recent: 10,
+            triggered_by: "auto_compression_threshold",
+          ) if LogStream.enabled?
+
+          messages_compressed
+        end
+
         # Trigger context_warning hooks
         #
         # Hooks have access to the chat instance via metadata[:chat]
@@ -263,9 +304,9 @@ module SwarmSDK
         def trigger_user_prompt(prompt, source: "user")
           return { halted: false, modified_prompt: prompt } unless @hook_executor
 
-          # Filter out delegation tools from tools list
-          actual_tools = if respond_to?(:tools) && @agent_context
-            tools.keys.reject { |tool_name| @agent_context.delegation_tool?(tool_name.to_s) }
+          # Get tool names without delegation tools using proper abstraction
+          actual_tools = if respond_to?(:non_delegation_tool_names) && @agent_context
+            non_delegation_tool_names
           else
             []
           end
@@ -281,9 +322,9 @@ module SwarmSDK
             event: :user_prompt,
             metadata: {
               prompt: prompt,
-              message_count: messages.size,
-              model: model.id,
-              provider: model.provider,
+              message_count: message_count,
+              model: model_id,
+              provider: model_provider,
               tools: actual_tools,
               delegates_to: delegate_agents,
               source: source,
@@ -371,16 +412,37 @@ module SwarmSDK
           digest = case tool_call.name
           when "Read"
             Tools::Stores::ReadTracker.get_read_files(@agent_context.name)[File.expand_path(path)]
-          when "MemoryRead"
-            # Only query if SwarmMemory is loaded (optional dependency)
-            if defined?(SwarmMemory::Core::StorageReadTracker)
-              SwarmMemory::Core::StorageReadTracker.get_read_entries(@agent_context.name)[path]
-            end
+          else
+            # Query registered plugins for digest (e.g., MemoryRead from SwarmMemory plugin)
+            query_plugin_for_digest(tool_call.name, path)
           end
 
           digest ? { read_digest: digest, read_path: path } : {}
         end
 
+        # Query registered plugins for a tool result digest
+        #
+        # This allows plugins to provide digest tracking for their own tools
+        # (e.g., MemoryRead tracking in SwarmMemory plugin).
+        #
+        # @param tool_name [String] Name of the tool
+        # @param path [String] Path or identifier of the resource
+        # @return [String, nil] Digest from first plugin that responds, or nil
+        def query_plugin_for_digest(tool_name, path)
+          return unless @agent_context
+
+          PluginRegistry.all.each do |plugin|
+            digest = plugin.get_tool_result_digest(
+              agent_name: @agent_context.name,
+              tool_name: tool_name,
+              path: path,
+            )
+            return digest if digest
+          end
+
+          nil
+        end
+
         # Wrap a tool result in our Hooks::ToolResult value object
         #
         # @param tool_call_id [String] Tool call ID

data/lib/swarm_sdk/agent/chat_helpers/instrumentation.rb

@@ -0,0 +1,78 @@
+# frozen_string_literal: true
+
+module SwarmSDK
+  module Agent
+    module ChatHelpers
+      # LLM instrumentation for API request/response logging
+      #
+      # Extracted from Chat to reduce class size and centralize observability logic.
+      module Instrumentation
+        private
+
+        # Inject LLM instrumentation middleware for API request/response logging
+        #
+        # @return [void]
+        def inject_llm_instrumentation
+          return unless @provider
+
+          faraday_conn = @provider.connection&.connection
+          return unless faraday_conn
+          return if @llm_instrumentation_injected
+
+          provider_name = @provider.class.name.split("::").last.downcase
+
+          faraday_conn.builder.insert(
+            0,
+            SwarmSDK::Agent::LLMInstrumentationMiddleware,
+            on_request: method(:handle_llm_api_request),
+            on_response: method(:handle_llm_api_response),
+            provider_name: provider_name,
+          )
+
+          @llm_instrumentation_injected = true
+
+          RubyLLM.logger.debug("SwarmSDK: Injected LLM instrumentation middleware for agent #{@agent_name}")
+        rescue StandardError => e
+          LogStream.emit_error(e, source: "instrumentation", context: "inject_middleware", agent: @agent_name)
+          RubyLLM.logger.debug("SwarmSDK: Failed to inject LLM instrumentation: #{e.message}")
+        end
+
+        # Handle LLM API request event
+        #
+        # @param data [Hash] Request data from middleware
+        def handle_llm_api_request(data)
+          return unless LogStream.emitter
+
+          LogStream.emit(
+            type: "llm_api_request",
+            agent: @agent_name,
+            swarm_id: @agent_context&.swarm_id,
+            parent_swarm_id: @agent_context&.parent_swarm_id,
+            **data,
+          )
+        rescue StandardError => e
+          LogStream.emit_error(e, source: "instrumentation", context: "emit_llm_api_request", agent: @agent_name)
+          RubyLLM.logger.debug("SwarmSDK: Error emitting llm_api_request event: #{e.message}")
+        end
+
+        # Handle LLM API response event
+        #
+        # @param data [Hash] Response data from middleware
+        def handle_llm_api_response(data)
+          return unless LogStream.emitter
+
+          LogStream.emit(
+            type: "llm_api_response",
+            agent: @agent_name,
+            swarm_id: @agent_context&.swarm_id,
+            parent_swarm_id: @agent_context&.parent_swarm_id,
+            **data,
+          )
+        rescue StandardError => e
+          LogStream.emit_error(e, source: "instrumentation", context: "emit_llm_api_response", agent: @agent_name)
+          RubyLLM.logger.debug("SwarmSDK: Error emitting llm_api_response event: #{e.message}")
+        end
+      end
+    end
+  end
+end