active_mcp 0.3.0 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 051dd51506aaadb0c6bf460f1f001049b5d70dc950a060a663d2256050224de7
-  data.tar.gz: e70d63d1618f882dd9d459de896a5ba086e29cbef9acbdf897f800a76266850e
+  metadata.gz: 862fda276ef5442ec0c9ee381134e33ce27b1dd797d386db7db95defc29fa1ee
+  data.tar.gz: 2d7892d5eceb1acc3643451e3786cc208cfa1fea1e4b440b05b6ec5a5c440ac8
 SHA512:
-  metadata.gz: 7b5c6aaa00d5f9138d2186f2ea4c988f621f5f231674f4c01eeda64418187e0502999b1385c9417824bd99b888a9252f2833c5f5f42026e8943ce07d8a8952a4
-  data.tar.gz: 31d7ea0b9f9cd7b6385a30ced3a9caf14d2105f366d614c1ae8fc5c227f617c547f1f86c63084e4816265cb838ed8e089a4997e9cd1d3e757f8304cd197717a7
+  metadata.gz: b14998a48988aef825e41603d34a3ec571df2ffb5dff804494ecfe137f4fadc138d55b462580dca5097341bc098fab890d1fe8456a90c29ee25b64a6337cf8d3
+  data.tar.gz: ab594bcf8c596d1f6f47c93f3a1c14163113b2767fb2dd9fb64cb9c92d97799144819d0b8870242562e02069c19fcee3eb0cde660b2c42c64f2783a35a300a4f

data/README.md

@@ -72,7 +72,7 @@ end
 
 #### with streamable HTTP
 
-Set MCP destination as `https:your-app.example.com/mcp`
+Set MCP destination to `https:your-app.example.com/mcp`
 
 #### with independent MCP Server
 

data/app/controllers/active_mcp/base_controller.rb

@@ -4,65 +4,56 @@ module ActiveMcp
   class BaseController < ActionController::Base
     protect_from_forgery with: :null_session
     skip_before_action :verify_authenticity_token
-    before_action :process_authentication, only: [:index]
+    before_action :authenticate, only: [:index]
 
     def index
+      if params[:jsonrpc]
+        process_request_from_mcp_client
+      else
+        process_request_from_mcp_server
+      end
+    end
+
+    private
+
+    def process_request_from_mcp_server
+      case params[:method]
+      when Method::TOOLS_LIST
+        result = Response::ToolsList::Json.call(
+          tools: Response::Tools.to_hash(auth_info: @auth_info)
+        )
+      when Method::TOOLS_CALL
+        result = Response::ToolsCall::Json.call(params:, auth_info: @auth_info)
+      else
+        result = Response::NoMethod.call
+      end
+
+      render json: result, status: 200
+    end
+
+    def process_request_from_mcp_client
       case params[:method]
       when Method::INITIALIZE
-        render_initialize
+        result = Response::Initialize.call(id: params[:id])
       when Method::INITIALIZED
-        render json: {
-          jsonrpc: "2.0",
-          method: "notifications/initialized"
-        }
+        result = Response::Initialized.call
       when Method::CANCELLED
-        render json: {
-          jsonrpc: "2.0",
-          method: "notifications/cancelled"
-        }
+        result = Response::Cancelled.call
       when Method::TOOLS_LIST
-        if params[:jsonrpc]
-          render_tools_list_as_jsonrpc
-        else
-          render_tools_list
-        end
+        result = Response::ToolsList::Jsonrpc.call(
+          id: params[:id],
+          tools: Response::Tools.to_hash(auth_info: @auth_info)
+        )
       when Method::TOOLS_CALL
-        if params[:jsonrpc]
-          call_tool_as_jsonrpc
-        else
-          render json: call_tool(params)
-        end
+        result = Response::ToolsCall::Jsonrpc.call(id: params[:id], params:, auth_info: @auth_info)
       else
