Wording

Author: JoannaaKL

pkg/github/__toolsnaps__/get_project.snap

@@ -3,7 +3,7 @@
     "title": "Get project",
     "readOnlyHint": true
   },
-  "description": "Get Project for an user or org",
+  "description": "Get Project for a user or org",
   "inputSchema": {
     "properties": {
       "owner": {

pkg/github/__toolsnaps__/list_project_fields.snap

@@ -3,7 +3,7 @@
     "title": "List project fields",
     "readOnlyHint": true
   },
-  "description": "List Project fields for an user or org",
+  "description": "List Project fields for a user or org",
   "inputSchema": {
     "properties": {
       "owner": {

pkg/github/__toolsnaps__/list_projects.snap

@@ -3,7 +3,7 @@
     "title": "List projects",
     "readOnlyHint": true
   },
-  "description": "List Projects for an user or org",
+  "description": "List Projects for a user or org",
   "inputSchema": {
     "properties": {
       "owner": {

pkg/github/projects.go

@@ -19,7 +19,7 @@ import (
 
 func ListProjects(getClient GetClientFn, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("list_projects",
-			mcp.WithDescription(t("TOOL_LIST_PROJECTS_DESCRIPTION", "List Projects for an user or org")),
+			mcp.WithDescription(t("TOOL_LIST_PROJECTS_DESCRIPTION", "List Projects for a user or org")),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{Title: t("TOOL_LIST_PROJECTS_USER_TITLE", "List projects"), ReadOnlyHint: ToBoolPtr(true)}),
 			mcp.WithString("owner_type", mcp.Required(), mcp.Description("Owner type"), mcp.Enum("user", "org")),
 			mcp.WithString("owner", mcp.Required(), mcp.Description("If owner_type == user it is the handle for the GitHub user account. If owner_type == org it is the name of the organization. The name is not case sensitive.")),
@@ -101,7 +101,7 @@ func ListProjects(getClient GetClientFn, t translations.TranslationHelperFunc) (
 
 func GetProject(getClient GetClientFn, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("get_project",
-			mcp.WithDescription(t("TOOL_GET_PROJECT_DESCRIPTION", "Get Project for an user or org")),
+			mcp.WithDescription(t("TOOL_GET_PROJECT_DESCRIPTION", "Get Project for a user or org")),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{Title: t("TOOL_GET_PROJECT_USER_TITLE", "Get project"), ReadOnlyHint: ToBoolPtr(true)}),
 			mcp.WithNumber("project_number", mcp.Required(), mcp.Description("The project's number")),
 			mcp.WithString("owner_type", mcp.Required(), mcp.Description("Owner type"), mcp.Enum("user", "org")),
@@ -170,7 +170,7 @@ func GetProject(getClient GetClientFn, t translations.TranslationHelperFunc) (to
 
 func ListProjectFields(getClient GetClientFn, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("list_project_fields",
-			mcp.WithDescription(t("TOOL_LIST_PROJECT_FIELDS_DESCRIPTION", "List Project fields for an user or org")),
+			mcp.WithDescription(t("TOOL_LIST_PROJECT_FIELDS_DESCRIPTION", "List Project fields for a user or org")),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{Title: t("TOOL_LIST_PROJECT_FIELDS_USER_TITLE", "List project fields"), ReadOnlyHint: ToBoolPtr(true)}),
 			mcp.WithString("owner_type", mcp.Required(), mcp.Description("Owner type"), mcp.Enum("user", "org")),
 			mcp.WithString("owner", mcp.Required(), mcp.Description("If owner_type == user it is the handle for the GitHub user account. If owner_type == org it is the name of the organization. The name is not case sensitive.")),

pkg/github/projects_test.go

@@ -75,7 +75,6 @@ func Test_ListProjects(t *testing.T) {
 					mock.EndpointPattern{Pattern: "/orgs/{org}/projectsV2", Method: http.MethodGet},
 					http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 						q := r.URL.Query()
-						// Assert query params present
 						if q.Get("per_page") == "50" && q.Get("q") == "roadmap" {
 							w.WriteHeader(http.StatusOK)
 							_, _ = w.Write(mock.MustMarshal(orgProjects))
@@ -142,7 +141,6 @@ func Test_ListProjects(t *testing.T) {
 				if tc.expectedErrMsg != "" {
 					assert.Contains(t, text, tc.expectedErrMsg)
 				}
-				// Parameter missing cases
 				if tc.name == "missing owner" {
 					assert.Contains(t, text, "missing required parameter: owner")
 				}
@@ -227,7 +225,7 @@ func Test_GetProject(t *testing.T) {
 				"owner_type":     "org",
 			},
 			expectError:    true,
-			expectedErrMsg: "failed to get project", // updated to match implementation
+			expectedErrMsg: "failed to get project",
 		},
 		{
 			name:         "missing project_number",
@@ -294,7 +292,6 @@ func Test_GetProject(t *testing.T) {
 }
 
 func Test_ListProjectFields(t *testing.T) {
-	// Verify tool definition & schema snapshot
 	mockClient := gh.NewClient(nil)
 	tool, _ := ListProjectFields(stubGetClientFn(mockClient), translations.NullTranslationHelper)
 	require.NoError(t, toolsnaps.Test(tool.Name, tool))
@@ -307,7 +304,6 @@ func Test_ListProjectFields(t *testing.T) {
 	assert.Contains(t, tool.InputSchema.Properties, "per_page")
 	assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner_type", "owner", "projectNumber"})
 
-	// Minimal field objects
 	orgFields := []map[string]any{
 		{"id": 101, "name": "Status", "dataType": "single_select"},
 	}