feat(slack): add DM channel listing support

[code-yeongyu]

Problem

agent-slack channel list --type dm returns an empty array because:

1. listChannels() only requests types: 'public_channel,private_channel' — im and mpim are not included
2. The --type dm branch in channel.ts hardcodes channels = []

Solution

• Added listDMs() method to SlackClient that calls conversations.list with types: 'im,mpim'
• Updated channel list --type dm to call listDMs() and return actual DM channels with id, user, and is_mpim fields

Notes

• conversations.history already works for DM channels (cookie is passed via WebClient headers), so message list <DM_ID> needs no changes
• All existing tests pass (bun test)

Changes

• src/platforms/slack/client.ts: Added listDMs() method
• src/platforms/slack/commands/channel.ts: Implemented --type dm listing

---

Summary by cubic

Adds DM and multi-person DM channel listing to the Slack integration. agent-slack channel list --type dm now returns real channels with id, user, and is_mpim, and respects --include-archived.

• New Features

  • Added listDMs() in SlackClient using conversations.list with types: 'im,mpim' and pagination.
  • Updated channel list --type dm to call listDMs() and print results.

• Bug Fixes

  • Fixed DM listing: include 'im,mpim' and remove empty result; skip public/private fetch for --type dm; respect --include-archived via Slack exclude_archived.

^{Written for commit 1fbd12b. Summary will update on new commits.}

[vercel]

@code-yeongyu is attempting to deploy a commit to the devxoul Team on Vercel.

A member of the Team first needs to authorize it.

[cubic-dev-ai]

2 issues found across 2 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/platforms/slack/commands/channel.ts">

<violation number="1" location="src/platforms/slack/commands/channel.ts:31">
P2: `--include-archived` is ignored when `--type dm` is used; DM listing always includes archived IM/MPIM conversations, making the flag inconsistent with public/private channel listing.</violation>

<violation number="2" location="src/platforms/slack/commands/channel.ts:31">
P2: `channel list --type dm` still performs `listChannels()` before the DM branch, so DM listing depends on the public/private channel list call succeeding and always incurs an unnecessary API request.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

P2: --include-archived is ignored when --type dm is used; DM listing always includes archived IM/MPIM conversations, making the flag inconsistent with public/private channel listing.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At src/platforms/slack/commands/channel.ts, line 31:

<comment>`--include-archived` is ignored when `--type dm` is used; DM listing always includes archived IM/MPIM conversations, making the flag inconsistent with public/private channel listing.</comment>

<file context>
@@ -28,7 +28,14 @@ async function listAction(options: { type?: string; includeArchived?: boolean; p
       channels = channels.filter((c) => c.is_private)
     } else if (options.type === 'dm') {
-      channels = []
+      const dms = await client.listDMs()
+      const dmOutput = dms.map((dm) => ({
+        id: dm.id,
</file context>

[cubic-dev-ai]

P2: channel list --type dm still performs listChannels() before the DM branch, so DM listing depends on the public/private channel list call succeeding and always incurs an unnecessary API request.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At src/platforms/slack/commands/channel.ts, line 31:

<comment>`channel list --type dm` still performs `listChannels()` before the DM branch, so DM listing depends on the public/private channel list call succeeding and always incurs an unnecessary API request.</comment>

<file context>
@@ -28,7 +28,14 @@ async function listAction(options: { type?: string; includeArchived?: boolean; p
       channels = channels.filter((c) => c.is_private)
     } else if (options.type === 'dm') {
-      channels = []
+      const dms = await client.listDMs()
+      const dmOutput = dms.map((dm) => ({
+        id: dm.id,
</file context>

[code-yeongyu]

Thanks for the review! Both issues fixed in 1fbd12b:

1. --include-archived now respected for DMs — listDMs() accepts excludeArchived option and passes it to Slack's exclude_archived API parameter
2. No unnecessary API call — DM branch moved before listChannels() so conversations.list for public/private channels is never called when --type dm is specified