channels prefixes fix, prompts fixes

Author: korotovsky

README.md

@@ -28,7 +28,7 @@ This feature-rich Slack MCP Server has:
 1. `conversations_history`
   - Get messages from the channel by channelID
   - Required inputs:
-    - `channel_id` (string): ID of the channel in format Cxxxxxxxxxx or its name starting with #... aka #general.
+    - `channel_id` (string): ID of the channel in format Cxxxxxxxxxx or its name starting with #... or @... aka #general or @username_dm.
     - `include_activity_messages` (bool, default: false): If true, the response will include activity messages such as 'channel_join' or 'channel_leave'. Default is boolean false.
     - `cursor` (string, default: ""): Cursor for pagination. Use the value of the last row and column in the response as next_cursor field returned from the previous request.
     - `limit` (string, default: 28): Limit of messages to fetch.
@@ -37,7 +37,7 @@ This feature-rich Slack MCP Server has:
 2. `conversations_replies`
   - Get a thread of messages posted to a conversation by channelID and thread_ts
   - Required inputs:
-    - `channel_id` (string): ID of the channel in format Cxxxxxxxxxx or its name starting with #... aka #general.
+    - `channel_id` (string): ID of the channel in format Cxxxxxxxxxx or its name starting with #... or @... aka #general or @username_dm.
     - `thread_ts` (string): Unique identifier of either a thread’s parent message or a message in the thread. ts must be the timestamp in format 1234567890.123456 of an existing message with 0 or more replies.
     - `include_activity_messages` (bool, default: false): If true, the response will include activity messages such as 'channel_join' or 'channel_leave'. Default is boolean false.
     - `cursor` (string, default: ""): Cursor for pagination. Use the value of the last row and column in the response as next_cursor field returned from the previous request.

pkg/handler/conversations.go

@@ -181,7 +181,7 @@ func (ch *ConversationsHandler) parseParams(request mcp.CallToolRequest) (*conve
 		}
 	}
 
-	if strings.HasPrefix(channel, "#") {
+	if strings.HasPrefix(channel, "#") || strings.HasPrefix(channel, "@") {
 		channelsMaps := ch.apiProvider.ProvideChannelsMaps()
 		chn, ok := channelsMaps.ChannelsInv[channel]
 		if !ok {

pkg/provider/api.go

@@ -367,7 +367,7 @@ func (ap *ApiProvider) GetChannels(ctx context.Context, channelTypes []string) [
 
 		for _, ch := range chans {
 			ap.channels[ch.ID] = ch
-			ap.channelsInv["#"+ch.Name] = ch.ID
+			ap.channelsInv[ch.Name] = ch.ID
 		}
 
 		if nextcur == "" {
@@ -496,7 +496,7 @@ func mapChannel(
 	isIM, isMpIM, isPrivate bool,
 	usersMap map[string]slack.User,
 ) Channel {
-	channelName := "#" + name
+	channelName := name
 	finalPurpose := purpose
 	finalTopic := topic
 	finalMemberCount := numMembers
@@ -526,6 +526,8 @@ func mapChannel(
 			finalPurpose = "Group DM with " + strings.Join(userNames, ", ")
 			finalTopic = ""
 		}
+	} else {
+		channelName = "#" + nameNormalized
 	}
 
 	return Channel{

pkg/server/server.go

@@ -16,18 +16,18 @@ type MCPServer struct {
 func NewMCPServer(provider *provider.ApiProvider) *MCPServer {
 	s := server.NewMCPServer(
 		"Slack MCP Server",
-		"1.1.15",
+		"1.1.17",
 		server.WithLogging(),
 		server.WithRecovery(),
 	)
 
 	conversationsHandler := handler.NewConversationsHandler(provider)
 
 	s.AddTool(mcp.NewTool("conversations_history",
-		mcp.WithDescription("Get messages from the channel by channel_id, the last row/column in the response is used as 'cursor' parameter for pagination if not empty"),
+		mcp.WithDescription("Get messages from the channel (or DM) by channel_id, the last row/column in the response is used as 'cursor' parameter for pagination if not empty"),
 		mcp.WithString("channel_id",
 			mcp.Required(),
-			mcp.Description("ID of the channel in format Cxxxxxxxxxx or its name starting with #... aka #general."),
+			mcp.Description("    - `channel_id` (string): ID of the channel in format Cxxxxxxxxxx or its name starting with #... or @... aka #general or @username_dm."),
 		),
 		mcp.WithBoolean("include_activity_messages",
 			mcp.Description("If true, the response will include activity messages such as 'channel_join' or 'channel_leave'. Default is boolean false."),
@@ -46,7 +46,7 @@ func NewMCPServer(provider *provider.ApiProvider) *MCPServer {
 		mcp.WithDescription("Get a thread of messages posted to a conversation by channelID and thread_ts, the last row/column in the response is used as 'cursor' parameter for pagination if not empty"),
 		mcp.WithString("channel_id",
 			mcp.Required(),
-			mcp.Description("ID of the channel in format Cxxxxxxxxxx or its name starting with #... aka #general."),
+			mcp.Description("ID of the channel in format Cxxxxxxxxxx or its name starting with #... or @... aka #general or @username_dm."),
 		),
 		mcp.WithString("thread_ts",
 			mcp.Required(),