agent-messenger 1.3.0 → 1.3.2

package/src/platforms/slack/commands/file.ts

@@ -1,27 +1,22 @@
 import { readFileSync } from 'node:fs'
 import { resolve } from 'node:path'
 import { Command } from 'commander'
-import { handleError } from '../../../shared/utils/error-handler'
-import { formatOutput } from '../../../shared/utils/output'
+import { handleError } from '@/shared/utils/error-handler'
+import { formatOutput } from '@/shared/utils/output'
 import { SlackClient } from '../client'
 import { CredentialManager } from '../credential-manager'
 
 async function uploadAction(
   channel: string,
   path: string,
-  options: { filename?: string; pretty?: boolean }
+  options: { filename?: string; pretty?: boolean },
 ): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -57,12 +52,7 @@ async function listAction(options: { channel?: string; pretty?: boolean }): Prom
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -93,12 +83,7 @@ async function infoAction(fileId: string, options: { pretty?: boolean }): Promis
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -137,17 +122,12 @@ export const fileCommand = new Command('file')
       .argument('<channel>', 'channel ID or name')
       .argument('<path>', 'file path')
       .option('--filename <name>', 'override filename')
-      .action(uploadAction)
+      .action(uploadAction),
   )
   .addCommand(
     new Command('list')
       .description('list files in workspace')
       .option('--channel <id>', 'filter by channel')
-      .action(listAction)
-  )
-  .addCommand(
-    new Command('info')
-      .description('show file details')
-      .argument('<file>', 'file ID')
-      .action(infoAction)
+      .action(listAction),
   )
+  .addCommand(new Command('info').description('show file details').argument('<file>', 'file ID').action(infoAction))

package/src/platforms/slack/commands/message.ts

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
-import { handleError } from '../../../shared/utils/error-handler'
-import { formatOutput } from '../../../shared/utils/output'
+import { handleError } from '@/shared/utils/error-handler'
+import { formatOutput } from '@/shared/utils/output'
 import { SlackClient } from '../client'
 import { CredentialManager } from '../credential-manager'
 import type { SlackMessage } from '../types'
@@ -8,19 +8,14 @@ import type { SlackMessage } from '../types'
 async function sendAction(
   channel: string,
   text: string,
-  options: { thread?: string; pretty?: boolean }
+  options: { thread?: string; pretty?: boolean },
 ): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -43,19 +38,14 @@ async function sendAction(
 
 async function listAction(
   channel: string,
-  options: { limit?: number; thread?: string; pretty?: boolean }
+  options: { limit?: number; thread?: string; pretty?: boolean },
 ): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -80,22 +70,13 @@ async function listAction(
   }
 }
 
-async function getAction(
-  channel: string,
-  ts: string,
-  options: { pretty?: boolean }
-): Promise<void> {
+async function getAction(channel: string, ts: string, options: { pretty?: boolean }): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -124,23 +105,13 @@ async function getAction(
   }
 }
 