-        render json: {error: "Method not found: #{params[:method]}"}, status: 404
+        result = Response::NoMethod.call
       end
-    end
 
-    private
-
-    def render_tools_list_as_jsonrpc
-      render json: {
-        jsonrpc: "2.0",
-        id: params[:id],
-        result: {tools:}
-      }
+      render json: result, status: 200
     end
 
-    def call_tool_as_jsonrpc
-      render json: {
-        jsonrpc: "2.0",
-        id: params[:id],
-        result: {
-          content: [
-            {
-              type: "text",
-              text: call_tool(params[:params])[:result]
-            }
-          ]
-        }
-      }
-    end
-
-    def process_authentication
+    def authenticate
       auth_header = request.headers["Authorization"]
       if auth_header.present?
         @auth_info = {
@@ -78,90 +69,5 @@ module ActiveMcp
         }
       end
     end
-
-    def render_initialize
-      render json: {
-          jsonrpc: "2.0",
-          id: params[:id],
-          result: {
-            protocolVersion: "2024-11-05",
-            capabilities: {
-              logging: {},
-              capabilities: {
-                resources: {
-                  subscribe: false,
-                  listChanged: false
-                },
-                tools: {
-                  listChanged: false
-                }
-              },
-            },
-            serverInfo: {
-              name: ActiveMcp::Configuration.config.server_name,
-              version: ActiveMcp::Configuration.config.server_version
-            }
-          }
-        }
-    end
-
-    def render_tools_list
-      render json: {result: tools}
-    end
-
-    def tools
-      Tool.registered_tools.select do |tool_class|
-        tool_class.authorized?(@auth_info)
-      end.map do |tool_class|
-        {
-          name: tool_class.tool_name,
-          description: tool_class.desc,
-          inputSchema: tool_class.schema
-        }
-      end
-    end
-
-    def call_tool(params)
-      tool_name = params[:name]
-
-      unless tool_name
-        return {error: "Invalid params: missing tool name"}
-      end
-
-      tool_class = Tool.registered_tools.find do |tc|
-        tc.tool_name == tool_name
-      end
-
-      unless tool_class
-        return {error: "Tool not found: #{tool_name}"}
-      end
-
-      unless tool_class.authorized?(@auth_info)
-        return {error: "Unauthorized: Access to tool '#{tool_name}' denied"}
-      end
-
-      if params[:arguments].is_a?(Hash)
-        arguments = params[:arguments].symbolize_keys
-      else
-        arguments = params[:arguments].permit!.to_hash.symbolize_keys.transform_values { _1.match(/^\d+$/) ? _1.to_i : _1 }
-      end
-
-      tool = tool_class.new
-      validation_result = tool.validate_arguments(arguments)
-
-      if validation_result.is_a?(Hash) && validation_result[:error]
-        return {result: validation_result[:error]}
-      end
-
-      begin
-        arguments[:auth_info] = @auth_info if @auth_info.present?
-
-        result = tool.call(**arguments.symbolize_keys)
-
-        return {result: result}
-      rescue => e
-        return {error: "Error: #{e.message}"}
-      end
-    end
   end
 end

data/app/models/active_mcp/response/cancelled.rb

@@ -0,0 +1,12 @@
+module ActiveMcp
+  module Response
+    class Cancelled
+      def self.call
+        {
+          jsonrpc: JSON_RPC_VERSION,
+          method: Method::CANCELLED
+        }
+      end
+    end
+  end
+end

data/app/models/active_mcp/response/initialize.rb

@@ -0,0 +1,31 @@
+module ActiveMcp
+  module Response
+    class Initialize
+      def self.call(id:)
+        {
+          jsonrpc: JSON_RPC_VERSION,
+          id:,
+          result: {
+            protocolVersion: PROTOCOL_VERSION,
+            capabilities: {
+              logging: {},
+              capabilities: {
+                resources: {
+                  subscribe: false,
+                  listChanged: false
+                },
+                tools: {
+                  listChanged: false
+                }
+              },
+            },
+            serverInfo: {
+              name: ActiveMcp.config.server_name,
+              version: ActiveMcp.config.server_version
+            }
+          }
+        }
+      end
+    end
+  end
+end

