actionmcp 0.22.0 → 0.24.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5c997c4c99f4ecf6409fb3749bb58efab0eb03622e70d9a447781125620e6c82
-  data.tar.gz: dd9fc6be1b7364f306a6cbcfe0cd9e8db3b00530e46056c6ad49e6995ab7810b
+  metadata.gz: 6b50744d4dcd16237d18d1b5c787f721e56281c82d1037c776201226a06cc60c
+  data.tar.gz: ed2b466652be17b4443508719da7b474b1bfc941047815f9a45bc4acf39ca2d6
 SHA512:
-  metadata.gz: c6de9245cf20071a23b0f35181d1eb20db9a2fb96298c6afc165a05aa31f14c3f8035b2b3dbd9ac9b04319f0981ce26609f8afde1ab67ef99a055a070ce3018c
-  data.tar.gz: e8b0006312a943b8228408ab0fe5b859d4a638bfbbe89ddcb6be0d38465476fdb9977f6d2b322676eb1ddc1608abcc7531c9352495c7bde11bb04868586edae1
+  metadata.gz: ba4f687a5368fb04a2f986b61859e19d05658c00e39dc2b600984db39f34fd3e0f56259e6d08c425d140c1f1641d55ee209ec0ae69109bde1e0ec5312b01cf57
+  data.tar.gz: 6daf198545188c392453e6a53b905fafd3aa6c84ab508114c563ef59ff1c8b0e44712d5c8c82dc8643497daef7136bc3c284baec482d9fd8193c3cf1a32d4fe0

data/app/models/action_mcp/session/message.rb

@@ -48,7 +48,7 @@ module ActionMCP
 
       after_create_commit :broadcast_message, if: :outgoing_message?
       # Set is_ping on responses if the original request was a ping
-      after_create :handle_ping_response, if: -> { %w[response error].include?(message_type) }
+      after_create :acknowledge_request, if: -> { %w[response error].include?(message_type) }
 
       # Scope to exclude both "ping" requests and their responses
       scope :without_pings, -> { where(is_ping: false) }
@@ -136,17 +136,22 @@ module ActionMCP
         end
       end
 
-      def handle_ping_response
+      def acknowledge_request
         return unless jsonrpc_id.present?
 
         request_message = session.messages.find_by(
           jsonrpc_id: jsonrpc_id,
           message_type: "request"
         )
-        return unless request_message&.is_ping
 
-        self.is_ping = true
+        return unless request_message
+
+        # Set is_ping based on the request
+        self.is_ping = request_message.is_ping
+
+        # Mark the request as acknowledged for all responses
         request_message.update(request_acknowledged: true)
+
         save! if changed?
       end
     end

data/lib/action_mcp/client/blueprint.rb

@@ -24,35 +24,16 @@ module ActionMCP
     #   # Generate a concrete URI from a blueprint with parameters
     #   uri = Blueprint.construct("file://{path}", { path: "/logs/app.log" })
     #
-    class Blueprint
+    class Blueprint < Collection
       # Initialize a new Blueprints collection with URI template definitions
       #
       # @param templates [Array<Hash>] Array of URI template definition hashes, each containing
       #   uriTemplate, name, description, and optionally mimeType keys
       # @param client [Object, nil] Optional client for lazy loading of templates
-      attr_reader :client
-
       def initialize(templates, client)
-        self.templates = templates
-        @client = client
-        @loaded = !templates.empty?
-      end
-
-      # Return all URI templates in the collection. If initialized with a client and templates
-      # haven't been loaded yet, this will trigger lazy loading from the client.
-      #
-      # @return [Array<Blueprint>] All blueprint objects in the collection
-      def all
-        load_templates unless @loaded
-        @templates
-      end
-
-      # Force reload all templates from the client and return them
-      #
-      # @return [Array<Blueprint>] All blueprint objects in the collection
-      def all!
-        load_templates(force: true)
-        @templates
+        super(templates, client)
+        self.templates = @collection_data
+        @load_method = :list_resource_templates
       end
 
       # Find a blueprint by its URI pattern
