Merge pull request #45 from arvoreeducacao/joaobarrs-/-fix-slack-channel-resolve

fix(slack-advanced): resolve channel name com scopes separados

Author: Joao208

packages/slack-advanced/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arvoretech/slack-advanced-mcp",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "Advanced Slack MCP Server with semantic user search, smart DMs, style analysis, thread extraction, audio transcription, and image analysis",
   "main": "dist/index.js",
   "type": "module",

packages/slack-advanced/src/slack-client.ts

@@ -212,25 +212,35 @@ export class SlackClient {
 
     const name = identifier.replace(/^#/, "");
 
-    let cursor: string | undefined;
-    do {
-      const params: Record<string, unknown> = { limit: 200, types: "public_channel,private_channel" };
-      if (cursor) params.cursor = cursor;
-
-      const res = await this.request<{
-        ok: boolean;
-        channels: Array<{ id: string; name: string }>;
-        response_metadata?: { next_cursor?: string };
-      }>("conversations.list", params);
-
-      const match = res.channels.find((c) => c.name === name);
-      if (match) return match.id;
-
-      cursor = res.response_metadata?.next_cursor || undefined;
-    } while (cursor);
+    const channelTypes = ["public_channel", "private_channel"];
+
+    for (const type of channelTypes) {
+      try {
+        let cursor: string | undefined;
+        do {
+          const params: Record<string, unknown> = { limit: 200, types: type };
+          if (cursor) params.cursor = cursor;
+
+          const res = await this.request<{
+            ok: boolean;
+            channels: Array<{ id: string; name: string }>;
+            response_metadata?: { next_cursor?: string };
+          }>("conversations.list", params);
+
+          const match = res.channels.find((c) => c.name === name);
+          if (match) return match.id;
+
+          cursor = res.response_metadata?.next_cursor || undefined;
+        } while (cursor);
+      } catch (error) {
+        const isScopeError =
+          error instanceof SlackAdvancedMCPError && error.message.includes("missing_scope");
+        if (!isScopeError) throw error;
+      }
+    }
 
     throw new SlackAdvancedMCPError(
-      `Could not resolve channel: ${identifier}`,
+      `Could not resolve channel: ${identifier}. Tried public and private channel lists. Ensure the token has channels:read and/or groups:read scopes.`,
       "CHANNEL_NOT_FOUND"
     );
   }