mirror of
https://gitea.com/gitea/gitea-mcp.git
synced 2026-03-14 00:45:13 +00:00
c3db4fb65f
Reduce token usage by slimming tool responses. Instead of returning full Gitea SDK objects (with nested user/repo objects, avatars, permissions, etc.), each operation now has a colocated `slim.go` that extracts only the fields an LLM needs. List endpoints return even fewer fields than single-item endpoints.
Other changes:
- Add `params` helpers to DRY parameter extraction across 40+ handlers
- Remove `{"Result": ...}` wrapper for flatter responses
- Reduce default pageSize from 100 to 30
Fixes: https://gitea.com/gitea/gitea-mcp/issues/128
*Created by Claude on behalf of @silverwind*
Reviewed-on: https://gitea.com/gitea/gitea-mcp/pulls/141
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com>
Co-authored-by: silverwind <me@silverwind.io>
Co-committed-by: silverwind <me@silverwind.io>
40 lines
878 B
Go
40 lines
878 B
Go
package user
|
|
|
|
import (
|
|
"testing"
|
|
|
|
gitea_sdk "code.gitea.io/sdk/gitea"
|
|
)
|
|
|
|
func TestSlimUserDetail(t *testing.T) {
|
|
u := &gitea_sdk.User{
|
|
ID: 42,
|
|
UserName: "alice",
|
|
FullName: "Alice Smith",
|
|
Email: "alice@example.com",
|
|
AvatarURL: "https://gitea.com/avatars/42",
|
|
HTMLURL: "https://gitea.com/alice",
|
|
IsAdmin: true,
|
|
}
|
|
m := slimUserDetail(u)
|
|
|
|
if m["id"] != int64(42) {
|
|
t.Errorf("expected id 42, got %v", m["id"])
|
|
}
|
|
if m["login"] != "alice" {
|
|
t.Errorf("expected login alice, got %v", m["login"])
|
|
}
|
|
if m["full_name"] != "Alice Smith" {
|
|
t.Errorf("expected full_name Alice Smith, got %v", m["full_name"])
|
|
}
|
|
if m["is_admin"] != true {
|
|
t.Errorf("expected is_admin true, got %v", m["is_admin"])
|
|
}
|
|
}
|
|
|
|
func TestSlimUserDetail_Nil(t *testing.T) {
|
|
if m := slimUserDetail(nil); m != nil {
|
|
t.Errorf("expected nil for nil user, got %v", m)
|
|
}
|
|
}
|