@@ -84,23 +65,6 @@ module ActionMCP
         blueprint.construct(params)
       end
 
-      # Filter blueprints based on a given block
-      #
-      # @yield [blueprint] Block that determines whether to include a blueprint
-      # @yieldparam blueprint [Blueprint] A blueprint from the collection
-      # @yieldreturn [Boolean] true to include the blueprint, false to exclude it
-      # @return [Array<Blueprint>] Blueprints that match the filter criteria
-      def filter(&block)
-        all.select(&block)
-      end
-
-      # Number of blueprints in the collection
-      #
-      # @return [Integer] The number of blueprints
-      def size
-        all.size
-      end
-
       # Check if the collection contains a blueprint with the given pattern
       #
       # @param pattern [String] The blueprint pattern to check for
@@ -116,41 +80,11 @@ module ActionMCP
         all.group_by(&:protocol)
       end
 
-      # Implements enumerable functionality for the collection
-      include Enumerable
-
-      # Yield each blueprint in the collection to the given block
-      #
-      # @yield [blueprint] Block to execute for each blueprint
-      # @yieldparam blueprint [Blueprint] A blueprint from the collection
-      # @return [Enumerator] If no block is given
-      def each(&block)
-        all.each(&block)
-      end
-
       # Convert raw template data into ResourceTemplate objects
       #
       # @param templates [Array<Hash>] Array of template definition hashes
       def templates=(templates)
-        @templates = templates.map { |template_data| ResourceTemplate.new(template_data) }
-      end
-
-      private
-
-      # Load or reload templates using the client
-      #
-      # @param force [Boolean] Whether to force reload even if templates are already loaded
-      # @return [void]
-      def load_templates(force: false)
-        return if @loaded && !force
-
-        begin
-          @client.list_resource_templates
-          @loaded = true
-        rescue StandardError => e
-          Rails.logger.error("Failed to load templates: #{e.message}")
-          @loaded = true unless @templates.empty?
-        end
+        @collection_data = templates.map { |template_data| ResourceTemplate.new(template_data) }
       end
 
       # Internal Blueprint class to represent individual URI templates

data/lib/action_mcp/client/catalog.rb

@@ -24,35 +24,16 @@ module ActionMCP
     #   # Get all resources matching a criteria
     #   rust_files = catalog.filter { |r| r.mime_type == "text/x-rust" }
     #
-    class Catalog
+    class Catalog < Collection
       # Initialize a new Catalog with resource definitions
       #
       # @param resources [Array<Hash>] Array of resource definition hashes, each containing
       #   uri, name, description, and mimeType keys
       # @param client [Object, nil] Optional client for lazy loading of resources
-      attr_reader :client
-
       def initialize(resources, client)
-        self.resources = resources
-        @client = client
-        @loaded = !resources.empty?
-      end
-
-      # Return all resources in the collection. If initialized with a client and resources
-      # haven't been loaded yet, this will trigger lazy loading from the client.
-      #
-      # @return [Array<Resource>] All resource objects in the collection
-      def all
-        load_resources unless @loaded
-        @resources
-      end
-
-      # Force reload all resources from the client and return them
-      #
-      # @return [Array<Resource>] All resource objects in the collection
-      def all!
-        load_resources(force: true)
-        @resources
+        super(resources, client)
+        self.resources = @collection_data
+        @load_method = :list_resources
       end
 
       # Find a resource by URI
@@ -79,16 +60,6 @@ module ActionMCP
         all.select { |resource| resource.mime_type == mime_type }
       end
 
-      # Filter resources based on a given block
-      #
-      # @yield [resource] Block that determines whether to include a resource
-      # @yieldparam resource [Resource] A resource from the collection
-      # @yieldreturn [Boolean] true to include the resource, false to exclude it
-      # @return [Array<Resource>] Resources that match the filter criteria
-      def filter(&block)
-        all.select(&block)
-      end
-
       # Get a list of all resource URIs
       #
       # @return [Array<String>] URIs of all resources in the collection
