agent-messenger 2.8.0 → 2.10.0

package/src/platforms/discord/commands/snapshot.test.ts

@@ -8,7 +8,7 @@ test('snapshot: command is defined', () => {
 })
 
 test('snapshot: command has correct description', () => {
-  expect(snapshotCommand.description()).toContain('server state')
+  expect(snapshotCommand.description()).toContain('server overview')
 })
 
 test('snapshot: command has --channels-only option', () => {

package/src/platforms/discord/commands/snapshot.ts

@@ -11,6 +11,7 @@ import type { DiscordChannel } from '../types'
 export async function snapshotAction(options: {
   channelsOnly?: boolean
   usersOnly?: boolean
+  full?: boolean
   limit?: number
   pretty?: boolean
 }): Promise<void> {
@@ -25,7 +26,6 @@ export async function snapshotAction(options: {
 
     const client = await new DiscordClient().login({ token: config.token as string })
     const serverId = config.current_server as string
-    const messageLimit = options.limit || 20
 
     const snapshot: Record<string, any> = {}
 
@@ -35,51 +35,65 @@ export async function snapshotAction(options: {
       name: server.name,
     }
 
-    if (!options.usersOnly) {
-      const channels = await client.listChannels(serverId)
+    const isFull = options.full || options.channelsOnly || options.usersOnly
+    if (isFull) {
+      const messageLimit = options.limit || 20
 
-      snapshot.channels = channels.map((ch) => ({
-        id: ch.id,
-        name: ch.name,
-        type: ch.type,
-        topic: ch.topic,
-      }))
+      if (!options.usersOnly) {
+        const channels = await client.listChannels(serverId)
 
-      if (!options.channelsOnly) {
-        const isTextChannel = (ch: DiscordChannel) => ch.type === 0 || ch.type === 5
-        const textChannels = channels.filter(isTextChannel)
-
-        const channelMessages = await parallelMap(
-          textChannels,
-          async (channel: DiscordChannel) => {
-            const messages = await client.getMessages(channel.id, messageLimit)
-            return messages.map((msg) => ({
-              ...msg,
-              channel_name: channel.name,
-            }))
-          },
-          5,
-        )
-
-        snapshot.recent_messages = channelMessages.flat().map((msg) => ({
-          channel_id: msg.channel_id,
-          channel_name: msg.channel_name,
-          id: msg.id,
-          author: msg.author.username,
-          content: msg.content,
-          timestamp: msg.timestamp,
+        snapshot.channels = channels.map((ch) => ({
+          id: ch.id,
+          name: ch.name,
+          type: ch.type,
+          topic: ch.topic,
         }))
+
+        if (!options.channelsOnly) {
+          const isTextChannel = (ch: DiscordChannel) => ch.type === 0 || ch.type === 5
+          const textChannels = channels.filter(isTextChannel)
+
+          const channelMessages = await parallelMap(
+            textChannels,
+            async (channel: DiscordChannel) => {
+              const messages = await client.getMessages(channel.id, messageLimit)
+              return messages.map((msg) => ({
+                ...msg,
+                channel_name: channel.name,
+              }))
+            },
+            5,
+          )
+
+          snapshot.recent_messages = channelMessages.flat().map((msg) => ({
+            channel_id: msg.channel_id,
+            channel_name: msg.channel_name,
+            id: msg.id,
+            author: msg.author.username,
+            content: msg.content,
+            timestamp: msg.timestamp,
+          }))
+        }
       }
-    }
 
-    if (!options.channelsOnly) {
-      const users = await client.listUsers(serverId)
+      if (!options.channelsOnly) {
+        const users = await client.listUsers(serverId)
+
+        snapshot.members = users.map((u) => ({
+          id: u.id,
+          username: u.username,
+          global_name: u.global_name || null,
+        }))
+      }
+    } else {
+      if (!options.usersOnly) {
+        const channels = await client.listChannels(serverId)
+        const textChannels = channels.filter((ch: DiscordChannel) => ch.type === 0 || ch.type === 5)
+        snapshot.channels = textChannels.map((ch) => ({ id: ch.id, name: ch.name }))
+      }
 
-      snapshot.members = users.map((u) => ({
-        id: u.id,
-        username: u.username,
-        global_name: u.global_name || null,
-      }))
+      snapshot.hint =
+        "Use 'message list <channel>' for messages, 'channel info <channel>' for channel details, 'user list' for members."
     }
 
     console.log(formatOutput(snapshot, options.pretty))
@@ -89,14 +103,16 @@ export async function snapshotAction(options: {
 }
 
 export const snapshotCommand = new Command('snapshot')
-  .description('Get comprehensive server state for AI agents')
+  .description('Get server overview for AI agents (brief by default, use --full for comprehensive data)')
+  .option('--full', 'Include messages and members (verbose)')
   .option('--channels-only', 'Include only channels (exclude messages and members)')
   .option('--users-only', 'Include only members (exclude channels and messages)')
-  .option('--limit <n>', 'Number of recent messages per channel (default: 20)', '20')
+  .option('--limit <n>', 'Number of recent messages per channel with --full (default: 20)', '20')
   .action(async (options) => {
     await snapshotAction({
       channelsOnly: options.channelsOnly,
       usersOnly: options.usersOnly,
+      full: options.full,
       limit: parseInt(options.limit, 10),
       pretty: options.pretty,
     })

package/src/platforms/discordbot/commands/snapshot.test.ts

@@ -86,23 +86,17 @@ describe('snapshot command', () => {
     process.env = originalEnv
   })
 
-  describe('default (channels + messages)', () => {
-    test('returns channels with recent messages', async () => {
+  describe('default (brief)', () => {
+    test('returns channels with id and name only', async () => {
       const manager = await setupManager(tempDir)
 
       const result = await snapshotAction({ _credManager: manager })
 
       expect(result.server_id).toBe('guild1')
       expect(result.channels).toHaveLength(2)
-      expect(result.channels?.[0].id).toBe('ch1')
-      expect(result.channels?.[0].name).toBe('general')
-      expect(result.channels?.[0].messages).toHaveLength(2)
-      expect(result.channels?.[0].messages?.[0]).toEqual({
-        id: 'msg1-ch1',
-        author: 'alice',
-        content: 'Hello from ch1',
-        timestamp: '2025-01-01T00:00:00Z',
-      })
+      expect(result.channels?.[0]).toEqual({ id: 'ch1', name: 'general' })
+      expect(result.channels?.[1]).toEqual({ id: 'ch2', name: 'random' })
+      expect(result.hint).toBeDefined()
     })
 
     test('filters out non-text channels', async () => {
@@ -114,11 +108,39 @@ describe('snapshot command', () => {
       expect(channelNames).not.toContain('voice')
     })
 
-    test('fetches messages in parallel for all text channels', async () => {
+    test('does not fetch messages', async () => {
       const manager = await setupManager(tempDir)
 
       await snapshotAction({ _credManager: manager })
 
+      expect(mockGetMessages).not.toHaveBeenCalled()
+    })
+  })
+
+  describe('--full (channels + messages)', () => {
+    test('returns channels with recent messages', async () => {
+      const manager = await setupManager(tempDir)
+
+      const result = await snapshotAction({ _credManager: manager, full: true })
+
+      expect(result.server_id).toBe('guild1')
+      expect(result.channels).toHaveLength(2)
+      expect(result.channels?.[0].id).toBe('ch1')
+      expect(result.channels?.[0].name).toBe('general')
+      expect(result.channels?.[0].messages).toHaveLength(2)
+      expect(result.channels?.[0].messages?.[0]).toEqual({
+        id: 'msg1-ch1',
+        author: 'alice',
+        content: 'Hello from ch1',
+        timestamp: '2025-01-01T00:00:00Z',
+      })
+    })
+
+    test('fetches messages in parallel for all text channels', async () => {
+      const manager = await setupManager(tempDir)
+
+      await snapshotAction({ _credManager: manager, full: true })
+
       expect(mockGetMessages).toHaveBeenCalledTimes(2)
       expect(mockGetMessages).toHaveBeenCalledWith('ch1', 5)
       expect(mockGetMessages).toHaveBeenCalledWith('ch2', 5)
@@ -127,7 +149,7 @@ describe('snapshot command', () => {
     test('respects --limit option', async () => {
       const manager = await setupManager(tempDir)
 
-      await snapshotAction({ _credManager: manager, limit: 10 })
+      await snapshotAction({ _credManager: manager, full: true, limit: 10 })
 
       expect(mockGetMessages).toHaveBeenCalledWith('ch1', 10)
       expect(mockGetMessages).toHaveBeenCalledWith('ch2', 10)
@@ -136,17 +158,17 @@ describe('snapshot command', () => {
     test('defaults limit to 5', async () => {
       const manager = await setupManager(tempDir)
 
-      await snapshotAction({ _credManager: manager })
+      await snapshotAction({ _credManager: manager, full: true })
 
       expect(mockGetMessages).toHaveBeenCalledWith('ch1', 5)
     })
   })
 
-  describe('--channels-only', () => {
+  describe('--full --channels-only', () => {
     test('returns only channel list without messages', async () => {
       const manager = await setupManager(tempDir)
 
-      const result = await snapshotAction({ _credManager: manager, channelsOnly: true })
+      const result = await snapshotAction({ _credManager: manager, full: true, channelsOnly: true })
 
       expect(result.server_id).toBe('guild1')
       expect(result.channels).toHaveLength(2)
@@ -156,11 +178,11 @@ describe('snapshot command', () => {
     })
   })
 
-  describe('--users-only', () => {
+  describe('--full --users-only', () => {
     test('returns only user list', async () => {
       const manager = await setupManager(tempDir)
 
-      const result = await snapshotAction({ _credManager: manager, usersOnly: true })
+      const result = await snapshotAction({ _credManager: manager, full: true, usersOnly: true })
 
       expect(result.server_id).toBe('guild1')
       expect(result.users).toHaveLength(3)

package/src/platforms/discordbot/commands/snapshot.ts

@@ -10,6 +10,7 @@ import { getClient } from './shared'
 interface SnapshotOption extends BotOption {
   channelsOnly?: boolean
   usersOnly?: boolean
+  full?: boolean
   limit?: number
 }
 
@@ -31,6 +32,7 @@ interface SnapshotResult {
     username: string
     global_name: string | null
   }>
+  hint?: string
   error?: string
 }
 
@@ -43,53 +45,66 @@ export async function snapshotAction(options: SnapshotOption): Promise<SnapshotR
     }
 
     const client = await getClient(options)
-    const limit = options.limit ?? 5
 
-    if (options.usersOnly) {
-      const users = await client.listUsers(serverId)
-      return {
-        server_id: serverId,
-        users: users.map((u) => ({
-          id: u.id,
-          username: u.username,
-          global_name: u.global_name ?? null,
-        })),
+    const isFull = options.full || options.channelsOnly || options.usersOnly
+    if (isFull) {
+      const limit = options.limit ?? 5
+
+      if (options.usersOnly) {
+        const users = await client.listUsers(serverId)
+        return {
+          server_id: serverId,
+          users: users.map((u) => ({
+            id: u.id,
+            username: u.username,
+            global_name: u.global_name ?? null,
+          })),
+        }
       }
-    }
 
-    const allChannels = await client.listChannels(serverId)
-    const textChannels = allChannels.filter((ch) => ch.type === 0)
+      const allChannels = await client.listChannels(serverId)
+      const textChannels = allChannels.filter((ch) => ch.type === 0)
+
+      if (options.channelsOnly) {
+        return {
+          server_id: serverId,
+          channels: textChannels.map((ch) => ({
+            id: ch.id,
+            name: ch.name,
+            type: ch.type,
+          })),
+        }
+      }
+
+      const channelsWithMessages = await Promise.all(
+        textChannels.map(async (ch) => {
+          const messages = await client.getMessages(ch.id, limit)
+          return {
+            id: ch.id,
+            name: ch.name,
+            messages: messages.map((msg) => ({
+              id: msg.id,
+              author: msg.author.username,
+              content: msg.content,
+              timestamp: msg.timestamp,
+            })),
+          }
+        }),
+      )
 
-    if (options.channelsOnly) {
       return {
         server_id: serverId,
-        channels: textChannels.map((ch) => ({
-          id: ch.id,
-          name: ch.name,
-          type: ch.type,
-        })),
+        channels: channelsWithMessages,
       }
     }
 
-    const channelsWithMessages = await Promise.all(
-      textChannels.map(async (ch) => {
-        const messages = await client.getMessages(ch.id, limit)
-        return {
-          id: ch.id,
-          name: ch.name,
-          messages: messages.map((msg) => ({
-            id: msg.id,
-            author: msg.author.username,
-            content: msg.content,
-            timestamp: msg.timestamp,
-          })),
-        }
-      }),
-    )
+    const allChannels = await client.listChannels(serverId)
+    const textChannels = allChannels.filter((ch) => ch.type === 0)
 
     return {
       server_id: serverId,
-      channels: channelsWithMessages,
+      channels: textChannels.map((ch) => ({ id: ch.id, name: ch.name })),
+      hint: "Use 'message list <channel>' for messages, 'channel info <channel>' for channel details, 'user list' for members.",
     }
   } catch (error) {
     return { error: (error as Error).message }
@@ -97,10 +112,11 @@ export async function snapshotAction(options: SnapshotOption): Promise<SnapshotR
 }
 
 export const snapshotCommand = new Command('snapshot')
-  .description('Server overview for AI agent context')
+  .description('Server overview for AI agents (brief by default, use --full for comprehensive data)')
+  .option('--full', 'Include messages and members (verbose)')
   .option('--channels-only', 'List channels only, skip messages')
   .option('--users-only', 'List users only')
-  .option('--limit <n>', 'Messages per channel (default: 5)', '5')
+  .option('--limit <n>', 'Messages per channel with --full (default: 5)', '5')
   .option('--server <id>', 'Use specific server')
   .option('--bot <id>', 'Use specific bot')
   .option('--pretty', 'Pretty print JSON output')
@@ -108,6 +124,7 @@ export const snapshotCommand = new Command('snapshot')
     try {
       const result = await snapshotAction({
         ...options,
+        full: options.full,
         limit: parseInt(options.limit, 10),
       })
       console.log(formatOutput(result, options.pretty))

package/src/platforms/slack/commands/snapshot.test.ts

@@ -34,6 +34,12 @@ test('snapshot command has --limit option', () => {
   expect(hasLimit).toBe(true)
 })
 
+test('snapshot command has --full option', () => {
+  const options = snapshotCommand.options
+  const hasFull = options.some((opt: any) => opt.long === '--full')
+  expect(hasFull).toBe(true)
+})
+
 // Test snapshot logic using spyOn (no global mock pollution)
 let credManagerSpy: ReturnType<typeof spyOn>
 let clientTestAuthSpy: ReturnType<typeof spyOn>
@@ -147,11 +153,66 @@ afterEach(() => {
   clientGetMessagesSpy?.mockRestore()
 })
 
+test('brief snapshot (default) returns workspace, channels as {id, name}, and hint', async () => {
+  const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
+
+  const auth = await client.testAuth()
+  const channels = await client.listChannels()
+  const active = channels.filter((ch) => !ch.is_archived)
+
+  const snapshot = {
+    workspace: {
+      id: auth.team_id,
+      name: auth.team,
+    },
+    channels: active.map((ch) => ({ id: ch.id, name: ch.name })),
+    hint: "Use 'message list <channel>' for messages, 'channel info <channel>' for channel details, 'user list' for users, 'usergroup list' for groups.",
+  }
+
+  expect(snapshot.workspace).toBeDefined()
+  expect(snapshot.workspace.id).toBe('T123')
+  expect(snapshot.workspace.name).toBe('Test Workspace')
+
+  expect(snapshot.channels).toBeDefined()
+  expect(snapshot.channels.length).toBe(2)
+  expect(snapshot.channels[0]).toEqual({ id: 'C123', name: 'general' })
+  expect(snapshot.channels[1]).toEqual({ id: 'C456', name: 'random' })
+
+  expect((snapshot as any).recent_messages).toBeUndefined()
+  expect((snapshot as any).users).toBeUndefined()
+  expect((snapshot as any).usergroups).toBeUndefined()
+
+  expect(snapshot.hint).toContain('message list')
+})
+
+test('brief snapshot excludes archived channels', async () => {
+  const channelsWithArchived: SlackChannel[] = [
+    ...mockChannels,
+    {
+      id: 'C789',
+      name: 'old-channel',
+      is_private: false,
+      is_archived: true,
+      created: 1234567800,
+      creator: 'U123',
+      topic: { value: 'Archived', creator: 'U123', last_set: 1234567800 },
+      purpose: { value: 'Old channel', creator: 'U123', last_set: 1234567800 },
+    },
+  ]
+
+  clientListChannelsSpy.mockResolvedValue(channelsWithArchived)
+
+  const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
+  const channels = await client.listChannels()
+  const active = channels.filter((ch) => !ch.is_archived)
+
+  expect(active.length).toBe(2)
+  expect(active.every((ch) => !ch.is_archived)).toBe(true)
+})
+
 test('full snapshot returns workspace, channels, messages, and users', async () => {
-  const credManager = new CredentialManager()
   const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
 
-  const _workspace = await credManager.getWorkspace()
   const auth = await client.testAuth()
   const channels = await client.listChannels()
   const users = await client.listUsers()
@@ -218,10 +279,8 @@ test('full snapshot returns workspace, channels, messages, and users', async ()
 })
 
 test('snapshot with --channels-only excludes messages and users', async () => {
-  const credManager = new CredentialManager()
   const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
 
-  const _workspace = await credManager.getWorkspace()
   const auth = await client.testAuth()
   const channels = await client.listChannels()
 
@@ -249,10 +308,8 @@ test('snapshot with --channels-only excludes messages and users', async () => {
 })
 
 test('snapshot with --users-only excludes channels and messages', async () => {
-  const credManager = new CredentialManager()
   const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
 
-  const _workspace = await credManager.getWorkspace()
   const auth = await client.testAuth()
   const users = await client.listUsers()
 
@@ -279,10 +336,8 @@ test('snapshot with --users-only excludes channels and messages', async () => {
 })
 
 test('snapshot respects --limit option for messages', async () => {
-  const credManager = new CredentialManager()
   const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
 
-  const _workspace = await credManager.getWorkspace()
   const auth = await client.testAuth()
   const channels = await client.listChannels()