skills/paymog__slack-cli
1
0
Fork 0
mirror of https://github.com/paymog/slack-cli.git synced 2026-09-18 23:43:49 +08:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #213 from kirecek/master

Browse Source 
Allow to filter out users_search tool via enabled-tools
This commit is contained in:
Dmitrii Korotovskii
2026-03-03 22:37:58 +01:00
committed by GitHub
parent d3800035c2 6f7acbdc3e
commit b24b1b08af
2 changed files with 20 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/server/server.go
+17 -13
View File
@@ -40,6 +40,7 @@ const (
ToolUsergroupsCreate = "usergroups_create"
ToolUsergroupsUpdate = "usergroups_update"
ToolUsergroupsUsersUpdate = "usergroups_users_update"
ToolUsersSearch = "users_search"
)
var ValidToolNames = []string{
@@ -58,6 +59,7 @@ var ValidToolNames = []string{
ToolUsergroupsCreate,
ToolUsergroupsUpdate,
ToolUsergroupsUsersUpdate,
ToolUsersSearch,
}
func ValidateEnabledTools(tools []string) error {
@@ -282,19 +284,21 @@ func NewMCPServer(provider *provider.ApiProvider, logger *zap.Logger, enabledToo
s.AddTool(conversationsSearchTool, conversationsHandler.ConversationsSearchHandler)
}
s.AddTool(mcp.NewTool("users_search",
mcp.WithDescription("Search for users by name, email, or display name. Returns user details and DM channel ID if available."),
mcp.WithTitleAnnotation("Search Users"),
mcp.WithReadOnlyHintAnnotation(true),
mcp.WithString("query",
mcp.Required(),
mcp.Description("Search query - matches against real name, display name, username, or email."),
),
mcp.WithNumber("limit",
mcp.DefaultNumber(10),
mcp.Description("Maximum number of results to return (1-100). Default is 10."),
),
), conversationsHandler.UsersSearchHandler)
if shouldAddTool(ToolUsersSearch, enabledTools, "") {
s.AddTool(mcp.NewTool(ToolUsersSearch,
mcp.WithDescription("Search for users by name, email, or display name. Returns user details and DM channel ID if available."),
mcp.WithTitleAnnotation("Search Users"),
mcp.WithReadOnlyHintAnnotation(true),
mcp.WithString("query",
mcp.Required(),
mcp.Description("Search query - matches against real name, display name, username, or email."),
),
mcp.WithNumber("limit",
mcp.DefaultNumber(10),
mcp.Description("Maximum number of results to return (1-100). Default is 10."),
),
), conversationsHandler.UsersSearchHandler)
}
// Register unreads tool - gets all unread messages across channels efficiently.
// Bot tokens (xoxb) don't support unread tracking, so exclude them (same pattern as search tool).
pkg/server/server_test.go
+3
View File
@@ -24,6 +24,7 @@ func TestShouldAddTool_ReadOnly_EmptyEnabledTools(t *testing.T) {
ToolConversationsReplies,
ToolConversationsSearchMessages,
ToolChannelsList,
ToolUsersSearch,
}
for _, tool := range readOnlyTools {
result := shouldAddTool(tool, []string{}, "")
@@ -108,6 +109,7 @@ func TestValidToolNames(t *testing.T) {
ToolUsergroupsCreate: true,
ToolUsergroupsUpdate: true,
ToolUsergroupsUsersUpdate: true,
ToolUsersSearch: true,
}
assert.Equal(t, len(expectedTools), len(ValidToolNames), "ValidToolNames should have %d tools", len(expectedTools))
@@ -133,6 +135,7 @@ func TestValidToolNames(t *testing.T) {
assert.Equal(t, "usergroups_create", ToolUsergroupsCreate)
assert.Equal(t, "usergroups_update", ToolUsergroupsUpdate)
assert.Equal(t, "usergroups_users_update", ToolUsergroupsUsersUpdate)
assert.Equal(t, "users_search", ToolUsersSearch)
})
}