-
Notifications
You must be signed in to change notification settings - Fork 118
MCP client - inputSchema containing anyOf #1491
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I'm not sure if this is an issue here in Quarkus Langchain4j, or if its an issue with the MCP server, as I don't see the |
What version are you using? I fixed that recently via langchain4j/langchain4j#2963 |
Yeah the MCP spec doesn't say anything about this, so I'm also a bit confused and I'm not sure if it's considered valid, but multiple MCP servers seem to be doing this, including |
I'm using 0.26.2, but will probably switch to the 0.27 CR today
----------
Sent from a mobile device
…On Thu, May 15, 2025, 12:52 AM Jan Martiska ***@***.***> wrote:
*jmartisk* left a comment (quarkiverse/quarkus-langchain4j#1491)
<#1491 (comment)>
Yeah the MCP spec doesn't say anything about this, so I'm also a bit
confused and I'm not sure if it's considered valid, but multiple MCP
servers seem to be doing this, including
@modelcontextprotocol/server-github which is kinda official
—
Reply to this email directly, view it on GitHub
<#1491 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACYXNY6SU4SGDNUZ3G7J6T26QMPBAVCNFSM6AAAAAB5D3SXKGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQOBSGMYTAMRXHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
0.27 CR2 is probably still too old, but I assume @geoand is going to release 1.0.0.CR1 which will contain langchain4j-bom 1.0.0, which contains langchain4j-mcp 1.0.0.Beta5, which fixes this (hopefully I didn't get too tangled in the versions) |
That is correct. I'll do that tomorrow morning most likely |
Then I will wait.... |
I'm trying to build an MCP client to https://github.com/GongRzhe/Office-Word-MCP-Server. At startup, when
ToolSpecificationHelper.toolSpecificationListFromMcpResponse
runs, one of the entries in the array looks like the following.You'll notice in the
inputSchema.properties
there are some properties that don't have atype
key directly. Instead they have"anyOf": ["type":]...
Quarkus langchain4j seems to blow up with this, specifically in
ToolSpecificationHelper.jsonNodeToJsonSchemaElement
. It doesn't know what to do withanyOf
.The text was updated successfully, but these errors were encountered: