agent-messenger 2.10.0 → 2.10.2

package/src/platforms/whatsappbot/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 mockSendTextMessage = mock(() =>
   Promise.resolve({
@@ -64,7 +64,7 @@ describe('message commands', () => {
   })
 
   describe('sendAction', () => {
-    test('sends a text message and returns result', async () => {
+    it('sends a text message and returns result', async () => {
       const result = await sendAction('+1234567890', 'Hello!', {})
 
       expect(result.messaging_product).toBe('whatsapp')
@@ -73,7 +73,7 @@ describe('message commands', () => {
       expect(mockSendTextMessage).toHaveBeenCalledWith('+1234567890', 'Hello!')
     })
 
-    test('returns error when client throws', async () => {
+    it('returns error when client throws', async () => {
       mockSendTextMessage.mockImplementationOnce(() => Promise.reject(new Error('Network error')))
 
       const result = await sendAction('+1234567890', 'Hello!', {})
@@ -83,7 +83,7 @@ describe('message commands', () => {
   })
 
   describe('sendTemplateAction', () => {
-    test('sends a template message with default language', async () => {
+    it('sends a template message with default language', async () => {
       const result = await sendTemplateAction('+1234567890', 'hello_world', {})
 
       expect(result.messaging_product).toBe('whatsapp')
@@ -91,14 +91,14 @@ describe('message commands', () => {
       expect(mockSendTemplateMessage).toHaveBeenCalledWith('+1234567890', 'hello_world', 'en_US', undefined)
     })
 
-    test('sends a template message with custom language', async () => {
+    it('sends a template message with custom language', async () => {
       const result = await sendTemplateAction('+1234567890', 'hello_world', { language: 'pt_BR' })
 
       expect(mockSendTemplateMessage).toHaveBeenCalledWith('+1234567890', 'hello_world', 'pt_BR', undefined)
       expect(result.error).toBeUndefined()
     })
 
-    test('parses and passes components JSON', async () => {
+    it('parses and passes components JSON', async () => {
       const components = [{ type: 'body', parameters: [{ type: 'text', text: 'World' }] }]
       const result = await sendTemplateAction('+1234567890', 'hello_world', {
         components: JSON.stringify(components),
@@ -108,7 +108,7 @@ describe('message commands', () => {
       expect(result.error).toBeUndefined()
     })
 
-    test('returns error for invalid components JSON', async () => {
+    it('returns error for invalid components JSON', async () => {
       const result = await sendTemplateAction('+1234567890', 'hello_world', {
         components: 'not-valid-json',
       })
@@ -116,7 +116,7 @@ describe('message commands', () => {
       expect(result.error).toBe('Invalid --components JSON')
     })
 
-    test('returns error when client throws', async () => {
+    it('returns error when client throws', async () => {
       mockSendTemplateMessage.mockImplementationOnce(() => Promise.reject(new Error('Template not found')))
 
       const result = await sendTemplateAction('+1234567890', 'missing_template', {})
@@ -126,7 +126,7 @@ describe('message commands', () => {
   })
 
   describe('sendReactionAction', () => {
-    test('sends a reaction and returns result', async () => {
+    it('sends a reaction and returns result', async () => {
       const result = await sendReactionAction('+1234567890', 'wamid.msg123', '👍', {})
 
       expect(result.messaging_product).toBe('whatsapp')
@@ -134,7 +134,7 @@ describe('message commands', () => {
       expect(mockSendReaction).toHaveBeenCalledWith('+1234567890', 'wamid.msg123', '👍')
     })
 
-    test('returns error when client throws', async () => {
+    it('returns error when client throws', async () => {
       mockSendReaction.mockImplementationOnce(() => Promise.reject(new Error('Message not found')))
 
       const result = await sendReactionAction('+1234567890', 'wamid.bad', '👍', {})
@@ -144,7 +144,7 @@ describe('message commands', () => {
   })
 
   describe('sendImageAction', () => {
-    test('sends an image message and returns result', async () => {
+    it('sends an image message and returns result', async () => {
       const result = await sendImageAction('+1234567890', 'https://example.com/image.jpg', {})
 
       expect(result.messaging_product).toBe('whatsapp')
@@ -152,14 +152,14 @@ describe('message commands', () => {
       expect(mockSendImageMessage).toHaveBeenCalledWith('+1234567890', 'https://example.com/image.jpg', undefined)
     })
 
-    test('passes caption when provided', async () => {
+    it('passes caption when provided', async () => {
       const result = await sendImageAction('+1234567890', 'https://example.com/image.jpg', { caption: 'My photo' })
 
       expect(mockSendImageMessage).toHaveBeenCalledWith('+1234567890', 'https://example.com/image.jpg', 'My photo')
       expect(result.error).toBeUndefined()
     })
 
-    test('returns error when client throws', async () => {
+    it('returns error when client throws', async () => {
       mockSendImageMessage.mockImplementationOnce(() => Promise.reject(new Error('Invalid URL')))
 
       const result = await sendImageAction('+1234567890', 'bad-url', {})
@@ -169,7 +169,7 @@ describe('message commands', () => {
   })
 
   describe('sendDocumentAction', () => {
-    test('sends a document message and returns result', async () => {
+    it('sends a document message and returns result', async () => {
       const result = await sendDocumentAction('+1234567890', 'https://example.com/doc.pdf', {})
 
       expect(result.messaging_product).toBe('whatsapp')
@@ -182,7 +182,7 @@ describe('message commands', () => {
       )
     })
 
-    test('passes filename and caption when provided', async () => {
+    it('passes filename and caption when provided', async () => {
       const result = await sendDocumentAction('+1234567890', 'https://example.com/doc.pdf', {
         filename: 'report.pdf',
         caption: 'Monthly report',
@@ -197,7 +197,7 @@ describe('message commands', () => {
       expect(result.error).toBeUndefined()
     })
 
-    test('returns error when client throws', async () => {
+    it('returns error when client throws', async () => {
       mockSendDocumentMessage.mockImplementationOnce(() => Promise.reject(new Error('Upload failed')))
 
       const result = await sendDocumentAction('+1234567890', 'https://example.com/doc.pdf', {})

package/src/platforms/whatsappbot/commands/template.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 type { WhatsAppBotTemplate } from '../types'
 
@@ -46,7 +46,7 @@ describe('template commands', () => {
   })
 
   describe('listAction', () => {
-    test('returns all templates', async () => {
+    it('returns all templates', async () => {
       const result = await listAction({})
 
       expect(result.templates).toHaveLength(2)
@@ -55,28 +55,28 @@ describe('template commands', () => {
       expect(mockListTemplates).toHaveBeenCalledWith(undefined)
     })
 
-    test('passes limit when provided', async () => {
+    it('passes limit when provided', async () => {
       const result = await listAction({ limit: '1' })
 
       expect(mockListTemplates).toHaveBeenCalledWith({ limit: 1 })
       expect(result.error).toBeUndefined()
     })
 
-    test('returns error for invalid limit', async () => {
+    it('returns error for invalid limit', async () => {
       const result = await listAction({ limit: 'abc' })
 
       expect(result.error).toBeDefined()
       expect(result.error).toContain('Invalid --limit')
     })
 
-    test('returns error for zero limit', async () => {
+    it('returns error for zero limit', async () => {
       const result = await listAction({ limit: '0' })
 
       expect(result.error).toBeDefined()
       expect(result.error).toContain('Invalid --limit')
     })
 
-    test('returns error when client throws', async () => {
+    it('returns error when client throws', async () => {
       mockListTemplates.mockImplementationOnce(() => Promise.reject(new Error('API error')))
 
       const result = await listAction({})
@@ -86,7 +86,7 @@ describe('template commands', () => {
   })
 
   describe('getAction', () => {
-    test('returns a specific template by name', async () => {
+    it('returns a specific template by name', async () => {
       const result = await getAction('hello_world', {})
 
       expect(result.template).toBeDefined()
@@ -95,13 +95,13 @@ describe('template commands', () => {
       expect(mockGetTemplate).toHaveBeenCalledWith('hello_world')
     })
 
-    test('returns error when template not found', async () => {
+    it('returns error when template not found', async () => {
       const result = await getAction('nonexistent_template', {})
 
       expect(result.error).toContain('nonexistent_template')
     })
 
-    test('returns error when client throws', async () => {
+    it('returns error when client throws', async () => {
       mockGetTemplate.mockImplementationOnce(() => Promise.reject(new Error('Network error')))
 
       const result = await getAction('hello_world', {})

package/src/platforms/whatsappbot/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'
@@ -35,7 +35,7 @@ describe('whoami command', () => {
     }
   })
 
-  test('returns phone number id, account name, and verified name', async () => {
+  it('returns phone number id, account name, and verified name', async () => {
     const manager = new WhatsAppBotCredentialManager(tempDir)
     await manager.setCredentials({
       phone_number_id: '12345678901',
@@ -51,7 +51,7 @@ describe('whoami command', () => {
     expect(result.error).toBeUndefined()
   })
 
-  test('returns info for specific --account', async () => {
+  it('returns info for specific --account', async () => {
     const manager = new WhatsAppBotCredentialManager(tempDir)
     await manager.setCredentials({
       phone_number_id: '11111111111',
@@ -71,7 +71,7 @@ describe('whoami command', () => {
     expect(result.verified_name).toBe('Test Business')
   })
 
-  test('returns error when verifyToken fails', async () => {
+  it('returns error when verifyToken fails', async () => {
     mockVerifyToken.mockImplementationOnce(() => Promise.reject(new Error('Invalid token')))
 
     const manager = new WhatsAppBotCredentialManager(tempDir)
@@ -87,7 +87,7 @@ describe('whoami command', () => {
     expect(result.error).toContain('Invalid token')
   })
 
-  test('returns error when no credentials', async () => {
+  it('returns error when no credentials', async () => {
     const manager = new WhatsAppBotCredentialManager(tempDir)
 
     const result = await whoamiAction({ _credManager: manager })

package/src/platforms/whatsappbot/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'
 
@@ -22,7 +22,7 @@ afterAll(() => {
 })
 
 describe('WhatsAppBotCredentialManager', () => {
-  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()
 
@@ -32,7 +32,7 @@ describe('WhatsAppBotCredentialManager', () => {
     })
   })
 
-  test('save creates file with correct permissions', async () => {
+  it('save creates file with correct permissions', async () => {
     const testConfigDir = join(
       import.meta.dir,
       `.test-whatsappbot-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
@@ -61,13 +61,13 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(loaded).toEqual(config)
   })
 
-  test('getCredentials returns null when not configured', async () => {
+  it('getCredentials returns null when not configured', async () => {
     const manager = setup()
     const creds = await manager.getCredentials()
     expect(creds).toBeNull()
   })
 
-  test('getCredentials returns credentials from env vars when E2E env vars are set', async () => {
+  it('getCredentials returns credentials from env vars when E2E env vars are set', async () => {
     const originalAccessToken = process.env.E2E_WHATSAPPBOT_ACCESS_TOKEN
     const originalPhoneNumberId = process.env.E2E_WHATSAPPBOT_PHONE_NUMBER_ID
 
@@ -96,7 +96,7 @@ describe('WhatsAppBotCredentialManager', () => {
     }
   })
 
-  test('getCredentials ignores env vars when accountId is provided', async () => {
+  it('getCredentials ignores env vars when accountId is provided', async () => {
     const originalAccessToken = process.env.E2E_WHATSAPPBOT_ACCESS_TOKEN
     const originalPhoneNumberId = process.env.E2E_WHATSAPPBOT_PHONE_NUMBER_ID
 
@@ -121,7 +121,7 @@ describe('WhatsAppBotCredentialManager', () => {
     }
   })
 
-  test('getCredentials returns specific account by accountId', async () => {
+  it('getCredentials returns specific account by accountId', async () => {
     const manager = setup()
     await manager.setCredentials({
       phone_number_id: 'phone-1',
@@ -142,13 +142,13 @@ describe('WhatsAppBotCredentialManager', () => {
     })
   })
 
-  test('getCredentials returns null for nonexistent accountId', async () => {
+  it('getCredentials returns null for nonexistent accountId', async () => {
     const manager = setup()
     const creds = await manager.getCredentials('nonexistent')
     expect(creds).toBeNull()
   })
 
-  test('setCredentials saves and sets as current', async () => {
+  it('setCredentials saves and sets as current', async () => {
     const manager = setup()
     await manager.setCredentials({
       phone_number_id: '123456789',
@@ -167,7 +167,7 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(config.current).toEqual({ account_id: '123456789' })
   })
 
-  test('removeAccount deletes account and adjusts current', async () => {
+  it('removeAccount deletes account and adjusts current', async () => {
     const manager = setup()
     await manager.setCredentials({
       phone_number_id: '123456789',
@@ -186,13 +186,13 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(config.accounts['123456789']).toBeUndefined()
   })
 
-  test('removeAccount returns false for non-existent account', async () => {
+  it('removeAccount returns false for non-existent account', async () => {
     const manager = setup()
     const removed = await manager.removeAccount('nonexistent')
     expect(removed).toBe(false)
   })
 
-  test('removeAccount does not clear current when a different account is removed', async () => {
+  it('removeAccount does not clear current when a different account is removed', async () => {
     const manager = setup()
     await manager.setCredentials({
       phone_number_id: 'phone-1',
@@ -212,7 +212,7 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(config.current).toEqual({ account_id: 'phone-2' })
   })
 
-  test('setCurrent switches active account', async () => {
+  it('setCurrent switches active account', async () => {
     const manager = setup()
     await manager.setCredentials({
       phone_number_id: 'phone-1',
@@ -232,13 +232,13 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(config.current).toEqual({ account_id: 'phone-1' })
   })
 
-  test('setCurrent returns false for non-existent account', async () => {
+  it('setCurrent returns false for non-existent account', async () => {
     const manager = setup()
     const result = await manager.setCurrent('nonexistent')
     expect(result).toBe(false)
   })
 
-  test('listAll returns all accounts with is_current flag', async () => {
+  it('listAll returns all accounts with is_current flag', async () => {
     const manager = setup()
     await manager.setCredentials({
       phone_number_id: 'phone-1',
@@ -261,13 +261,13 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(phone2?.is_current).toBe(true)
   })
 
-  test('listAll returns empty array when no accounts', async () => {
+  it('listAll returns empty array when no accounts', async () => {
     const manager = setup()
     const accounts = await manager.listAll()
     expect(accounts).toHaveLength(0)
   })
 
-  test('clearCredentials resets everything', async () => {
+  it('clearCredentials resets everything', async () => {
     const manager = setup()
     await manager.setCredentials({
       phone_number_id: '123456789',
@@ -282,7 +282,7 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(config.accounts).toEqual({})
   })
 
-  test('round-trip: set → get → remove → get null', async () => {
+  it('completes round-trip: set → get → remove → get null', async () => {
     const manager = setup()
 
     await manager.setCredentials({

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

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   WhatsAppBotAccountEntrySchema,
@@ -9,26 +9,26 @@ import {
   WhatsAppBotError,
 } from '@/platforms/whatsappbot/index'
 
-test('WhatsAppBotClient is exported from barrel', () => {
+it('WhatsAppBotClient is exported from barrel', () => {
   expect(typeof WhatsAppBotClient).toBe('function')
 })
 
-test('WhatsAppBotCredentialManager is exported from barrel', () => {
+it('WhatsAppBotCredentialManager is exported from barrel', () => {
   expect(typeof WhatsAppBotCredentialManager).toBe('function')
 })
 
-test('WhatsAppBotError is exported from barrel', () => {
+it('WhatsAppBotError is exported from barrel', () => {
   expect(typeof WhatsAppBotError).toBe('function')
 })
 
-test('WhatsAppBotAccountEntrySchema is exported from barrel', () => {
+it('WhatsAppBotAccountEntrySchema is exported from barrel', () => {
   expect(typeof WhatsAppBotAccountEntrySchema.parse).toBe('function')
 })
 
-test('WhatsAppBotConfigSchema is exported from barrel', () => {
+it('WhatsAppBotConfigSchema is exported from barrel', () => {
   expect(typeof WhatsAppBotConfigSchema.parse).toBe('function')
 })
 
-test('WhatsAppBotCredentialsSchema is exported from barrel', () => {
+it('WhatsAppBotCredentialsSchema is exported from barrel', () => {
   expect(typeof WhatsAppBotCredentialsSchema.parse).toBe('function')
 })

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

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   WhatsAppBotAccountEntrySchema,
@@ -7,7 +7,7 @@ import {
   WhatsAppBotError,
 } from '@/platforms/whatsappbot/types'
 
-test('WhatsAppBotAccountEntrySchema validates correct data', () => {
+it('WhatsAppBotAccountEntrySchema validates correct data', () => {
   const result = WhatsAppBotAccountEntrySchema.safeParse({
     phone_number_id: '123456789',
     account_name: 'Test Business',
@@ -16,7 +16,7 @@ test('WhatsAppBotAccountEntrySchema validates correct data', () => {
   expect(result.success).toBe(true)
 })
 
-test('WhatsAppBotAccountEntrySchema rejects missing phone_number_id', () => {
+it('WhatsAppBotAccountEntrySchema rejects missing phone_number_id', () => {
   const result = WhatsAppBotAccountEntrySchema.safeParse({
     account_name: 'Test Business',
     access_token: 'EAAtest123',
@@ -24,7 +24,7 @@ test('WhatsAppBotAccountEntrySchema rejects missing phone_number_id', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotAccountEntrySchema rejects missing account_name', () => {
+it('WhatsAppBotAccountEntrySchema rejects missing account_name', () => {
   const result = WhatsAppBotAccountEntrySchema.safeParse({
     phone_number_id: '123456789',
     access_token: 'EAAtest123',
@@ -32,7 +32,7 @@ test('WhatsAppBotAccountEntrySchema rejects missing account_name', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotAccountEntrySchema rejects missing access_token', () => {
+it('WhatsAppBotAccountEntrySchema rejects missing access_token', () => {
   const result = WhatsAppBotAccountEntrySchema.safeParse({
     phone_number_id: '123456789',
     account_name: 'Test Business',
@@ -40,7 +40,7 @@ test('WhatsAppBotAccountEntrySchema rejects missing access_token', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotConfigSchema validates with current account', () => {
+it('WhatsAppBotConfigSchema validates with current account', () => {
   const result = WhatsAppBotConfigSchema.safeParse({
     current: { account_id: '123456789' },
     accounts: {
@@ -54,7 +54,7 @@ test('WhatsAppBotConfigSchema validates with current account', () => {
   expect(result.success).toBe(true)
 })
 
-test('WhatsAppBotConfigSchema validates with current null and empty accounts', () => {
+it('WhatsAppBotConfigSchema validates with current null and empty accounts', () => {
   const result = WhatsAppBotConfigSchema.safeParse({
     current: null,
     accounts: {},
@@ -62,7 +62,7 @@ test('WhatsAppBotConfigSchema validates with current null and empty accounts', (
   expect(result.success).toBe(true)
 })
 
-test('WhatsAppBotConfigSchema validates with multiple accounts', () => {
+it('WhatsAppBotConfigSchema validates with multiple accounts', () => {
   const result = WhatsAppBotConfigSchema.safeParse({
     current: { account_id: 'phone-1' },
     accounts: {
@@ -73,21 +73,21 @@ test('WhatsAppBotConfigSchema validates with multiple accounts', () => {
   expect(result.success).toBe(true)
 })
 
-test('WhatsAppBotConfigSchema rejects missing accounts field', () => {
+it('WhatsAppBotConfigSchema rejects missing accounts field', () => {
   const result = WhatsAppBotConfigSchema.safeParse({
     current: null,
   })
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotConfigSchema rejects missing current field', () => {
+it('WhatsAppBotConfigSchema rejects missing current field', () => {
   const result = WhatsAppBotConfigSchema.safeParse({
     accounts: {},
   })
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotCredentialsSchema validates correct data', () => {
+it('WhatsAppBotCredentialsSchema validates correct data', () => {
   const result = WhatsAppBotCredentialsSchema.safeParse({
     phone_number_id: '123456789',
     account_name: 'Test Business',
@@ -96,7 +96,7 @@ test('WhatsAppBotCredentialsSchema validates correct data', () => {
   expect(result.success).toBe(true)
 })
 
-test('WhatsAppBotCredentialsSchema rejects missing phone_number_id', () => {
+it('WhatsAppBotCredentialsSchema rejects missing phone_number_id', () => {
   const result = WhatsAppBotCredentialsSchema.safeParse({
     account_name: 'Test Business',
     access_token: 'EAAtest123',
@@ -104,7 +104,7 @@ test('WhatsAppBotCredentialsSchema rejects missing phone_number_id', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotCredentialsSchema rejects missing account_name', () => {
+it('WhatsAppBotCredentialsSchema rejects missing account_name', () => {
   const result = WhatsAppBotCredentialsSchema.safeParse({
     phone_number_id: '123456789',
     access_token: 'EAAtest123',
@@ -112,7 +112,7 @@ test('WhatsAppBotCredentialsSchema rejects missing account_name', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotCredentialsSchema rejects missing access_token', () => {
+it('WhatsAppBotCredentialsSchema rejects missing access_token', () => {
   const result = WhatsAppBotCredentialsSchema.safeParse({
     phone_number_id: '123456789',
     account_name: 'Test Business',
@@ -120,22 +120,22 @@ test('WhatsAppBotCredentialsSchema rejects missing access_token', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotError has correct name', () => {
+it('WhatsAppBotError has correct name', () => {
   const error = new WhatsAppBotError('Test error', 'TEST_CODE')
   expect(error.name).toBe('WhatsAppBotError')
 })
 
-test('WhatsAppBotError has correct message', () => {
+it('WhatsAppBotError has correct message', () => {
   const error = new WhatsAppBotError('Test error', 'TEST_CODE')
   expect(error.message).toBe('Test error')
 })
 
-test('WhatsAppBotError has correct code', () => {
+it('WhatsAppBotError has correct code', () => {
   const error = new WhatsAppBotError('Test error', 'TEST_CODE')
   expect(error.code).toBe('TEST_CODE')
 })
 
-test('WhatsAppBotError is instance of Error', () => {
+it('WhatsAppBotError is instance of Error', () => {
   const error = new WhatsAppBotError('Test error', 'TEST_CODE')
   expect(error instanceof Error).toBe(true)
 })