@@ -96,13 +67,6 @@ module ActionMCP
         all.map(&:uri)
       end
 
-      # Number of resources in the collection
-      #
-      # @return [Integer] The number of resources
-      def size
-        all.size
-      end
-
       # Check if the collection contains a resource with the given URI
       #
       # @param uri [String] The resource URI to check for
@@ -130,41 +94,11 @@ module ActionMCP
         end
       end
 
-      # Implements enumerable functionality for the collection
-      include Enumerable
-
-      # Yield each resource in the collection to the given block
-      #
-      # @yield [resource] Block to execute for each resource
-      # @yieldparam resource [Resource] A resource from the collection
-      # @return [Enumerator] If no block is given
-      def each(&block)
-        all.each(&block)
-      end
-
       # Convert raw resource data into Resource objects
       #
       # @param raw_resources [Array<Hash>] Array of resource definition hashes
       def resources=(raw_resources)
-        @resources = raw_resources.map { |resource_data| Resource.new(resource_data) }
-      end
-
-      private
-
-      # Load or reload resources using the client
-      #
-      # @param force [Boolean] Whether to force reload even if resources are already loaded
-      # @return [void]
-      def load_resources(force: false)
-        return if @loaded && !force
-
-        begin
-          @client.list_resources
-          @loaded = true
-        rescue StandardError => e
-          Rails.logger.error("Failed to load resources: #{e.message}")
-          @loaded = true unless @resources.empty?
-        end
+        @collection_data = raw_resources.map { |resource_data| Resource.new(resource_data) }
       end
 
       # Internal Resource class to represent individual resources

data/lib/action_mcp/client/collection.rb

@@ -0,0 +1,93 @@
+# frozen_string_literal: true
+
+module ActionMCP
+  module Client
+    # Base collection class for MCP client collections
+    class Collection
+      include RequestTimeouts
+
+      attr_reader :client, :loaded
+
+      def initialize(items, client, silence_sql: true)
+        @collection_data = items || []
+        @client = client
+        @loaded = !(@collection_data.empty?)
+        @silence_sql = silence_sql
+      end
+
+      def all
+        silence_logs { load_items unless @loaded }
+        @collection_data
+      end
+
+      def all!(timeout: DEFAULT_TIMEOUT)
+        silence_logs { load_items(force: true, timeout: timeout) }
+        @collection_data
+      end
+
+      # Filter items based on a given block
+      #
+      # @yield [item] Block that determines whether to include an item
+      # @yieldparam item [Object] An item from the collection
+      # @yieldreturn [Boolean] true to include the item, false to exclude it
+      # @return [Array<Object>] Items that match the filter criteria
+      def filter(&block)
+        all.select(&block)
+      end
+
+      # Number of items in the collection
+      #
+      # @return [Integer] The number of items
+      def size
+        all.size
+      end
+
+      # Implements enumerable functionality
+      include Enumerable
+
+      def each(&block)
+        all.each(&block)
+      end
+
+      alias loaded? loaded
+
+      protected
+
+      def load_items(force: false, timeout: DEFAULT_TIMEOUT)
+        return if @loaded && !force
+
+        # Make sure @load_method is defined in the subclass
+        raise NotImplementedError, "Subclass must define @load_method" unless defined?(@load_method)
+
+        # Use the RequestTimeouts module to handle the request
+        load_with_timeout(@load_method, force: force, timeout: timeout)
+      end
+
+      private
+
+      def silence_logs
+        return yield unless @silence_sql
+
+        original_log_level = Session.logger&.level
+        begin
+          # Temporarily increase log level to suppress SQL queries
+          Session.logger.level = Logger::WARN if Session.logger
+          yield
+        ensure
+          # Restore original log level
+          Session.logger.level = original_log_level if Session.logger
+        end
+      end
+
+      def log_error(message)
+        # Safely handle logging - don't assume Rails.logger exists
+        if defined?(Rails) && Rails.respond_to?(:logger) && Rails.logger
+          Rails.logger.error("[#{self.class.name}] #{message}")
+        else
+          # Fall back to puts if Rails.logger is not available
+          puts "[ERROR] [#{self.class.name}] #{message}"
+        end
+      end
+    end
+  end
+end

