Validate positive comment IDs

Reject non-positive issue and pull request comment IDs in handlers and add the missing schema minimum for pull request review comment IDs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: SamMorrowDrums

pkg/github/__toolsnaps__/add_reply_to_pull_request_comment.snap

@@ -11,6 +11,7 @@
       },
       "commentId": {
         "description": "The numeric ID of the pull request review comment to reply or react to. Use the number from a #discussion_r... anchor, not the GraphQL thread node ID (PRRT_...).",
+        "minimum": 1,
         "type": "number"
       },
       "owner": {

pkg/github/issues.go

@@ -1166,6 +1166,9 @@ func AddIssueComment(t translations.TranslationHelperFunc) inventory.ServerTool
 				if err != nil {
 					return utils.NewToolResultError(err.Error()), nil, nil
 				}
+				if commentID < 1 {
+					return utils.NewToolResultError("comment_id must be greater than 0"), nil, nil
+				}
 				hasCommentID = true
 			}
 			body, hasBody, err := OptionalParamOK[string](args, "body")

pkg/github/issues_test.go

@@ -4353,6 +4353,18 @@ func TestAddIssueComment(t *testing.T) {
 			expectToolError:    true,
 			expectedToolErrMsg: "comment_id can only be provided when reaction is provided",
 		},
+		{
+			name: "negative comment_id",
+			requestArgs: map[string]any{
+				"owner":        "owner",
+				"repo":         "repo",
+				"issue_number": float64(42),
+				"comment_id":   float64(-1),
+				"reaction":     "heart",
+			},
+			expectToolError:    true,
+			expectedToolErrMsg: "comment_id must be greater than 0",
+		},
 		{
 			name: "comment_id with body",
 			requestArgs: map[string]any{

pkg/github/pullrequests.go

@@ -1216,6 +1216,7 @@ func AddReplyToPullRequestComment(t translations.TranslationHelperFunc) inventor
 			"commentId": {
 				Type:        "number",
 				Description: "The numeric ID of the pull request review comment to reply or react to. Use the number from a #discussion_r... anchor, not the GraphQL thread node ID (PRRT_...).",
+				Minimum:     jsonschema.Ptr(1.0),
 			},
 			"body": {
 				Type:        "string",
@@ -1255,6 +1256,9 @@ func AddReplyToPullRequestComment(t translations.TranslationHelperFunc) inventor
 			if err != nil {
 				return utils.NewToolResultError(err.Error()), nil, nil
 			}
+			if commentID < 1 {
+				return utils.NewToolResultError("commentId must be greater than 0"), nil, nil
+			}
 			body, hasBody, err := OptionalParamOK[string](args, "body")
 			if err != nil {
 				return utils.NewToolResultError(err.Error()), nil, nil

pkg/github/pullrequests_test.go

@@ -4094,6 +4094,17 @@ func TestAddReplyToPullRequestComment(t *testing.T) {
 			expectToolError:    true,
 			expectedToolErrMsg: "missing required parameter: commentId",
 		},
+		{
+			name: "negative commentId",
+			requestArgs: map[string]any{
+				"owner":     "owner",
+				"repo":      "repo",
+				"commentId": float64(-123),
+				"reaction":  "rocket",
+			},
+			expectToolError:    true,
+			expectedToolErrMsg: "commentId must be greater than 0",
+		},
 		{
 			name: "missing body and reaction",
 			requestArgs: map[string]any{