Merge pull request #2 from yevheniidehtiar/feature/create-milestone

fix: Make state parameter required for createMilestone

Author: yevheniidehtiar

pkg/github/__toolsnaps__/create_milestone.snap

@@ -38,7 +38,8 @@
     "required": [
       "owner",
       "repo",
-      "title"
+      "title",
+      "state"
     ],
     "type": "object"
   },

pkg/github/issues.go

@@ -227,6 +227,7 @@ func CreateMilestone(getClient GetClientFn, t translations.TranslationHelperFunc
 				mcp.Description("Milestone title"),
 			),
 			mcp.WithString("state",
+				mcp.Required(),
 				mcp.Description("Milestone state"),
 				mcp.Enum("open", "closed"),
 			),

pkg/github/issues_test.go

@@ -136,7 +136,7 @@ func Test_CreateMilestone(t *testing.T) {
 	assert.Contains(t, tool.InputSchema.Properties, "state")
 	assert.Contains(t, tool.InputSchema.Properties, "description")
 	assert.Contains(t, tool.InputSchema.Properties, "due_on")
-	assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "repo", "title"})
+	assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "repo", "title", "state"})
 
 	// Setup mock milestone for success case
 	dueOn, _ := time.Parse(time.RFC3339, "2025-01-01T00:00:00Z")