data/lib/action_mcp/client/json_rpc_handler.rb

@@ -70,25 +70,30 @@ module ActionMCP
 
       def process_response(id, result)
         if transport.id == id
-          ## This initialize the transport
+          ## This initializes the transport
           client.server = Client::Server.new(result)
           return send_initialized_notification
         end
+
         request = transport.messages.requests.find_by(jsonrpc_id: id)
         return unless request
+
+        # Mark the request as acknowledged
+        request.update(request_acknowledged: true)
+
         case request.rpc_method
         when "tools/list"
-           client.toolbox.tools = result["tools"]
-           return client.toolbox.all
+          client.toolbox.tools = result["tools"]
+          return true
         when "prompts/list"
           client.prompt_book.prompts = result["prompts"]
-          return client.prompt_book.all
+          return true
         when "resources/list"
           client.catalog.resources = result["resources"]
-          return client.catalog.all
+          return true
         when "resources/templates/list"
           client.blueprint.templates = result["resourceTemplates"]
-          return client.blueprint.all
+          return true
         end
 
         puts "\e[31mUnknown response: #{id} #{result}\e[0m"

data/lib/action_mcp/client/prompt_book.rb

@@ -24,27 +24,16 @@ module ActionMCP
     #   # Get all prompts matching a criteria
     #   text_prompts = book.filter { |p| p.name.include?("text") }
     #
-    class PromptBook
+    class PromptBook < Collection
       # Initialize a new PromptBook with prompt definitions
       #
       # @param prompts [Array<Hash>] Array of prompt definition hashes, each containing
       #   name, description, and arguments keys
       # @param client [Object, nil] Optional client for lazy loading of prompts
-      attr_reader :client
-
       def initialize(prompts, client)
-        self.prompts = prompts
-        @client = client
-        @loaded = !prompts.empty?
-      end
-
-      # Return all prompts in the collection. If initialized with a client and prompts
-      # haven't been loaded yet, this will trigger lazy loading from the client.
-      #
-      # @return [Array<Prompt>] All prompt objects in the collection
-      def all
-        load_prompts unless @loaded
-        @prompts
+        super(prompts, client)
+        self.prompts = @collection_data
+        @load_method = :list_prompts
       end
 
       # Find a prompt by name
@@ -55,16 +44,6 @@ module ActionMCP
         all.find { |prompt| prompt.name == name }
       end
 
-      # Filter prompts based on a given block
-      #
-      # @yield [prompt] Block that determines whether to include a prompt
-      # @yieldparam prompt [Prompt] A prompt from the collection
-      # @yieldreturn [Boolean] true to include the prompt, false to exclude it
-      # @return [Array<Prompt>] Prompts that match the filter criteria
-      def filter(&block)
-        all.select(&block)
-      end
-
       # Get a list of all prompt names
       #
       # @return [Array<String>] Names of all prompts in the collection
@@ -72,13 +51,6 @@ module ActionMCP
         all.map(&:name)
       end
 
-      # Number of prompts in the collection
-      #
-      # @return [Integer] The number of prompts
-      def size
-        all.size
-      end
-
       # Check if the collection contains a prompt with the given name
       #
       # @param name [String] The prompt name to check for
@@ -87,49 +59,11 @@ module ActionMCP
         all.any? { |prompt| prompt.name == name }
       end
 
-      # Implements enumerable functionality for the collection
-      include Enumerable
-
-      # Yield each prompt in the collection to the given block
-      #
-      # @yield [prompt] Block to execute for each prompt
-      # @yieldparam prompt [Prompt] A prompt from the collection
-      # @return [Enumerator] If no block is given
-      def each(&block)
-        all.each(&block)
-      end
-
-      # Force reload all prompts from the client and return them
-      #
-      # @return [Array<Prompt>] All prompt objects in the collection
-      def all!
-        load_prompts(force: true)
-        all
-      end
-
       # Convert raw prompt data into Prompt objects
       #
       # @param prompts [Array<Hash>] Array of prompt definition hashes
       def prompts=(prompts)
