Merge pull request #23983 from opf/dependabot/bundler/dev/mcp-0.20.0

Bump mcp from 0.18.0 to 0.20.0

Author: NobodysNightmare

Gemfile

@@ -162,7 +162,7 @@ gem "ttfunk", "~> 1.7.0" # remove after https://github.com/prawnpdf/prawn/issues
 # prawn implicitly depends on matrix gem no longer in ruby core with 3.1
 gem "matrix", "~> 0.4.3"
 
-gem "mcp", "~> 0.18.0"
+gem "mcp", "~> 0.20.0"
 
 gem "meta-tags", "~> 2.23.0"
 

Gemfile.lock

@@ -836,8 +836,8 @@ GEM
     marcel (1.0.4)
     markly (0.16.0)
     matrix (0.4.3)
-    mcp (0.18.0)
-      json-schema (>= 4.1)
+    mcp (0.20.0)
+      json_schemer (>= 2.4)
     messagebird-rest (5.0.0)
       jwt (< 4)
     meta-tags (2.23.0)
@@ -1664,7 +1664,7 @@ DEPENDENCIES
   mail (= 2.9.0)
   markly (~> 0.15)
   matrix (~> 0.4.3)
-  mcp (~> 0.18.0)
+  mcp (~> 0.20.0)
   md_to_pdf!
   meta-tags (~> 2.23.0)
   mini_magick (~> 5.3.0)
@@ -2023,7 +2023,7 @@ CHECKSUMS
   marcel (1.0.4) sha256=0d5649feb64b8f19f3d3468b96c680bae9746335d02194270287868a661516a4
   markly (0.16.0) sha256=6f70d79e385b1efc9e171f74c81628826259039fe6c778e03c3924c71dac5511
   matrix (0.4.3) sha256=a0d5ab7ddcc1973ff690ab361b67f359acbb16958d1dc072b8b956a286564c5b
-  mcp (0.18.0) sha256=4d8ec572c1536f53370196539d5272355fb6480ed969ea58a8d24917ee2562c8
+  mcp (0.20.0) sha256=6b71bfc9a19f6dca34953bded1bd89cba75176a4f8bb293ce82ace065af584f9
   md_to_pdf (0.2.6)
   messagebird-rest (5.0.0) sha256=da4cc1efba3d5e4aa021fad07426c2cb6b326ce5670da5104bb8f6056a39d59c
   meta-tags (2.23.0) sha256=ffe78b5bee398de4ff5ac3316f5a786049538a651643b8476def06c3acc762c1

app/services/mcp_tools/base.rb

@@ -174,8 +174,7 @@ def handle_request(**)
       if Rails.env.local? && @tool_context.output_schema
         # We are only validating the output during development, so we can see errors during dev, but do not break the
         # API in production due to minor schema differences.
-        @tool_context.output_schema.validate_result(result.to_json)
-        validate_root_output_schema!(@tool_context.output_schema)
+        @tool_context.output_schema.validate_result(JSON.parse(result.to_json))
       end
 
       format_response(result)
@@ -206,13 +205,6 @@ def current_user
       @server_context[:current_user]
     end
 
-    def validate_root_output_schema!(output_schema)
-      root_type = output_schema.schema.fetch(:type, "object")
-      return if root_type == "object"
-
-      raise "MCP tools must respond with a JSON object as the root element. #{self.class} responds in #{root_type}."
-    end
-
     def render_plain_content?
       %i[full content_only].include?(Setting.mcp_tool_response_format)
     end