data/app/models/active_mcp/response/initialized.rb

@@ -0,0 +1,12 @@
+module ActiveMcp
+  module Response
+    class Initialized
+      def self.call
+        {
+          jsonrpc: JSON_RPC_VERSION,
+          method: Method::INITIALIZED
+        }
+      end
+    end
+  end
+end

data/app/models/active_mcp/response/no_method.rb

@@ -0,0 +1,11 @@
+module ActiveMcp
+  module Response
+    class NoMethod
+      def self.call
+        {
+          error: "Method not found"
+        }
+      end
+    end
+  end
+end

data/app/models/active_mcp/response/tools.rb

@@ -0,0 +1,17 @@
+module ActiveMcp
+  module Response
+    class Tools
+      def self.to_hash(auth_info:)
+        Tool.registered_tools.select do |tool_class|
+          tool_class.authorized?(auth_info)
+        end.map do |tool_class|
+          {
+            name: tool_class.tool_name,
+            description: tool_class.desc,
+            inputSchema: tool_class.schema
+          }
+        end
+      end
+    end
+  end
+end

data/app/models/active_mcp/response/tools_call/json.rb

@@ -0,0 +1,11 @@
+module ActiveMcp
+  module Response
+    module ToolsCall
+      class Json
+        def self.call(params:, auth_info:)
+          ActiveMcp::ToolExecutor.call(params:, auth_info:)
+        end
+      end
+    end
+  end
+end

data/app/models/active_mcp/response/tools_call/jsonrpc.rb

@@ -0,0 +1,16 @@
+module ActiveMcp
+  module Response
+    module ToolsCall
+      class Jsonrpc
+        def self.call(id:, params:, auth_info:)
+          result = ActiveMcp::ToolExecutor.call(params: params[:params], auth_info:)
+          {
+            jsonrpc: JSON_RPC_VERSION,
+            id:,
+            result:
+          }
+        end
+      end
+    end
+  end
+end

data/app/models/active_mcp/response/tools_list/json.rb

@@ -0,0 +1,13 @@
+module ActiveMcp
+  module Response
+    module ToolsList
+      class Json
+        def self.call(tools:)
+          {
+            result: tools
+          }
+        end
+      end
+    end
+  end
+end

data/app/models/active_mcp/response/tools_list/jsonrpc.rb

@@ -0,0 +1,15 @@
+module ActiveMcp
+  module Response
+    module ToolsList
+      class Jsonrpc
+        def self.call(id:, tools:)
+          {
+            jsonrpc: JSON_RPC_VERSION,
+            id:,
+            result: {tools:}
+          }
+        end
+      end
+    end
+  end
+end

data/app/models/active_mcp/tool_executor.rb

