modelcontextprotocol/java-sdk
Watch
1
0
Fork
You've already forked java-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor: change int to Integer for optional numeric fields (#604)

Browse source 
Convert primitive int types to Integer wrapper class for:
- JSONRPCError.code
- CreateMessageRequest.maxTokens (including constructor and builder)

This allows these fields to be nullable, properly representing
optional parameters in the MCP protocol schema.

Signed-off-by: Christian Tzolov <christian.tzolov@broadcom.com>
This commit is contained in:
Christian Tzolov 2025-10-06 23:04:42 +02:00 • committed by Christian Tzolov
commit 61f7be222b
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
mcp-core/src/main/java/io/modelcontextprotocol/spec/McpSchema.java
View file

@ -306,7 +306,7 @@ public final class McpSchema {
@JsonInclude(JsonInclude.Include.NON_ABSENT) @JsonInclude(JsonInclude.Include.NON_ABSENT)
@JsonIgnoreProperties(ignoreUnknown = true) @JsonIgnoreProperties(ignoreUnknown = true)
public record JSONRPCError( // @formatter:off public record JSONRPCError( // @formatter:off
@JsonProperty("code") int code, @JsonProperty("code") Integer code,
@JsonProperty("message") String message, @JsonProperty("message") String message,
@JsonProperty("data") Object data) { // @formatter:on @JsonProperty("data") Object data) { // @formatter:on
} }
@ -1822,14 +1822,14 @@ public final class McpSchema {
@JsonProperty("systemPrompt") String systemPrompt, @JsonProperty("systemPrompt") String systemPrompt,
@JsonProperty("includeContext") ContextInclusionStrategy includeContext, @JsonProperty("includeContext") ContextInclusionStrategy includeContext,
@JsonProperty("temperature") Double temperature, @JsonProperty("temperature") Double temperature,
@JsonProperty("maxTokens") int maxTokens, @JsonProperty("maxTokens") Integer maxTokens,
@JsonProperty("stopSequences") List<String> stopSequences, @JsonProperty("stopSequences") List<String> stopSequences,
@JsonProperty("metadata") Map<String, Object> metadata, @JsonProperty("metadata") Map<String, Object> metadata,
@JsonProperty("_meta") Map<String, Object> meta) implements Request { // @formatter:on @JsonProperty("_meta") Map<String, Object> meta) implements Request { // @formatter:on
// backwards compatibility constructor // backwards compatibility constructor
public CreateMessageRequest(List<SamplingMessage> messages, ModelPreferences modelPreferences, public CreateMessageRequest(List<SamplingMessage> messages, ModelPreferences modelPreferences,
String systemPrompt, ContextInclusionStrategy includeContext, Double temperature, int maxTokens, String systemPrompt, ContextInclusionStrategy includeContext, Double temperature, Integer maxTokens,
List<String> stopSequences, Map<String, Object> metadata) { List<String> stopSequences, Map<String, Object> metadata) {
this(messages, modelPreferences, systemPrompt, includeContext, temperature, maxTokens, stopSequences, this(messages, modelPreferences, systemPrompt, includeContext, temperature, maxTokens, stopSequences,
metadata, null); metadata, null);
@ -1859,7 +1859,7 @@ public final class McpSchema {
private Double temperature; private Double temperature;
private int maxTokens; private Integer maxTokens;
private List<String> stopSequences; private List<String> stopSequences;