-        @prompts = prompts.map { |data| Prompt.new(data) }
-      end
-
-      private
-
-      # Load or reload prompts using the client
-      #
-      # @param force [Boolean] Whether to force reload even if prompts are already loaded
-      # @return [void]
-      def load_prompts(force: false)
-        return if @loaded && !force
-
-        begin
-          @client.list_prompts
-          @loaded = true
-        rescue StandardError => e
-          Rails.logger.error("Failed to load prompts: #{e.message}")
-          @loaded = true unless @prompts.empty?
-        end
+        @collection_data = prompts.map { |data| Prompt.new(data) }
       end
 
       # Internal Prompt class to represent individual prompts

data/lib/action_mcp/client/prompts.rb

@@ -4,18 +4,21 @@ module ActionMCP
   module Client
     module Prompts
       # List all available prompts from the server
-      # @return [Array<Hash>] List of available prompts with their metadata
+      # @return [String] Request ID for tracking the request
       def list_prompts
         request_id = SecureRandom.uuid_v7
 
         # Send request
         send_jsonrpc_request("prompts/list", id: request_id)
+
+        # Return request ID for tracking the request
+        request_id
       end
 
       # Get a specific prompt with arguments
       # @param name [String] Name of the prompt to get
       # @param arguments [Hash] Arguments to pass to the prompt
-      # @return [Hash] Prompt content with messages
+      # @return [String] Request ID for tracking the request
       def get_prompt(name, arguments = {})
         request_id = SecureRandom.uuid_v7
 
@@ -27,6 +30,9 @@ module ActionMCP
                              },
                              id: request_id
         )
+
+        # Return request ID for tracking the request
+        request_id
       end
     end
   end

data/lib/action_mcp/client/request_timeouts.rb

@@ -0,0 +1,76 @@
+# frozen_string_literal: true
+
+module ActionMCP
+  module Client
+    module RequestTimeouts
+      # Default timeout in seconds
+      DEFAULT_TIMEOUT = 1.0
+
+      # Load resources with timeout support - blocking until response or timeout
+      # @param method_name [Symbol] The method to call for loading (e.g., :list_resources)
+      # @param force [Boolean] Whether to force reload even if already loaded
+      # @param timeout [Float] Timeout in seconds
+      # @return [Boolean] Success status
+      def load_with_timeout(method_name, force: false, timeout: DEFAULT_TIMEOUT)
+        return true if @loaded && !force
+
+        # Make the request and store its ID
+        request_id = client.send(method_name)
+
+        start_time = Time.now
+
+        # Wait until either:
+        # 1. The collection is loaded (@loaded becomes true from JsonRpcHandler)
+        # 2. The timeout is reached
+        while !@loaded && (Time.now - start_time) < timeout
+          sleep(0.1)
+        end
+
+        # If we timed out
+        unless @loaded
+          request = client.session.messages.requests.find_by(jsonrpc_id: request_id)
+
+          if request && !request.request_acknowledged?
+            # Send cancel notification
+            client.send_jsonrpc_notification("notifications/cancelled", {
+              requestId: request_id,
+              reason: "Request timed out after #{timeout} seconds"
+            })
+
+            # Mark as cancelled in the database
+            request.update(request_cancelled: true)
+
+            log_error("Request #{method_name} timed out after #{timeout} seconds")
+          end
+
+          # Mark as loaded even though we timed out
+          @loaded = true
+          return false
+        end
+
+        # Collection was successfully loaded
+        true
+      end
+
+      private
+
+      def handle_timeout(request_id, method_name, timeout)
+        # Find the request
+        request = client.session.messages.requests.find_by(jsonrpc_id: request_id)
+
+        if request && !request.request_acknowledged?
+          # Send cancel notification
+          client.send_jsonrpc_notification("notifications/cancelled", {
+            requestId: request_id,
+            reason: "Request timed out after #{timeout} seconds"
+          })
+
+          # Mark as cancelled in the database
+          request.update(request_cancelled: true)
+
+          log_error("Request #{method_name} timed out after #{timeout} seconds")
+        end
+      end
+    end
+  end
+end

