preserve typed tool compatibility

Author: kerobbi

README.md

@@ -1518,7 +1518,7 @@ For negotiated supported protocol versions `2026-07-28` or newer, the consolidat
 
 - **search_repositories** - Search repositories
   - **OAuth Challenge Scopes**: `repo`
-  - `minimal_output`: Return minimal repository information (default: true). When false, returns full GitHub API repository objects. (boolean, optional)
+  - `minimal_output`: Return minimal repository information (default: true). When false, modern clients receive additional curated repository details, not the complete GitHub API object. Legacy clients retain full GitHub API repository objects. (boolean, optional)
   - `order`: Sort order (string, optional)
   - `page`: Page number for pagination (min 1) (number, optional)
   - `perPage`: Results per page for pagination (min 1, max 100) (number, optional)

docs/feature-flags.md

@@ -402,8 +402,8 @@ as output formatting) won't appear here.
   - `confidence_threshold`: Minimum similarity threshold a candidate must meet to be returned; higher values are stricter. When omitted, the API's high-precision default is used. The scale is defined by the API, so no client-side bounds are enforced. (number, optional)
   - `issue_number`: The number of the existing issue to find duplicates for (number, required)
   - `owner`: The owner of the repository (string, required)
-  - `page`: Page number for pagination (min 1) (number, optional)
-  - `perPage`: Results per page for pagination (min 1, max 100) (number, optional)
+  - `page`: Page number for pagination (min 0). Zero is forwarded for the GitHub API default. (number, optional)
+  - `perPage`: Results per page for pagination (min 0, max 100). Zero is forwarded for the GitHub API default. (number, optional)
   - `repo`: The name of the repository (string, required)
 
 ### `thread_resolution_reason`

pkg/github/__toolsnaps__/add_pull_request_review_comment_reaction.snap

@@ -63,27 +63,18 @@
   "outputSchema": {
     "additionalProperties": false,
     "properties": {
-      "content": {
-        "description": "Reaction type",
-        "enum": [
-          "+1",
-          "-1",
-          "laugh",
-          "confused",
-          "heart",
-          "hooray",
-          "rocket",
-          "eyes"
-        ],
-        "type": "string"
-      },
       "id": {
         "description": "Reaction ID",
-        "type": "integer"
+        "type": "string"
+      },
+      "url": {
+        "description": "Reaction URL",
+        "type": "string"
       }
     },
     "required": [
-      "id"
+      "id",
+      "url"
     ],
     "type": "object"
   }

pkg/github/__toolsnaps__/find_duplicate_ff_duplicate_detection.snap

@@ -20,14 +20,14 @@
         "type": "string"
       },
       "page": {
-        "description": "Page number for pagination (min 1)",
-        "minimum": 1,
+        "description": "Page number for pagination (min 0). Zero is forwarded for the GitHub API default.",
+        "minimum": 0,
         "type": "number"
       },
       "perPage": {
-        "description": "Results per page for pagination (min 1, max 100)",
+        "description": "Results per page for pagination (min 0, max 100). Zero is forwarded for the GitHub API default.",
         "maximum": 100,
-        "minimum": 1,
+        "minimum": 0,
         "type": "number"
       },
       "repo": {

pkg/github/__toolsnaps__/get_file_contents.snap

@@ -83,6 +83,12 @@
                 "string"
               ]
             },
+            "git_url": {
+              "type": [
+                "null",
+                "string"
+              ]
+            },
             "html_url": {
               "type": [
                 "null",
@@ -132,6 +138,12 @@
                 "null",
                 "string"
               ]
+            },
+            "url": {
+              "type": [
+                "null",
+                "string"
+              ]
             }
           },
           "type": [

pkg/github/__toolsnaps__/get_gist.snap

@@ -50,6 +50,9 @@
                 "language": {
                   "type": "string"
                 },
+                "raw_url": {
+                  "type": "string"
+                },
                 "size": {
                   "description": "File size in bytes.",
                   "type": "integer"
@@ -63,6 +66,9 @@
             },
             "type": "object"
           },
+          "git_pull_url": {
+            "type": "string"
+          },
           "html_url": {
             "type": "string"
           },

pkg/github/__toolsnaps__/list_gists.snap

@@ -63,6 +63,9 @@
                   "language": {
                     "type": "string"
                   },
+                  "raw_url": {
+                    "type": "string"
+                  },
                   "size": {
                     "description": "File size in bytes.",
                     "type": "integer"
@@ -76,6 +79,9 @@
               },
               "type": "object"
             },
+            "git_pull_url": {
+              "type": "string"
+            },
             "html_url": {
               "type": "string"
             },

pkg/github/__toolsnaps__/search_repositories.snap

@@ -9,7 +9,7 @@
     "properties": {
       "minimal_output": {
         "default": true,
-        "description": "Return minimal repository information (default: true). When false, returns full GitHub API repository objects.",
+        "description": "Return minimal repository information (default: true). When false, modern clients receive additional curated repository details, not the complete GitHub API object. Legacy clients retain full GitHub API repository objects.",
         "type": "boolean"
       },
       "order": {

pkg/github/consolidated_pull_request_types.go

@@ -538,6 +538,13 @@ func normalizePullRequestArguments(kind string) func(json.RawMessage) (json.RawM
 		if spec.methods != nil && !slices.Contains(spec.methods, args["method"].(string)) {
 			return nil, fmt.Errorf("unknown method: %s", args["method"])
 		}
+		if kind == "read" {
+			if args["method"] != "get_review_comments" {
+				delete(args, "after")
+			} else if _, err := OptionalParam[string](args, "after"); err != nil {
+				return nil, err
+			}
+		}
 		if kind == "reply" {
 			_, hasBody := args["body"]
 			_, hasReaction := args["reaction"]

pkg/github/find_duplicate.go

@@ -62,6 +62,10 @@ func FindDuplicate(t translations.TranslationHelperFunc) inventory.ServerTool {
 		Required: []string{"owner", "repo", "issue_number"},
 	}
 	WithPagination(schema)
+	schema.Properties["page"].Minimum = new(0.0)
+	schema.Properties["page"].Description = "Page number for pagination (min 0). Zero is forwarded for the GitHub API default."
+	schema.Properties["perPage"].Minimum = new(0.0)
+	schema.Properties["perPage"].Description = "Results per page for pagination (min 0, max 100). Zero is forwarded for the GitHub API default."
 	st := NewToolWithSchemaOptions[FindDuplicateInput, []DuplicateCandidate](
 		ToolsetMetadataIssues,
 		mcp.Tool{