Simplify tool description

Author: JoannaaKL

pkg/github/__toolsnaps__/list_repository_collaborators.snap

@@ -3,7 +3,7 @@
     "readOnlyHint": true,
     "title": "List repository collaborators"
   },
-  "description": "List collaborators of a GitHub repository. Lists users who have access to the repository. Includes their username, ID, and role name.",
+  "description": "List collaborators of a GitHub repository.",
   "inputSchema": {
     "properties": {
       "affiliation": {

pkg/github/repositories.go

@@ -2247,7 +2247,7 @@ func ListRepositoryCollaborators(t translations.TranslationHelperFunc) inventory
 		ToolsetMetadataRepos,
 		mcp.Tool{
 			Name:        "list_repository_collaborators",
-			Description: t("TOOL_LIST_REPOSITORY_COLLABORATORS_DESCRIPTION", "List collaborators of a GitHub repository. Lists users who have access to the repository. Includes their username, ID, and role name."),
+			Description: t("TOOL_LIST_REPOSITORY_COLLABORATORS_DESCRIPTION", "List collaborators of a GitHub repository."),
 			Annotations: &mcp.ToolAnnotations{
 				Title:        t("TOOL_LIST_REPOSITORY_COLLABORATORS_USER_TITLE", "List repository collaborators"),
 				ReadOnlyHint: true,
@@ -2322,7 +2322,7 @@ func ListRepositoryCollaborators(t translations.TranslationHelperFunc) inventory
 				return ghErrors.NewGitHubAPIStatusErrorResponse(ctx, "failed to list collaborators", resp, body), nil, nil
 			}
 
-			var result []MinimalCollaborator
+			result := make([]MinimalCollaborator, 0, len(collaborators))
 			for _, c := range collaborators {
 				result = append(result, MinimalCollaborator{
 					Login:    c.GetLogin(),

pkg/github/repositories_test.go

@@ -4459,6 +4459,19 @@ func Test_ListRepositoryCollaborators(t *testing.T) {
 			mockResponses: []MockBackendOption{},
 			errContains:   "missing required parameter: repo",
 		},
+		{
+			name: "empty collaborators returns empty array",
+			args: map[string]any{
+				"owner": "owner",
+				"repo":  "repo",
+			},
+			mockResponses: []MockBackendOption{
+				WithRequestMatch(
+					GetReposCollaboratorsByOwnerByRepo,
+					[]*github.User{},
+				),
+			},
+		},
 	}
 
 	for _, tt := range tests {
@@ -4486,6 +4499,13 @@ func Test_ListRepositoryCollaborators(t *testing.T) {
 			var collaborators []MinimalCollaborator
 			err = json.Unmarshal([]byte(textContent.Text), &collaborators)
 			require.NoError(t, err)
+
+			if tt.name == "empty collaborators returns empty array" {
+				assert.Empty(t, collaborators)
+				assert.Equal(t, "[]", textContent.Text)
+				return
+			}
+
 			assert.Len(t, collaborators, 2)
 			assert.Equal(t, "user1", collaborators[0].Login)
 			assert.Equal(t, int64(101), collaborators[0].ID)