-async function updateAction(
-  channel: string,
-  ts: string,
-  text: string,
-  options: { pretty?: boolean }
-): Promise<void> {
+async function updateAction(channel: string, ts: string, text: string, options: { pretty?: boolean }): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -163,19 +134,14 @@ async function updateAction(
 async function deleteAction(
   channel: string,
   ts: string,
-  options: { force?: boolean; pretty?: boolean }
+  options: { force?: boolean; pretty?: boolean },
 ): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -195,19 +161,14 @@ async function deleteAction(
 
 async function searchAction(
   query: string,
-  options: { sort?: string; sortDir?: string; limit?: number; pretty?: boolean }
+  options: { sort?: string; sortDir?: string; limit?: number; pretty?: boolean },
 ): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -237,19 +198,14 @@ async function searchAction(
 async function repliesAction(
   channel: string,
   threadTs: string,
-  options: { limit?: number; oldest?: string; latest?: string; cursor?: string; pretty?: boolean }
+  options: { limit?: number; oldest?: string; latest?: string; cursor?: string; pretty?: boolean },
 ): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -287,7 +243,7 @@ export const messageCommand = new Command('message')
       .argument('<text>', 'Message text')
       .option('--thread <ts>', 'Thread timestamp for replies')
       .option('--pretty', 'Pretty print JSON output')
-      .action(sendAction)
+      .action(sendAction),
   )
   .addCommand(
     new Command('list')
@@ -302,7 +258,7 @@ export const messageCommand = new Command('message')
           thread: options.thread,
           pretty: options.pretty,
         })
-      })
+      }),
   )
   .addCommand(
     new Command('get')
@@ -310,7 +266,7 @@ export const messageCommand = new Command('message')
       .argument('<channel>', 'Channel ID or name')
       .argument('<ts>', 'Message timestamp')
       .option('--pretty', 'Pretty print JSON output')
-      .action(getAction)
+      .action(getAction),
   )
   .addCommand(
     new Command('update')
@@ -319,7 +275,7 @@ export const messageCommand = new Command('message')
       .argument('<ts>', 'Message timestamp')
       .argument('<text>', 'New message text')
       .option('--pretty', 'Pretty print JSON output')
-      .action(updateAction)
+      .action(updateAction),
   )
   .addCommand(
     new Command('delete')
@@ -328,7 +284,7 @@ export const messageCommand = new Command('message')
       .argument('<ts>', 'Message timestamp')
       .option('--force', 'Skip confirmation')
       .option('--pretty', 'Pretty print JSON output')
-      .action(deleteAction)
+      .action(deleteAction),
   )
   .addCommand(
     new Command('search')
@@ -345,7 +301,7 @@ export const messageCommand = new Command('message')
           limit: parseInt(options.limit, 10),
           pretty: options.pretty,
         })
-      })
+      }),
   )
   .addCommand(
     new Command('replies')
@@ -365,5 +321,5 @@ export const messageCommand = new Command('message')
           cursor: options.cursor,
           pretty: options.pretty,
         })
-      })
+      }),
   )

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

