modelcontextprotocol/ruby-sdk
Watch
1
0
Fork
You've already forked ruby-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Combine redundant any? calls

Browse source 
Calling `parameters.any?` twice is redundant.
This commit is contained in:
Koichi ITO 2025-09-07 10:02:45 +09:00
commit b9f1c58899
1 changed files with 1 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lib/mcp/server.rb
View file

@ -320,10 +320,8 @@ module MCP
def accepts_server_context?(method_object)
parameters = method_object.parameters
accepts_server_context = parameters.any? { |_type, name| name == :server_context }
has_kwargs = parameters.any? { |type, _| type == :keyrest }
accepts_server_context || has_kwargs
parameters.any? { |type, name| type == :keyrest || name == :server_context }
end
def call_tool_with_args(tool, arguments)