add state_reason param

Author: kerobbi

README.md

@@ -597,6 +597,7 @@ The following sets of tools are available (all are on by default):
   - `owner`: Repository owner (string, required)
   - `repo`: Repository name (string, required)
   - `state`: New state (string, optional)
+  - `state_reason`: Reason for the state change, ignored unless state is changed. (string, optional)
   - `title`: New title (string, optional)
   - `type`: New issue type (string, optional)
 

pkg/github/__toolsnaps__/update_issue.snap

@@ -48,6 +48,16 @@
         ],
         "type": "string"
       },
+      "state_reason": {
+        "description": "Reason for the state change, ignored unless state is changed.",
+        "enum": [
+          "completed",
+          "not_planned",
+          "duplicate",
+          "reopened"
+        ],
+        "type": "string"
+      },
       "title": {
         "description": "New title",
         "type": "string"

pkg/github/issues.go

@@ -1128,6 +1128,10 @@ func UpdateIssue(getClient GetClientFn, t translations.TranslationHelperFunc) (t
 				mcp.Description("New state"),
 				mcp.Enum("open", "closed"),
 			),
+			mcp.WithString("state_reason",
+				mcp.Description("Reason for the state change, ignored unless state is changed."),
+				mcp.Enum("completed", "not_planned", "duplicate", "reopened"),
+			),
 			mcp.WithArray("labels",
 				mcp.Description("New labels"),
 				mcp.Items(
@@ -1193,6 +1197,14 @@ func UpdateIssue(getClient GetClientFn, t translations.TranslationHelperFunc) (t
 				issueRequest.State = github.Ptr(state)
 			}
 
+			stateReason, err := OptionalParam[string](request, "state_reason")
+			if err != nil {
+				return mcp.NewToolResultError(err.Error()), nil
+			}
+			if stateReason != "" {
+				issueRequest.StateReason = github.Ptr(stateReason)
+			}
+
 			// Get labels
 			labels, err := OptionalStringArrayParam(request, "labels")
 			if err != nil {

pkg/github/issues_test.go

@@ -1044,6 +1044,7 @@ func Test_UpdateIssue(t *testing.T) {
 	assert.Contains(t, tool.InputSchema.Properties, "title")
 	assert.Contains(t, tool.InputSchema.Properties, "body")
 	assert.Contains(t, tool.InputSchema.Properties, "state")
+	assert.Contains(t, tool.InputSchema.Properties, "state_reason")
 	assert.Contains(t, tool.InputSchema.Properties, "labels")
 	assert.Contains(t, tool.InputSchema.Properties, "assignees")
 	assert.Contains(t, tool.InputSchema.Properties, "milestone")
@@ -1174,6 +1175,41 @@ func Test_UpdateIssue(t *testing.T) {
 			expectError:    true,
 			expectedErrMsg: "failed to update issue",
 		},
+		{
+			name: "close issue as not planned",
+			mockedClient: mock.NewMockedHTTPClient(
+				mock.WithRequestMatchHandler(
+					mock.PatchReposIssuesByOwnerByRepoByIssueNumber,
+					expectRequestBody(t, map[string]any{
+						"state":        "closed",
+						"state_reason": "not_planned",
+					}).andThen(
+						mockResponse(t, http.StatusOK, &github.Issue{
+							Number:      github.Ptr(123),
+							Title:       github.Ptr("Test Issue"),
+							HTMLURL:     github.Ptr("https://github.com/owner/repo/issues/123"),
+							State:       github.Ptr("closed"),
+							StateReason: github.Ptr("not_planned"),
+						}),
+					),
+				),
+			),
+			requestArgs: map[string]interface{}{
+				"owner":        "owner",
+				"repo":         "repo",
+				"issue_number": float64(123),
+				"state":        "closed",
+				"state_reason": "not_planned",
+			},
+			expectError: false,
+			expectedIssue: &github.Issue{
+				Number:      github.Ptr(123),
+				Title:       github.Ptr("Test Issue"),
+				HTMLURL:     github.Ptr("https://github.com/owner/repo/issues/123"),
+				State:       github.Ptr("closed"),
+				StateReason: github.Ptr("not_planned"),
+			},
+		},
 	}
 
 	for _, tc := range tests {