Add JPMS compatibility to Jackson JSON mapper

未關閉

#753 建立於 2026年1月16日 0 則留言 0 個 reaction 已指派 0 人在 GitHub 檢視

還沒有人認領這個 Issue。

評估

難度: 2/5
預估耗時: 1-3 小時
新手友好度: 35/100

Issue 類型: 缺陷
描述清晰度: 描述清楚
活躍度: 停滯
技術堆疊: java
領域: api, backend

研究方向

從 JacksonMcpJsonMapperSupplier 開始，檢查其 ObjectMapper 如何處理 record 建構函式和參數名稱。對照 JPMS 範例驗證 mapper 設定，然後確認 record 反序列化無需 --add-opens 旗標即可正常運作，並保留現有行為。

由索引模型根據 Issue 內容生成。

描述

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.

主要語言: Java
星號: 3.7k
分支: 1.1k
平均合併: 1 天 15 小時
30 天內合併 PR: 9

貢獻指南

開啟貢獻指南

從這裡開始

先讀完整個 Issue，再讀專案的貢獻指南。
在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
Fork 儲存庫，在一個分支上完成修改。
送出 Pull Request，並在描述裡引用這個 Issue 編號。

modelcontextprotocol/java-sdk 的其他 Issue

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

area/transport bug P2

難度 2/5 1-3 小時新手友好度 88/100

modelcontextprotocol/java-sdk#1136 · 2026年9月18日建立
Client request handlers that complete empty send no JSON-RPC response 未關閉

area/client bug P2

難度 2/5 1-3 小時新手友好度 84/100

modelcontextprotocol/java-sdk#1124 · 1 則留言 · 2026年9月3日建立
ServerCapabilities.logging is added unconditionally, overriding the caller's explicit capabilities 未關閉

bug P2 ready for work

難度 2/5 1-3 小時新手友好度 68/100

modelcontextprotocol/java-sdk#1086 · 1 則留言 · 2026年8月11日建立
Reject listRoots if not supported by client, without sending any request 未關閉

enhancement good first issue P3

難度 2/5 1-3 小時新手友好度 82/100

modelcontextprotocol/java-sdk#1067 · 2026年7月19日建立
StdioClientTransport missing explicit UTF-8 charset in InputStreamReader (same issue as #295, but on client side) 未關閉

bug P2 ready for work

難度 2/5 1-3 小時新手友好度 74/100

modelcontextprotocol/java-sdk#898 · 1 則留言 · 2026年4月2日建立

查看 modelcontextprotocol/java-sdk 的全部 Issue

相似的 Issue

[Bug][UI] Change the method of opening the Action Detail 未關閉

難度 2/5 1-3 小時新手友好度 65/100

Netcracker/qubership-integration-platform#913 · 2026年9月23日建立
SEC-CORS-006 declares MEDIUM but can only emit HIGH or LOW 未關閉

bug

難度 2/5 1-3 小時新手友好度 75/100

jdubois/boot-ui#1100 · 2026年9月23日建立
build-scan-xunit: accept ISO offset timestamps in test reports 未關閉

難度 2/5 1-3 小時新手友好度 75/100

elastic/gradle-plugins#157 · 2026年9月23日建立
OpenAPI document marks optional request body of `PUT /api/users/me` as required 未關閉

難度 2/5 1-3 小時新手友好度 75/100

cryptomator/hub#497 · 2026年9月23日建立
No test covers the claim that only the last command's state is checked 未關閉

難度 2/5 1-3 小時新手友好度 75/100

johanhaleby/occurrent#1120 · 2026年9月23日建立

更多 Java Issue