Merge pull request #139 from koic/remove_useless_block_argument

Remove unused block parameters

Author: koic

lib/mcp/server.rb

@@ -253,7 +253,7 @@ def init(request)
     end
 
     def list_tools(request)
-      @tools.map { |_, tool| tool.to_h }
+      @tools.values.map(&:to_h)
     end
 
     def call_tool(request)
@@ -293,7 +293,7 @@ def call_tool(request)
     end
 
     def list_prompts(request)
-      @prompts.map { |_, prompt| prompt.to_h }
+      @prompts.values.map(&:to_h)
     end
 
     def get_prompt(request)

test/mcp/server/transports/stdio_notification_integration_test.rb

@@ -210,7 +210,7 @@ def puts(message)
           @server.define_prompt(
             name: "test_prompt",
             description: "Test prompt",
-          ) do |_args, _server_context:|
+          ) do
             MCP::PromptResponse.new(messages: [{ role: "user", content: "Test" }])
           end
 

test/mcp/server_test.rb

@@ -37,7 +37,7 @@ class ServerTest < ActiveSupport::TestCase
         arguments: [
           Prompt::Argument.new(name: "test_argument", description: "Test argument", required: true),
         ],
-      ) do |_|
+      ) do
         Prompt::Result.new(
           description: "Hello, world!",
           messages: [

test/mcp/tool_test.rb

@@ -123,7 +123,7 @@ class InputSchemaTool < Tool
         name: "mock_tool",
         title: "Mock Tool",
         description: "a mock tool for testing",
-      ) do |_|
+      ) do
         Tool::Response.new([{ type: "text", content: "OK" }])
       end
 
@@ -141,7 +141,7 @@ class InputSchemaTool < Tool
           read_only_hint: true,
           title: "Mock Tool",
         },
-      ) do |_|
+      ) do
         Tool::Response.new([{ type: "text", content: "OK" }])
       end
 
@@ -347,7 +347,7 @@ class OutputSchemaObjectTool < Tool
         title: "Mock Tool",
         description: "a mock tool for testing",
         output_schema: { properties: { result: { type: "string" } }, required: ["result"] },
-      ) do |_|
+      ) do
         Tool::Response.new([{ type: "text", content: "OK" }])
       end