Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Mcp-core has compile-time dependency on mcp-json #612

Copy link
Copy link
@scottslewis

Description

@scottslewis
Issue body actions

mcp-core project has a compile-time dependency on mcp-json due to the location of these three classes:

/mcp-json/src/main/java/io/modelcontextprotocol/json/McpJsonMapper.java

/mcp-json/src/main/java/io/modelcontextprotocol/json/TypeRef.java

/mcp-json/src/main/java/io/modelcontextprotocol/json/schema/JsonSchemaValidator.java

I would argue that since these classes are required at all times, it doesn't make sense for them to be in a separate jar...as mcp-core jar is useless without mcp-json jar also present.

I would suggest moving these three classes (at least) back to mcp-core (e.g. in package io.modelcontextprotocol.sdk.util or io.modelcontextprotocol.sdk.json. This would allow a clear separation between mapper and schema validator interfaces (in core) and their implementation (default or other).

Reactions are currently unavailable

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestNew feature or request

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      Morty Proxy This is a proxified and sanitized view of the page, visit original site.