agent-messenger 2.10.1 → 2.10.2

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

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 
@@ -17,14 +17,14 @@ describe('Emoji Commands', () => {
   })
 
   describe('emoji list', () => {
-    test('lists custom emoji', async () => {
+    it('lists custom emoji', async () => {
       const emoji = await (mockClient as SlackClient).listEmoji()
       expect(emoji['party_blob']).toBe('https://example.com/party_blob.gif')
       expect(emoji['cool_sunglasses']).toBe('https://example.com/cool_sunglasses.png')
       expect(Object.keys(emoji)).toHaveLength(2)
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.listEmoji = mock(async () => {
         throw new Error('invalid_auth')
       })

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

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import type { SlackFile } from '@/platforms/slack/types'
@@ -72,7 +72,7 @@ describe('File Commands', () => {
   })
 
   describe('file upload', () => {
-    test('uploads file to channel', async () => {
+    it('uploads file to channel', async () => {
       const channel = 'C123'
       const fileBuffer = Buffer.from('test content')
       const filename = 'test.txt'
@@ -85,7 +85,7 @@ describe('File Commands', () => {
       expect(result.channels).toContain(channel)
     })
 
-    test('supports --filename override', async () => {
+    it('supports --filename override', async () => {
       const channel = 'C123'
       const fileBuffer = Buffer.from('content')
       const customFilename = 'custom-name.txt'
@@ -95,7 +95,7 @@ describe('File Commands', () => {
       expect(result.name).toBe(customFilename)
     })
 
-    test('returns file with metadata', () => {
+    it('returns file with metadata', () => {
       const file: SlackFile = {
         id: 'F123',
         name: 'test.txt',
@@ -115,7 +115,7 @@ describe('File Commands', () => {
   })
 
   describe('file list', () => {
-    test('lists all files in workspace', async () => {
+    it('lists all files in workspace', async () => {
       const files = await mockClient.listFiles()
 
       expect(files).toHaveLength(2)
@@ -123,7 +123,7 @@ describe('File Commands', () => {
       expect(files[1].name).toBe('document.pdf')
     })
 
-    test('filters files by channel', async () => {
+    it('filters files by channel', async () => {
       const channel = 'C123'
 
       const files = await mockClient.listFiles(channel)
@@ -132,7 +132,7 @@ describe('File Commands', () => {
       expect(files.length).toBeGreaterThan(0)
     })
 
-    test('returns files with metadata', () => {
+    it('returns files with metadata', () => {
       const files: SlackFile[] = [
         {
           id: 'F123',
@@ -165,7 +165,7 @@ describe('File Commands', () => {
   })
 
   describe('file info', () => {
-    test('shows file details', async () => {
+    it('shows file details', async () => {
       const fileId = 'F123'
 
       const files = await mockClient.listFiles()
@@ -178,7 +178,7 @@ describe('File Commands', () => {
   })
 
   describe('file download', () => {
-    test('downloads file by ID', async () => {
+    it('downloads file by ID', async () => {
       const fileId = 'F123'
       const result = await mockClient.downloadFile(fileId)
 
@@ -187,7 +187,7 @@ describe('File Commands', () => {
       expect(result.buffer.toString()).toBe('downloaded content')
     })
 
-    test('sanitizes filename to prevent path traversal', () => {
+    it('sanitizes filename to prevent path traversal', () => {
       const { basename } = require('node:path')
       const sanitize = (name: string) => basename(name.replace(/\\/g, '/'))
 
@@ -199,7 +199,7 @@ describe('File Commands', () => {
   })
 
   describe('output formatting', () => {
-    test('includes file fields in output', () => {
+    it('includes file fields in output', () => {
       const file: SlackFile = {
         id: 'F123',
         name: 'test.txt',
@@ -217,7 +217,7 @@ describe('File Commands', () => {
       expect(output.name).toBe('test.txt')
     })
 
-    test('formats multiple files', () => {
+    it('formats multiple files', () => {
       const files: SlackFile[] = [
         {
           id: 'F123',

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

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import type { SlackMessage } from '@/platforms/slack/types'
@@ -97,7 +97,7 @@ describe('Message Commands', () => {
   })
 
   describe('message send', () => {
-    test('sends message to channel', async () => {
+    it('sends message to channel', async () => {
       // Given: A channel and message text
       const channel = 'C123'
       const text = 'Hello, world!'
@@ -111,7 +111,7 @@ describe('Message Commands', () => {
       expect(result.type).toBe('message')
     })
 
-    test('sends message to thread', async () => {
+    it('sends message to thread', async () => {
       // Given: A channel, message text, and thread ts
       const channel = 'C123'
       const text = 'Reply in thread'
@@ -127,7 +127,7 @@ describe('Message Commands', () => {
   })
 
   describe('message list', () => {
-    test('lists messages from channel', async () => {
+    it('lists messages from channel', async () => {
       // Given: A channel
       const channel = 'C123'
 
@@ -140,7 +140,7 @@ describe('Message Commands', () => {
       expect(messages[1].text).toBe('Second message')
     })
 
-    test('respects limit parameter', async () => {
+    it('respects limit parameter', async () => {
       // Given: A channel and limit
       const channel = 'C123'
       const limit = 10
@@ -152,7 +152,7 @@ describe('Message Commands', () => {
       expect(messages).toBeDefined()
     })
 
-    test('filters messages by thread', async () => {
+    it('filters messages by thread', async () => {
       // Given: A channel and thread ts
       const channel = 'C123'
       const _threadTs = '1234567890.123456'
@@ -166,7 +166,7 @@ describe('Message Commands', () => {
   })
 
   describe('message update', () => {
-    test('updates message text', async () => {
+    it('updates message text', async () => {
       // Given: A channel, message ts, and new text
       const channel = 'C123'
       const ts = '1234567890.123456'
@@ -182,7 +182,7 @@ describe('Message Commands', () => {
   })
 
   describe('message delete', () => {
-    test('deletes message', async () => {
+    it('deletes message', async () => {
       // Given: A channel and message ts
       const channel = 'C123'
       const ts = '1234567890.123456'
@@ -196,7 +196,7 @@ describe('Message Commands', () => {
   })
 
   describe('message get', () => {
-    test('gets single message by timestamp', async () => {
+    it('gets single message by timestamp', async () => {
       // Given: A channel and message ts
       const channel = 'C123'
       const ts = '1234567890.123456'
@@ -210,7 +210,7 @@ describe('Message Commands', () => {
       expect(message?.text).toBe('Found single message')
     })
 
-    test('returns null for non-existent message', async () => {
+    it('returns null for non-existent message', async () => {
       // Given: A channel and non-existent ts
       const channel = 'C123'
       const ts = '9999999999.999999'
@@ -224,7 +224,7 @@ describe('Message Commands', () => {
   })
 
   describe('message search', () => {
-    test('searches messages across workspace', async () => {
+    it('searches messages across workspace', async () => {
       // Given: A search query
       const query = 'hello world'
 
@@ -238,7 +238,7 @@ describe('Message Commands', () => {
       expect(results[0].permalink).toBeDefined()
     })
 
-    test('returns channel info with each result', async () => {
+    it('returns channel info with each result', async () => {
       // Given: A search query
       const query = 'test'
 
@@ -255,7 +255,7 @@ describe('Message Commands', () => {
   })
 
   describe('message replies', () => {
-    test('gets thread replies including parent message', async () => {
+    it('gets thread replies including parent message', async () => {
       // Given: A channel and thread ts
       const channel = 'C123'
       const threadTs = '1234567890.123456'
@@ -271,7 +271,7 @@ describe('Message Commands', () => {
       expect(result.messages[2].text).toBe('Second reply')
     })
 
-    test('returns has_more flag for pagination', async () => {
+    it('returns has_more flag for pagination', async () => {
       // Given: A channel and thread ts
       const channel = 'C123'
       const threadTs = '1234567890.123456'
@@ -283,7 +283,7 @@ describe('Message Commands', () => {
       expect(result.has_more).toBe(false)
     })
 
-    test('all replies have same thread_ts as parent', async () => {
+    it('all replies have same thread_ts as parent', async () => {
       // Given: A channel and thread ts
       const channel = 'C123'
       const threadTs = '1234567890.123456'
@@ -299,7 +299,7 @@ describe('Message Commands', () => {
   })
 
   describe('output formatting', () => {
-    test('formats message output', () => {
+    it('formats message output', () => {
       // Given: A message
       const message: SlackMessage = {
         ts: '1234567890.123456',
@@ -316,7 +316,7 @@ describe('Message Commands', () => {
       expect(output.text).toBe('Hello')
     })
 
-    test('formats multiple messages', () => {
+    it('formats multiple messages', () => {
       // Given: Multiple messages
       const messages: SlackMessage[] = [
         {
@@ -342,7 +342,7 @@ describe('Message Commands', () => {
       expect(output[1].text).toBe('Second')
     })
 
-    test('formats message with files', () => {
+    it('formats message with files', () => {
       // Given: A message with file attachments
       const message: SlackMessage = {
         ts: '1234567890.123456',
@@ -375,7 +375,7 @@ describe('Message Commands', () => {
       expect(output.files![0].url_private).toContain('files.slack.com')
     })
 
-    test('message without files has undefined files field', () => {
+    it('message without files has undefined files field', () => {
       // Given: A message without file attachments
       const message: SlackMessage = {
         ts: '1234567890.123456',

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

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import type { SlackPin } from '@/platforms/slack/types'
@@ -29,12 +29,12 @@ describe('Pin Commands', () => {
   })
 
   describe('pin add', () => {
-    test('pins a message successfully', async () => {
+    it('pins a message successfully', async () => {
       await (mockClient as SlackClient).pinMessage('C001', '1234567890.123456')
       expect(mockClient.pinMessage).toHaveBeenCalledWith('C001', '1234567890.123456')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.pinMessage = mock(async () => {
         throw new Error('already_pinned')
       })
@@ -43,12 +43,12 @@ describe('Pin Commands', () => {
   })
 
   describe('pin remove', () => {
-    test('unpins a message successfully', async () => {
+    it('unpins a message successfully', async () => {
       await (mockClient as SlackClient).unpinMessage('C001', '1234567890.123456')
       expect(mockClient.unpinMessage).toHaveBeenCalledWith('C001', '1234567890.123456')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.unpinMessage = mock(async () => {
         throw new Error('no_pin')
       })
@@ -57,14 +57,14 @@ describe('Pin Commands', () => {
   })
 
   describe('pin list', () => {
-    test('lists pinned messages', async () => {
+    it('lists pinned messages', async () => {
       const pins = await (mockClient as SlackClient).listPins('C001')
       expect(pins).toHaveLength(1)
       expect(pins[0].message.text).toBe('Pinned message')
       expect(pins[0].created_by).toBe('U001')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.listPins = mock(async () => {
         throw new Error('channel_not_found')
       })

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

@@ -1,4 +1,4 @@
-import { describe, expect, mock, test } from 'bun:test'
+import { describe, expect, mock, it } from 'bun:test'
 
 import { Command } from 'commander'
 
@@ -6,7 +6,7 @@ import { reactionCommand } from '@/platforms/slack/commands/reaction'
 
 describe('reaction command', () => {
   describe('add subcommand', () => {
-    test('adds reaction to message with channel and message ts', async () => {
+    it('adds reaction to message with channel and message ts', async () => {
       const channelId = 'C123'
       const messageTs = '1234567890.123456'
       const emoji = 'thumbsup'
@@ -18,7 +18,7 @@ describe('reaction command', () => {
       expect(mockAddReaction).toHaveBeenCalledTimes(1)
     })
 
-    test('handles emoji without colons', async () => {
+    it('handles emoji without colons', async () => {
       const channelId = 'C123'
       const messageTs = '1234567890.123456'
       const emoji = 'thumbsup'
@@ -31,7 +31,7 @@ describe('reaction command', () => {
   })
 
   describe('remove subcommand', () => {
-    test('removes reaction from message with channel and message ts', async () => {
+    it('removes reaction from message with channel and message ts', async () => {
       const channelId = 'C123'
       const messageTs = '1234567890.123456'
       const emoji = 'thumbsup'
@@ -45,7 +45,7 @@ describe('reaction command', () => {
   })
 
   describe('list subcommand', () => {
-    test('lists reactions on a message', async () => {
+    it('lists reactions on a message', async () => {
       const messageTs = '1234567890.123456'
       const mockMessage = {
         ts: messageTs,
@@ -66,7 +66,7 @@ describe('reaction command', () => {
       expect(result!.reactions).toHaveLength(2)
     })
 
-    test('returns empty reactions list when no reactions', async () => {
+    it('returns empty reactions list when no reactions', async () => {
       const messageTs = '1234567890.123456'
       const mockMessage = {
         ts: messageTs,
@@ -83,7 +83,7 @@ describe('reaction command', () => {
       expect(result!.reactions).toEqual([])
     })
 
-    test('returns null when message not found', async () => {
+    it('returns null when message not found', async () => {
       const mockGetMessage = mock(() => Promise.resolve(null))
       const result = await mockGetMessage()
 
@@ -92,15 +92,15 @@ describe('reaction command', () => {
   })
 
   describe('command structure', () => {
-    test('reaction command exists', () => {
+    it('reaction command exists', () => {
       expect(reactionCommand).toBeInstanceOf(Command)
     })
 
-    test('reaction command has correct name', () => {
+    it('reaction command has correct name', () => {
       expect(reactionCommand.name()).toBe('reaction')
     })
 
-    test('reaction command has description', () => {
+    it('reaction command has description', () => {
       expect(reactionCommand.description()).toBe('Reaction commands')
     })
   })

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

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import type { SlackReminder } from '@/platforms/slack/types'
@@ -26,13 +26,13 @@ describe('Reminder Commands', () => {
   })
 
   describe('reminder add', () => {
-    test('adds a reminder successfully', async () => {
+    it('adds a reminder successfully', async () => {
       const result = await (mockClient as SlackClient).addReminder('Do something important', 1700000000)
       expect(result.id).toBe('Rm001')
       expect(result.text).toBe('Do something important')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.addReminder = mock(async () => {
         throw new Error('invalid_time')
       })
@@ -41,14 +41,14 @@ describe('Reminder Commands', () => {
   })
 
   describe('reminder list', () => {
-    test('lists all reminders', async () => {
+    it('lists all reminders', async () => {
       const reminders = await (mockClient as SlackClient).listReminders()
       expect(reminders).toHaveLength(1)
       expect(reminders[0].text).toBe('Do something important')
       expect(reminders[0].recurring).toBe(false)
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.listReminders = mock(async () => {
         throw new Error('invalid_auth')
       })
@@ -57,12 +57,12 @@ describe('Reminder Commands', () => {
   })
 
   describe('reminder complete', () => {
-    test('completes a reminder successfully', async () => {
+    it('completes a reminder successfully', async () => {
       await (mockClient as SlackClient).completeReminder('Rm001')
       expect(mockClient.completeReminder).toHaveBeenCalledWith('Rm001')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.completeReminder = mock(async () => {
         throw new Error('reminder_not_found')
       })
@@ -71,12 +71,12 @@ describe('Reminder Commands', () => {
   })
 
   describe('reminder delete', () => {
-    test('deletes a reminder successfully', async () => {
+    it('deletes a reminder successfully', async () => {
       await (mockClient as SlackClient).deleteReminder('Rm001')
       expect(mockClient.deleteReminder).toHaveBeenCalledWith('Rm001')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.deleteReminder = mock(async () => {
         throw new Error('reminder_not_found')
       })

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

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import type { SlackSavedItem } from '@/platforms/slack/types'
@@ -73,7 +73,7 @@ describe('Saved Commands', () => {
   })
 
   describe('saved list', () => {
-    test('lists saved items', async () => {
+    it('lists saved items', async () => {
       // Given: SlackClient returns saved items
       // When: Listing saved items
       const result = await mockClient.getSavedItems()
@@ -86,7 +86,7 @@ describe('Saved Commands', () => {
       expect(result.next_cursor).toBe('next_page')
     })
 
-    test('supports pagination with cursor', async () => {
+    it('supports pagination with cursor', async () => {
       // Given: SlackClient returns paginated results
       // When: Fetching with cursor
       const result = await mockClient.getSavedItems('next_page')
@@ -97,7 +97,7 @@ describe('Saved Commands', () => {
       expect(result.has_more).toBe(false)
     })
 
-    test('returns channel information with saved items', async () => {
+    it('returns channel information with saved items', async () => {
       // Given: SlackClient returns saved items with channel info
       // When: Listing saved items
       const result = await mockClient.getSavedItems()
@@ -109,7 +109,7 @@ describe('Saved Commands', () => {
       expect(result.items[1].channel.name).toBe('random')
     })
 
-    test('includes date_created timestamp', async () => {
+    it('includes date_created timestamp', async () => {
       // Given: SlackClient returns saved items
       // When: Listing saved items
       const result = await mockClient.getSavedItems()
@@ -121,14 +121,14 @@ describe('Saved Commands', () => {
   })
 
   describe('CLI command: saved list', () => {
-    test('supports --limit option', async () => {
+    it('supports --limit option', async () => {
       // Given: CLI command with --limit 50
       // When: Executing saved list --limit 50
       // Then: Should pass limit to API (tested via integration)
       expect(true).toBe(true)
     })
 
-    test('supports --cursor option for pagination', async () => {
+    it('supports --cursor option for pagination', async () => {
       // Given: CLI command with --cursor next_page
       // When: Executing saved list --cursor next_page
       const result = await mockClient.getSavedItems('next_page')

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

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 
@@ -35,7 +35,7 @@ describe('Sections Commands', () => {
   })
 
   describe('sections list', () => {
-    test('lists all channel sections', async () => {
+    it('lists all channel sections', async () => {
       // Given: SlackClient returns channel sections
       // When: Listing sections
       const sections = await mockClient.getChannelSections()
@@ -47,7 +47,7 @@ describe('Sections Commands', () => {
       expect(sections[2].name).toBe('Archive')
     })
 
-    test('returns sections with channel IDs', async () => {
+    it('returns sections with channel IDs', async () => {
       // Given: SlackClient returns sections with channels
       // When: Listing sections
       const sections = await mockClient.getChannelSections()
@@ -58,7 +58,7 @@ describe('Sections Commands', () => {
       expect(sections[1].channel_ids).toHaveLength(2)
     })
 
-    test('returns sections with timestamps', async () => {
+    it('returns sections with timestamps', async () => {
       // Given: SlackClient returns sections
       // When: Listing sections
       const sections = await mockClient.getChannelSections()
@@ -68,7 +68,7 @@ describe('Sections Commands', () => {
       expect(sections[0].date_updated).toBe(1234567900)
     })
 
-    test('handles empty sections', async () => {
+    it('handles empty sections', async () => {
       // Given: SlackClient returns section with no channels
       // When: Listing sections
       const sections = await mockClient.getChannelSections()