mirror of
https://gitea.com/gitea/gitea-mcp.git
synced 2026-08-27 18:47:44 +00:00
Compare commits
1 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| bfc1ff17da |
@@ -49,6 +49,7 @@ var (
|
|||||||
tool.Array("milestones", tool.Description("milestone name or ID filter"), tool.Items(map[string]any{"type": "string"})),
|
tool.Array("milestones", tool.Description("milestone name or ID filter"), tool.Items(map[string]any{"type": "string"})),
|
||||||
tool.String("since", tool.Description("updated after ISO 8601")),
|
tool.String("since", tool.Description("updated after ISO 8601")),
|
||||||
tool.String("before", tool.Description("updated before ISO 8601")),
|
tool.String("before", tool.Description("updated before ISO 8601")),
|
||||||
|
tool.String("assigned_by", tool.Description("filter by the user who assigned the issue")),
|
||||||
tool.Number("page", tool.Description(params.PageDesc), tool.Default(1)),
|
tool.Number("page", tool.Description(params.PageDesc), tool.Default(1)),
|
||||||
tool.Number("per_page", tool.Description(params.PaginationDesc), tool.Default(30)),
|
tool.Number("per_page", tool.Description(params.PaginationDesc), tool.Default(30)),
|
||||||
)
|
)
|
||||||
@@ -204,6 +205,9 @@ func listRepoIssuesFn(ctx context.Context, args map[string]any) (*mcp.CallToolRe
|
|||||||
if t := params.GetOptionalTime(args, "before"); t != nil {
|
if t := params.GetOptionalTime(args, "before"); t != nil {
|
||||||
opt.Before = *t
|
opt.Before = *t
|
||||||
}
|
}
|
||||||
|
if assignedBy, ok := args["assigned_by"].(string); ok {
|
||||||
|
opt.AssignedBy = assignedBy
|
||||||
|
}
|
||||||
client, err := gitea.ClientFromContext(ctx)
|
client, err := gitea.ClientFromContext(ctx)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
||||||
|
|||||||
@@ -61,12 +61,13 @@ func Test_listRepoIssuesFn_filters(t *testing.T) {
|
|||||||
}()
|
}()
|
||||||
|
|
||||||
args := map[string]any{
|
args := map[string]any{
|
||||||
"owner": owner,
|
"owner": owner,
|
||||||
"repo": repo,
|
"repo": repo,
|
||||||
"type": "issues",
|
"type": "issues",
|
||||||
"labels": []any{"bug", "enhancement"},
|
"labels": []any{"bug", "enhancement"},
|
||||||
"milestones": []any{"v1.0", "2"},
|
"milestones": []any{"v1.0", "2"},
|
||||||
"since": "2026-01-01T00:00:00Z",
|
"since": "2026-01-01T00:00:00Z",
|
||||||
|
"assigned_by": "octocat",
|
||||||
}
|
}
|
||||||
|
|
||||||
_, err := listRepoIssuesFn(context.Background(), args)
|
_, err := listRepoIssuesFn(context.Background(), args)
|
||||||
@@ -89,6 +90,9 @@ func Test_listRepoIssuesFn_filters(t *testing.T) {
|
|||||||
if !strings.Contains(gotQuery, "type=issues") {
|
if !strings.Contains(gotQuery, "type=issues") {
|
||||||
t.Fatalf("expected type query param, got %s", gotQuery)
|
t.Fatalf("expected type query param, got %s", gotQuery)
|
||||||
}
|
}
|
||||||
|
if !strings.Contains(gotQuery, "assigned_by=octocat") {
|
||||||
|
t.Fatalf("expected assigned_by query param, got %s", gotQuery)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func Test_listRepoIssuesFn_includesMilestone(t *testing.T) {
|
func Test_listRepoIssuesFn_includesMilestone(t *testing.T) {
|
||||||
@@ -138,6 +142,52 @@ func Test_listRepoIssuesFn_includesMilestone(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func Test_listRepoIssuesFn_includesAssignees(t *testing.T) {
|
||||||
|
const (
|
||||||
|
owner = "octo"
|
||||||
|
repo = "demo"
|
||||||
|
)
|
||||||
|
|
||||||
|
handler := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
||||||
|
switch r.URL.Path {
|
||||||
|
case "/api/v1/version":
|
||||||
|
w.Header().Set("Content-Type", "application/json")
|
||||||
|
_, _ = w.Write([]byte(`{"version":"1.12.0"}`))
|
||||||
|
case fmt.Sprintf("/api/v1/repos/%s/%s", owner, repo):
|
||||||
|
w.Header().Set("Content-Type", "application/json")
|
||||||
|
_, _ = w.Write([]byte(`{"private":false}`))
|
||||||
|
case fmt.Sprintf("/api/v1/repos/%s/%s/issues", owner, repo):
|
||||||
|
w.Header().Set("Content-Type", "application/json")
|
||||||
|
_, _ = w.Write([]byte(`[
|
||||||
|
{"number": 1, "title": "with assignees", "state": "open", "assignees": [{"login": "octocat"}]}
|
||||||
|
]`))
|
||||||
|
default:
|
||||||
|
http.NotFound(w, r)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
server := httptest.NewServer(handler)
|
||||||
|
defer server.Close()
|
||||||
|
|
||||||
|
origHost, origToken, origVersion := flag.Host, flag.Token, flag.Version
|
||||||
|
flag.Host, flag.Token, flag.Version = server.URL, "", "test"
|
||||||
|
defer func() { flag.Host, flag.Token, flag.Version = origHost, origToken, origVersion }()
|
||||||
|
|
||||||
|
args := map[string]any{
|
||||||
|
"owner": owner, "repo": repo,
|
||||||
|
}
|
||||||
|
res, err := listRepoIssuesFn(context.Background(), args)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("listRepoIssuesFn() error = %v", err)
|
||||||
|
}
|
||||||
|
if res.IsError {
|
||||||
|
t.Fatalf("unexpected error result: %v", res.Content)
|
||||||
|
}
|
||||||
|
body := res.Content[0].(*mcp.TextContent).Text
|
||||||
|
if !strings.Contains(body, `"assignees"`) || !strings.Contains(body, `"octocat"`) {
|
||||||
|
t.Fatalf("expected assignees in list output, got: %s", body)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func Test_createIssueFn_labels(t *testing.T) {
|
func Test_createIssueFn_labels(t *testing.T) {
|
||||||
const (
|
const (
|
||||||
owner = "octo"
|
owner = "octo"
|
||||||
|
|||||||
@@ -63,6 +63,9 @@ func slimIssues(issues []*gitea_sdk.Issue) []map[string]any {
|
|||||||
if len(i.Labels) > 0 {
|
if len(i.Labels) > 0 {
|
||||||
m["labels"] = slim.LabelNames(i.Labels)
|
m["labels"] = slim.LabelNames(i.Labels)
|
||||||
}
|
}
|
||||||
|
if len(i.Assignees) > 0 {
|
||||||
|
m["assignees"] = slim.UserLogins(i.Assignees)
|
||||||
|
}
|
||||||
if i.Milestone != nil {
|
if i.Milestone != nil {
|
||||||
m["milestone"] = map[string]any{
|
m["milestone"] = map[string]any{
|
||||||
"id": i.Milestone.ID,
|
"id": i.Milestone.ID,
|
||||||
|
|||||||
+32
-29
@@ -69,26 +69,28 @@ func init() {
|
|||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
type createBranchArgs struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
Repo string `json:"repo" required:"true"`
|
|
||||||
Branch string `json:"branch" required:"true"`
|
|
||||||
OldBranch string `json:"old_branch"`
|
|
||||||
}
|
|
||||||
|
|
||||||
func CreateBranchFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
func CreateBranchFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
||||||
var in createBranchArgs
|
owner, err := params.GetString(args, "owner")
|
||||||
if err := params.Bind(args, &in); err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(err)
|
return to.ErrorResult(err)
|
||||||
}
|
}
|
||||||
|
repo, err := params.GetString(args, "repo")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
branch, err := params.GetString(args, "branch")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
oldBranch, _ := args["old_branch"].(string)
|
||||||
|
|
||||||
client, err := gitea.ClientFromContext(ctx)
|
client, err := gitea.ClientFromContext(ctx)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
||||||
}
|
}
|
||||||
_, _, err = client.Repositories.CreateBranch(ctx, in.Owner, in.Repo, gitea_sdk.CreateBranchOption{
|
_, _, err = client.Repositories.CreateBranch(ctx, owner, repo, gitea_sdk.CreateBranchOption{
|
||||||
BranchName: in.Branch,
|
BranchName: branch,
|
||||||
OldBranchName: in.OldBranch,
|
OldBranchName: oldBranch,
|
||||||
})
|
})
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("create branch error: %v", err))
|
return to.ErrorResult(fmt.Errorf("create branch error: %v", err))
|
||||||
@@ -97,22 +99,24 @@ func CreateBranchFn(ctx context.Context, args map[string]any) (*mcp.CallToolResu
|
|||||||
return to.TextResult("Branch Created")
|
return to.TextResult("Branch Created")
|
||||||
}
|
}
|
||||||
|
|
||||||
type deleteBranchArgs struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
Repo string `json:"repo" required:"true"`
|
|
||||||
Branch string `json:"branch" required:"true"`
|
|
||||||
}
|
|
||||||
|
|
||||||
func DeleteBranchFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
func DeleteBranchFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
||||||
var in deleteBranchArgs
|
owner, err := params.GetString(args, "owner")
|
||||||
if err := params.Bind(args, &in); err != nil {
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
repo, err := params.GetString(args, "repo")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
branch, err := params.GetString(args, "branch")
|
||||||
|
if err != nil {
|
||||||
return to.ErrorResult(err)
|
return to.ErrorResult(err)
|
||||||
}
|
}
|
||||||
client, err := gitea.ClientFromContext(ctx)
|
client, err := gitea.ClientFromContext(ctx)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
||||||
}
|
}
|
||||||
_, _, err = client.Repositories.DeleteRepoBranch(ctx, in.Owner, in.Repo, in.Branch)
|
_, _, err = client.Repositories.DeleteRepoBranch(ctx, owner, repo, branch)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("delete branch error: %v", err))
|
return to.ErrorResult(fmt.Errorf("delete branch error: %v", err))
|
||||||
}
|
}
|
||||||
@@ -120,14 +124,13 @@ func DeleteBranchFn(ctx context.Context, args map[string]any) (*mcp.CallToolResu
|
|||||||
return to.TextResult("Branch Deleted")
|
return to.TextResult("Branch Deleted")
|
||||||
}
|
}
|
||||||
|
|
||||||
type listBranchesArgs struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
Repo string `json:"repo" required:"true"`
|
|
||||||
}
|
|
||||||
|
|
||||||
func ListBranchesFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
func ListBranchesFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
||||||
var in listBranchesArgs
|
owner, err := params.GetString(args, "owner")
|
||||||
if err := params.Bind(args, &in); err != nil {
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
repo, err := params.GetString(args, "repo")
|
||||||
|
if err != nil {
|
||||||
return to.ErrorResult(err)
|
return to.ErrorResult(err)
|
||||||
}
|
}
|
||||||
page, pageSize := params.GetPagination(args, 30)
|
page, pageSize := params.GetPagination(args, 30)
|
||||||
@@ -141,7 +144,7 @@ func ListBranchesFn(ctx context.Context, args map[string]any) (*mcp.CallToolResu
|
|||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
||||||
}
|
}
|
||||||
branches, _, err := client.Repositories.ListRepoBranches(ctx, in.Owner, in.Repo, opt)
|
branches, _, err := client.Repositories.ListRepoBranches(ctx, owner, repo, opt)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("list branches error: %v", err))
|
return to.ErrorResult(fmt.Errorf("list branches error: %v", err))
|
||||||
}
|
}
|
||||||
|
|||||||
+71
-60
@@ -102,28 +102,30 @@ type ContentLine struct {
|
|||||||
Content string `json:"content"`
|
Content string `json:"content"`
|
||||||
}
|
}
|
||||||
|
|
||||||
type getFileContentArgs struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
Repo string `json:"repo" required:"true"`
|
|
||||||
Ref string `json:"ref"`
|
|
||||||
Path string `json:"path" required:"true"`
|
|
||||||
WithLines bool `json:"withLines"`
|
|
||||||
}
|
|
||||||
|
|
||||||
func GetFileContentFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
func GetFileContentFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
||||||
var in getFileContentArgs
|
owner, err := params.GetString(args, "owner")
|
||||||
if err := params.Bind(args, &in); err != nil {
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
repo, err := params.GetString(args, "repo")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
ref, _ := args["ref"].(string)
|
||||||
|
filePath, err := params.GetString(args, "path")
|
||||||
|
if err != nil {
|
||||||
return to.ErrorResult(err)
|
return to.ErrorResult(err)
|
||||||
}
|
}
|
||||||
client, err := gitea.ClientFromContext(ctx)
|
client, err := gitea.ClientFromContext(ctx)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
||||||
}
|
}
|
||||||
content, _, err := client.Repositories.GetContents(ctx, in.Owner, in.Repo, in.Ref, in.Path)
|
content, _, err := client.Repositories.GetContents(ctx, owner, repo, ref, filePath)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get file err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get file err: %v", err))
|
||||||
}
|
}
|
||||||
if in.WithLines {
|
withLines, _ := args["withLines"].(bool)
|
||||||
|
if withLines {
|
||||||
rawContent, err := base64.StdEncoding.DecodeString(*content.Content)
|
rawContent, err := base64.StdEncoding.DecodeString(*content.Content)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("decode base64 content err: %v", err))
|
return to.ErrorResult(fmt.Errorf("decode base64 content err: %v", err))
|
||||||
@@ -162,45 +164,49 @@ func GetFileContentFn(ctx context.Context, args map[string]any) (*mcp.CallToolRe
|
|||||||
return to.TextResult(slimContents(content))
|
return to.TextResult(slimContents(content))
|
||||||
}
|
}
|
||||||
|
|
||||||
type getDirContentArgs struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
Repo string `json:"repo" required:"true"`
|
|
||||||
Ref string `json:"ref"`
|
|
||||||
Path string `json:"path" required:"true"`
|
|
||||||
}
|
|
||||||
|
|
||||||
func GetDirContentFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
func GetDirContentFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
||||||
var in getDirContentArgs
|
owner, err := params.GetString(args, "owner")
|
||||||
if err := params.Bind(args, &in); err != nil {
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
repo, err := params.GetString(args, "repo")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
ref, _ := args["ref"].(string)
|
||||||
|
filePath, err := params.GetString(args, "path")
|
||||||
|
if err != nil {
|
||||||
return to.ErrorResult(err)
|
return to.ErrorResult(err)
|
||||||
}
|
}
|
||||||
client, err := gitea.ClientFromContext(ctx)
|
client, err := gitea.ClientFromContext(ctx)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
||||||
}
|
}
|
||||||
content, _, err := client.Repositories.ListContents(ctx, in.Owner, in.Repo, in.Ref, in.Path)
|
content, _, err := client.Repositories.ListContents(ctx, owner, repo, ref, filePath)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get dir content err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get dir content err: %v", err))
|
||||||
}
|
}
|
||||||
return to.TextResult(slimDirEntries(content))
|
return to.TextResult(slimDirEntries(content))
|
||||||
}
|
}
|
||||||
|
|
||||||
type createOrUpdateFileArgs struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
Repo string `json:"repo" required:"true"`
|
|
||||||
Path string `json:"path" required:"true"`
|
|
||||||
Content string `json:"content"`
|
|
||||||
Message string `json:"message"`
|
|
||||||
BranchName string `json:"branch_name"`
|
|
||||||
NewBranchName string `json:"new_branch_name"`
|
|
||||||
SHA string `json:"sha"`
|
|
||||||
}
|
|
||||||
|
|
||||||
func CreateOrUpdateFileFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
func CreateOrUpdateFileFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
||||||
var in createOrUpdateFileArgs
|
owner, err := params.GetString(args, "owner")
|
||||||
if err := params.Bind(args, &in); err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(err)
|
return to.ErrorResult(err)
|
||||||
}
|
}
|
||||||
|
repo, err := params.GetString(args, "repo")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
filePath, err := params.GetString(args, "path")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
content, _ := args["content"].(string)
|
||||||
|
message, _ := args["message"].(string)
|
||||||
|
branchName, _ := args["branch_name"].(string)
|
||||||
|
newBranchName, _ := args["new_branch_name"].(string)
|
||||||
|
sha, _ := args["sha"].(string)
|
||||||
|
|
||||||
client, err := gitea.ClientFromContext(ctx)
|
client, err := gitea.ClientFromContext(ctx)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
@@ -208,20 +214,20 @@ func CreateOrUpdateFileFn(ctx context.Context, args map[string]any) (*mcp.CallTo
|
|||||||
}
|
}
|
||||||
|
|
||||||
fileOpt := gitea_sdk.FileOptions{
|
fileOpt := gitea_sdk.FileOptions{
|
||||||
Message: in.Message,
|
Message: message,
|
||||||
BranchName: in.BranchName,
|
BranchName: branchName,
|
||||||
NewBranchName: in.NewBranchName,
|
NewBranchName: newBranchName,
|
||||||
}
|
}
|
||||||
targetBranch := cmp.Or(in.NewBranchName, in.BranchName)
|
targetBranch := cmp.Or(newBranchName, branchName)
|
||||||
|
|
||||||
if in.SHA != "" {
|
if sha != "" {
|
||||||
// Update existing file
|
// Update existing file
|
||||||
opt := gitea_sdk.UpdateFileOptions{
|
opt := gitea_sdk.UpdateFileOptions{
|
||||||
SHA: in.SHA,
|
SHA: sha,
|
||||||
Content: base64.StdEncoding.EncodeToString([]byte(in.Content)),
|
Content: base64.StdEncoding.EncodeToString([]byte(content)),
|
||||||
FileOptions: fileOpt,
|
FileOptions: fileOpt,
|
||||||
}
|
}
|
||||||
_, _, err = client.Repositories.UpdateFile(ctx, in.Owner, in.Repo, in.Path, opt)
|
_, _, err = client.Repositories.UpdateFile(ctx, owner, repo, filePath, opt)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("update file err: %v", err))
|
return to.ErrorResult(fmt.Errorf("update file err: %v", err))
|
||||||
}
|
}
|
||||||
@@ -230,42 +236,47 @@ func CreateOrUpdateFileFn(ctx context.Context, args map[string]any) (*mcp.CallTo
|
|||||||
|
|
||||||
// Create new file
|
// Create new file
|
||||||
opt := gitea_sdk.CreateFileOptions{
|
opt := gitea_sdk.CreateFileOptions{
|
||||||
Content: base64.StdEncoding.EncodeToString([]byte(in.Content)),
|
Content: base64.StdEncoding.EncodeToString([]byte(content)),
|
||||||
FileOptions: fileOpt,
|
FileOptions: fileOpt,
|
||||||
}
|
}
|
||||||
_, _, err = client.Repositories.CreateFile(ctx, in.Owner, in.Repo, in.Path, opt)
|
_, _, err = client.Repositories.CreateFile(ctx, owner, repo, filePath, opt)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("create file err: %v", err))
|
return to.ErrorResult(fmt.Errorf("create file err: %v", err))
|
||||||
}
|
}
|
||||||
return to.TextResult("Create file success on branch " + targetBranch)
|
return to.TextResult("Create file success on branch " + targetBranch)
|
||||||
}
|
}
|
||||||
|
|
||||||
type deleteFileArgs struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
Repo string `json:"repo" required:"true"`
|
|
||||||
Path string `json:"path" required:"true"`
|
|
||||||
Message string `json:"message"`
|
|
||||||
BranchName string `json:"branch_name"`
|
|
||||||
SHA string `json:"sha" required:"true"`
|
|
||||||
}
|
|
||||||
|
|
||||||
func DeleteFileFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
func DeleteFileFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
||||||
var in deleteFileArgs
|
owner, err := params.GetString(args, "owner")
|
||||||
if err := params.Bind(args, &in); err != nil {
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
repo, err := params.GetString(args, "repo")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
filePath, err := params.GetString(args, "path")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
message, _ := args["message"].(string)
|
||||||
|
branchName, _ := args["branch_name"].(string)
|
||||||
|
sha, err := params.GetString(args, "sha")
|
||||||
|
if err != nil {
|
||||||
return to.ErrorResult(err)
|
return to.ErrorResult(err)
|
||||||
}
|
}
|
||||||
opt := gitea_sdk.DeleteFileOptions{
|
opt := gitea_sdk.DeleteFileOptions{
|
||||||
FileOptions: gitea_sdk.FileOptions{
|
FileOptions: gitea_sdk.FileOptions{
|
||||||
Message: in.Message,
|
Message: message,
|
||||||
BranchName: in.BranchName,
|
BranchName: branchName,
|
||||||
},
|
},
|
||||||
SHA: in.SHA,
|
SHA: sha,
|
||||||
}
|
}
|
||||||
client, err := gitea.ClientFromContext(ctx)
|
client, err := gitea.ClientFromContext(ctx)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
||||||
}
|
}
|
||||||
_, err = client.Repositories.DeleteFile(ctx, in.Owner, in.Repo, in.Path, opt)
|
_, err = client.Repositories.DeleteFile(ctx, owner, repo, filePath, opt)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("delete file err: %v", err))
|
return to.ErrorResult(fmt.Errorf("delete file err: %v", err))
|
||||||
}
|
}
|
||||||
|
|||||||
+14
-12
@@ -37,18 +37,20 @@ func init() {
|
|||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
type getRepoTreeArgs struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
Repo string `json:"repo" required:"true"`
|
|
||||||
TreeSHA string `json:"tree_sha" required:"true"`
|
|
||||||
Recursive bool `json:"recursive"`
|
|
||||||
}
|
|
||||||
|
|
||||||
func GetRepoTreeFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
func GetRepoTreeFn(ctx context.Context, args map[string]any) (*mcp.CallToolResult, error) {
|
||||||
var in getRepoTreeArgs
|
owner, err := params.GetString(args, "owner")
|
||||||
if err := params.Bind(args, &in); err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(err)
|
return to.ErrorResult(err)
|
||||||
}
|
}
|
||||||
|
repo, err := params.GetString(args, "repo")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
treeSHA, err := params.GetString(args, "tree_sha")
|
||||||
|
if err != nil {
|
||||||
|
return to.ErrorResult(err)
|
||||||
|
}
|
||||||
|
recursive, _ := args["recursive"].(bool)
|
||||||
page, pageSize := params.GetPagination(args, 30)
|
page, pageSize := params.GetPagination(args, 30)
|
||||||
|
|
||||||
opt := gitea_sdk.ListTreeOptions{
|
opt := gitea_sdk.ListTreeOptions{
|
||||||
@@ -56,14 +58,14 @@ func GetRepoTreeFn(ctx context.Context, args map[string]any) (*mcp.CallToolResul
|
|||||||
Page: page,
|
Page: page,
|
||||||
PageSize: pageSize,
|
PageSize: pageSize,
|
||||||
},
|
},
|
||||||
Ref: in.TreeSHA,
|
Ref: treeSHA,
|
||||||
Recursive: in.Recursive,
|
Recursive: recursive,
|
||||||
}
|
}
|
||||||
client, err := gitea.ClientFromContext(ctx)
|
client, err := gitea.ClientFromContext(ctx)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
||||||
}
|
}
|
||||||
tree, _, err := client.Git.GetTrees(ctx, in.Owner, in.Repo, opt)
|
tree, _, err := client.Git.GetTrees(ctx, owner, repo, opt)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return to.ErrorResult(fmt.Errorf("get repository tree err: %v", err))
|
return to.ErrorResult(fmt.Errorf("get repository tree err: %v", err))
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,55 +0,0 @@
|
|||||||
package params
|
|
||||||
|
|
||||||
import (
|
|
||||||
"encoding/json"
|
|
||||||
"fmt"
|
|
||||||
"reflect"
|
|
||||||
"strings"
|
|
||||||
)
|
|
||||||
|
|
||||||
// Bind decodes args into out, a pointer to a struct, replacing the repeated
|
|
||||||
// args["x"].(string)/!ok extraction pattern. It round-trips args through JSON
|
|
||||||
// so JSON numbers land in the correct Go numeric field types.
|
|
||||||
//
|
|
||||||
// Struct fields tagged `required:"true"` must be present in args; a missing
|
|
||||||
// key, or an empty string value for a string field, returns an error naming
|
|
||||||
// the field's json tag.
|
|
||||||
func Bind(args map[string]any, out any) error {
|
|
||||||
v := reflect.ValueOf(out)
|
|
||||||
if v.Kind() != reflect.Pointer || v.Elem().Kind() != reflect.Struct {
|
|
||||||
return fmt.Errorf("params.Bind: out must be a pointer to a struct, got %T", out)
|
|
||||||
}
|
|
||||||
|
|
||||||
if err := checkRequiredFields(args, v.Elem().Type()); err != nil {
|
|
||||||
return err
|
|
||||||
}
|
|
||||||
|
|
||||||
data, err := json.Marshal(args)
|
|
||||||
if err != nil {
|
|
||||||
return fmt.Errorf("params.Bind: marshal args: %w", err)
|
|
||||||
}
|
|
||||||
if err := json.Unmarshal(data, out); err != nil {
|
|
||||||
return fmt.Errorf("params.Bind: %w", err)
|
|
||||||
}
|
|
||||||
return nil
|
|
||||||
}
|
|
||||||
|
|
||||||
func checkRequiredFields(args map[string]any, t reflect.Type) error {
|
|
||||||
for field := range t.Fields() {
|
|
||||||
if field.Tag.Get("required") != "true" {
|
|
||||||
continue
|
|
||||||
}
|
|
||||||
name, _, _ := strings.Cut(field.Tag.Get("json"), ",")
|
|
||||||
if name == "" || name == "-" {
|
|
||||||
continue
|
|
||||||
}
|
|
||||||
val, ok := args[name]
|
|
||||||
if !ok {
|
|
||||||
return fmt.Errorf("%s is required", name)
|
|
||||||
}
|
|
||||||
if s, isString := val.(string); isString && s == "" {
|
|
||||||
return fmt.Errorf("%s is required", name)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
return nil
|
|
||||||
}
|
|
||||||
@@ -1,132 +0,0 @@
|
|||||||
package params
|
|
||||||
|
|
||||||
import (
|
|
||||||
"strings"
|
|
||||||
"testing"
|
|
||||||
)
|
|
||||||
|
|
||||||
func TestBind_RequiredFieldsPresent(t *testing.T) {
|
|
||||||
type args struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
Repo string `json:"repo" required:"true"`
|
|
||||||
}
|
|
||||||
var out args
|
|
||||||
err := Bind(map[string]any{"owner": "gitea", "repo": "gitea-mcp"}, &out)
|
|
||||||
if err != nil {
|
|
||||||
t.Fatalf("Bind() unexpected error = %v", err)
|
|
||||||
}
|
|
||||||
if out.Owner != "gitea" || out.Repo != "gitea-mcp" {
|
|
||||||
t.Errorf("Bind() = %+v, want Owner=gitea Repo=gitea-mcp", out)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestBind_RequiredFieldMissing(t *testing.T) {
|
|
||||||
type args struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
}
|
|
||||||
var out args
|
|
||||||
err := Bind(map[string]any{}, &out)
|
|
||||||
if err == nil {
|
|
||||||
t.Fatal("Bind() expected error, got nil")
|
|
||||||
}
|
|
||||||
if !strings.Contains(err.Error(), "owner") {
|
|
||||||
t.Errorf("Bind() error = %v, want mentioning %q", err, "owner")
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestBind_RequiredStringFieldEmpty(t *testing.T) {
|
|
||||||
type args struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
}
|
|
||||||
var out args
|
|
||||||
err := Bind(map[string]any{"owner": ""}, &out)
|
|
||||||
if err == nil {
|
|
||||||
t.Fatal("Bind() expected error for empty required string, got nil")
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestBind_OptionalFieldDefaultsToZeroValue(t *testing.T) {
|
|
||||||
type args struct {
|
|
||||||
Owner string `json:"owner" required:"true"`
|
|
||||||
OldBranch string `json:"old_branch"`
|
|
||||||
}
|
|
||||||
var out args
|
|
||||||
err := Bind(map[string]any{"owner": "gitea"}, &out)
|
|
||||||
if err != nil {
|
|
||||||
t.Fatalf("Bind() unexpected error = %v", err)
|
|
||||||
}
|
|
||||||
if out.OldBranch != "" {
|
|
||||||
t.Errorf("Bind() OldBranch = %q, want empty", out.OldBranch)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestBind_NumericConversion(t *testing.T) {
|
|
||||||
type args struct {
|
|
||||||
Page int `json:"page"`
|
|
||||||
PerPage int64 `json:"per_page"`
|
|
||||||
}
|
|
||||||
var out args
|
|
||||||
err := Bind(map[string]any{"page": float64(2), "per_page": float64(40)}, &out)
|
|
||||||
if err != nil {
|
|
||||||
t.Fatalf("Bind() unexpected error = %v", err)
|
|
||||||
}
|
|
||||||
if out.Page != 2 || out.PerPage != 40 {
|
|
||||||
t.Errorf("Bind() = %+v, want Page=2 PerPage=40", out)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestBind_Boolean(t *testing.T) {
|
|
||||||
type args struct {
|
|
||||||
Recursive bool `json:"recursive"`
|
|
||||||
}
|
|
||||||
var out args
|
|
||||||
err := Bind(map[string]any{"recursive": true}, &out)
|
|
||||||
if err != nil {
|
|
||||||
t.Fatalf("Bind() unexpected error = %v", err)
|
|
||||||
}
|
|
||||||
if !out.Recursive {
|
|
||||||
t.Errorf("Bind() Recursive = false, want true")
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestBind_Array(t *testing.T) {
|
|
||||||
type args struct {
|
|
||||||
Labels []string `json:"labels"`
|
|
||||||
IDs []int64 `json:"ids"`
|
|
||||||
}
|
|
||||||
var out args
|
|
||||||
err := Bind(map[string]any{
|
|
||||||
"labels": []any{"bug", "help wanted"},
|
|
||||||
"ids": []any{float64(1), float64(2)},
|
|
||||||
}, &out)
|
|
||||||
if err != nil {
|
|
||||||
t.Fatalf("Bind() unexpected error = %v", err)
|
|
||||||
}
|
|
||||||
if len(out.Labels) != 2 || out.Labels[0] != "bug" || out.Labels[1] != "help wanted" {
|
|
||||||
t.Errorf("Bind() Labels = %v, want [bug help wanted]", out.Labels)
|
|
||||||
}
|
|
||||||
if len(out.IDs) != 2 || out.IDs[0] != 1 || out.IDs[1] != 2 {
|
|
||||||
t.Errorf("Bind() IDs = %v, want [1 2]", out.IDs)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestBind_InvalidFieldType(t *testing.T) {
|
|
||||||
type args struct {
|
|
||||||
Page int `json:"page"`
|
|
||||||
}
|
|
||||||
var out args
|
|
||||||
err := Bind(map[string]any{"page": "not-a-number"}, &out)
|
|
||||||
if err == nil {
|
|
||||||
t.Fatal("Bind() expected error for invalid numeric field, got nil")
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestBind_NonPointerRejected(t *testing.T) {
|
|
||||||
type args struct {
|
|
||||||
Owner string `json:"owner"`
|
|
||||||
}
|
|
||||||
err := Bind(map[string]any{}, args{})
|
|
||||||
if err == nil {
|
|
||||||
t.Fatal("Bind() expected error for non-pointer out, got nil")
|
|
||||||
}
|
|
||||||
}
|
|
||||||
Reference in New Issue
Block a user