@@ -0,0 +1,104 @@
+module ActiveMcp
+  module ToolExecutor
+    def self.call(params:, auth_info:)
+      tool_name = params[:name]
+
+      unless tool_name
+        return {
+          isError: true,
+          content: [
+            {
+              type: "text",
+              text: "Invalid params: missing tool name",
+            }
+          ]
+        }
+      end
+
+      tool_class = Tool.registered_tools.find do |tc|
+        tc.tool_name == tool_name
+      end
+
+      unless tool_class
+        return {
+          isError: true,
+          content: [
+            {
+              type: "text",
+              text: "Tool not found: #{tool_name}",
+            }
+          ]
+        }
+      end
+
+      unless tool_class.authorized?(auth_info)
+        return {
+          isError: true,
+          content: [
+            {
+              type: "text",
+              text: "Unauthorized: Access to tool '#{tool_name}' denied",
+            }
+          ]
+        }
+      end
+
+      arguments = params[:arguments].permit!.to_hash.symbolize_keys.transform_values do |value|
+        if !value.is_a?(String)
+          value
+        else
+          value.match(/^\d+$/) ? value.to_i : value
+        end
+      end
+
+      tool = tool_class.new
+      validation_result = tool.validate_arguments(arguments)
+
+      if validation_result.is_a?(Hash) && validation_result[:error]
+        return {
+          isError: true,
+          content: [
+            {
+              type: "text",
+              text: validation_result[:error],
+            }
+          ]
+        }
+      end
+
+      begin
+        arguments[:auth_info] = auth_info if auth_info.present?
+
+        return {
+          content: [
+            {
+              type: "text",
+              text: formatted(tool.call(**arguments.symbolize_keys))
+            }
+          ]
+        }
+      rescue => e
+        return {
+          isError: true,
+          content: [
+            {
+              type: "text",
+              text: "Error: #{e.message}",
+            }
+          ]
+        }
+      end
+    end
+
+    def self.formatted(object)
+      case object
+      when String
+        object
+      when Hash
+        object.to_json
+      else
+        object.to_s
+      end
+    end
+  end
+end

data/lib/active_mcp/server/tool_manager.rb

@@ -36,7 +36,6 @@ module ActiveMcp
       def invoke_tool(name, arguments)
         require "net/http"
         