@@ -45,43 +45,47 @@ describe('reaction command', () => {
   describe('list subcommand', () => {
     test('lists reactions on a message', async () => {
       const messageTs = '1234567890.123456'
-      const mockMessages = [
-        {
-          ts: messageTs,
-          text: 'Hello',
-          type: 'message',
-          user: 'U456',
-          reactions: [
-            { name: 'thumbsup', count: 2, users: ['U123', 'U456'] },
-            { name: 'heart', count: 1, users: ['U789'] },
-          ],
-        },
-      ]
-
-      const mockGetMessages = mock(() => Promise.resolve(mockMessages))
-      const result = await mockGetMessages()
-
-      expect(result).toHaveLength(1)
-      expect(result[0].ts).toBe(messageTs)
+      const mockMessage = {
+        ts: messageTs,
+        text: 'Hello',
+        type: 'message',
+        user: 'U456',
+        reactions: [
+          { name: 'thumbsup', count: 2, users: ['U123', 'U456'] },
+          { name: 'heart', count: 1, users: ['U789'] },
+        ],
+      }
+
+      const mockGetMessage = mock(() => Promise.resolve(mockMessage))
+      const result = await mockGetMessage()
+
+      expect(result).not.toBeNull()
+      expect(result!.ts).toBe(messageTs)
+      expect(result!.reactions).toHaveLength(2)
     })
 
     test('returns empty reactions list when no reactions', async () => {
       const messageTs = '1234567890.123456'
-      const mockMessages = [
-        {
-          ts: messageTs,
-          text: 'Hello',
-          type: 'message',
-          user: 'U456',
-          reactions: [],
-        },
-      ]
-
-      const mockGetMessages = mock(() => Promise.resolve(mockMessages))
-      const result = await mockGetMessages()
-
-      expect(result).toHaveLength(1)
-      expect(result[0].reactions).toEqual([])
+      const mockMessage = {
+        ts: messageTs,
+        text: 'Hello',
+        type: 'message',
+        user: 'U456',
+        reactions: [],
+      }
+
+      const mockGetMessage = mock(() => Promise.resolve(mockMessage))
+      const result = await mockGetMessage()
+
+      expect(result).not.toBeNull()
+      expect(result!.reactions).toEqual([])
+    })
+
+    test('returns null when message not found', async () => {
+      const mockGetMessage = mock(() => Promise.resolve(null))
+      const result = await mockGetMessage()
+
+      expect(result).toBeNull()
     })
   })
 

package/src/platforms/slack/commands/reaction.ts

@@ -1,26 +1,16 @@
 import { Command } from 'commander'
-import { handleError } from '../../../shared/utils/error-handler'
-import { formatOutput } from '../../../shared/utils/output'
+import { handleError } from '@/shared/utils/error-handler'
+import { formatOutput } from '@/shared/utils/output'
 import { SlackClient } from '../client'
 import { CredentialManager } from '../credential-manager'
 
-async function addAction(
-  channel: string,
-  ts: string,
-  emoji: string,
-  options: { pretty?: boolean }
-): Promise<void> {
+async function addAction(channel: string, ts: string, emoji: string, options: { pretty?: boolean }): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const ws = await credManager.getWorkspace()
 
     if (!ws) {
-      console.log(
-        formatOutput(
-          { error: 'No workspace configured. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No workspace configured. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -35,31 +25,21 @@ async function addAction(
           ts,
           emoji,
         },
-        options.pretty
-      )
+        options.pretty,
+      ),
     )
   } catch (error) {
     handleError(error as Error)
   }
 }
 
-async function removeAction(
-  channel: string,
-  ts: string,
-  emoji: string,
-  options: { pretty?: boolean }
-): Promise<void> {
+async function removeAction(channel: string, ts: string, emoji: string, options: { pretty?: boolean }): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const ws = await credManager.getWorkspace()
 
     if (!ws) {
-      console.log(
-        formatOutput(
-          { error: 'No workspace configured. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No workspace configured. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -74,36 +54,26 @@ async function removeAction(
           ts,
           emoji,
         },
-        options.pretty
-      )
+        options.pretty,
+      ),
     )
   } catch (error) {
     handleError(error as Error)
   }
 }
 
-async function listAction(
-  channel: string,
-  ts: string,
-  options: { pretty?: boolean }
-): Promise<void> {
+async function listAction(channel: string, ts: string, options: { pretty?: boolean }): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const ws = await credManager.getWorkspace()
 
     if (!ws) {
-      console.log(
-        formatOutput(
-          { error: 'No workspace configured. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No workspace configured. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
     const client = new SlackClient(ws.token, ws.cookie)
-    const messages = await client.getMessages(channel, 1)
-    const message = messages.find((m) => m.ts === ts)
+    const message = await client.getMessage(channel, ts)
 
     if (!message) {
       console.log(
@@ -113,13 +83,13 @@ async function listAction(
             channel,
             ts,
           },
-          options.pretty
-        )
+          options.pretty,
+        ),
       )
       process.exit(1)
     }
 
-    const reactions = (message as any).reactions || []
+    const reactions = message.reactions || []
 
     console.log(
       formatOutput(
@@ -128,8 +98,8 @@ async function listAction(
           ts,
           reactions,
         },
-        options.pretty
-      )
+        options.pretty,
+      ),
     )
   } catch (error) {
     handleError(error as Error)
@@ -145,7 +115,7 @@ export const reactionCommand = new Command('reaction')
       .argument('<ts>', 'Message timestamp')
       .argument('<emoji>', 'Emoji name (without colons)')
       .option('--pretty', 'Pretty print JSON output')
-      .action(addAction)
+      .action(addAction),
   )
   .addCommand(
     new Command('remove')
@@ -154,7 +124,7 @@ export const reactionCommand = new Command('reaction')
       .argument('<ts>', 'Message timestamp')
       .argument('<emoji>', 'Emoji name (without colons)')
       .option('--pretty', 'Pretty print JSON output')
-      .action(removeAction)
+      .action(removeAction),
   )
   .addCommand(
     new Command('list')
@@ -162,5 +132,5 @@ export const reactionCommand = new Command('reaction')
       .argument('<channel>', 'Channel ID or name')
       .argument('<ts>', 'Message timestamp')
       .option('--pretty', 'Pretty print JSON output')
-      .action(listAction)
+      .action(listAction),
   )

package/src/platforms/slack/commands/saved.ts

@@ -1,25 +1,16 @@
 import { Command } from 'commander'
-import { handleError } from '../../../shared/utils/error-handler'
-import { formatOutput } from '../../../shared/utils/output'
+import { handleError } from '@/shared/utils/error-handler'
+import { formatOutput } from '@/shared/utils/output'
 import { SlackClient } from '../client'
 import { CredentialManager } from '../credential-manager'
 
-async function listAction(options: {
-  limit?: number
-  cursor?: string
-  pretty?: boolean
-}): Promise<void> {
+async function listAction(options: { limit?: number; cursor?: string; pretty?: boolean }): Promise<void> {
   try {
     const credManager = new CredentialManager()
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -67,5 +58,5 @@ export const savedCommand = new Command('saved').description('Saved items comman
         cursor: options.cursor,
         pretty: options.pretty,
       })
-    })
+    }),
 )

