robot_lab 0.5.1 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7830657dbfe2508c38884f5a117e3b1eb6db6223c0c5336ba8fdbdd3c8718732
-  data.tar.gz: 3df367453aef2881e24e7e718c7778416e932e4f265deca27961bbe1952db483
+  metadata.gz: c0a38d5eaca5ddd1d558581dd57db81c94c461b0d9bf7bffbbf325f529e5a8ef
+  data.tar.gz: 7541f4d1fd133333d667a528f8f96da5287a76a4c6642a3677334496740934b6
 SHA512:
-  metadata.gz: 5f12198c217fa3465274de33a0dfac869eecc51e679f5f65d6d0ee3f0ad18ce9cb14e346b12f35fff6f0fc948ec8dc19e2dfc4eae66afd94a6e4f469a9d2458b
-  data.tar.gz: 7e91d8448bc45512216c1e53ba4dee007c0f40f225a6a34b1b8a51238962b99fd6b26757a6dc842d332ed34ccc73f7a381e404d622ad1e847bfeff5da29c1df0
+  metadata.gz: 6501bd24004fc28a105555a74ca5ae40d4e713d4485d1c581bfb6a77e810119b8e5a9c939c29fd19ec770e7b51eb10b43825c6ebddb7ef34fd60c4915e93639d
+  data.tar.gz: 70b65d1afb7fa35c48c278455ed7dd33fb7ff3c506eaf259ae885bb2b681efbeb0f4d9d8dfa7f99676817f07328c40ca7428ddf0ffe2110503bdf290524bf65c

data/CHANGELOG.md

@@ -10,6 +10,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
+- **Tool classes in `local_tools:`.** A `RubyLLM::Tool` subclass passed as a class (`local_tools: [SharedTools::Tools::DnsTool]`) reached the chat as `dns`, but robot_lab's own checks used the Ruby class name `"SharedTools::Tools::DnsTool"`. As a result, a `tools: %w[dns]` allowlist dropped the tool without an error. The robot now instantiates tool classes on construction, so every check uses the tool's name. The new `ToolConfig.instantiate` does this here and for template `tools:` front matter, and `ToolConfig.tool_name` (now public) reads `tool_name` from a tool class.
+
+### Added
+
+- **MCP server classes by constant name.** `RobotLab::MCP.build`, `name_of`, `Registry#add` and a robot's per-turn server selection accept a String naming a `RubyLLM::MCP` subclass. Template front matter can now list `mcp: ["SharedTools::MCP::GitHub"]` in the same way `tools:` lists tool classes. The new `RobotLab::MCP.resolve` does the lookup and raises `MCPError` for an undefined constant or a class that is not a `RubyLLM::MCP`.
+- **Class-level descriptions in `mcp_discovery`.** `ServerDiscovery.description_for` now falls back to a server's class when the instance has no `description`. Plain `RubyLLM::MCP` subclasses that declare a description at class level, such as the shared_tools 0.7.1 MCP clients, are scored instead of looking blank.
+
+## [0.5.1] - 2026-11-01
+
+### Fixed
+
 - **OAuth `grant:` from YAML.** An MCP server config such as `oauth: { grant: client_credentials }` raised `ArgumentError: Unknown OAuth grant`, because ruby_llm accepts the grant only as a Symbol. `RobotLab::MCP.define` now converts a String grant to a Symbol.
 
 ## [0.5.0] - 2026-11-01

data/lib/robot_lab/mcp/registry.rb

@@ -24,9 +24,10 @@ module RobotLab
 
       # Register a server (or return the one already registered for +spec+).
       #
-      # @param spec [Hash, RubyLLM::MCP, Class<RubyLLM::MCP>]
+      # @param spec [Hash, RubyLLM::MCP, Class<RubyLLM::MCP>, String]
       # @return [RubyLLM::MCP]
       def add(spec)
+        spec     = MCP.resolve(spec)
         key      = MCP.name_of(spec).to_sym
         existing = @servers[key]
         return existing if existing && same_spec?(key, spec, existing)

data/lib/robot_lab/mcp/server_discovery.rb

@@ -101,12 +101,18 @@ module RobotLab
         name.to_s.strip
       end
 
+      # An instance without its own +description+ falls back to its class's,
+      # so plain +RubyLLM::MCP+ subclasses that declare one at class level
+      # (SharedTools::MCP::GitHub, ...) are scored too.
+      #
       # @param server [Hash, RubyLLM::MCP, Class<RubyLLM::MCP>]
       # @return [String] description or empty string
       def self.description_for(server)
         desc = case server
                when Hash then server[:description] || server['description']
-               else           (server.description if server.respond_to?(:description))
+               else
+                 owner = [server, server.class].find { it.respond_to?(:description) }
+                 owner&.description
                end
         desc.to_s.strip
       end

data/lib/robot_lab/mcp.rb

@@ -54,12 +54,14 @@ module RobotLab
     module_function
 
     # Build a server from a config Hash, or pass through an instance or class.
+    # A String names a +RubyLLM::MCP+ subclass constant, the form template
+    # front matter uses (+mcp: ["SharedTools::MCP::GitHub"]+).
     #
-    # @param spec [Hash, RubyLLM::MCP, Class<RubyLLM::MCP>]
+    # @param spec [Hash, RubyLLM::MCP, Class<RubyLLM::MCP>, String]
     # @return [RubyLLM::MCP]
     # @raise [MCPError] for an unusable spec
     def build(spec)
