Skip to content

Commit 3e61ad2

Browse files
committed
✏️ Format cp tool name
issue #64
1 parent 3bbf6c1 commit 3e61ad2

File tree

3 files changed

+39
-23
lines changed

3 files changed

+39
-23
lines changed

cmd/activity/list.go

Lines changed: 33 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -14,39 +14,45 @@ import (
1414
var defaultParts = []string{"id", "snippet", "contentDetails"}
1515

1616
var listTool = mcp.NewTool(
17-
"activity.list",
17+
"activity-list",
1818
mcp.WithTitleAnnotation("List Activities"),
1919
mcp.WithOpenWorldHintAnnotation(true),
2020
mcp.WithDescription(long),
2121
mcp.WithString(
22-
"channelId", mcp.DefaultString(""), mcp.Description(ciUsage),
22+
"channelId", mcp.DefaultString(""), mcp.Description(ciUsage), mcp.Required(),
2323
),
2424
mcp.WithString(
2525
"home", mcp.Enum("true", "false", ""),
26-
mcp.DefaultString(""), mcp.Description(homeUsage),
26+
mcp.DefaultString(""), mcp.Description(homeUsage), mcp.Required(),
2727
),
2828
mcp.WithNumber(
29-
"maxResults", mcp.DefaultNumber(5), mcp.Description(mrUsage),
29+
"maxResults", mcp.DefaultNumber(5), mcp.Description(mrUsage), mcp.Required(),
3030
),
3131
mcp.WithString(
3232
"mine", mcp.Enum("true", "false", ""),
33-
mcp.DefaultString("true"), mcp.Description(mineUsage),
33+
mcp.DefaultString("true"), mcp.Description(mineUsage), mcp.Required(),
3434
),
3535
mcp.WithString(
3636
"publishedAfter", mcp.DefaultString(""),
37-
mcp.Description(paUsage),
37+
mcp.Description(paUsage), mcp.Required(),
3838
),
3939
mcp.WithString(
4040
"publishedBefore", mcp.DefaultString(""),
41-
mcp.Description(pbUsage),
41+
mcp.Description(pbUsage), mcp.Required(),
4242
),
4343
mcp.WithString(
44-
"regionCode", mcp.DefaultString(""), mcp.Description(rcUsage),
44+
"regionCode", mcp.DefaultString(""),
45+
mcp.Description(rcUsage), mcp.Required(),
4546
),
4647
mcp.WithArray(
47-
"parts", mcp.DefaultArray(defaultParts), mcp.Description(partsUsage),
48+
"parts", mcp.DefaultArray(defaultParts),
49+
mcp.Items(map[string]any{"type": "string"}),
50+
mcp.Description(partsUsage), mcp.Required(),
51+
),
52+
mcp.WithString(
53+
"output", mcp.DefaultString(""), mcp.Description(outputUsage),
54+
mcp.Required(),
4855
),
49-
mcp.WithString("output", mcp.DefaultString(""), mcp.Description(outputUsage)),
5056
)
5157

5258
func run(writer io.Writer) error {
@@ -99,18 +105,25 @@ func listHandler(ctx context.Context, request mcp.CallToolRequest) (
99105
*mcp.CallToolResult, error,
100106
) {
101107
args := request.GetArguments()
102-
channelId = args["channelId"].(string)
103-
home = utils.BoolPtr(args["home"].(string))
104-
maxResults = int64(args["maxResults"].(float64))
105-
mine = utils.BoolPtr(args["mine"].(string))
106-
publishedAfter = args["publishedAfter"].(string)
107-
publishedBefore = args["publishedBefore"].(string)
108-
regionCode = args["regionCode"].(string)
109-
parts = make([]string, len(args["parts"].([]interface{})))
110-
for i, part := range args["parts"].([]interface{}) {
108+
channelId, _ = args["channelId"].(string)
109+
homeRaw, _ := args["home"].(string)
110+
home = utils.BoolPtr(homeRaw)
111+
maxResultsRaw, _ := args["maxResults"].(float64)
112+
maxResults = int64(maxResultsRaw)
113+
mineRaw, ok := args["mine"].(string)
114+
if !ok {
115+
mineRaw = "true" // Default to true if not provided
116+
}
117+
mine = utils.BoolPtr(mineRaw)
118+
publishedAfter, _ = args["publishedAfter"].(string)
119+
publishedBefore, _ = args["publishedBefore"].(string)
120+
regionCode, _ = args["regionCode"].(string)
121+
partsRaw, _ := args["parts"].([]interface{})
122+
parts = make([]string, len(partsRaw))
123+
for i, part := range partsRaw {
111124
parts[i] = part.(string)
112125
}
113-
output = args["output"].(string)
126+
output, _ = args["output"].(string)
114127

115128
var writer bytes.Buffer
116129
err := run(&writer)

cmd/mcp.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,5 +59,5 @@ func init() {
5959
RootCmd.AddCommand(mcpCmd)
6060

6161
mcpCmd.Flags().StringVarP(&mode, "mode", "m", "stdio", modeUsage)
62-
mcpCmd.Flags().IntVarP(&port, "port", "p", 8080, portUsage)
62+
mcpCmd.Flags().IntVarP(&port, "port", "p", 8216, portUsage)
6363
}

pkg/activity/activity.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,10 +99,13 @@ func (a *activity) List(
9999
tb.SetOutputMirror(writer)
100100
tb.SetStyle(table.StyleLight)
101101
tb.SetAutoIndex(true)
102-
tb.AppendHeader(table.Row{"ID", "Title", "Type"})
102+
tb.AppendHeader(table.Row{"ID", "Title", "Type", "Time"})
103103
for _, activity := range activities {
104104
tb.AppendRow(
105-
table.Row{activity.Id, activity.Snippet.Title, activity.Snippet.Type},
105+
table.Row{
106+
activity.Id, activity.Snippet.Title,
107+
activity.Snippet.Type, activity.Snippet.PublishedAt,
108+
},
106109
)
107110
}
108111
}

0 commit comments

Comments
 (0)