-        # URIの検証
         unless @uri.is_a?(URI) || @uri.is_a?(String)
           log_error("Invalid URI type", StandardError.new("URI must be a String or URI object"))
           return {
@@ -48,7 +47,6 @@ module ActiveMcp
         begin
           uri = URI.parse(@uri.to_s)
           
-          # 有効なスキームとホストの検証
           unless uri.scheme =~ /\Ahttps?\z/ && !uri.host.nil?
             log_error("Invalid URI", StandardError.new("URI must have a valid scheme and host"))
             return {
@@ -57,7 +55,6 @@ module ActiveMcp
             }
           end
           
-          # 本番環境ではHTTPSを強制
           if defined?(Rails) && Rails.env.production? && uri.scheme != "https"
             return {
               isError: true,
@@ -87,15 +84,7 @@ module ActiveMcp
           end
 
           if response.code == "200"
-            body = JSON.parse(response.body, symbolize_names: true)
-            if body[:error]
-              {
-                isError: true,
-                content: [{type: "text", text: body[:error]}]
-              }
-            else
-              format_result(body[:result])
-            end
+            JSON.parse(response.body, symbolize_names: true)
           else
             {
               isError: true,
@@ -103,7 +92,6 @@ module ActiveMcp
             }
           end
         rescue => e
-          # ログに詳細を記録
           log_error("Error calling tool", e)
           {
             isError: true,
@@ -117,7 +105,6 @@ module ActiveMcp
 
         require "net/http"
         
-        # URIの検証
         unless @uri.is_a?(URI) || @uri.is_a?(String)
           log_error("Invalid URI type", StandardError.new("URI must be a String or URI object"))
           return
@@ -126,13 +113,11 @@ module ActiveMcp
         begin
           uri = URI.parse(@uri.to_s)
           
-          # 有効なスキームとホストの検証
           unless uri.scheme =~ /\Ahttps?\z/ && !uri.host.nil?
             log_error("Invalid URI", StandardError.new("URI must have a valid scheme and host"))
             return
           end
           
-          # 本番環境ではHTTPSを強制
           if defined?(Rails) && Rails.env.production? && uri.scheme != "https"
             log_error("HTTPS is required in production environment", StandardError.new("Non-HTTPS URI in production"))
             return
@@ -162,17 +147,6 @@ module ActiveMcp
           @tools = []
         end
       end
-
-      def format_result(result)
-        case result
-        when String
-          {content: [{type: "text", text: result}]}
-        when Hash
-          {content: [{type: "text", text: result.to_json}]}
-        else
-          {content: [{type: "text", text: result.to_s}]}
-        end
-      end
       
       def log_error(message, error)
         error_details = "#{message}: #{error.message}\n"
@@ -181,7 +155,6 @@ module ActiveMcp
         if defined?(Rails)
           Rails.logger.error(error_details)
         else
-          # Fallback to standard error output if Rails is not available
           $stderr.puts(error_details)
         end
       end

data/lib/active_mcp/version.rb

@@ -1,3 +1,3 @@
 module ActiveMcp
-  VERSION = "0.3.0"
+  VERSION = "0.3.2"
 end

data/lib/active_mcp.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require_relative "active_mcp/version"
-require_relative "active_mcp/config"
+require_relative "active_mcp/configuration"
 require_relative "active_mcp/tool"
 require_relative "active_mcp/server"
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: active_mcp
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.2
 platform: ruby
 authors:
-- Your Name
+- Moeki Kawakami
 bindir: exe
 cert_chain: []
-date: 2025-04-05 00:00:00.000000000 Z
+date: 2025-04-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -45,7 +45,7 @@ dependencies:
         version: '0'
 description: A Rails engine that provides MCP capabilities to your Rails application
 email:
-- your.email@example.com
+- hi@moeki.org
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -54,9 +54,19 @@ files:
 - README.md
 - Rakefile
 - app/controllers/active_mcp/base_controller.rb
+- app/models/active_mcp/response/cancelled.rb
+- app/models/active_mcp/response/initialize.rb
+- app/models/active_mcp/response/initialized.rb
+- app/models/active_mcp/response/no_method.rb
+- app/models/active_mcp/response/tools.rb
+- app/models/active_mcp/response/tools_call/json.rb
+- app/models/active_mcp/response/tools_call/jsonrpc.rb
+- app/models/active_mcp/response/tools_list/json.rb
+- app/models/active_mcp/response/tools_list/jsonrpc.rb
+- app/models/active_mcp/tool_executor.rb
 - config/routes.rb
 - lib/active_mcp.rb
-- lib/active_mcp/config.rb
+- lib/active_mcp/configuration.rb
 - lib/active_mcp/engine.rb
 - lib/active_mcp/server.rb
 - lib/active_mcp/server/error_codes.rb
@@ -68,7 +78,6 @@ files:
 - lib/active_mcp/version.rb
 - lib/generators/active_mcp/install/install_generator.rb
 - lib/generators/active_mcp/install/templates/initializer.rb
-- lib/generators/active_mcp/install/templates/mcp_server.rb
 - lib/generators/active_mcp/tool/templates/tool.rb.erb
 - lib/generators/active_mcp/tool/tool_generator.rb
 homepage: https://github.com/moekiorg/active_mcp

data/lib/generators/active_mcp/install/templates/mcp_server.rb

@@ -1,31 +0,0 @@
-#!/usr/bin/env ruby
-# MCP Server script for Active MCP
-require 'active_mcp'
-
-# Load Rails application
-ENV['RAILS_ENV'] ||= 'development'
-require File.expand_path('../config/environment', __dir__)
-
-# Initialize MCP server
-server = ActiveMcp::Server.new(
-  name: "<%= Rails.application.class.module_parent_name %> MCP Server",
-  uri: '<%= Rails.application.config.action_controller.default_url_options&.fetch(:host, 'http://localhost:3000') %>/mcp'
-)
-
-# Optional authentication
-<% if ActiveMcp.config.auth_enabled %>
-server.auth = {
-  type: :bearer,
-  token: ENV['MCP_AUTH_TOKEN'] || '<%= ActiveMcp.config.auth_token %>'
-}
-<% end %>
-
-# Start the server
-puts "Starting MCP server for <%= Rails.application.class.module_parent_name %>..."
-puts "Connect to this server in MCP clients using the following configuration:"
-puts
-puts "   Command: #{File.expand_path(__FILE__)}"
-puts "   Args: []"
-puts
-puts "Press Ctrl+C to stop the server"
-server.start