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 GitHub
commit a63d32a5ea
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
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)
@JsonIgnoreProperties(ignoreUnknown = true)
public record JSONRPCError( // @formatter:off
@JsonProperty("code") int code,
@JsonProperty("code") Integer code,
@JsonProperty("message") String message,
@JsonProperty("data") Object data) { // @formatter:on
}
@ -1822,14 +1822,14 @@ public final class McpSchema {
@JsonProperty("systemPrompt") String systemPrompt,
@JsonProperty("includeContext") ContextInclusionStrategy includeContext,
@JsonProperty("temperature") Double temperature,
@JsonProperty("maxTokens") int maxTokens,
@JsonProperty("maxTokens") Integer maxTokens,
@JsonProperty("stopSequences") List<String> stopSequences,
@JsonProperty("metadata") Map<String, Object> metadata,
@JsonProperty("_meta") Map<String, Object> meta) implements Request { // @formatter:on
// backwards compatibility constructor
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) {
this(messages, modelPreferences, systemPrompt, includeContext, temperature, maxTokens, stopSequences,
metadata, null);
@ -1859,7 +1859,7 @@ public final class McpSchema {
private Double temperature;
private int maxTokens;
private Integer maxTokens;
private List<String> stopSequences;