return MinimalUser

LuluBeatson · SamMorrowDrums · commit 37d1ed6fd8e4 · 2025-07-04T14:58:38.000+02:00
diff --git a/pkg/github/context_tools.go b/pkg/github/context_tools.go
@@ -38,10 +38,19 @@ func GetMe(getClient GetClientFn, t translations.TranslationHelperFunc) (mcp.Too
 			), nil
 		}
 
-		// Set nil to omit from output
-		user.SiteAdmin = nil
+		// Create minimal user representation instead of returning full user object
+		minimalUser := MinimalUser{
+			Login: user.GetLogin(),
+			ID:    user.GetID(),
+		}
+		if user.HTMLURL != nil {
+			minimalUser.ProfileURL = *user.HTMLURL
+		}
+		if user.AvatarURL != nil {
+			minimalUser.AvatarURL = *user.AvatarURL
+		}
 
-		return MarshalledTextResult(user), nil
+		return MarshalledTextResult(minimalUser), nil
 	})
 
 	return tool, handler
diff --git a/pkg/github/context_tools_test.go b/pkg/github/context_tools_test.go
@@ -117,17 +117,13 @@ func Test_GetMe(t *testing.T) {
 			}
 
 			// Unmarshal and verify the result
-			var returnedUser github.User
+			var returnedUser MinimalUser
 			err = json.Unmarshal([]byte(textContent.Text), &returnedUser)
 			require.NoError(t, err)
 
-			// Verify user details
-			assert.Equal(t, *tc.expectedUser.Login, *returnedUser.Login)
-			assert.Equal(t, *tc.expectedUser.Name, *returnedUser.Name)
-			assert.Equal(t, *tc.expectedUser.Email, *returnedUser.Email)
-			assert.Equal(t, *tc.expectedUser.Bio, *returnedUser.Bio)
-			assert.Equal(t, *tc.expectedUser.HTMLURL, *returnedUser.HTMLURL)
-			assert.Equal(t, *tc.expectedUser.Type, *returnedUser.Type)
+			// Verify minimal user details
+			assert.Equal(t, *tc.expectedUser.Login, returnedUser.Login)
+			assert.Equal(t, *tc.expectedUser.HTMLURL, returnedUser.ProfileURL)
 		})
 	}
 }
