GetTeams - rename "login" field to "org"

Author: LuluBeatson

pkg/github/context_tools.go

@@ -92,18 +92,18 @@ func GetMe(getClient GetClientFn, t translations.TranslationHelperFunc) (mcp.Too
 	return tool, handler
 }
 
-func GetTeams(getClient GetClientFn, getGQLClient GetGQLClientFn, t translations.TranslationHelperFunc) (mcp.Tool, server.ToolHandlerFunc) {
-	type TeamInfo struct {
-		Name        string `json:"name"`
-		Slug        string `json:"slug"`
-		Description string `json:"description"`
-	}
+type TeamInfo struct {
+	Name        string `json:"name"`
+	Slug        string `json:"slug"`
+	Description string `json:"description"`
+}
 
-	type OrganizationTeams struct {
-		Login string     `json:"login"`
-		Teams []TeamInfo `json:"teams"`
-	}
+type OrganizationTeams struct {
+	Org   string     `json:"org"`
+	Teams []TeamInfo `json:"teams"`
+}
 
+func GetTeams(getClient GetClientFn, getGQLClient GetGQLClientFn, t translations.TranslationHelperFunc) (mcp.Tool, server.ToolHandlerFunc) {
 	return mcp.NewTool("get_teams",
 			mcp.WithDescription(t("TOOL_GET_TEAMS_DESCRIPTION", "Get details of the teams the user is a member of. Limited to organizations accessible with current credentials")),
 			mcp.WithString("user",
@@ -171,7 +171,7 @@ func GetTeams(getClient GetClientFn, getGQLClient GetGQLClientFn, t translations
 			var organizations []OrganizationTeams
 			for _, org := range q.User.Organizations.Nodes {
 				orgTeams := OrganizationTeams{
-					Login: string(org.Login),
+					Org:   string(org.Login),
 					Teams: make([]TeamInfo, 0, len(org.Teams.Nodes)),
 				}
 

pkg/github/context_tools_test.go

@@ -345,28 +345,21 @@ func Test_GetTeams(t *testing.T) {
 				return
 			}
 
-			var organizations []struct {
-				Login string `json:"login"`
-				Teams []struct {
-					Name        string `json:"name"`
-					Slug        string `json:"slug"`
-					Description string `json:"description"`
-				} `json:"teams"`
-			}
+			var organizations []OrganizationTeams
 			err = json.Unmarshal([]byte(textContent.Text), &organizations)
 			require.NoError(t, err)
 
 			assert.Len(t, organizations, tc.expectedTeamsCount)
 
 			if tc.expectedTeamsCount > 0 {
-				assert.Equal(t, "testorg1", organizations[0].Login)
+				assert.Equal(t, "testorg1", organizations[0].Org)
 				assert.Len(t, organizations[0].Teams, 2)
 				assert.Equal(t, "team1", organizations[0].Teams[0].Name)
 				assert.Equal(t, "team1", organizations[0].Teams[0].Slug)
 				assert.Equal(t, "Team 1", organizations[0].Teams[0].Description)
 
 				if tc.expectedTeamsCount > 1 {
-					assert.Equal(t, "testorg2", organizations[1].Login)
+					assert.Equal(t, "testorg2", organizations[1].Org)
 					assert.Len(t, organizations[1].Teams, 1)
 					assert.Equal(t, "team3", organizations[1].Teams[0].Name)
 					assert.Equal(t, "team3", organizations[1].Teams[0].Slug)