Add JPMS compatibility to Jackson JSON mapper

Offen

#753 erstellt am 16.1.2026 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: java
Bereich: api, backend

Rechercherichtung

Beginne bei JacksonMcpJsonMapperSupplier und untersuche, wie sein ObjectMapper mit Record-Konstruktoren und Parameternamen umgeht. Überprüfe die Mapper-Konfiguration anhand des JPMS-Beispiels und bestätige anschließend, dass die Deserialisierung von Records ohne --add-opens-Flags funktioniert und das bestehende Verhalten beibehält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug P2

Expected Behavior

Applications using the MCP Java SDK should work with JPMS (Java Platform Module System) without requiring --add-opens JVM flags. Record deserialization should work through public constructors and parameter name discovery rather than reflective access.

  // Should work without --add-opens flags
  McpJsonMapper mapper = new JacksonMcpJsonMapperSupplier().get();
  MyRecord record = mapper.readValue(json, MyRecord.class);

Current Behavior

The JacksonMcpJsonMapperSupplier creates a plain ObjectMapper that uses reflection to access record constructors. This requires --add-opens JVM flags like:

  --add-opens java.base/java.lang.reflect=ALL-UNNAMED

Without these flags, deserialization of Java records fails with InaccessibleObjectException.

Context

We're integrating the MCP SDK into a JPMS-modularized application and cannot use --add-opens flags as they break module encapsulation.

The fix is straightforward:

Disable MapperFeature.CAN_OVERRIDE_ACCESS_MODIFIERS to prevent setAccessible() calls
Add ParameterNamesModule to discover constructor parameters from bytecode

The SDK already compiles with -parameters, so ParameterNamesModule will work without additional configuration.

Workaround: We currently maintain a custom MCP client with JPMS-compatible Jackson configuration, but would prefer to use the upstream SDK directly.

I have a PR.

Vorherrschende Sprache: Java
Sterne: 3.7k
Forks: 1.1k
Ø Merge: 1 T. 15 Std.
Gemergte PRs (30 T.): 9

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus modelcontextprotocol/java-sdk

SSE client silently truncates a data: line at U+2028/U+2029/U+0085, then fails with "Error parsing JSON-RPC message" Offen

area/transport bug P2

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100

modelcontextprotocol/java-sdk#1136 · Erstellt am 18.9.2026
Client request handlers that complete empty send no JSON-RPC response Offen

area/client bug P2

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100

modelcontextprotocol/java-sdk#1124 · 1 Kommentar · Erstellt am 03.9.2026
ServerCapabilities.logging is added unconditionally, overriding the caller's explicit capabilities Offen

bug P2 ready for work

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

modelcontextprotocol/java-sdk#1086 · 1 Kommentar · Erstellt am 11.8.2026
Reject listRoots if not supported by client, without sending any request Offen

enhancement good first issue P3

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100

modelcontextprotocol/java-sdk#1067 · Erstellt am 19.7.2026
StdioClientTransport missing explicit UTF-8 charset in InputStreamReader (same issue as #295, but on client side) Offen

bug P2 ready for work

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100

modelcontextprotocol/java-sdk#898 · 1 Kommentar · Erstellt am 02.4.2026

Alle Issues in modelcontextprotocol/java-sdk