mirror of
https://gitea.com/gitea/gitea-mcp.git
synced 2026-08-27 10:37: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.String("since", tool.Description("updated after 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("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 {
|
||||
opt.Before = *t
|
||||
}
|
||||
if assignedBy, ok := args["assigned_by"].(string); ok {
|
||||
opt.AssignedBy = assignedBy
|
||||
}
|
||||
client, err := gitea.ClientFromContext(ctx)
|
||||
if err != nil {
|
||||
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{
|
||||
"owner": owner,
|
||||
"repo": repo,
|
||||
"type": "issues",
|
||||
"labels": []any{"bug", "enhancement"},
|
||||
"milestones": []any{"v1.0", "2"},
|
||||
"since": "2026-01-01T00:00:00Z",
|
||||
"owner": owner,
|
||||
"repo": repo,
|
||||
"type": "issues",
|
||||
"labels": []any{"bug", "enhancement"},
|
||||
"milestones": []any{"v1.0", "2"},
|
||||
"since": "2026-01-01T00:00:00Z",
|
||||
"assigned_by": "octocat",
|
||||
}
|
||||
|
||||
_, err := listRepoIssuesFn(context.Background(), args)
|
||||
@@ -89,6 +90,9 @@ func Test_listRepoIssuesFn_filters(t *testing.T) {
|
||||
if !strings.Contains(gotQuery, "type=issues") {
|
||||
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) {
|
||||
@@ -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) {
|
||||
const (
|
||||
owner = "octo"
|
||||
|
||||
@@ -63,6 +63,9 @@ func slimIssues(issues []*gitea_sdk.Issue) []map[string]any {
|
||||
if len(i.Labels) > 0 {
|
||||
m["labels"] = slim.LabelNames(i.Labels)
|
||||
}
|
||||
if len(i.Assignees) > 0 {
|
||||
m["assignees"] = slim.UserLogins(i.Assignees)
|
||||
}
|
||||
if i.Milestone != nil {
|
||||
m["milestone"] = map[string]any{
|
||||
"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) {
|
||||
var in createBranchArgs
|
||||
if err := params.Bind(args, &in); err != nil {
|
||||
owner, err := params.GetString(args, "owner")
|
||||
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)
|
||||
}
|
||||
oldBranch, _ := args["old_branch"].(string)
|
||||
|
||||
client, err := gitea.ClientFromContext(ctx)
|
||||
if err != nil {
|
||||
return to.ErrorResult(fmt.Errorf("get gitea client err: %v", err))
|
||||
}
|
||||
_, _, err = client.Repositories.CreateBranch(ctx, in.Owner, in.Repo, gitea_sdk.CreateBranchOption{
|
||||
BranchName: in.Branch,
|
||||
OldBranchName: in.OldBranch,
|
||||
_, _, err = client.Repositories.CreateBranch(ctx, owner, repo, gitea_sdk.CreateBranchOption{
|
||||
BranchName: branch,
|
||||
OldBranchName: oldBranch,
|
||||
})
|
||||
if err != nil {
|
||||
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")
|
||||
}
|
||||
|
||||
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) {
|
||||
var in deleteBranchArgs
|
||||
if err := params.Bind(args, &in); err != nil {
|
||||
owner, err := params.GetString(args, "owner")
|
||||
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)
|
||||
}
|
||||
client, err := gitea.ClientFromContext(ctx)
|
||||
if err != nil {
|
||||
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 {
|
||||
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")
|
||||
}
|
||||
|
||||
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) {
|
||||
var in listBranchesArgs
|
||||
if err := params.Bind(args, &in); err != nil {
|
||||
owner, err := params.GetString(args, "owner")
|
||||
if err != nil {
|
||||
return to.ErrorResult(err)
|
||||
}
|
||||
repo, err := params.GetString(args, "repo")
|
||||
if err != nil {
|
||||
return to.ErrorResult(err)
|
||||
}
|
||||
page, pageSize := params.GetPagination(args, 30)
|
||||
@@ -141,7 +144,7 @@ func ListBranchesFn(ctx context.Context, args map[string]any) (*mcp.CallToolResu
|
||||
if err != nil {
|
||||
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 {
|
||||
return to.ErrorResult(fmt.Errorf("list branches error: %v", err))
|
||||
}
|
||||
|
||||
+71
-60
@@ -102,28 +102,30 @@ type ContentLine struct {
|
||||
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) {
|
||||
var in getFileContentArgs
|
||||
if err := params.Bind(args, &in); err != nil {
|
||||
owner, err := params.GetString(args, "owner")
|
||||
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)
|
||||
}
|
||||
client, err := gitea.ClientFromContext(ctx)
|
||||
if err != nil {
|
||||
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 {
|
||||
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)
|
||||
if err != nil {
|
||||
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))
|
||||
}
|
||||
|
||||
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) {
|
||||
var in getDirContentArgs
|
||||
if err := params.Bind(args, &in); err != nil {
|
||||
owner, err := params.GetString(args, "owner")
|
||||
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)
|
||||
}
|
||||
client, err := gitea.ClientFromContext(ctx)
|
||||
if err != nil {
|
||||
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 {
|
||||
return to.ErrorResult(fmt.Errorf("get dir content err: %v", err))
|
||||
}
|
||||
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) {
|
||||
var in createOrUpdateFileArgs
|
||||
if err := params.Bind(args, &in); err != nil {
|
||||
owner, err := params.GetString(args, "owner")
|
||||
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)
|
||||
}
|
||||
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)
|
||||
if err != nil {
|
||||
@@ -208,20 +214,20 @@ func CreateOrUpdateFileFn(ctx context.Context, args map[string]any) (*mcp.CallTo
|
||||
}
|
||||
|
||||
fileOpt := gitea_sdk.FileOptions{
|
||||
Message: in.Message,
|
||||
BranchName: in.BranchName,
|
||||
NewBranchName: in.NewBranchName,
|
||||
Message: message,
|
||||
BranchName: branchName,
|
||||
NewBranchName: newBranchName,
|
||||
}
|
||||
targetBranch := cmp.Or(in.NewBranchName, in.BranchName)
|
||||
targetBranch := cmp.Or(newBranchName, branchName)
|
||||
|
||||
if in.SHA != "" {
|
||||
if sha != "" {
|
||||
// Update existing file
|
||||
opt := gitea_sdk.UpdateFileOptions{
|
||||
SHA: in.SHA,
|
||||
Content: base64.StdEncoding.EncodeToString([]byte(in.Content)),
|
||||
SHA: sha,
|
||||
Content: base64.StdEncoding.EncodeToString([]byte(content)),
|
||||
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 {
|
||||
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
|
||||
opt := gitea_sdk.CreateFileOptions{
|
||||
Content: base64.StdEncoding.EncodeToString([]byte(in.Content)),
|
||||
Content: base64.StdEncoding.EncodeToString([]byte(content)),
|
||||
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 {
|
||||
return to.ErrorResult(fmt.Errorf("create file err: %v", err))
|
||||
}
|
||||
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) {
|
||||
var in deleteFileArgs
|
||||
if err := params.Bind(args, &in); err != nil {
|
||||
owner, err := params.GetString(args, "owner")
|
||||
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)
|
||||
}
|
||||
opt := gitea_sdk.DeleteFileOptions{
|
||||
FileOptions: gitea_sdk.FileOptions{
|
||||
Message: in.Message,
|
||||
BranchName: in.BranchName,
|
||||
Message: message,
|
||||
BranchName: branchName,
|
||||
},
|
||||
SHA: in.SHA,
|
||||
SHA: sha,
|
||||
}
|
||||
client, err := gitea.ClientFromContext(ctx)
|
||||
if err != nil {
|
||||
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 {
|
||||
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) {
|
||||
var in getRepoTreeArgs
|
||||
if err := params.Bind(args, &in); err != nil {
|
||||
owner, err := params.GetString(args, "owner")
|
||||
if err != nil {
|
||||
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)
|
||||
|
||||
opt := gitea_sdk.ListTreeOptions{
|
||||
@@ -56,14 +58,14 @@ func GetRepoTreeFn(ctx context.Context, args map[string]any) (*mcp.CallToolResul
|
||||
Page: page,
|
||||
PageSize: pageSize,
|
||||
},
|
||||
Ref: in.TreeSHA,
|
||||
Recursive: in.Recursive,
|
||||
Ref: treeSHA,
|
||||
Recursive: recursive,
|
||||
}
|
||||
client, err := gitea.ClientFromContext(ctx)
|
||||
if err != nil {
|
||||
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 {
|
||||
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