agent-messenger 2.10.1 → 2.10.2

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 const originalConsoleLog = console.log
 
@@ -86,7 +86,7 @@ describe('auth commands', () => {
 
   // list has no throwing tests — run first to avoid Commander state corruption
   describe('list', () => {
-    test('outputs empty array when no accounts', async () => {
+    it('outputs empty array when no accounts', async () => {
       mockListAccounts.mockImplementation(() => Promise.resolve([]))
 
       await authCommand.parseAsync(['list'], { from: 'user' })
@@ -95,7 +95,7 @@ describe('auth commands', () => {
       expect(output).toEqual([])
     })
 
-    test('outputs accounts list with is_current flag', async () => {
+    it('outputs accounts list with is_current flag', async () => {
       mockListAccounts.mockImplementation(() =>
         Promise.resolve([
           {
@@ -130,7 +130,7 @@ describe('auth commands', () => {
 
   // use: success test first, then throwing test last
   describe('use', () => {
-    test('switches to specified account and outputs success', async () => {
+    it('switches to specified account and outputs success', async () => {
       mockSetCurrent.mockImplementation(() => Promise.resolve(true))
       mockGetAccount.mockImplementation(() =>
         Promise.resolve({
@@ -150,7 +150,7 @@ describe('auth commands', () => {
       expect(output.account_id).toBe('plus-12025551234')
     })
 
-    test('outputs error and exits when account not found', async () => {
+    it('outputs error and exits when account not found', async () => {
       mockSetCurrent.mockImplementation(() => Promise.resolve(false))
 
       await expect(authCommand.parseAsync(['use', 'nonexistent'], { from: 'user' })).rejects.toThrow('process.exit(1)')
@@ -163,7 +163,7 @@ describe('auth commands', () => {
 
   // status: no-account tests first, --account tests last (avoids Commander option caching)
   describe('status', () => {
-    test('outputs account info when account exists', async () => {
+    it('outputs account info when account exists', async () => {
       mockGetAccount.mockImplementation(() =>
         Promise.resolve({
           account_id: 'plus-12025551234',
@@ -182,7 +182,7 @@ describe('auth commands', () => {
       expect(output.name).toBe('Test User')
     })
 
-    test('outputs error and exits when no account configured', async () => {
+    it('outputs error and exits when no account configured', async () => {
       mockGetAccount.mockImplementation(() => Promise.resolve(null))
 
       await expect(authCommand.parseAsync(['status'], { from: 'user' })).rejects.toThrow('process.exit(1)')
@@ -193,7 +193,7 @@ describe('auth commands', () => {
       expect(output.error).toContain('No WhatsApp account configured')
     })
 
-    test('passes --account option to getAccount', async () => {
+    it('passes --account option to getAccount', async () => {
       mockGetAccount.mockImplementation((id?: string) => {
         if (id === 'plus-19995551234') {
           return Promise.resolve({
@@ -214,7 +214,7 @@ describe('auth commands', () => {
       expect(output.account_id).toBe('plus-19995551234')
     })
 
-    test('outputs error for specific missing account', async () => {
+    it('outputs error for specific missing account', async () => {
       mockGetAccount.mockImplementation(() => Promise.resolve(null))
 
       await expect(authCommand.parseAsync(['status', '--account', 'missing-id'], { from: 'user' })).rejects.toThrow(
@@ -229,7 +229,7 @@ describe('auth commands', () => {
 
   // logout: no-account tests first, --account test last (avoids Commander option caching)
   describe('logout', () => {
-    test('removes current account and outputs success', async () => {
+    it('removes current account and outputs success', async () => {
       mockGetAccount.mockImplementation(() =>
         Promise.resolve({
           account_id: 'plus-12025551234',
@@ -250,7 +250,7 @@ describe('auth commands', () => {
       expect(output.logged_out).toBe(true)
     })
 
-    test('outputs error and exits when no account configured', async () => {
+    it('outputs error and exits when no account configured', async () => {
       mockGetAccount.mockImplementation(() => Promise.resolve(null))
 
       await expect(authCommand.parseAsync(['logout'], { from: 'user' })).rejects.toThrow('process.exit(1)')
@@ -260,7 +260,7 @@ describe('auth commands', () => {
       expect(output.error).toBeDefined()
     })
 
-    test('proceeds with local cleanup even when client connection fails', async () => {
+    it('proceeds with local cleanup even when client connection fails', async () => {
       mockGetAccount.mockImplementation(() =>
         Promise.resolve({
           account_id: 'plus-12025551234',
@@ -280,7 +280,7 @@ describe('auth commands', () => {
       expect(output.success).toBe(true)
     })
 
-    test('removes specific account with --account flag', async () => {
+    it('removes specific account with --account flag', async () => {
       mockGetAccount.mockImplementation((id?: string) => {
         if (id === 'plus-19995551234') {
           return Promise.resolve({

package/src/platforms/whatsapp/commands/chat.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 const originalConsoleLog = console.log
 
@@ -130,7 +130,7 @@ describe('chat commands', () => {
   })
 
   describe('list', () => {
-    test('lists chats with default limit', async () => {
+    it('lists chats with default limit', async () => {
       await expect(chatCommand.parseAsync(['list'], { from: 'user' })).rejects.toThrow('process.exit(0)')
 
       expect(mockListChats).toHaveBeenCalledWith(20)
@@ -140,7 +140,7 @@ describe('chat commands', () => {
       expect(output[0].name).toBe('Alice')
     })
 
-    test('respects --limit option', async () => {
+    it('respects --limit option', async () => {
       await expect(chatCommand.parseAsync(['list', '--limit', '5'], { from: 'user' })).rejects.toThrow(
         'process.exit(0)',
       )
@@ -148,7 +148,7 @@ describe('chat commands', () => {
       expect(mockListChats).toHaveBeenCalledWith(5)
     })
 
-    test('passes account option to credential manager', async () => {
+    it('passes account option to credential manager', async () => {
       await expect(chatCommand.parseAsync(['list', '--account', 'my-account'], { from: 'user' })).rejects.toThrow(
         'process.exit(0)',
       )
@@ -156,7 +156,7 @@ describe('chat commands', () => {
       expect(mockGetAccount).toHaveBeenCalledWith('my-account')
     })
 
-    test('exits with error when no account configured', async () => {
+    it('exits with error when no account configured', async () => {
       mockGetAccount.mockImplementation(() => Promise.resolve(null))
 
       await expect(chatCommand.parseAsync(['list'], { from: 'user' })).rejects.toThrow('process.exit(1)')
@@ -166,7 +166,7 @@ describe('chat commands', () => {
   })
 
   describe('search', () => {
-    test('searches chats by query', async () => {
+    it('searches chats by query', async () => {
       await expect(chatCommand.parseAsync(['search', 'Bob'], { from: 'user' })).rejects.toThrow('process.exit(0)')
 
       expect(mockSearchChats).toHaveBeenCalledWith('Bob', 20)
@@ -176,7 +176,7 @@ describe('chat commands', () => {
       expect(output[0].name).toBe('Bob')
     })
 
-    test('respects --limit option', async () => {
+    it('respects --limit option', async () => {
       await expect(chatCommand.parseAsync(['search', 'Alice', '--limit', '3'], { from: 'user' })).rejects.toThrow(
         'process.exit(0)',
       )
@@ -184,7 +184,7 @@ describe('chat commands', () => {
       expect(mockSearchChats).toHaveBeenCalledWith('Alice', 3)
     })
 
-    test('passes account option to credential manager', async () => {
+    it('passes account option to credential manager', async () => {
       await expect(
         chatCommand.parseAsync(['search', 'test', '--account', 'my-account'], { from: 'user' }),
       ).rejects.toThrow('process.exit(0)')

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 const originalConsoleLog = console.log
 
@@ -130,7 +130,7 @@ describe('message commands', () => {
   })
 
   describe('list', () => {
-    test('fetches messages for a chat', async () => {
+    it('fetches messages for a chat', async () => {
       await expect(messageCommand.parseAsync(['list', '12025551234@s.whatsapp.net'], { from: 'user' })).rejects.toThrow(
         'process.exit(0)',
       )
@@ -142,7 +142,7 @@ describe('message commands', () => {
       expect(output[0].text).toBe('Hello')
     })
 
-    test('respects --limit option', async () => {
+    it('respects --limit option', async () => {
       await expect(
         messageCommand.parseAsync(['list', '12025551234@s.whatsapp.net', '--limit', '10'], { from: 'user' }),
       ).rejects.toThrow('process.exit(0)')
@@ -150,7 +150,7 @@ describe('message commands', () => {
       expect(mockGetMessages).toHaveBeenCalledWith('12025551234@s.whatsapp.net', 10)
     })
 
-    test('passes account option to credential manager', async () => {
+    it('passes account option to credential manager', async () => {
       await expect(
         messageCommand.parseAsync(['list', '12025551234@s.whatsapp.net', '--account', 'my-account'], { from: 'user' }),
       ).rejects.toThrow('process.exit(0)')
@@ -158,7 +158,7 @@ describe('message commands', () => {
       expect(mockGetAccount).toHaveBeenCalledWith('my-account')
     })
 
-    test('exits with error when no account configured', async () => {
+    it('exits with error when no account configured', async () => {
       mockGetAccount.mockImplementation(() => Promise.resolve(null))
 
       await expect(messageCommand.parseAsync(['list', '12025551234@s.whatsapp.net'], { from: 'user' })).rejects.toThrow(
@@ -170,7 +170,7 @@ describe('message commands', () => {
   })
 
   describe('send', () => {
-    test('sends a message to a chat', async () => {
+    it('sends a message to a chat', async () => {
       await expect(
         messageCommand.parseAsync(['send', '12025551234@s.whatsapp.net', 'Hello world'], { from: 'user' }),
       ).rejects.toThrow('process.exit(0)')
@@ -181,7 +181,7 @@ describe('message commands', () => {
       expect(output.text).toBe('Hi there')
     })
 
-    test('passes account option to credential manager', async () => {
+    it('passes account option to credential manager', async () => {
       await expect(
         messageCommand.parseAsync(['send', '12025551234@s.whatsapp.net', 'Hi', '--account', 'my-account'], {
           from: 'user',
@@ -193,7 +193,7 @@ describe('message commands', () => {
   })
 
   describe('react', () => {
-    test('sends a reaction to a message', async () => {
+    it('sends a reaction to a message', async () => {
       await expect(
         messageCommand.parseAsync(['react', '12025551234@s.whatsapp.net', 'msg-1', '👍'], { from: 'user' }),
       ).rejects.toThrow('process.exit(0)')
@@ -206,7 +206,7 @@ describe('message commands', () => {
       expect(output.emoji).toBe('👍')
     })
 
-    test('passes --from-me flag to sendReaction', async () => {
+    it('passes --from-me flag to sendReaction', async () => {
       await expect(
         messageCommand.parseAsync(['react', '12025551234@s.whatsapp.net', 'msg-1', '❤️', '--from-me'], {
           from: 'user',
@@ -216,7 +216,7 @@ describe('message commands', () => {
       expect(mockSendReaction).toHaveBeenCalledWith('12025551234@s.whatsapp.net', 'msg-1', '❤️', true)
     })
 
-    test('passes account option to credential manager', async () => {
+    it('passes account option to credential manager', async () => {
       await expect(
         messageCommand.parseAsync(['react', '12025551234@s.whatsapp.net', 'msg-1', '👍', '--account', 'my-account'], {
           from: 'user',

package/src/platforms/whatsapp/commands/whoami.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 type { WhatsAppClient } from '../client'
 import * as sharedModule from './shared'
@@ -32,7 +32,7 @@ describe('whoami command', () => {
     consoleLogSpy?.mockRestore()
   })
 
-  test('outputs profile information', async () => {
+  it('outputs profile information', async () => {
     await whoamiAction({})
 
     expect(consoleLogSpy).toHaveBeenCalledTimes(1)
@@ -42,7 +42,7 @@ describe('whoami command', () => {
     expect(output.phone_number).toBe('12025551234')
   })
 
-  test('outputs profile with null name', async () => {
+  it('outputs profile with null name', async () => {
     mockProfileData = {
       id: '12025551234@s.whatsapp.net',
       name: null,

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

@@ -1,4 +1,4 @@
-import { afterAll, describe, expect, test } from 'bun:test'
+import { afterAll, describe, expect, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { join } from 'node:path'
 
@@ -35,7 +35,7 @@ afterAll(() => {
 
 describe('WhatsAppCredentialManager', () => {
   describe('loadConfig', () => {
-    test('returns default config when file does not exist', async () => {
+    it('returns default config when file does not exist', async () => {
       const manager = setup()
       const config = await manager.loadConfig()
 
@@ -44,7 +44,7 @@ describe('WhatsAppCredentialManager', () => {
   })
 
   describe('saveConfig', () => {
-    test('creates file and can be re-read via loadConfig', async () => {
+    it('creates file and can be re-read via loadConfig', async () => {
       const manager = setup()
       const config = {
         current: 'test-account',
@@ -61,14 +61,14 @@ describe('WhatsAppCredentialManager', () => {
   })
 
   describe('getAccount', () => {
-    test('returns null when no accounts exist', async () => {
+    it('returns null when no accounts exist', async () => {
       const manager = setup()
       const account = await manager.getAccount()
 
       expect(account).toBeNull()
     })
 
-    test('returns null for specific accountId when no accounts exist', async () => {
+    it('returns null for specific accountId when no accounts exist', async () => {
       const manager = setup()
       const account = await manager.getAccount('nonexistent')
 
@@ -77,7 +77,7 @@ describe('WhatsAppCredentialManager', () => {
   })
 
   describe('setAccount', () => {
-    test('round-trips: set then get returns same account', async () => {
+    it('round-trips: set then get returns same account', async () => {
       const manager = setup()
       const account = makeAccount()
 
@@ -87,7 +87,7 @@ describe('WhatsAppCredentialManager', () => {
       expect(retrieved).toEqual(account)
     })
 
-    test('sets first account as current automatically', async () => {
+    it('sets first account as current automatically', async () => {
       const manager = setup()
       const account = makeAccount()
 
@@ -97,7 +97,7 @@ describe('WhatsAppCredentialManager', () => {
       expect(current).toEqual(account)
     })
 
-    test('does not override current when it is already set', async () => {
+    it('does not override current when it is already set', async () => {
       const manager = setup()
       const first = makeAccount({ account_id: 'first-account' })
       const second = makeAccount({ account_id: 'second-account' })
@@ -109,7 +109,7 @@ describe('WhatsAppCredentialManager', () => {
       expect(current?.account_id).toBe('first-account')
     })
 
-    test('getAccount with normalized ID lookup via createAccountId', async () => {
+    it('looks up account by normalized ID via createAccountId', async () => {
       const manager = setup()
       const account = makeAccount({ account_id: 'plus-12025551234' })
 
@@ -121,14 +121,14 @@ describe('WhatsAppCredentialManager', () => {
   })
 
   describe('listAccounts', () => {
-    test('returns empty array when no accounts', async () => {
+    it('returns empty array when no accounts', async () => {
       const manager = setup()
       const accounts = await manager.listAccounts()
 
       expect(accounts).toEqual([])
     })
 
-    test('returns all accounts with is_current flag', async () => {
+    it('returns all accounts with is_current flag', async () => {
       const manager = setup()
       const first = makeAccount({ account_id: 'first-account' })
       const second = makeAccount({ account_id: 'second-account' })
@@ -147,7 +147,7 @@ describe('WhatsAppCredentialManager', () => {
   })
 
   describe('setCurrent', () => {
-    test('switches active account and returns true', async () => {
+    it('switches active account and returns true', async () => {
       const manager = setup()
       const first = makeAccount({ account_id: 'first-account' })
       const second = makeAccount({ account_id: 'second-account' })
@@ -162,7 +162,7 @@ describe('WhatsAppCredentialManager', () => {
       expect(current?.account_id).toBe('second-account')
     })
 
-    test('returns false for non-existent account', async () => {
+    it('returns false for non-existent account', async () => {
       const manager = setup()
 
       const result = await manager.setCurrent('nonexistent')
@@ -172,7 +172,7 @@ describe('WhatsAppCredentialManager', () => {
   })
 
   describe('removeAccount', () => {
-    test('removes account and adjusts current to next available', async () => {
+    it('removes account and adjusts current to next available', async () => {
       const manager = setup()
       const first = makeAccount({ account_id: 'first-account' })
       const second = makeAccount({ account_id: 'second-account' })
@@ -191,7 +191,7 @@ describe('WhatsAppCredentialManager', () => {
       expect(current?.account_id).toBe('second-account')
     })
 
-    test('returns false for non-existent account', async () => {
+    it('returns false for non-existent account', async () => {
       const manager = setup()
 
       const result = await manager.removeAccount('nonexistent')
@@ -199,7 +199,7 @@ describe('WhatsAppCredentialManager', () => {
       expect(result).toBe(false)
     })
 
-    test('deletes session directory even when account is not in config', async () => {
+    it('deletes session directory even when account is not in config', async () => {
       const manager = setup()
 
       const paths = await manager.ensureAccountPaths('orphan-account')
@@ -211,7 +211,7 @@ describe('WhatsAppCredentialManager', () => {
       expect(existsSync(paths.account_dir)).toBe(false)
     })
 
-    test('returns false when neither config entry nor directory exists', async () => {
+    it('returns false when neither config entry nor directory exists', async () => {
       const manager = setup()
 
       const result = await manager.removeAccount('ghost-account')
@@ -221,7 +221,7 @@ describe('WhatsAppCredentialManager', () => {
   })
 
   describe('clearCredentials', () => {
-    test('removes credentials file when it exists', async () => {
+    it('removes credentials file when it exists', async () => {
       const manager = setup()
       const account = makeAccount()
       await manager.setAccount(account)
@@ -235,7 +235,7 @@ describe('WhatsAppCredentialManager', () => {
       expect(existsSync(credentialsPath)).toBe(false)
     })
 
-    test('does not throw when credentials file does not exist', async () => {
+    it('does not throw when credentials file does not exist', async () => {
       const manager = setup()
 
       await expect(manager.clearCredentials()).resolves.toBeUndefined()
@@ -243,7 +243,7 @@ describe('WhatsAppCredentialManager', () => {
   })
 
   describe('getAccountPaths', () => {
-    test('returns correct paths structure for account ID', async () => {
+    it('returns correct paths structure for account ID', async () => {
       const testConfigDir = join(
         import.meta.dir,
         `.test-whatsapp-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
@@ -257,7 +257,7 @@ describe('WhatsAppCredentialManager', () => {
       expect(paths.auth_dir).toBe(join(testConfigDir, 'whatsapp', 'test-account', 'auth'))
     })
 
-    test('normalizes account ID via createAccountId', async () => {
+    it('normalizes account ID via createAccountId', async () => {
       const testConfigDir = join(
         import.meta.dir,
         `.test-whatsapp-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
@@ -273,7 +273,7 @@ describe('WhatsAppCredentialManager', () => {
   })
 
   describe('ensureAccountPaths', () => {
-    test('creates directories', async () => {
+    it('creates directories', async () => {
       const manager = setup()
 
       const paths = await manager.ensureAccountPaths('test-account')
@@ -281,7 +281,7 @@ describe('WhatsAppCredentialManager', () => {
       expect(existsSync(paths.auth_dir)).toBe(true)
     })
 
-    test('returns paths structure', async () => {
+    it('returns paths structure', async () => {
       const testConfigDir = join(
         import.meta.dir,
         `.test-whatsapp-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,

package/src/platforms/whatsapp/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 * as fs from 'node:fs'
 
 import { WhatsAppCredentialManager } from '@/platforms/whatsapp/credential-manager'
@@ -47,7 +47,7 @@ afterEach(() => {
 })
 
 describe('ensureWhatsAppAuth', () => {
-  test('exits with error when no account configured', async () => {
+  it('exits with error when no account configured', async () => {
     // given
     getAccountSpy.mockResolvedValue(null)
 
@@ -57,7 +57,7 @@ describe('ensureWhatsAppAuth', () => {
     expect(consoleSpy).toHaveBeenCalled()
   })
 
-  test('exits with error when creds file missing', async () => {
+  it('exits with error when creds file missing', async () => {
     // given
     getAccountSpy.mockResolvedValue(validAccount)
     existsSyncSpy.mockReturnValue(false)
@@ -68,7 +68,7 @@ describe('ensureWhatsAppAuth', () => {
     expect(consoleSpy).toHaveBeenCalled()
   })
 
-  test('succeeds when account and creds file exist', async () => {
+  it('succeeds when account and creds file exist', async () => {
     // given
     getAccountSpy.mockResolvedValue(validAccount)
     existsSyncSpy.mockReturnValue(true)

package/src/platforms/whatsapp/index.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   createAccountId,
@@ -10,30 +10,30 @@ import {
   WhatsAppError,
 } from '@/platforms/whatsapp/index'
 
-test('WhatsAppClient is exported from barrel', () => {
+it('WhatsAppClient is exported from barrel', () => {
   expect(typeof WhatsAppClient).toBe('function')
 })
 
-test('WhatsAppCredentialManager is exported from barrel', () => {
+it('WhatsAppCredentialManager is exported from barrel', () => {
   expect(typeof WhatsAppCredentialManager).toBe('function')
 })
 
-test('WhatsAppError is exported from barrel', () => {
+it('WhatsAppError is exported from barrel', () => {
   expect(typeof WhatsAppError).toBe('function')
 })
 
-test('createAccountId is exported from barrel', () => {
+it('createAccountId is exported from barrel', () => {
   expect(typeof createAccountId).toBe('function')
 })
 
-test('jidToType is exported from barrel', () => {
+it('jidToType is exported from barrel', () => {
   expect(typeof jidToType).toBe('function')
 })
 
-test('extractMessageText is exported from barrel', () => {
+it('extractMessageText is exported from barrel', () => {
   expect(typeof extractMessageText).toBe('function')
 })
 
-test('getMessageType is exported from barrel', () => {
+it('getMessageType is exported from barrel', () => {
   expect(typeof getMessageType).toBe('function')
 })