agent-messenger 2.10.1 → 2.10.2

package/src/platforms/slackbot/client.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 { SlackBotClient } from './client'
 import { SlackBotError } from './types'
@@ -123,30 +123,30 @@ describe('SlackBotClient', () => {
   })
 
   describe('login', () => {
-    test('accepts bot tokens (xoxb-)', async () => {
+    it('accepts bot tokens (xoxb-)', async () => {
       // when/then: should not throw
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
       expect(client).toBeDefined()
     })
 
-    test('rejects user tokens (xoxp-)', async () => {
+    it('rejects user tokens (xoxp-)', async () => {
       // when/then
       await expect(new SlackBotClient().login({ token: 'xoxp-user-token' })).rejects.toThrow(SlackBotError)
     })
 
-    test('rejects empty token', async () => {
+    it('rejects empty token', async () => {
       // when/then
       await expect(new SlackBotClient().login({ token: '' })).rejects.toThrow(SlackBotError)
     })
 
-    test('rejects non-bot tokens', async () => {
+    it('rejects non-bot tokens', async () => {
       // when/then
       await expect(new SlackBotClient().login({ token: 'invalid-token' })).rejects.toThrow(SlackBotError)
     })
   })
 
   describe('testAuth', () => {
-    test('returns auth info for valid token', async () => {
+    it('returns auth info for valid token', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -161,7 +161,7 @@ describe('SlackBotClient', () => {
   })
 
   describe('postMessage', () => {
-    test('sends message and returns timestamp', async () => {
+    it('sends message and returns timestamp', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -175,7 +175,7 @@ describe('SlackBotClient', () => {
   })
 
   describe('getConversationHistory', () => {
-    test('returns messages', async () => {
+    it('returns messages', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -189,7 +189,7 @@ describe('SlackBotClient', () => {
   })
 
   describe('getMessage', () => {
-    test('returns single message', async () => {
+    it('returns single message', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -203,7 +203,7 @@ describe('SlackBotClient', () => {
   })
 
   describe('addReaction', () => {
-    test('adds reaction to message', async () => {
+    it('adds reaction to message', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -214,7 +214,7 @@ describe('SlackBotClient', () => {
   })
 
   describe('removeReaction', () => {
-    test('removes reaction from message', async () => {
+    it('removes reaction from message', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -225,7 +225,7 @@ describe('SlackBotClient', () => {
   })
 
   describe('listChannels', () => {
-    test('returns list of channels', async () => {
+    it('returns list of channels', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -240,7 +240,7 @@ describe('SlackBotClient', () => {
   })
 
   describe('getChannelInfo', () => {
-    test('returns channel details', async () => {
+    it('returns channel details', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -254,7 +254,7 @@ describe('SlackBotClient', () => {
   })
 
   describe('resolveChannel', () => {
-    test('returns channel ID unchanged when it starts with C', async () => {
+    it('returns channel ID unchanged when it starts with C', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -266,7 +266,7 @@ describe('SlackBotClient', () => {
       expect(mockConversations.list).not.toHaveBeenCalled()
     })
 
-    test('returns channel ID unchanged when it starts with D', async () => {
+    it('returns channel ID unchanged when it starts with D', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -278,7 +278,7 @@ describe('SlackBotClient', () => {
       expect(mockConversations.list).not.toHaveBeenCalled()
     })
 
-    test('returns channel ID unchanged when it starts with G', async () => {
+    it('returns channel ID unchanged when it starts with G', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -290,7 +290,7 @@ describe('SlackBotClient', () => {
       expect(mockConversations.list).not.toHaveBeenCalled()
     })
 
-    test('resolves channel name to ID', async () => {
+    it('resolves channel name to ID', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -302,7 +302,7 @@ describe('SlackBotClient', () => {
       expect(mockConversations.list).toHaveBeenCalled()
     })
 
-    test('strips leading # from channel name', async () => {
+    it('strips leading # from channel name', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -314,7 +314,7 @@ describe('SlackBotClient', () => {
       expect(mockConversations.list).toHaveBeenCalled()
     })
 
-    test('returns channel ID unchanged when input is #C prefixed ID', async () => {
+    it('returns channel ID unchanged when input is #C prefixed ID', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -325,7 +325,7 @@ describe('SlackBotClient', () => {
       expect(channel).toBe('C123ABC')
     })
 
-    test('throws channel_not_found error when name is not found', async () => {
+    it('throws channel_not_found error when name is not found', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -341,7 +341,7 @@ describe('SlackBotClient', () => {
   })
 
   describe('listUsers', () => {
-    test('returns list of users', async () => {
+    it('returns list of users', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -355,7 +355,7 @@ describe('SlackBotClient', () => {
   })
 
   describe('getUserInfo', () => {
-    test('returns user details', async () => {
+    it('returns user details', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 

package/src/platforms/slackbot/commands/auth.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { mkdir } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
@@ -46,7 +46,7 @@ describe('auth commands', () => {
   })
 
   describe('setAction', () => {
-    test('validates and stores bot token with default bot_id from auth', async () => {
+    it('validates and stores bot token with default bot_id from auth', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
 
       const result = await setAction('xoxb-test-token', { _credManager: manager })
@@ -60,7 +60,7 @@ describe('auth commands', () => {
       expect(creds?.bot_id).toBe('B789')
     })
 
-    test('uses --bot flag as bot_id', async () => {
+    it('uses --bot flag as bot_id', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
 
       const result = await setAction('xoxb-test-token', { bot: 'deploy', _credManager: manager })
@@ -70,7 +70,7 @@ describe('auth commands', () => {
       expect(creds?.token).toBe('xoxb-test-token')
     })
 
-    test('rejects user tokens', async () => {
+    it('rejects user tokens', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
 
       const result = await setAction('xoxp-user-token', { _credManager: manager })
@@ -79,7 +79,7 @@ describe('auth commands', () => {
       expect(result.error).toContain('bot token')
     })
 
-    test('rejects invalid token format', async () => {
+    it('rejects invalid token format', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
 
       const result = await setAction('invalid-token', { _credManager: manager })
@@ -89,7 +89,7 @@ describe('auth commands', () => {
   })
 
   describe('clearAction', () => {
-    test('removes all stored credentials', async () => {
+    it('removes all stored credentials', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'xoxb-token',
@@ -107,7 +107,7 @@ describe('auth commands', () => {
   })
 
   describe('statusAction', () => {
-    test('returns no credentials when none set', async () => {
+    it('returns no credentials when none set', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
 
       const result = await statusAction({ _credManager: manager })
@@ -116,7 +116,7 @@ describe('auth commands', () => {
       expect(result.error).toBeDefined()
     })
 
-    test('returns valid status for current bot', async () => {
+    it('returns valid status for current bot', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'xoxb-token',
@@ -133,7 +133,7 @@ describe('auth commands', () => {
       expect(result.bot_id).toBe('mybot')
     })
 
-    test('returns status for specific --bot', async () => {
+    it('returns status for specific --bot', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'xoxb-token',
@@ -158,7 +158,7 @@ describe('auth commands', () => {
   })
 
   describe('listAction', () => {
-    test('returns all stored bots', async () => {
+    it('returns all stored bots', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'xoxb-a',
@@ -183,7 +183,7 @@ describe('auth commands', () => {
   })
 
   describe('useAction', () => {
-    test('switches current bot', async () => {
+    it('switches current bot', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'xoxb-a',
@@ -206,7 +206,7 @@ describe('auth commands', () => {
       expect(result.bot_id).toBe('deploy')
     })
 
-    test('returns error for unknown bot', async () => {
+    it('returns error for unknown bot', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
 
       const result = await useAction('nonexistent', { _credManager: manager })
@@ -216,7 +216,7 @@ describe('auth commands', () => {
   })
 
   describe('removeAction', () => {
-    test('removes a stored bot', async () => {
+    it('removes a stored bot', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'xoxb-a',
@@ -232,7 +232,7 @@ describe('auth commands', () => {
       expect(await manager.getCredentials('deploy')).toBeNull()
     })
 
-    test('returns error for unknown bot', async () => {
+    it('returns error for unknown bot', async () => {
       const manager = new SlackBotCredentialManager(tempDir)
 
       const result = await removeAction('nonexistent', { _credManager: manager })

package/src/platforms/slackbot/commands/channel.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 const mockResolveChannel = mock((_channel: string) => Promise.resolve('C123456'))
 const mockListChannels = mock(() =>
@@ -55,7 +55,7 @@ describe('channel commands', () => {
   })
 
   describe('listChannels', () => {
-    test('returns list of channels', async () => {
+    it('returns list of channels', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -68,7 +68,7 @@ describe('channel commands', () => {
       expect(channels[1].name).toBe('random')
     })
 
-    test('includes channel metadata', async () => {
+    it('includes channel metadata', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -84,7 +84,7 @@ describe('channel commands', () => {
       expect(channel.created).toBeDefined()
     })
 
-    test('passes limit option', async () => {
+    it('passes limit option', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -97,7 +97,7 @@ describe('channel commands', () => {
   })
 
   describe('getChannelInfo', () => {
-    test('returns channel details', async () => {
+    it('returns channel details', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -110,7 +110,7 @@ describe('channel commands', () => {
       expect(channel.is_private).toBe(false)
     })
 
-    test('includes topic and purpose', async () => {
+    it('includes topic and purpose', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -124,7 +124,7 @@ describe('channel commands', () => {
   })
 
   describe('resolveChannel', () => {
-    test('resolves channel name to ID', async () => {
+    it('resolves channel name to ID', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 

package/src/platforms/slackbot/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'
 
 const mockResolveChannel = mock((_channel: string) => Promise.resolve('C123456'))
 const mockPostMessage = mock(() =>
@@ -68,7 +68,7 @@ describe('message commands', () => {
   })
 
   describe('postMessage', () => {
-    test('sends message to channel', async () => {
+    it('sends message to channel', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -81,7 +81,7 @@ describe('message commands', () => {
       expect(result.type).toBe('message')
     })
 
-    test('sends message with thread_ts', async () => {
+    it('sends message with thread_ts', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -94,7 +94,7 @@ describe('message commands', () => {
   })
 
   describe('getConversationHistory', () => {
-    test('returns messages from channel', async () => {
+    it('returns messages from channel', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -107,7 +107,7 @@ describe('message commands', () => {
       expect(messages[1].text).toBe('World')
     })
 
-    test('passes limit option', async () => {
+    it('passes limit option', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -120,7 +120,7 @@ describe('message commands', () => {
   })
 
   describe('getMessage', () => {
-    test('returns single message by ts', async () => {
+    it('returns single message by ts', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -133,7 +133,7 @@ describe('message commands', () => {
       expect(message?.text).toBe('Hello world')
     })
 
-    test('returns null when message not found', async () => {
+    it('returns null when message not found', async () => {
       // given
       mockGetMessage.mockResolvedValueOnce(null)
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
@@ -147,7 +147,7 @@ describe('message commands', () => {
   })
 
   describe('updateMessage', () => {
-    test('updates message text', async () => {
+    it('updates message text', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -159,7 +159,7 @@ describe('message commands', () => {
       expect(result.text).toBe('Updated text')
     })
 
-    test('called with correct arguments', async () => {
+    it('passes correct arguments', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -172,7 +172,7 @@ describe('message commands', () => {
   })
 
   describe('deleteMessage', () => {
-    test('deletes message by ts', async () => {
+    it('deletes message by ts', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -185,7 +185,7 @@ describe('message commands', () => {
   })
 
   describe('getThreadReplies', () => {
-    test('returns thread replies', async () => {
+    it('returns thread replies', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -198,7 +198,7 @@ describe('message commands', () => {
       expect(replies[1].text).toBe('Thread reply 2')
     })
 
-    test('passes limit option', async () => {
+    it('passes limit option', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -211,7 +211,7 @@ describe('message commands', () => {
   })
 
   describe('resolveChannel', () => {
-    test('resolves channel name to ID', async () => {
+    it('resolves channel name to ID', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 

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

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 const mockResolveChannel = mock((_channel: string) => Promise.resolve('C123456'))
 const mockAddReaction = mock(() => Promise.resolve())
@@ -25,7 +25,7 @@ describe('reaction commands', () => {
   })
 
   describe('addReaction', () => {
-    test('adds emoji reaction to message', async () => {
+    it('adds emoji reaction to message', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -36,7 +36,7 @@ describe('reaction commands', () => {
       expect(mockAddReaction).toHaveBeenCalledWith('C123456', '1234567890.000100', 'thumbsup')
     })
 
-    test('called with correct arguments', async () => {
+    it('passes correct arguments', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -50,7 +50,7 @@ describe('reaction commands', () => {
   })
 
   describe('removeReaction', () => {
-    test('removes emoji reaction from message', async () => {
+    it('removes emoji reaction from message', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -61,7 +61,7 @@ describe('reaction commands', () => {
       expect(mockRemoveReaction).toHaveBeenCalledWith('C123456', '1234567890.000100', 'thumbsup')
     })
 
-    test('called with correct arguments', async () => {
+    it('passes correct arguments', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -75,7 +75,7 @@ describe('reaction commands', () => {
   })
 
   describe('resolveChannel', () => {
-    test('resolves channel name before reaction', async () => {
+    it('resolves channel name before reaction', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 

package/src/platforms/slackbot/commands/user.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 const mockListUsers = mock(() =>
   Promise.resolve([
@@ -61,7 +61,7 @@ describe('user commands', () => {
   })
 
   describe('listUsers', () => {
-    test('returns list of users', async () => {
+    it('returns list of users', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -74,7 +74,7 @@ describe('user commands', () => {
       expect(users[1].name).toBe('bob')
     })
 
-    test('includes user metadata', async () => {
+    it('includes user metadata', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -90,7 +90,7 @@ describe('user commands', () => {
       expect(user.is_admin).toBe(false)
     })
 
-    test('passes limit option', async () => {
+    it('passes limit option', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -103,7 +103,7 @@ describe('user commands', () => {
   })
 
   describe('getUserInfo', () => {
-    test('returns user details by ID', async () => {
+    it('returns user details by ID', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -116,7 +116,7 @@ describe('user commands', () => {
       expect(user.real_name).toBe('Alice Smith')
     })
 
-    test('includes profile information', async () => {
+    it('includes profile information', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
@@ -129,7 +129,7 @@ describe('user commands', () => {
       expect(user.profile?.status_text).toBe('Working from home')
     })
 
-    test('called with correct user ID', async () => {
+    it('passes the user ID to getUserInfo', async () => {
       // given
       const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 

package/src/platforms/slackbot/commands/whoami.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { mkdir } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
@@ -45,7 +45,7 @@ describe('whoami command', () => {
     process.env = originalEnv
   })
 
-  test('returns auth info for current bot', async () => {
+  it('returns auth info for current bot', async () => {
     const manager = new SlackBotCredentialManager(tempDir)
     await manager.setCredentials({
       token: 'xoxb-test-token',
@@ -65,7 +65,7 @@ describe('whoami command', () => {
     expect(result.error).toBeUndefined()
   })
 
-  test('returns auth info for specific --bot', async () => {
+  it('returns auth info for specific --bot', async () => {
     const manager = new SlackBotCredentialManager(tempDir)
     await manager.setCredentials({
       token: 'xoxb-test-token',
@@ -82,7 +82,7 @@ describe('whoami command', () => {
     expect(mockTestAuth).toHaveBeenCalledTimes(1)
   })
 
-  test('returns error when client throws', async () => {
+  it('returns error when client throws', async () => {
     mockTestAuth.mockImplementationOnce(() => Promise.reject(new Error('API Error')))
 
     const manager = new SlackBotCredentialManager(tempDir)