agent-messenger 2.10.1 → 2.11.0

package/src/platforms/discord/types.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   DiscordChannelSchema,
@@ -12,7 +12,7 @@ import {
   DiscordUserSchema,
 } from './types'
 
-test('DiscordGuildSchema validates correct guild', () => {
+it('DiscordGuildSchema validates correct guild', () => {
   const result = DiscordGuildSchema.safeParse({
     id: '123456789012345678',
     name: 'Test Guild',
@@ -20,21 +20,21 @@ test('DiscordGuildSchema validates correct guild', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordGuildSchema rejects missing id', () => {
+it('DiscordGuildSchema rejects missing id', () => {
   const result = DiscordGuildSchema.safeParse({
     name: 'Test Guild',
   })
   expect(result.success).toBe(false)
 })
 
-test('DiscordGuildSchema rejects missing name', () => {
+it('DiscordGuildSchema rejects missing name', () => {
   const result = DiscordGuildSchema.safeParse({
     id: '123456789012345678',
   })
   expect(result.success).toBe(false)
 })
 
-test('DiscordChannelSchema validates correct channel', () => {
+it('DiscordChannelSchema validates correct channel', () => {
   const result = DiscordChannelSchema.safeParse({
     id: '987654321098765432',
     guild_id: '123456789012345678',
@@ -44,7 +44,7 @@ test('DiscordChannelSchema validates correct channel', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordChannelSchema validates channel with optional fields', () => {
+it('DiscordChannelSchema validates channel with optional fields', () => {
   const result = DiscordChannelSchema.safeParse({
     id: '987654321098765432',
     guild_id: '123456789012345678',
@@ -56,7 +56,7 @@ test('DiscordChannelSchema validates channel with optional fields', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordChannelSchema rejects missing required fields', () => {
+it('DiscordChannelSchema rejects missing required fields', () => {
   const result = DiscordChannelSchema.safeParse({
     id: '987654321098765432',
     name: 'general',
@@ -64,7 +64,7 @@ test('DiscordChannelSchema rejects missing required fields', () => {
   expect(result.success).toBe(false)
 })
 
-test('DiscordMessageSchema validates correct message', () => {
+it('DiscordMessageSchema validates correct message', () => {
   const result = DiscordMessageSchema.safeParse({
     id: '111222333444555666',
     channel_id: '987654321098765432',
@@ -78,7 +78,7 @@ test('DiscordMessageSchema validates correct message', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordMessageSchema validates message with optional fields', () => {
+it('DiscordMessageSchema validates message with optional fields', () => {
   const result = DiscordMessageSchema.safeParse({
     id: '111222333444555666',
     channel_id: '987654321098765432',
@@ -94,7 +94,7 @@ test('DiscordMessageSchema validates message with optional fields', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordMessageSchema rejects missing required fields', () => {
+it('DiscordMessageSchema rejects missing required fields', () => {
   const result = DiscordMessageSchema.safeParse({
     id: '111222333444555666',
     channel_id: '987654321098765432',
@@ -103,7 +103,7 @@ test('DiscordMessageSchema rejects missing required fields', () => {
   expect(result.success).toBe(false)
 })
 
-test('DiscordUserSchema validates correct user', () => {
+it('DiscordUserSchema validates correct user', () => {
   const result = DiscordUserSchema.safeParse({
     id: '222333444555666777',
     username: 'testuser',
@@ -111,7 +111,7 @@ test('DiscordUserSchema validates correct user', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordUserSchema validates user with optional fields', () => {
+it('DiscordUserSchema validates user with optional fields', () => {
   const result = DiscordUserSchema.safeParse({
     id: '222333444555666777',
     username: 'testuser',
@@ -122,14 +122,14 @@ test('DiscordUserSchema validates user with optional fields', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordUserSchema rejects missing required fields', () => {
+it('DiscordUserSchema rejects missing required fields', () => {
   const result = DiscordUserSchema.safeParse({
     id: '222333444555666777',
   })
   expect(result.success).toBe(false)
 })
 
-test('DiscordReactionSchema validates correct reaction', () => {
+it('DiscordReactionSchema validates correct reaction', () => {
   const result = DiscordReactionSchema.safeParse({
     emoji: {
       name: 'thumbsup',
@@ -139,7 +139,7 @@ test('DiscordReactionSchema validates correct reaction', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordReactionSchema validates reaction with custom emoji', () => {
+it('DiscordReactionSchema validates reaction with custom emoji', () => {
   const result = DiscordReactionSchema.safeParse({
     emoji: {
       id: '123456789012345678',
@@ -150,7 +150,7 @@ test('DiscordReactionSchema validates reaction with custom emoji', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordReactionSchema rejects missing required fields', () => {
+it('DiscordReactionSchema rejects missing required fields', () => {
   const result = DiscordReactionSchema.safeParse({
     emoji: {
       name: 'thumbsup',
@@ -159,7 +159,7 @@ test('DiscordReactionSchema rejects missing required fields', () => {
   expect(result.success).toBe(false)
 })
 
-test('DiscordFileSchema validates correct file', () => {
+it('DiscordFileSchema validates correct file', () => {
   const result = DiscordFileSchema.safeParse({
     id: '444555666777888999',
     filename: 'document.pdf',
@@ -169,7 +169,7 @@ test('DiscordFileSchema validates correct file', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordFileSchema validates file with optional fields', () => {
+it('DiscordFileSchema validates file with optional fields', () => {
   const result = DiscordFileSchema.safeParse({
     id: '444555666777888999',
     filename: 'document.pdf',
@@ -182,7 +182,7 @@ test('DiscordFileSchema validates file with optional fields', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordFileSchema rejects missing required fields', () => {
+it('DiscordFileSchema rejects missing required fields', () => {
   const result = DiscordFileSchema.safeParse({
     id: '444555666777888999',
     filename: 'document.pdf',
@@ -190,19 +190,19 @@ test('DiscordFileSchema rejects missing required fields', () => {
   expect(result.success).toBe(false)
 })
 
-test('DiscordCredentialsSchema validates correct credentials', () => {
+it('DiscordCredentialsSchema validates correct credentials', () => {
   const result = DiscordCredentialsSchema.safeParse({
     token: 'token_value',
   })
   expect(result.success).toBe(true)
 })
 
-test('DiscordCredentialsSchema rejects missing token', () => {
+it('DiscordCredentialsSchema rejects missing token', () => {
   const result = DiscordCredentialsSchema.safeParse({})
   expect(result.success).toBe(false)
 })
 
-test('DiscordConfigSchema validates correct config', () => {
+it('DiscordConfigSchema validates correct config', () => {
   const result = DiscordConfigSchema.safeParse({
     current_server: null,
     token: 'token_value',
@@ -211,7 +211,7 @@ test('DiscordConfigSchema validates correct config', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordConfigSchema validates config with servers', () => {
+it('DiscordConfigSchema validates config with servers', () => {
   const result = DiscordConfigSchema.safeParse({
     current_server: '123456789012345678',
     token: 'token_value',
@@ -225,7 +225,7 @@ test('DiscordConfigSchema validates config with servers', () => {
   expect(result.success).toBe(true)
 })
 
-test('DiscordConfigSchema rejects missing required fields', () => {
+it('DiscordConfigSchema rejects missing required fields', () => {
   const result = DiscordConfigSchema.safeParse({
     current_server: null,
     token: 'token_value',
@@ -233,14 +233,14 @@ test('DiscordConfigSchema rejects missing required fields', () => {
   expect(result.success).toBe(false)
 })
 
-test('DiscordError has correct name and code', () => {
+it('DiscordError has correct name and code', () => {
   const error = new DiscordError('Test error', 'TEST_CODE')
   expect(error.name).toBe('DiscordError')
   expect(error.message).toBe('Test error')
   expect(error.code).toBe('TEST_CODE')
 })
 
-test('DiscordError is instance of Error', () => {
+it('DiscordError is instance of Error', () => {
   const error = new DiscordError('Test error', 'TEST_CODE')
   expect(error instanceof Error).toBe(true)
 })

package/src/platforms/discordbot/cli.ts

@@ -22,8 +22,18 @@ program
   .name('agent-discordbot')
   .description('CLI tool for Discord bot integration using bot tokens')
   .version(pkg.version)
+  .option('--pretty', 'Pretty-print JSON output')
   .option('--bot <id>', 'Bot ID to use')
   .option('--server <id>', 'Server ID to use')
+  .hook('preAction', (thisCmd, actionCmd) => {
+    for (const [key, value] of Object.entries(thisCmd.opts())) {
+      if (value === undefined) continue
+      const source = actionCmd.getOptionValueSource(key)
+      if (source === undefined || source === 'default') {
+        actionCmd.setOptionValue(key, value)
+      }
+    }
+  })
 
 program.addCommand(authCommand)
 program.addCommand(whoamiCommand)

package/src/platforms/discordbot/client.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, it } from 'bun:test'
 
 import { DiscordBotClient } from './client'
 import { DiscordBotError } from './types'
@@ -48,19 +48,19 @@ describe('DiscordBotClient', () => {
   }
 
   describe('constructor', () => {
-    test('requires token', async () => {
+    it('requires token', async () => {
       await expect(new DiscordBotClient().login({ token: '' })).rejects.toThrow(DiscordBotError)
       await expect(new DiscordBotClient().login({ token: '' })).rejects.toThrow('Token is required')
     })
 
-    test('accepts valid token', async () => {
+    it('accepts valid token', async () => {
       const client = await new DiscordBotClient().login({ token: 'bot-test-token' })
       expect(client).toBeInstanceOf(DiscordBotClient)
     })
   })
 
   describe('testAuth', () => {
-    test('returns current user and uses Bot auth header', async () => {
+    it('returns current user and uses Bot auth header', async () => {
       mockResponse({
         id: '123456789',
         username: 'testbot',
@@ -81,7 +81,7 @@ describe('DiscordBotClient', () => {
       expect(headers['User-Agent']).toContain('DiscordBot')
     })
 
-    test('throws DiscordBotError on API error', async () => {
+    it('throws DiscordBotError on API error', async () => {
       mockResponse({ message: 'Unauthorized', code: 401 }, 401)
 
       const client = await new DiscordBotClient().login({ token: 'bad-token' })
@@ -90,7 +90,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('listGuilds', () => {
-    test('returns list of guilds', async () => {
+    it('returns list of guilds', async () => {
       mockResponse([
         { id: '111', name: 'Guild One' },
         { id: '222', name: 'Guild Two' },
@@ -106,7 +106,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('getGuild', () => {
-    test('returns guild info', async () => {
+    it('returns guild info', async () => {
       mockResponse({ id: '111', name: 'Test Guild' })
 
       const client = await new DiscordBotClient().login({ token: 'bot-token' })
@@ -119,7 +119,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('listChannels', () => {
-    test('returns channels for guild', async () => {
+    it('returns channels for guild', async () => {
       mockResponse([
         { id: 'ch1', guild_id: '111', name: 'general', type: 0 },
         { id: 'ch2', guild_id: '111', name: 'random', type: 0 },
@@ -135,7 +135,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('sendMessage', () => {
-    test('sends message to channel', async () => {
+    it('sends message to channel', async () => {
       mockResponse({
         id: 'msg1',
         channel_id: 'ch1',
@@ -153,7 +153,7 @@ describe('DiscordBotClient', () => {
       expect(fetchCalls[0].options?.body).toBe(JSON.stringify({ content: 'Hello world' }))
     })
 
-    test('includes thread_id when provided', async () => {
+    it('includes thread_id when provided', async () => {
       mockResponse({
         id: 'msg1',
         channel_id: 'ch1',
@@ -170,7 +170,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('editMessage', () => {
-    test('edits message with PATCH', async () => {
+    it('edits message with PATCH', async () => {
       mockResponse({
         id: 'msg1',
         channel_id: 'ch1',
@@ -191,7 +191,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('deleteMessage', () => {
-    test('deletes message and returns void', async () => {
+    it('deletes message and returns void', async () => {
       mockResponse(null, 204)
 
       const client = await new DiscordBotClient().login({ token: 'bot-token' })
@@ -204,7 +204,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('addReaction', () => {
-    test('adds reaction with encoded emoji', async () => {
+    it('adds reaction with encoded emoji', async () => {
       mockResponse(null, 204)
 
       const client = await new DiscordBotClient().login({ token: 'bot-token' })
@@ -218,7 +218,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('removeReaction', () => {
-    test('removes reaction with DELETE', async () => {
+    it('removes reaction with DELETE', async () => {
       mockResponse(null, 204)
 
       const client = await new DiscordBotClient().login({ token: 'bot-token' })
@@ -232,7 +232,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('listUsers', () => {
-    test('maps guild members to users', async () => {
+    it('maps guild members to users', async () => {
       mockResponse([{ user: { id: 'u1', username: 'user1' } }, { user: { id: 'u2', username: 'user2' } }])
 
       const client = await new DiscordBotClient().login({ token: 'bot-token' })
@@ -245,7 +245,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('getUser', () => {
-    test('returns user info', async () => {
+    it('returns user info', async () => {
       mockResponse({ id: 'u1', username: 'testuser' })
 
       const client = await new DiscordBotClient().login({ token: 'bot-token' })
@@ -258,7 +258,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('uploadFile', () => {
-    test('uploads file to channel', async () => {
+    it('uploads file to channel', async () => {
       const { tmpdir } = await import('node:os')
       const { join } = await import('node:path')
       const tempFile = join(tmpdir(), 'test-discordbot-upload.txt')
@@ -293,7 +293,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('listFiles', () => {
-    test('returns files from recent messages', async () => {
+    it('returns files from recent messages', async () => {
       mockResponse([
         {
           id: 'msg1',
@@ -322,7 +322,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('createThread', () => {
-    test('creates thread in channel', async () => {
+    it('creates thread in channel', async () => {
       mockResponse({
         id: 'thread1',
         guild_id: '111',
@@ -342,7 +342,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('archiveThread', () => {
-    test('archives thread with PATCH', async () => {
+    it('archives thread with PATCH', async () => {
       mockResponse({
         id: 'thread1',
         guild_id: '111',
@@ -361,7 +361,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('resolveChannel', () => {
-    test('returns channel ID directly if all digits', async () => {
+    it('returns channel ID directly if all digits', async () => {
       const client = await new DiscordBotClient().login({ token: 'bot-token' })
       const id = await client.resolveChannel('111', '123456789')
 
@@ -369,7 +369,7 @@ describe('DiscordBotClient', () => {
       expect(fetchCalls.length).toBe(0)
     })
 
-    test('looks up channel by name', async () => {
+    it('looks up channel by name', async () => {
       mockResponse([
         { id: 'ch1', guild_id: '111', name: 'general', type: 0 },
         { id: 'ch2', guild_id: '111', name: 'random', type: 0 },
@@ -381,7 +381,7 @@ describe('DiscordBotClient', () => {
       expect(id).toBe('ch2')
     })
 
-    test('strips # prefix when looking up by name', async () => {
+    it('strips # prefix when looking up by name', async () => {
       mockResponse([{ id: 'ch1', guild_id: '111', name: 'general', type: 0 }])
 
       const client = await new DiscordBotClient().login({ token: 'bot-token' })
@@ -390,7 +390,7 @@ describe('DiscordBotClient', () => {
       expect(id).toBe('ch1')
     })
 
-    test('throws when channel name not found', async () => {
+    it('throws when channel name not found', async () => {
       mockResponse([{ id: 'ch1', guild_id: '111', name: 'general', type: 0 }])
 
       const client = await new DiscordBotClient().login({ token: 'bot-token' })
@@ -406,7 +406,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('rate limiting', () => {
-    test('waits when bucket is exhausted', async () => {
+    it('waits when bucket is exhausted', async () => {
       mockResponse({ id: '1', username: 'bot' }, 200, {
         'X-RateLimit-Remaining': '0',
         'X-RateLimit-Reset': String(Date.now() / 1000 + 0.1),
@@ -429,7 +429,7 @@ describe('DiscordBotClient', () => {
       expect(fetchCalls.length).toBe(2)
     })
 
-    test('retries on 429 with Retry-After header', async () => {
+    it('retries on 429 with Retry-After header', async () => {
       mockResponse({ message: 'Rate limited', retry_after: 0.1 }, 429, { 'Retry-After': '0.1' })
       mockResponse({ id: '123', username: 'bot' })
 
@@ -440,7 +440,7 @@ describe('DiscordBotClient', () => {
       expect(fetchCalls.length).toBe(2)
     })
 
-    test('handles global rate limit', async () => {
+    it('handles global rate limit', async () => {
       mockResponse({ message: 'Global rate limited', global: true }, 429, {
         'Retry-After': '0.1',
         'X-RateLimit-Global': 'true',
@@ -454,7 +454,7 @@ describe('DiscordBotClient', () => {
       expect(fetchCalls.length).toBe(2)
     })
 
-    test('throws after max retries exceeded', async () => {
+    it('throws after max retries exceeded', async () => {
       for (let i = 0; i <= 3; i++) {
         mockResponse({ message: 'Rate limited' }, 429, { 'Retry-After': '0.01' })
       }
@@ -466,7 +466,7 @@ describe('DiscordBotClient', () => {
   })
 
   describe('retry logic', () => {
-    test('retries on 500 server error', async () => {
+    it('retries on 500 server error', async () => {
       mockResponse({ message: 'Internal Server Error' }, 500)
       mockResponse({ id: '123', username: 'bot' })
 
@@ -477,7 +477,7 @@ describe('DiscordBotClient', () => {
       expect(fetchCalls.length).toBe(2)
     })
 
-    test('does not retry on 4xx client errors (except 429)', async () => {
+    it('does not retry on 4xx client errors (except 429)', async () => {
       mockResponse({ message: 'Not Found' }, 404)
 
       const client = await new DiscordBotClient().login({ token: 'bot-token' })
@@ -485,7 +485,7 @@ describe('DiscordBotClient', () => {
       expect(fetchCalls.length).toBe(1)
     })
 
-    test('exponential backoff increases delay', async () => {
+    it('exponential backoff increases delay', async () => {
       mockResponse({ message: 'Error' }, 500)
       mockResponse({ message: 'Error' }, 500)
       mockResponse({ id: '123', username: 'bot' })

package/src/platforms/discordbot/client.ts

@@ -347,6 +347,10 @@ export class DiscordBotClient {
     return this.request<DiscordChannel>('PATCH', `/channels/${threadId}`, { archived })
   }
 
+  async gatewayConnect(): Promise<{ token: string }> {
+    return { token: this.ensureAuth() }
+  }
+
   async resolveChannel(guildId: string, channel: string): Promise<string> {
     if (/^\d+$/.test(channel)) return channel
 

package/src/platforms/discordbot/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'
@@ -55,7 +55,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 DiscordBotCredentialManager(tempDir)
 
       const result = await setAction('token123', { _credManager: manager })
@@ -71,7 +71,7 @@ describe('auth commands', () => {
       expect(creds?.bot_id).toBe('bot123')
     })
 
-    test('uses --bot flag as bot_id', async () => {
+    it('uses --bot flag as bot_id', async () => {
       const manager = new DiscordBotCredentialManager(tempDir)
 
       const result = await setAction('token123', { bot: 'mybot', _credManager: manager })
@@ -81,7 +81,7 @@ describe('auth commands', () => {
       expect(creds?.token).toBe('token123')
     })
 
-    test('rejects user tokens (bot: false)', async () => {
+    it('rejects user tokens (bot: false)', async () => {
       mockTestAuth.mockImplementationOnce(() =>
         Promise.resolve({
           id: 'user123',
@@ -98,7 +98,7 @@ describe('auth commands', () => {
       expect(result.error).toContain('not a bot token')
     })
 
-    test('rejects user tokens (bot: undefined)', async () => {
+    it('rejects user tokens (bot: undefined)', async () => {
       mockTestAuth.mockImplementationOnce(() =>
         Promise.resolve({
           id: 'user123',
@@ -114,7 +114,7 @@ describe('auth commands', () => {
       expect(result.error).toContain('not a bot token')
     })
 
-    test('handles client errors', async () => {
+    it('handles client errors', async () => {
       mockTestAuth.mockImplementationOnce(() => Promise.reject(new Error('Invalid token')))
 
       const manager = new DiscordBotCredentialManager(tempDir)
@@ -127,7 +127,7 @@ describe('auth commands', () => {
   })
 
   describe('clearAction', () => {
-    test('removes all stored credentials', async () => {
+    it('removes all stored credentials', async () => {
       const manager = new DiscordBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'token123',
@@ -143,7 +143,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 DiscordBotCredentialManager(tempDir)
 
       const result = await statusAction({ _credManager: manager })
@@ -152,7 +152,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 DiscordBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'token123',
@@ -169,7 +169,7 @@ describe('auth commands', () => {
       expect(result.bot_name).toBe('testbot')
     })
 
-    test('returns status for specific --bot', async () => {
+    it('returns status for specific --bot', async () => {
       const manager = new DiscordBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'token1',
@@ -188,7 +188,7 @@ describe('auth commands', () => {
       expect(result.bot_id).toBe('bot123')
     })
 
-    test('returns invalid when token test fails', async () => {
+    it('returns invalid when token test fails', async () => {
       mockTestAuth.mockImplementationOnce(() => Promise.reject(new Error('Unauthorized')))
 
       const manager = new DiscordBotCredentialManager(tempDir)
@@ -203,7 +203,7 @@ describe('auth commands', () => {
       expect(result.valid).toBe(false)
     })
 
-    test('returns invalid when bot field is false', async () => {
+    it('returns invalid when bot field is false', async () => {
       mockTestAuth.mockImplementationOnce(() =>
         Promise.resolve({
           id: 'user123',
@@ -226,7 +226,7 @@ describe('auth commands', () => {
   })
 
   describe('listAction', () => {
-    test('returns all stored bots', async () => {
+    it('returns all stored bots', async () => {
       const manager = new DiscordBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'token1',
@@ -245,7 +245,7 @@ describe('auth commands', () => {
       expect(result.bots?.find((b) => b.bot_id === 'bot2')?.is_current).toBe(true)
     })
 
-    test('returns empty list when no bots stored', async () => {
+    it('returns empty list when no bots stored', async () => {
       const manager = new DiscordBotCredentialManager(tempDir)
 
       const result = await listAction({ _credManager: manager })
@@ -255,7 +255,7 @@ describe('auth commands', () => {
   })
 
   describe('useAction', () => {
-    test('switches current bot', async () => {
+    it('switches current bot', async () => {
       const manager = new DiscordBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'token1',
@@ -274,7 +274,7 @@ describe('auth commands', () => {
       expect(result.bot_id).toBe('bot1')
     })
 
-    test('returns error for unknown bot', async () => {
+    it('returns error for unknown bot', async () => {
       const manager = new DiscordBotCredentialManager(tempDir)
 
       const result = await useAction('nonexistent', { _credManager: manager })
@@ -284,7 +284,7 @@ describe('auth commands', () => {
   })
 
   describe('removeAction', () => {
-    test('removes a stored bot', async () => {
+    it('removes a stored bot', async () => {
       const manager = new DiscordBotCredentialManager(tempDir)
       await manager.setCredentials({
         token: 'token1',
@@ -298,7 +298,7 @@ describe('auth commands', () => {
       expect(await manager.getCredentials('bot1')).toBeNull()
     })
 
-    test('returns error for unknown bot', async () => {
+    it('returns error for unknown bot', async () => {
       const manager = new DiscordBotCredentialManager(tempDir)
 
       const result = await removeAction('nonexistent', { _credManager: manager })