agent-messenger 2.10.0 → 2.10.2

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, mock, spyOn, it } from 'bun:test'
 
 import { DiscordClient } from '../client'
 import { DiscordCredentialManager } from '../credential-manager'
@@ -93,7 +93,7 @@ afterEach(() => {
   credManagerLoadSpy?.mockRestore()
 })
 
-test('send: returns message with id', async () => {
+it('send: returns message with id', async () => {
   const consoleSpy = mock((_msg: string) => {})
   console.log = consoleSpy
 
@@ -104,7 +104,7 @@ test('send: returns message with id', async () => {
   expect(output).toContain('msg_123')
 })
 
-test('list: returns array of messages', async () => {
+it('list: returns array of messages', async () => {
   const consoleSpy = mock((_msg: string) => {})
   console.log = consoleSpy
 
@@ -116,7 +116,7 @@ test('list: returns array of messages', async () => {
   expect(output).toContain('msg_124')
 })
 
-test('get: returns single message', async () => {
+it('get: returns single message', async () => {
   const consoleSpy = mock((_msg: string) => {})
   console.log = consoleSpy
 
@@ -127,7 +127,7 @@ test('get: returns single message', async () => {
   expect(output).toContain('msg_123')
 })
 
-test('delete: returns success', async () => {
+it('delete: returns success', async () => {
   const consoleSpy = mock((_msg: string) => {})
   console.log = consoleSpy
 
@@ -138,7 +138,7 @@ test('delete: returns success', async () => {
   expect(output).toContain('deleted')
 })
 
-test('ack: returns success', async () => {
+it('ack: returns success', async () => {
   const consoleSpy = mock((_msg: string) => {})
   console.log = consoleSpy
 
@@ -149,7 +149,7 @@ test('ack: returns success', async () => {
   expect(output).toContain('acknowledged')
 })
 
-test('search: returns search results with query', async () => {
+it('search: returns search results with query', async () => {
   const consoleSpy = mock((_msg: string) => {})
   console.log = consoleSpy
 
@@ -162,7 +162,7 @@ test('search: returns search results with query', async () => {
   expect(output).toContain('Hello world')
 })
 
-test('search: includes total_results in output', async () => {
+it('search: includes total_results in output', async () => {
   const consoleSpy = mock((_msg: string) => {})
   console.log = consoleSpy
 
@@ -174,7 +174,7 @@ test('search: includes total_results in output', async () => {
   expect(output).toContain('2')
 })
 
-test('search: passes options to client', async () => {
+it('search: passes options to client', async () => {
   const consoleSpy = mock((_msg: string) => {})
   console.log = consoleSpy
 

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

@@ -1,4 +1,4 @@
-import { expect, mock, test } from 'bun:test'
+import { expect, mock, it } from 'bun:test'
 
 // Mock DiscordClient
 const mockClient = {
@@ -21,7 +21,7 @@ const mockClient = {
   }),
 }
 
-test('get returns note when it exists', async () => {
+it('get returns note when it exists', async () => {
   // given: user with a note
   const userId = 'user_with_note'
 
@@ -34,7 +34,7 @@ test('get returns note when it exists', async () => {
   expect(result?.note).toBe('This is a test note')
 })
 
-test('get returns null when note does not exist', async () => {
+it('get returns null when note does not exist', async () => {
   // given: user without a note
   const userId = 'user_without_note'
 
@@ -45,7 +45,7 @@ test('get returns null when note does not exist', async () => {
   expect(result).toBeNull()
 })
 
-test('set creates or updates note', async () => {
+it('set creates or updates note', async () => {
   // given: user id and note content
   const userId = 'user123'
   const noteContent = 'Important person to remember'
@@ -58,7 +58,7 @@ test('set creates or updates note', async () => {
   expect(result.note).toBe('Important person to remember')
 })
 
-test('set can update existing note', async () => {
+it('set can update existing note', async () => {
   // given: user with existing note
   const userId = 'user_with_note'
   const newNote = 'Updated note content'
@@ -71,7 +71,7 @@ test('set can update existing note', async () => {
   expect(result.note).toBe('Updated note content')
 })
 
-test('set can clear note with empty string', async () => {
+it('set can clear note with empty string', async () => {
   // given: user with existing note
   const userId = 'user_with_note'
 

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

@@ -1,4 +1,4 @@
-import { expect, mock, test } from 'bun:test'
+import { expect, mock, it } from 'bun:test'
 
 // Mock DiscordClient
 const mockClient = {
@@ -38,7 +38,7 @@ const mockClient = {
   })),
 }
 
-test('get returns user profile with all fields', async () => {
+it('get returns user profile with all fields', async () => {
   // given: user id
   const userId = 'user123'
 
@@ -57,7 +57,7 @@ test('get returns user profile with all fields', async () => {
   expect(profile.mutual_guilds?.[0].nick).toBe('TestNick')
 })
 
-test('get returns profile with minimal fields', async () => {
+it('get returns profile with minimal fields', async () => {
   // given: mock with minimal profile
   const minimalMock = {
     getUserProfile: mock(async (userId: string) => ({
@@ -81,7 +81,7 @@ test('get returns profile with minimal fields', async () => {
   expect(profile.mutual_guilds).toBeUndefined()
 })
 
-test('get formats connected accounts correctly', async () => {
+it('get formats connected accounts correctly', async () => {
   // given: user with connected accounts
   const userId = 'user789'
   const profile = await mockClient.getUserProfile(userId)

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, mock, spyOn, it } from 'bun:test'
 
 import { DiscordClient } from '../client'
 import { DiscordCredentialManager } from '../credential-manager'
@@ -48,7 +48,7 @@ afterEach(() => {
   credManagerLoadSpy?.mockRestore()
 })
 
-test('add: sends correct PUT request with emoji', async () => {
+it('add: sends correct PUT request with emoji', async () => {
   const consoleSpy = mock((_msg: string) => {})
   const originalLog = console.log
   console.log = consoleSpy
@@ -66,7 +66,7 @@ test('add: sends correct PUT request with emoji', async () => {
   }
 })
 
-test('remove: sends correct DELETE request with emoji', async () => {
+it('remove: sends correct DELETE request with emoji', async () => {
   const consoleSpy = mock((_msg: string) => {})
   const originalLog = console.log
   console.log = consoleSpy
@@ -84,7 +84,7 @@ test('remove: sends correct DELETE request with emoji', async () => {
   }
 })
 
-test('list: extracts reactions from message', async () => {
+it('list: extracts reactions from message', async () => {
   const consoleSpy = mock((_msg: string) => {})
   const originalLog = console.log
   console.log = consoleSpy
@@ -104,7 +104,7 @@ test('list: extracts reactions from message', async () => {
   }
 })
 
-test('add: handles missing token gracefully', async () => {
+it('add: handles missing token gracefully', async () => {
   // Temporarily override the credential manager spy to return null token
   credManagerLoadSpy?.mockResolvedValue({
     token: null,

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, spyOn, it } from 'bun:test'
 
 import { DiscordClient } from '../client'
 import { DiscordCredentialManager } from '../credential-manager'
@@ -53,7 +53,7 @@ afterEach(() => {
   credManagerGetCurrentServerSpy?.mockRestore()
 })
 
-test('list: returns servers with current marker', async () => {
+it('list: returns servers with current marker', async () => {
   // given: credential manager with servers
   const credManager = new DiscordCredentialManager()
   const config = await credManager.load()
@@ -67,7 +67,7 @@ test('list: returns servers with current marker', async () => {
   expect(config.servers['server-2']).toBeDefined()
 })
 
-test('list: marks current server', async () => {
+it('list: marks current server', async () => {
   // given: credential manager with current server set
   const credManager = new DiscordCredentialManager()
   const config = await credManager.load()
@@ -80,7 +80,7 @@ test('list: marks current server', async () => {
   expect(config.current_server).toBe('server-1')
 })
 
-test('info: returns server details', async () => {
+it('info: returns server details', async () => {
   // given: discord client with server data
   const client = await new DiscordClient().login({ token: 'test-token' })
   const server = await client.getServer('server-1')
@@ -95,7 +95,7 @@ test('info: returns server details', async () => {
   expect(server.owner).toBe(true)
 })
 
-test('info: throws error for non-existent server', async () => {
+it('info: throws error for non-existent server', async () => {
   // given: discord client
   const client = await new DiscordClient().login({ token: 'test-token' })
 
@@ -109,7 +109,7 @@ test('info: throws error for non-existent server', async () => {
   }
 })
 
-test('switch: updates current server', async () => {
+it('switch: updates current server', async () => {
   // given: credential manager
   const credManager = new DiscordCredentialManager()
 
@@ -120,7 +120,7 @@ test('switch: updates current server', async () => {
   expect(credManager.setCurrentServer).toHaveBeenCalledWith('server-2')
 })
 
-test('current: returns current server info', async () => {
+it('current: returns current server info', async () => {
   // given: credential manager with current server
   const credManager = new DiscordCredentialManager()
   const config = await credManager.load()

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

@@ -1,29 +1,29 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import { snapshotCommand } from './snapshot'
 
-test('snapshot: command is defined', () => {
+it('snapshot: command is defined', () => {
   expect(snapshotCommand).toBeDefined()
   expect(snapshotCommand.name()).toBe('snapshot')
 })
 
-test('snapshot: command has correct description', () => {
+it('snapshot: command has correct description', () => {
   expect(snapshotCommand.description()).toContain('server overview')
 })
 
-test('snapshot: command has --channels-only option', () => {
+it('snapshot: command has --channels-only option', () => {
   const options = snapshotCommand.options
   const channelsOnlyOption = options.find((opt) => opt.long === '--channels-only')
   expect(channelsOnlyOption).toBeDefined()
 })
 
-test('snapshot: command has --users-only option', () => {
+it('snapshot: command has --users-only option', () => {
   const options = snapshotCommand.options
   const usersOnlyOption = options.find((opt) => opt.long === '--users-only')
   expect(usersOnlyOption).toBeDefined()
 })
 
-test('snapshot: command has --limit option', () => {
+it('snapshot: command has --limit option', () => {
   const options = snapshotCommand.options
   const limitOption = options.find((opt) => opt.long === '--limit')
   expect(limitOption).toBeDefined()

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, spyOn, it } from 'bun:test'
 
 import { DiscordClient } from '../client'
 import { DiscordCredentialManager } from '../credential-manager'
@@ -44,7 +44,7 @@ afterEach(() => {
   credManagerLoadSpy?.mockRestore()
 })
 
-test('createThread: creates thread with name', async () => {
+it('createThread: creates thread with name', async () => {
   // given: discord client
   const client = await new DiscordClient().login({ token: 'test-token' })
 
@@ -59,7 +59,7 @@ test('createThread: creates thread with name', async () => {
   expect(thread.parent_id).toBe('ch-1')
 })
 
-test('createThread: creates thread with auto_archive_duration', async () => {
+it('createThread: creates thread with auto_archive_duration', async () => {
   // given: discord client
   const client = await new DiscordClient().login({ token: 'test-token' })
 
@@ -72,7 +72,7 @@ test('createThread: creates thread with auto_archive_duration', async () => {
   })
 })
 
-test('createThread: creates thread with rate_limit_per_user', async () => {
+it('createThread: creates thread with rate_limit_per_user', async () => {
   // given: discord client
   const client = await new DiscordClient().login({ token: 'test-token' })
 
@@ -85,7 +85,7 @@ test('createThread: creates thread with rate_limit_per_user', async () => {
   })
 })
 
-test('archiveThread: archives thread', async () => {
+it('archiveThread: archives thread', async () => {
   // given: discord client
   const client = await new DiscordClient().login({ token: 'test-token' })
 
@@ -98,7 +98,7 @@ test('archiveThread: archives thread', async () => {
   expect(thread.thread_metadata?.archived).toBe(true)
 })
 
-test('archiveThread: unarchives thread when archived=false', async () => {
+it('archiveThread: unarchives thread when archived=false', async () => {
   // given: discord client with unarchive mock
   clientArchiveThreadSpy.mockResolvedValue({
     id: 'thread-1',

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

@@ -1,4 +1,4 @@
-import { expect, mock, test } from 'bun:test'
+import { expect, mock, it } from 'bun:test'
 
 // Mock DiscordClient
 const mockClient = {
@@ -41,7 +41,7 @@ const mockClient = {
   ]),
 }
 
-test('me returns current user info', async () => {
+it('me returns current user info', async () => {
   // given: authenticated user
   const user = await mockClient.testAuth()
 
@@ -61,7 +61,7 @@ test('me returns current user info', async () => {
   expect(result.bot).toBe(false)
 })
 
-test('info returns user details by id', async () => {
+it('info returns user details by id', async () => {
   // given: user id
   const userId = 'user123'
 
@@ -80,7 +80,7 @@ test('info returns user details by id', async () => {
   expect(result.username).toBe('testuser')
 })
 
-test('list returns server members', async () => {
+it('list returns server members', async () => {
   // given: server id
   const serverId = 'server123'
 
@@ -101,7 +101,7 @@ test('list returns server members', async () => {
   expect(result[2].username).toBe('bob')
 })
 
-test('list filters out bots when flag not set', async () => {
+it('list filters out bots when flag not set', async () => {
   // given: server id and users with bots
   const serverId = 'server123'
   const users = await mockClient.listUsers(serverId)

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, spyOn, it } from 'bun:test'
 
 import { DiscordClient } from '@/platforms/discord/client'
 import { whoamiAction, whoamiCommand } from '@/platforms/discord/commands/whoami'
@@ -35,19 +35,19 @@ afterEach(() => {
   processExitSpy?.mockRestore()
 })
 
-test('whoami command is defined with correct name and description', () => {
+it('whoami command is defined with correct name and description', () => {
   expect(whoamiCommand).toBeDefined()
   expect(whoamiCommand.name()).toBe('whoami')
   expect(whoamiCommand.description()).toBe('Show current authenticated user')
 })
 
-test('whoami command has --pretty option', () => {
+it('whoami command has --pretty option', () => {
   const options = whoamiCommand.options
   const hasPretty = options.some((opt: { long?: string }) => opt.long === '--pretty')
   expect(hasPretty).toBe(true)
 })
 
-test('whoami outputs expected user fields', async () => {
+it('whoami outputs expected user fields', async () => {
   await whoamiAction({})
 
   expect(consoleLogSpy).toHaveBeenCalledWith(
@@ -61,7 +61,7 @@ test('whoami outputs expected user fields', async () => {
   )
 })
 
-test('whoami outputs pretty-printed JSON when pretty is true', async () => {
+it('whoami outputs pretty-printed JSON when pretty is true', async () => {
   await whoamiAction({ pretty: true })
 
   expect(consoleLogSpy).toHaveBeenCalledWith(
@@ -79,7 +79,7 @@ test('whoami outputs pretty-printed JSON when pretty is true', async () => {
   )
 })
 
-test('whoami exits with error when not authenticated', async () => {
+it('whoami exits with error when not authenticated', async () => {
   credManagerSpy.mockResolvedValue({ token: undefined })
 
   await whoamiAction({})

package/src/platforms/discord/credential-manager.test.ts

@@ -1,4 +1,4 @@
-import { afterAll, afterEach, beforeEach, describe, expect, test } from 'bun:test'
+import { afterAll, afterEach, beforeEach, describe, expect, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { join } from 'node:path'
 
@@ -39,7 +39,7 @@ afterAll(() => {
 })
 
 describe('DiscordCredentialManager', () => {
-  test('load returns default config when file does not exist', async () => {
+  it('load returns default config when file does not exist', async () => {
     const manager = setup()
     const config = await manager.load()
 
@@ -50,7 +50,7 @@ describe('DiscordCredentialManager', () => {
     })
   })
 
-  test('save creates config file with correct permissions', async () => {
+  it('save creates config file with correct permissions', async () => {
     const testConfigDir = join(
       import.meta.dir,
       `.test-discord-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
@@ -76,13 +76,13 @@ describe('DiscordCredentialManager', () => {
     expect(loaded).toEqual(config)
   })
 
-  test('getToken returns null when not authenticated', async () => {
+  it('getToken returns null when not authenticated', async () => {
     const manager = setup()
     const token = await manager.getToken()
     expect(token).toBeNull()
   })
 
-  test('setToken saves token to config', async () => {
+  it('setToken saves token to config', async () => {
     const manager = setup()
     await manager.setToken('test-token-123')
 
@@ -90,7 +90,7 @@ describe('DiscordCredentialManager', () => {
     expect(token).toBe('test-token-123')
   })
 
-  test('getToken returns previously set token', async () => {
+  it('getToken returns previously set token', async () => {
     const manager = setup()
     await manager.setToken('my-token')
 
@@ -98,7 +98,7 @@ describe('DiscordCredentialManager', () => {
     expect(token).toBe('my-token')
   })
 
-  test('clearToken removes token from config', async () => {
+  it('clearToken removes token from config', async () => {
     const manager = setup()
     await manager.setToken('test-token')
     await manager.clearToken()
@@ -107,13 +107,13 @@ describe('DiscordCredentialManager', () => {
     expect(token).toBeNull()
   })
 
-  test('getCurrentServer returns null when not set', async () => {
+  it('getCurrentServer returns null when not set', async () => {
     const manager = setup()
     const server = await manager.getCurrentServer()
     expect(server).toBeNull()
   })
 
-  test('setCurrentServer saves server id', async () => {
+  it('setCurrentServer saves server id', async () => {
     const manager = setup()
     await manager.setCurrentServer('server-456')
 
@@ -121,13 +121,13 @@ describe('DiscordCredentialManager', () => {
     expect(server).toBe('server-456')
   })
 
-  test('getServers returns empty object when no servers set', async () => {
+  it('getServers returns empty object when no servers set', async () => {
     const manager = setup()
     const servers = await manager.getServers()
     expect(servers).toEqual({})
   })
 
-  test('setServers saves servers to config', async () => {
+  it('setServers saves servers to config', async () => {
     const manager = setup()
     const servers = {
       'server-1': { server_id: 'server-1', server_name: 'Server One' },
@@ -140,13 +140,13 @@ describe('DiscordCredentialManager', () => {
     expect(loaded).toEqual(servers)
   })
 
-  test('getCredentials returns null when not authenticated', async () => {
+  it('getCredentials returns null when not authenticated', async () => {
     const manager = setup()
     const creds = await manager.getCredentials()
     expect(creds).toBeNull()
   })
 
-  test('getCredentials returns null when token exists but no server selected', async () => {
+  it('getCredentials returns null when token exists but no server selected', async () => {
     const manager = setup()
     await manager.setToken('test-token')
 
@@ -154,7 +154,7 @@ describe('DiscordCredentialManager', () => {
     expect(creds).toBeNull()
   })
 
-  test('getCredentials returns null when server selected but no token', async () => {
+  it('getCredentials returns null when server selected but no token', async () => {
     const manager = setup()
     await manager.setCurrentServer('server-123')
 
@@ -162,7 +162,7 @@ describe('DiscordCredentialManager', () => {
     expect(creds).toBeNull()
   })
 
-  test('getCredentials returns token and serverId when both are set', async () => {
+  it('getCredentials returns token and serverId when both are set', async () => {
     const manager = setup()
     await manager.setToken('test-token-xyz')
     await manager.setCurrentServer('server-789')
@@ -174,7 +174,7 @@ describe('DiscordCredentialManager', () => {
     })
   })
 
-  test('multiple operations preserve existing data', async () => {
+  it('multiple operations preserve existing data', async () => {
     const manager = setup()
     await manager.setToken('token-1')
     await manager.setCurrentServer('server-1')

package/src/platforms/discord/ensure-auth.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, spyOn, it } from 'bun:test'
 
 import { DiscordClient } from './client'
 import { DiscordCredentialManager } from './credential-manager'
@@ -42,7 +42,7 @@ afterEach(() => {
 })
 
 describe('ensureDiscordAuth', () => {
-  test('skips extraction when token already exists', async () => {
+  it('skips extraction when token already exists', async () => {
     // given
     getTokenSpy.mockResolvedValue('existing-token')
 
@@ -53,7 +53,7 @@ describe('ensureDiscordAuth', () => {
     expect(extractSpy).not.toHaveBeenCalled()
   })
 
-  test('extracts and saves credentials when no token', async () => {
+  it('extracts and saves credentials when no token', async () => {
     // when
     await ensureDiscordAuth()
 
@@ -71,7 +71,7 @@ describe('ensureDiscordAuth', () => {
     })
   })
 
-  test('sets first server as current', async () => {
+  it('sets first server as current', async () => {
     // when
     await ensureDiscordAuth()
 
@@ -79,7 +79,7 @@ describe('ensureDiscordAuth', () => {
     expect(saveSpy).toHaveBeenCalledWith(expect.objectContaining({ current_server: 'server-1' }))
   })
 
-  test('handles no servers with null current_server', async () => {
+  it('handles no servers with null current_server', async () => {
     // given
     listServersSpy.mockResolvedValue([])
 
@@ -90,7 +90,7 @@ describe('ensureDiscordAuth', () => {
     expect(saveSpy).toHaveBeenCalledWith(expect.objectContaining({ current_server: null, servers: {} }))
   })
 
-  test('does not save when extraction returns null', async () => {
+  it('does not save when extraction returns null', async () => {
     // given
     extractSpy.mockResolvedValue([])
 
@@ -102,7 +102,7 @@ describe('ensureDiscordAuth', () => {
     expect(saveSpy).not.toHaveBeenCalled()
   })
 
-  test('silently handles extraction failure', async () => {
+  it('silently handles extraction failure', async () => {
     // given
     extractSpy.mockRejectedValue(new Error('Discord not found'))
 
@@ -113,7 +113,7 @@ describe('ensureDiscordAuth', () => {
     expect(saveSpy).not.toHaveBeenCalled()
   })
 
-  test('silently handles auth validation failure', async () => {
+  it('silently handles auth validation failure', async () => {
     // given
     testAuthSpy.mockRejectedValue(new Error('401 Unauthorized'))