-      case spec
+      case (spec = resolve(spec))
       when RubyLLM::MCP then spec
       when Class
         raise MCPError, "#{spec} is not a RubyLLM::MCP subclass" unless spec <= RubyLLM::MCP
@@ -68,10 +70,29 @@ module RobotLab
       when Hash then define(spec).new
       else
         raise MCPError, "Cannot build an MCP server from #{spec.class}; " \
-                        'pass a Hash, a RubyLLM::MCP, or a RubyLLM::MCP subclass'
+                        'pass a Hash, a RubyLLM::MCP, a RubyLLM::MCP subclass, or its constant name'
       end
     end
 
+    # Turn a String spec into the +RubyLLM::MCP+ subclass it names; any other
+    # spec is returned unchanged.
+    #
+    # @param spec [Hash, RubyLLM::MCP, Class<RubyLLM::MCP>, String]
+    # @return [Hash, RubyLLM::MCP, Class<RubyLLM::MCP>]
+    # @raise [MCPError] when the String does not name a RubyLLM::MCP subclass
+    def resolve(spec)
+      return spec unless spec.is_a?(String)
+
+      klass = begin
+        Object.const_get(spec)
+      rescue NameError
+        raise MCPError, "MCP server class '#{spec}' is not defined; require the file that defines it"
+      end
+      raise MCPError, "#{spec} is not a RubyLLM::MCP subclass" unless klass.is_a?(Class) && klass <= RubyLLM::MCP
+
+      klass
+    end
+
     # Turn a config Hash into an anonymous {Server} subclass with ruby_llm's
     # class-level DSL applied. Public so hosts can build classes once and
     # instantiate them per request (ruby_llm's recommended pattern).
@@ -88,11 +109,11 @@ module RobotLab
 
     # The name a spec will be registered under.
     #
-    # @param spec [Hash, RubyLLM::MCP, Class<RubyLLM::MCP>]
+    # @param spec [Hash, RubyLLM::MCP, Class<RubyLLM::MCP>, String]
     # @return [String]
     # @raise [MCPError] when the spec has no name
     def name_of(spec)
-      name = case spec
+      name = case (spec = resolve(spec))
              when Hash then spec[:name] || spec['name']
              when Class then spec.default_name
              when RubyLLM::MCP then spec.name

data/lib/robot_lab/robot/mcp_management.rb

@@ -136,7 +136,7 @@ module RobotLab
       # @param message [String, RubyLLM::MCP::Prompt, nil] this run's request
       # @return [Array<RubyLLM::MCP>]
       def servers_for(resolved, message: nil)
-        specs = Array(resolved)
+        specs = Array(resolved).map { MCP.resolve(it) }
         specs = MCP::ServerDiscovery.select(discovery_query(message), from: specs) if @mcp_discovery && message
         specs.map { |spec| @mcp_registry.add(spec) }
       end

data/lib/robot_lab/robot/template_rendering.rb

@@ -323,8 +323,7 @@ module RobotLab
           case name
           when String
             begin
-              const = Object.const_get(name)
-              const.is_a?(Class) && const < RubyLLM::Tool ? const.new : const
+              ToolConfig.instantiate(Object.const_get(name))
             rescue NameError
               RobotLab.config.logger.warn("Robot '#{@name}': tool '#{name}' not found, skipping")
               nil

data/lib/robot_lab/robot.rb

@@ -565,7 +565,7 @@ module RobotLab
       @system_prompt = system_prompt
       @build_context = context
       @description = description
-      @local_tools = Array(local_tools)
+      @local_tools = Array(local_tools).map { ToolConfig.instantiate(it) }
       @skills = skills ? Array(skills).map(&:to_sym) : nil
       @expanded_skills = nil
       @pending_agent_skills = []

data/lib/robot_lab/tool_config.rb

@@ -98,12 +98,28 @@ module RobotLab
         tools.select { |tool| allowed_set.include?(tool_name(tool)) }
       end
 
-      private
+      # Turn a RubyLLM::Tool subclass into an instance; anything else is
+      # returned unchanged. Robots hold tool instances so every name check
+      # sees the tool's name (+current_date_time+), not its Ruby class name
+      # (+SharedTools::Tools::CurrentDateTimeTool+).
+      #
+      # @param tool [Class<RubyLLM::Tool>, RubyLLM::Tool, Object]
+      # @return [Object]
+      #
+      def instantiate(tool)
+        tool.is_a?(Class) && tool < RubyLLM::Tool ? tool.new : tool
+      end
 
+      # The name a tool is offered to the LLM under.
+      #
+      # @param tool [String, Symbol, Class<RubyLLM::Tool>, #name]
+      # @return [String]
+      #
       def tool_name(tool)
         case tool
         when String then tool
         when Symbol then tool.to_s
+        when Class  then (tool.respond_to?(:tool_name) ? tool.tool_name : tool.name).to_s
         else tool.respond_to?(:name) ? tool.name.to_s : tool.to_s
         end
       end

data/lib/robot_lab/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RobotLab
-  VERSION = "0.5.1"
+  VERSION = "0.6.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: robot_lab
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.6.0
 platform: ruby
 authors:
 - Dewayne VanHoozer