data/lib/action_mcp/client/resources.rb

@@ -4,26 +4,32 @@ module ActionMCP
   module Client
     module Resources
       # List all available resources from the server
-      # @return [Array<Hash>] List of available resources with their metadata
+      # @return [String] Request ID for tracking the request
       def list_resources
         request_id = SecureRandom.uuid_v7
 
         # Send request
         send_jsonrpc_request("resources/list", id: request_id)
+
+        # Return request ID for tracking the request
+        request_id
       end
 
       # List resource templates from the server
-      # @return [Array<Hash>] List of resource templates
+      # @return [String] Request ID for tracking the request
       def list_resource_templates
         request_id = SecureRandom.uuid_v7
 
         # Send request
         send_jsonrpc_request("resources/templates/list", id: request_id)
+
+        # Return request ID for tracking the request
+        request_id
       end
 
       # Read a specific resource
       # @param uri [String] URI of the resource to read
-      # @return [Hash] Resource content
+      # @return [String] Request ID for tracking the request
       def read_resource(uri)
         request_id = SecureRandom.uuid_v7
 
@@ -32,12 +38,15 @@ module ActionMCP
                              params: { uri: uri },
                              id: request_id
         )
+
+        # Return request ID for tracking the request
+        request_id
       end
 
       # Subscribe to updates for a specific resource
       # @param uri [String] URI of the resource to subscribe to
       # @param update_callback [Proc] Callback for resource updates
-      # @return [Boolean] Success status
+      # @return [String] Request ID for tracking the request
       def subscribe_resource(uri, update_callback)
         @resource_subscriptions ||= {}
         @resource_subscriptions[uri] = update_callback
@@ -49,11 +58,14 @@ module ActionMCP
                              params: { uri: uri },
                              id: request_id
         )
+
+        # Return request ID for tracking the request
+        request_id
       end
 
       # Unsubscribe from updates for a specific resource
       # @param uri [String] URI of the resource to unsubscribe from
-      # @return [Boolean] Success status
+      # @return [String] Request ID for tracking the request
       def unsubscribe_resource(uri)
         @resource_subscriptions&.delete(uri)
 
@@ -64,6 +76,9 @@ module ActionMCP
                              params: { uri: uri },
                              id: request_id
         )
+
+        # Return request ID for tracking the request
+        request_id
       end
     end
   end

data/lib/action_mcp/client/toolbox.rb

@@ -18,30 +18,15 @@ module ActionMCP
     #   # Get all tools matching a criteria
     #   calculation_tools = toolbox.filter { |t| t.name.include?("calculate") }
     #
-    class Toolbox
-      attr_reader :client
-
+    class Toolbox < Collection
       # Initialize a new Toolbox with tool definitions
       #
       # @param tools [Array<Hash>] Array of tool definition hashes, each containing
       #   name, description, and inputSchema keys
       def initialize(tools, client)
-        self.tools = tools
-        @client = client
-        @loaded = !tools.empty?
-      end
-
-      # Return all tools in the collection
-      #
-      # @return [Array<Tool>] All tool objects in the collection
-      def all
-        load_tools unless @loaded
-        @tools
-      end
-
-      def all!
-        load_tools(force: true)
-        @tools
+        super(tools, client)
+        self.tools = @collection_data
+        @load_method = :list_tools
       end
 
       # Find a tool by name
@@ -89,8 +74,9 @@ module ActionMCP
       # @param keyword [String] Keyword to search for in tool names and descriptions
       # @return [Array<Tool>] Tools containing the keyword
       def search(keyword)
