fix: add @JsonIgnore to InputOptions, rollback McpServerConfig/ModelLimits changes

Co-authored-by: edburns <75821+edburns@users.noreply.github.com>

Author: Copilot

src/main/java/com/github/copilot/sdk/json/InputOptions.java

@@ -4,6 +4,7 @@
 
 package com.github.copilot.sdk.json;
 
+import com.fasterxml.jackson.annotation.JsonIgnore;
 import java.util.OptionalInt;
 
 /**
@@ -49,6 +50,7 @@ public InputOptions setDescription(String description) {
     }
 
     /** Gets the minimum character length. @return the min length */
+    @JsonIgnore
     public OptionalInt getMinLength() {
         return minLength == null ? OptionalInt.empty() : OptionalInt.of(minLength);
     }
@@ -73,6 +75,7 @@ public InputOptions clearMinLength() {
     }
 
     /** Gets the maximum character length. @return the max length */
+    @JsonIgnore
     public OptionalInt getMaxLength() {
         return maxLength == null ? OptionalInt.empty() : OptionalInt.of(maxLength);
     }

src/main/java/com/github/copilot/sdk/json/McpHttpServerConfig.java

@@ -99,7 +99,7 @@ public McpHttpServerConfig setTools(List<String> tools) {
     }
 
     @Override
-    public McpHttpServerConfig setTimeout(int timeout) {
+    public McpHttpServerConfig setTimeout(Integer timeout) {
         super.setTimeout(timeout);
         return this;
     }

src/main/java/com/github/copilot/sdk/json/McpServerConfig.java

@@ -11,8 +11,6 @@
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.annotation.JsonSubTypes;
 import com.fasterxml.jackson.annotation.JsonTypeInfo;
-import com.fasterxml.jackson.annotation.JsonIgnore;
-import java.util.OptionalInt;
 
 /**
  * Abstract base class for MCP (Model Context Protocol) server configurations.
@@ -70,34 +68,21 @@ public McpServerConfig setTools(List<String> tools) {
     /**
      * Gets the optional timeout in milliseconds for tool calls to this server.
      *
-     * @return an {@link OptionalInt} containing the timeout in milliseconds, or
-     *         empty for the default
+     * @return the timeout in milliseconds, or {@code null} for the default
      */
-    @JsonIgnore
-    public OptionalInt getTimeout() {
-        return timeout == null ? OptionalInt.empty() : OptionalInt.of(timeout);
+    public Integer getTimeout() {
+        return timeout;
     }
 
     /**
      * Sets an optional timeout in milliseconds for tool calls to this server.
      *
      * @param timeout
-     *            the timeout in milliseconds
+     *            the timeout in milliseconds, or {@code null} for the default
      * @return this config for method chaining
      */
-    public McpServerConfig setTimeout(int timeout) {
+    public McpServerConfig setTimeout(Integer timeout) {
         this.timeout = timeout;
         return this;
     }
-
-    /**
-     * Clears the timeout setting, reverting to the default behavior.
-     *
-     * @return this instance for method chaining
-     */
-    public McpServerConfig clearTimeout() {
-        this.timeout = null;
-        return this;
-    }
-
 }

src/main/java/com/github/copilot/sdk/json/McpStdioServerConfig.java

@@ -148,7 +148,7 @@ public McpStdioServerConfig setTools(List<String> tools) {
     }
 
     @Override
-    public McpStdioServerConfig setTimeout(int timeout) {
+    public McpStdioServerConfig setTimeout(Integer timeout) {
         super.setTimeout(timeout);
         return this;
     }

src/main/java/com/github/copilot/sdk/json/ModelLimits.java

@@ -6,8 +6,6 @@
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
-import com.fasterxml.jackson.annotation.JsonIgnore;
-import java.util.OptionalInt;
 
 /**
  * Model limits.
@@ -26,26 +24,15 @@ public class ModelLimits {
     @JsonProperty("vision")
     private ModelVisionLimits vision;
 
-    @JsonIgnore
-    public OptionalInt getMaxPromptTokens() {
-        return maxPromptTokens == null ? OptionalInt.empty() : OptionalInt.of(maxPromptTokens);
+    public Integer getMaxPromptTokens() {
+        return maxPromptTokens;
     }
 
-    public ModelLimits setMaxPromptTokens(int maxPromptTokens) {
+    public ModelLimits setMaxPromptTokens(Integer maxPromptTokens) {
         this.maxPromptTokens = maxPromptTokens;
         return this;
     }
 
-    /**
-     * Clears the maxPromptTokens setting, reverting to the default behavior.
-     *
-     * @return this instance for method chaining
-     */
-    public ModelLimits clearMaxPromptTokens() {
-        this.maxPromptTokens = null;
-        return this;
-    }
-
     public int getMaxContextWindowTokens() {
         return maxContextWindowTokens;
     }

src/test/java/com/github/copilot/sdk/DataObjectCoverageTest.java

@@ -194,7 +194,7 @@ void mcpHttpServerConfigCoversGettersAndFluentSetters() {
         assertEquals("https://mcp.example.com/sse", cfg.getUrl());
         assertEquals("Bearer token", cfg.getHeaders().get("Authorization"));
         assertEquals(tools, cfg.getTools());
-        assertEquals(45, cfg.getTimeout().getAsInt());
+        assertEquals(45, cfg.getTimeout());
     }
 
     @Test
@@ -212,7 +212,7 @@ void mcpStdioServerConfigCoversGettersAndFluentSetters() {
         assertEquals("1", cfg.getEnv().get("DEBUG"));
         assertEquals("/tmp", cfg.getWorkingDirectory());
         assertEquals(tools, cfg.getTools());
-        assertEquals(30, cfg.getTimeout().getAsInt());
+        assertEquals(30, cfg.getTimeout());
     }
 
     @Test