ref(slack): Remove channel member listing tool

Remove the slackChannelListMembers tool and its underlying Slack action path because the tool only exposed member IDs and provided low user value.

Prune related status labels, mock fixtures, and integration/unit coverage tied to the removed tool to keep the Slack tool surface smaller and explicit.

Co-Authored-By: GPT-5 Codex <noreply@openai.com>

Author: dcramer

packages/junior/src/chat/respond.ts

@@ -223,7 +223,6 @@ function formatToolStatus(toolName: string): string {
     webFetch: "Reading source pages",
     slackChannelPostMessage: "Posting message to channel",
     slackMessageAddReaction: "Adding emoji reaction",
-    slackChannelListMembers: "Listing channel members",
     slackChannelListMessages: "Listing channel messages",
     slackCanvasCreate: "Creating detailed brief",
     slackCanvasUpdate: "Updating detailed brief",
@@ -278,7 +277,6 @@ function formatToolResultStatus(toolName: string): string {
     webFetch: "Reviewing page content",
     slackChannelPostMessage: "Posted message to channel",
     slackMessageAddReaction: "Added emoji reaction",
-    slackChannelListMembers: "Reviewed channel members",
     slackChannelListMessages: "Reviewed channel messages",
     slackCanvasCreate: "Preparing canvas response",
     slackCanvasUpdate: "Preparing canvas update",

packages/junior/src/chat/slack-actions/channel.ts

@@ -30,10 +30,6 @@ export interface SlackThreadReply {
   files?: SlackFileRef[];
 }
 
-export interface SlackChannelMemberProfile {
-  user_id: string;
-}
-
 export async function postMessageToChannel(input: {
   channelId: string;
   text: string;
@@ -191,34 +187,6 @@ export async function listChannelMessages(input: {
   };
 }
 
-export async function listChannelMembers(input: {
-  channelId: string;
-  limit: number;
-  cursor?: string;
-}): Promise<{ members: SlackChannelMemberProfile[]; nextCursor?: string }> {
-  const client = getSlackClient();
-  const channelId = normalizeSlackConversationId(input.channelId);
-  if (!channelId) {
-    throw new Error("Slack channel member lookup requires a valid channel ID");
-  }
-  const targetLimit = Math.max(1, Math.min(input.limit, 200));
-  const response = await withSlackRetries(() =>
-    client.conversations.members({
-      channel: channelId,
-      limit: targetLimit,
-      cursor: input.cursor
-    }),
-    3,
-    { action: "conversations.members" }
-  );
-
-  const members = (response.members ?? []).slice(0, targetLimit);
-  return {
-    members: members.map((userId) => ({ user_id: userId })),
-    nextCursor: response.response_metadata?.next_cursor || undefined
-  };
-}
-
 export async function listThreadReplies(input: {
   channelId: string;
   threadTs: string;

packages/junior/src/chat/tools/index.ts

@@ -3,7 +3,6 @@ import type { SkillMetadata } from "@/chat/skills";
 import { createImageGenerateTool } from "@/chat/tools/image-generate";
 import { createLoadSkillTool } from "@/chat/tools/load-skill";
 import { createReadFileTool } from "@/chat/tools/read-file";
-import { createSlackChannelListMembersTool } from "@/chat/tools/slack-channel-list-members";
 import { createSlackChannelListMessagesTool } from "@/chat/tools/slack-channel-list-messages";
 import { createSlackChannelPostMessageTool } from "@/chat/tools/slack-channel-post-message";
 import { createSlackMessageAddReactionTool } from "@/chat/tools/slack-message-add-reaction";
@@ -136,11 +135,6 @@ export function createTools(
       createSlackChannelPostMessageTool(context, state),
       hooks
     );
-    tools.slackChannelListMembers = wrapToolExecution(
-      "slackChannelListMembers",
-      createSlackChannelListMembersTool(context),
-      hooks
-    );
     tools.slackChannelListMessages = wrapToolExecution(
       "slackChannelListMessages",
       createSlackChannelListMessagesTool(context),

packages/junior/src/chat/tools/slack-channel-list-members.ts

@@ -61,19 +61,6 @@ export function conversationsHistoryPage(input: {
   });
 }
 
-export function conversationsMembersPage(input: { members?: string[]; nextCursor?: string } = {}): {
-  ok: true;
-  members: string[];
-  response_metadata: { next_cursor: string };
-} {
-  return slackOk({
-    members: input.members ?? [TEST_USER_ID],
-    response_metadata: {
-      next_cursor: input.nextCursor ?? ""
-    }
-  });
-}
-
 export function conversationsRepliesPage(input: {
   messages?: Array<Record<string, unknown>>;
   nextCursor?: string;

packages/junior/tests/fixtures/slack/factories/api.ts

@@ -1,5 +1,4 @@
 import { describe, expect, it } from "vitest";
-import { createSlackChannelListMembersTool } from "@/chat/tools/slack-channel-list-members";
 import { createSlackChannelListMessagesTool } from "@/chat/tools/slack-channel-list-messages";
 import { createSlackChannelPostMessageTool } from "@/chat/tools/slack-channel-post-message";
 import { createSlackMessageAddReactionTool } from "@/chat/tools/slack-message-add-reaction";
@@ -8,7 +7,6 @@ import {
   chatGetPermalinkOk,
   chatPostMessageOk,
   conversationsHistoryPage,
-  conversationsMembersPage,
   reactionsAddOk
 } from "../fixtures/slack/factories/api";
 import { getCapturedSlackApiCalls, queueSlackApiError, queueSlackApiResponse } from "../msw/handlers/slack-api";
@@ -122,37 +120,6 @@ describe("slack channel tools", () => {
     expect(getCapturedSlackApiCalls("chat.getPermalink")).toHaveLength(1);
   });
 
-  it("lists channel members and forwards request parameters", async () => {
-    queueSlackApiResponse("conversations.members", {
-      body: conversationsMembersPage({
-        members: ["U1"],
-        nextCursor: "next-members"
-      })
-    });
-    const tool = createSlackChannelListMembersTool(createContext("who is in this channel?"));
-
-    const result = await executeTool(tool, {
-      limit: 25
-    });
-
-    expect(result).toMatchObject({
-      ok: true,
-      channel_id: "C123",
-      count: 1,
-      next_cursor: "next-members"
-    });
-    expect(result).toMatchObject({
-      members: [{ user_id: "U1" }]
-    });
-
-    const memberCalls = getCapturedSlackApiCalls("conversations.members");
-    expect(memberCalls).toHaveLength(1);
-    expect(memberCalls[0]?.params).toMatchObject({
-      channel: "C123"
-    });
-    expect(String(memberCalls[0]?.params.limit)).toBe("25");
-  });
-
   it("lists channel messages across history parameters and forwards filters", async () => {
     queueSlackApiResponse("conversations.history", {
       body: conversationsHistoryPage({
@@ -306,23 +273,4 @@ describe("slack channel tools", () => {
     expect(getCapturedSlackApiCalls("reactions.add")).toHaveLength(1);
   });
 
-  it("propagates missing_scope when members API fails", async () => {
-    queueSlackApiError("conversations.members", {
-      error: "missing_scope",
-      needed: "channels:read",
-      provided: "chat:write"
-    });
-    const tool = createSlackChannelListMembersTool(createContext("who is in this channel?"));
-
-    await expect(
-      executeTool(tool, {
-        limit: 10
-      })
-    ).rejects.toMatchObject({
-      name: "SlackActionError",
-      code: "missing_scope",
-      needed: "channels:read",
-      provided: "chat:write"
-    });
-  });
 });

packages/junior/tests/integration/slack-channel-tools.test.ts

@@ -7,7 +7,6 @@ import {
   chatPostMessageOk,
   conversationsCanvasesCreateOk,
   conversationsHistoryPage,
-  conversationsMembersPage,
   conversationsRepliesPage,
   filesCompleteUploadOk,
   filesGetUploadUrlOk,
@@ -29,7 +28,6 @@ export const SUPPORTED_SLACK_API_METHODS = [
   "reactions.add",
   "reactions.remove",
   "conversations.history",
-  "conversations.members",
   "conversations.replies",
   "canvases.create",
   "conversations.canvases.create",
@@ -170,8 +168,6 @@ function defaultSlackApiResponse(method: SlackApiMethod): SlackMockHttpResponse
       return { body: reactionsAddOk() };
     case "conversations.history":
       return { body: conversationsHistoryPage() };
-    case "conversations.members":
-      return { body: conversationsMembersPage() };
     case "conversations.replies":
       return { body: conversationsRepliesPage() };
     case "canvases.create":

packages/junior/tests/msw/handlers/slack-api.ts

@@ -8,7 +8,6 @@ describe("Slack tool registration", () => {
     const tools = createTools([], {}, { channelId: "D12345", sandbox: noopSandbox });
 
     expect(tools).not.toHaveProperty("slackChannelPostMessage");
-    expect(tools).not.toHaveProperty("slackChannelListMembers");
     expect(tools).not.toHaveProperty("slackChannelListMessages");
     expect(tools).toHaveProperty("slackMessageAddReaction");
     expect(tools).toHaveProperty("slackCanvasCreate");
@@ -18,7 +17,6 @@ describe("Slack tool registration", () => {
     const tools = createTools([], {}, { channelId: "C12345", sandbox: noopSandbox });
 
     expect(tools).toHaveProperty("slackChannelPostMessage");
-    expect(tools).toHaveProperty("slackChannelListMembers");
     expect(tools).toHaveProperty("slackChannelListMessages");
     expect(tools).toHaveProperty("slackMessageAddReaction");
     expect(tools).toHaveProperty("slackCanvasCreate");
@@ -29,7 +27,6 @@ describe("Slack tool registration", () => {
 
     expect(tools).not.toHaveProperty("slackCanvasCreate");
     expect(tools).not.toHaveProperty("slackChannelPostMessage");
-    expect(tools).not.toHaveProperty("slackChannelListMembers");
     expect(tools).not.toHaveProperty("slackChannelListMessages");
     expect(tools).not.toHaveProperty("slackMessageAddReaction");
   });