From 073d88b29a0161a33550a0dbb28990c908a565cc Mon Sep 17 00:00:00 2001 From: Mathias Bergqvist Date: Wed, 6 May 2026 22:38:15 +0200 Subject: [PATCH] feat(tools): branch_list --- internal/gitea/files.go | 22 ++++++++++ internal/gitea/files_test.go | 22 ++++++++++ internal/tools/branch_list.go | 67 ++++++++++++++++++++++++++++++ internal/tools/branch_list_test.go | 43 +++++++++++++++++++ 4 files changed, 154 insertions(+) create mode 100644 internal/tools/branch_list.go create mode 100644 internal/tools/branch_list_test.go diff --git a/internal/gitea/files.go b/internal/gitea/files.go index 3fcd229..bcc0b37 100644 --- a/internal/gitea/files.go +++ b/internal/gitea/files.go @@ -92,6 +92,28 @@ type FileWriteResult struct { } `json:"commit"` } +func (c *Client) ListBranches(ctx context.Context, owner, repo string, page, limit int) ([]Branch, error) { + if page < 1 { + page = 1 + } + if limit < 1 { + limit = 30 + } + p := fmt.Sprintf("/api/v1/repos/%s/%s/branches?page=%d&limit=%d", owner, repo, page, limit) + body, status, err := c.GetJSON(ctx, p) + if err != nil { + return nil, err + } + if err := MapStatus(status, body); err != nil { + return nil, err + } + var branches []Branch + if err := json.Unmarshal(body, &branches); err != nil { + return nil, err + } + return branches, nil +} + // UpsertFile creates a file when args.Sha is empty (POST) or updates an existing // file when args.Sha is set (PUT). Gitea routes both operations by HTTP method on // the same /contents/{path} URL, and rejects PUT without a sha. diff --git a/internal/gitea/files_test.go b/internal/gitea/files_test.go index 6f36d34..873eec7 100644 --- a/internal/gitea/files_test.go +++ b/internal/gitea/files_test.go @@ -82,6 +82,28 @@ func TestCreateBranchSendsPayload(t *testing.T) { assert.Equal(t, "main", payload["old_branch_name"]) } +func TestListBranches(t *testing.T) { + srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + assert.Equal(t, "/api/v1/repos/o/r/branches", r.URL.Path) + assert.Equal(t, "1", r.URL.Query().Get("page")) + assert.Equal(t, "30", r.URL.Query().Get("limit")) + w.Header().Set("Content-Type", "application/json") + _, _ = w.Write([]byte(`[ + {"name":"main","commit":{"id":"abc","url":"http://example.com"}}, + {"name":"feat/x","commit":{"id":"def","url":"http://example.com"}} + ]`)) + })) + defer srv.Close() + + c := gitea.NewClient(srv.URL, "tok") + branches, err := c.ListBranches(context.Background(), "o", "r", 0, 0) + require.NoError(t, err) + require.Len(t, branches, 2) + assert.Equal(t, "main", branches[0].Name) + assert.Equal(t, "abc", branches[0].Commit.ID) + assert.Equal(t, "feat/x", branches[1].Name) +} + func TestUpsertFileSendsPayloadAndDecodesResult(t *testing.T) { var captured []byte srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { diff --git a/internal/tools/branch_list.go b/internal/tools/branch_list.go new file mode 100644 index 0000000..89adb94 --- /dev/null +++ b/internal/tools/branch_list.go @@ -0,0 +1,67 @@ +package tools + +import ( + "context" + "encoding/json" + + "gitea.d-ma.be/mathias/gitea-mcp/internal/allowlist" + "gitea.d-ma.be/mathias/gitea-mcp/internal/gitea" + "gitea.d-ma.be/mathias/gitea-mcp/internal/registry" +) + +type BranchList struct { + c *gitea.Client + a *allowlist.Allowlist +} + +func NewBranchList(c *gitea.Client, a *allowlist.Allowlist) *BranchList { + return &BranchList{c: c, a: a} +} + +func (t *BranchList) Descriptor() registry.ToolDescriptor { + return registry.ToolDescriptor{ + Name: "branch_list", + Description: "List branches in a repository.", + InputSchema: json.RawMessage(`{ + "type":"object", + "properties":{ + "owner":{"type":"string"}, + "name":{"type":"string"}, + "page":{"type":"integer"}, + "limit":{"type":"integer"} + }, + "required":["owner","name"] + }`), + } +} + +type branchListArgs struct { + Owner string `json:"owner"` + Name string `json:"name"` + Page int `json:"page"` + Limit int `json:"limit"` +} + +func (t *BranchList) Call(ctx context.Context, raw json.RawMessage) (json.RawMessage, error) { + var args branchListArgs + if err := parseArgs(raw, &args); err != nil { + return nil, err + } + if err := t.a.Check(args.Owner); err != nil { + return nil, err + } + + branches, err := t.c.ListBranches(ctx, args.Owner, args.Name, args.Page, capLimit(args.Limit, 30)) + if err != nil { + return nil, err + } + + result := make([]map[string]any, len(branches)) + for i, b := range branches { + result[i] = map[string]any{ + "name": b.Name, + "sha": b.Commit.ID, + } + } + return textOK(result) +} diff --git a/internal/tools/branch_list_test.go b/internal/tools/branch_list_test.go new file mode 100644 index 0000000..d486360 --- /dev/null +++ b/internal/tools/branch_list_test.go @@ -0,0 +1,43 @@ +package tools_test + +import ( + "context" + "encoding/json" + "net/http" + "net/http/httptest" + "testing" + + "gitea.d-ma.be/mathias/gitea-mcp/internal/allowlist" + "gitea.d-ma.be/mathias/gitea-mcp/internal/gitea" + "gitea.d-ma.be/mathias/gitea-mcp/internal/tools" + "github.com/stretchr/testify/assert" + "github.com/stretchr/testify/require" +) + +func TestBranchListReturnsNames(t *testing.T) { + srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + w.Header().Set("Content-Type", "application/json") + _, _ = w.Write([]byte(`[ + {"name":"main","commit":{"id":"abc","url":""}}, + {"name":"feat/x","commit":{"id":"def","url":""}} + ]`)) + })) + defer srv.Close() + + tool := tools.NewBranchList(gitea.NewClient(srv.URL, "tok"), allowlist.New([]string{"owner"})) + out, err := tool.Call(context.Background(), json.RawMessage(`{"owner":"owner","name":"repo"}`)) + require.NoError(t, err) + + var result []map[string]any + require.NoError(t, json.Unmarshal(out, &result)) + require.Len(t, result, 2) + assert.Equal(t, "main", result[0]["name"]) + assert.Equal(t, "abc", result[0]["sha"]) + assert.Equal(t, "feat/x", result[1]["name"]) +} + +func TestBranchListAllowlistRejects(t *testing.T) { + tool := tools.NewBranchList(gitea.NewClient("http://unused", ""), allowlist.New([]string{"allowed"})) + _, err := tool.Call(context.Background(), json.RawMessage(`{"owner":"evil","name":"repo"}`)) + require.Error(t, err) +}