package/src/platforms/slack/commands/sections.ts

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
-import { handleError } from '../../../shared/utils/error-handler'
-import { formatOutput } from '../../../shared/utils/output'
+import { handleError } from '@/shared/utils/error-handler'
+import { formatOutput } from '@/shared/utils/output'
 import { SlackClient } from '../client'
 import { CredentialManager } from '../credential-manager'
 
@@ -10,12 +10,7 @@ async function listAction(options: { pretty?: boolean }): Promise<void> {
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -37,9 +32,7 @@ async function listAction(options: { pretty?: boolean }): Promise<void> {
   }
 }
 
-const sections = new Command('sections').description(
-  'Manage Slack channel sections (sidebar folders)'
-)
+const sections = new Command('sections').description('Manage Slack channel sections (sidebar folders)')
 
 sections
   .command('list')

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

@@ -132,9 +132,7 @@ beforeEach(() => {
     team: 'Test Workspace',
   })
 
-  clientListChannelsSpy = spyOn(SlackClient.prototype, 'listChannels').mockResolvedValue(
-    mockChannels
-  )
+  clientListChannelsSpy = spyOn(SlackClient.prototype, 'listChannels').mockResolvedValue(mockChannels)
   clientListUsersSpy = spyOn(SlackClient.prototype, 'listUsers').mockResolvedValue(mockUsers)
   clientGetMessagesSpy = spyOn(SlackClient.prototype, 'getMessages').mockResolvedValue(mockMessages)
 })

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

@@ -1,7 +1,7 @@
 import { Command } from 'commander'
-import { parallelMap } from '../../../shared/utils/concurrency'
-import { handleError } from '../../../shared/utils/error-handler'
-import { formatOutput } from '../../../shared/utils/output'
+import { parallelMap } from '@/shared/utils/concurrency'
+import { handleError } from '@/shared/utils/error-handler'
+import { formatOutput } from '@/shared/utils/output'
 import { SlackClient } from '../client'
 import { CredentialManager } from '../credential-manager'
 import type { SlackChannel } from '../types'
@@ -17,12 +17,7 @@ async function snapshotAction(options: {
     const workspace = await credManager.getWorkspace()
 
     if (!workspace) {
-      console.log(
-        formatOutput(
-          { error: 'No current workspace set. Run "auth extract" first.' },
-          options.pretty
-        )
-      )
+      console.log(formatOutput({ error: 'No current workspace set. Run "auth extract" first.' }, options.pretty))
       process.exit(1)
     }
 
@@ -64,7 +59,7 @@ async function snapshotAction(options: {
               channel_name: channel.name,
             }))
           },
-          5
+          5,
         )
 
         snapshot.recent_messages = channelMessages.flat().map((msg) => ({