-        @tools.select do |tool|
-          tool.name.include?(keyword) || tool.description.downcase.include?(keyword.downcase)
+        all.select do |tool|
+          tool.name.include?(keyword) ||
+            (tool.description && tool.description.downcase.include?(keyword.downcase))
         end
       end
 
@@ -102,46 +88,18 @@ module ActionMCP
         case provider
         when :claude
           # Claude format
-          { "tools" => @tools.map(&:to_claude_h) }
+          { "tools" => all.map(&:to_claude_h) }
         when :openai
           # OpenAI format
-          { "tools" => @tools.map(&:to_openai_h) }
+          { "tools" => all.map(&:to_openai_h) }
         else
           # Default format (same as original)
-          { "tools" => @tools.map(&:to_h) }
+          { "tools" => all.map(&:to_h) }
         end
       end
 
-      # Implements enumerable functionality for the collection
-      include Enumerable
-
-      # Yield each tool in the collection to the given block
-      #
-      # @yield [tool] Block to execute for each tool
-      # @yieldparam tool [Tool] A tool from the collection
-      # @return [Enumerator] If no block is given
-      def each(&block)
-        @tools.each(&block)
-      end
-
       def tools=(tools)
-        @tools = tools.map { |tool_data| Tool.new(tool_data) }
-      end
-
-      private
-
-      def load_tools(force: false)
-        return if @loaded && !force
-
-        begin
-          @client.list_tools
-          @loaded = true
-        rescue StandardError => e
-          # Handle error appropriately
-          Rails.logger.error("Failed to load tools: #{e.message}")
-          # Still mark as loaded but with empty list?
-          @loaded = true unless @tools.empty?
-        end
+        @collection_data = tools.map { |tool_data| Tool.new(tool_data) }
       end
 
       # Internal Tool class to represent individual tools

data/lib/action_mcp/client/tools.rb

@@ -4,18 +4,21 @@ module ActionMCP
   module Client
     module Tools
       # List all available tools from the server
-      # @return [Array<Hash>] List of available tools with their metadata
+      # @return [String] Request ID for tracking the request
       def list_tools
         request_id = SecureRandom.uuid_v7
 
         # Send request
         send_jsonrpc_request("tools/list", id: request_id)
+
+        # Return request ID for timeout tracking
+        request_id
       end
 
       # Call a specific tool on the server
       # @param name [String] Name of the tool to call
       # @param arguments [Hash] Arguments to pass to the tool
-      # @return [Hash] The result of the tool execution
+      # @return [String] Request ID for tracking the request
       def call_tool(name, arguments)
         request_id = SecureRandom.uuid_v7
 
@@ -27,6 +30,9 @@ module ActionMCP
                              },
                              id: request_id
         )
+
+        # Return request ID for tracking the request
+        request_id
       end
     end
   end

data/lib/action_mcp/version.rb

@@ -2,7 +2,7 @@
 
 require_relative "gem_version"
 module ActionMCP
-  VERSION = "0.22.0"
+  VERSION = "0.24.0"
 
   class << self
     alias version gem_version

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: actionmcp
 version: !ruby/object:Gem::Version
-  version: 0.22.0
+  version: 0.24.0
 platform: ruby
 authors:
 - Abdelkader Boudih
@@ -115,11 +115,13 @@ files:
 - lib/action_mcp/client/base.rb
 - lib/action_mcp/client/blueprint.rb
 - lib/action_mcp/client/catalog.rb
+- lib/action_mcp/client/collection.rb
 - lib/action_mcp/client/json_rpc_handler.rb
 - lib/action_mcp/client/logging.rb
 - lib/action_mcp/client/messaging.rb
 - lib/action_mcp/client/prompt_book.rb
 - lib/action_mcp/client/prompts.rb
+- lib/action_mcp/client/request_timeouts.rb
 - lib/action_mcp/client/resources.rb
 - lib/action_mcp/client/roots.rb
 - lib/action_mcp/client/server.rb