agent-messenger 2.10.1 → 2.10.2

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

@@ -1,4 +1,4 @@
-import { describe, expect, test } from 'bun:test'
+import { describe, expect, it } from 'bun:test'
 
 import {
   createAccountId,
@@ -9,182 +9,182 @@ import {
 } from '@/platforms/whatsapp/types'
 
 describe('WhatsAppError', () => {
-  test('name is WhatsAppError', () => {
+  it('has name WhatsAppError', () => {
     const err = new WhatsAppError('test message')
     expect(err.name).toBe('WhatsAppError')
   })
 
-  test('message is set correctly', () => {
+  it('sets message correctly', () => {
     const err = new WhatsAppError('something went wrong')
     expect(err.message).toBe('something went wrong')
   })
 
-  test('default code is whatsapp_error', () => {
+  it('defaults code to whatsapp_error', () => {
     const err = new WhatsAppError('test')
     expect(err.code).toBe('whatsapp_error')
   })
 
-  test('custom string code is preserved', () => {
+  it('preserves custom string code', () => {
     const err = new WhatsAppError('test', 'auth_failed')
     expect(err.code).toBe('auth_failed')
   })
 
-  test('numeric code is preserved', () => {
+  it('preserves numeric code', () => {
     const err = new WhatsAppError('test', 404)
     expect(err.code).toBe(404)
   })
 
-  test('is instanceof Error', () => {
+  it('is instanceof Error', () => {
     const err = new WhatsAppError('test')
     expect(err instanceof Error).toBe(true)
   })
 
-  test('is instanceof WhatsAppError', () => {
+  it('is instanceof WhatsAppError', () => {
     const err = new WhatsAppError('test')
     expect(err instanceof WhatsAppError).toBe(true)
   })
 })
 
 describe('createAccountId', () => {
-  test('converts phone with leading + to plus- prefix', () => {
+  it('converts phone with leading + to plus- prefix', () => {
     expect(createAccountId('+12025551234')).toBe('plus-12025551234')
   })
 
-  test('trims and lowercases with spaces becoming dashes', () => {
+  it('trims and lowercases with spaces becoming dashes', () => {
     expect(createAccountId('  My Account  ')).toBe('my-account')
   })
 
-  test('replaces special chars like @ with dashes', () => {
+  it('replaces special chars like @ with dashes', () => {
     expect(createAccountId('hello@world')).toBe('hello-world')
   })
 
-  test('empty string returns default', () => {
+  it('returns default for empty string', () => {
     expect(createAccountId('')).toBe('default')
   })
 
-  test('whitespace-only string returns default', () => {
+  it('returns default for whitespace-only string', () => {
     expect(createAccountId('   ')).toBe('default')
   })
 
-  test('already normalized input is unchanged', () => {
+  it('returns already normalized input unchanged', () => {
     expect(createAccountId('my-account')).toBe('my-account')
   })
 
-  test('collapses multiple special chars into single dash', () => {
+  it('collapses multiple special chars into single dash', () => {
     expect(createAccountId('foo!!bar')).toBe('foo-bar')
   })
 })
 
 describe('jidToType', () => {
-  test('individual JID returns individual', () => {
+  it('returns individual for individual JID', () => {
     expect(jidToType('1234@s.whatsapp.net')).toBe('individual')
   })
 
-  test('group JID returns group', () => {
+  it('returns group for group JID', () => {
     expect(jidToType('1234-5678@g.us')).toBe('group')
   })
 
-  test('status@broadcast returns status', () => {
+  it('returns status for status@broadcast JID', () => {
     expect(jidToType('status@broadcast')).toBe('status')
   })
 
-  test('other @broadcast JID returns broadcast', () => {
+  it('returns broadcast for other @broadcast JID', () => {
     expect(jidToType('1234@broadcast')).toBe('broadcast')
   })
 })
 
 describe('extractMessageText', () => {
-  test('null returns undefined', () => {
+  it('returns undefined for null', () => {
     expect(extractMessageText(null)).toBeUndefined()
   })
 
-  test('undefined returns undefined', () => {
+  it('returns undefined for undefined input', () => {
     expect(extractMessageText(undefined)).toBeUndefined()
   })
 
-  test('conversation returns conversation text', () => {
+  it('returns conversation text for conversation message', () => {
     expect(extractMessageText({ conversation: 'hello' })).toBe('hello')
   })
 
-  test('extendedTextMessage returns text', () => {
+  it('returns text for extendedTextMessage', () => {
     expect(extractMessageText({ extendedTextMessage: { text: 'hello' } })).toBe('hello')
   })
 
-  test('imageMessage returns caption', () => {
+  it('returns caption for imageMessage', () => {
     expect(extractMessageText({ imageMessage: { caption: 'photo' } })).toBe('photo')
   })
 
-  test('videoMessage returns caption', () => {
+  it('returns caption for videoMessage', () => {
     expect(extractMessageText({ videoMessage: { caption: 'vid' } })).toBe('vid')
   })
 
-  test('reactionMessage returns text', () => {
+  it('returns text for reactionMessage', () => {
     expect(extractMessageText({ reactionMessage: { text: '👍' } })).toBe('👍')
   })
 
-  test('empty message returns undefined', () => {
+  it('returns undefined for empty message', () => {
     expect(extractMessageText({})).toBeUndefined()
   })
 })
 
 describe('getMessageType', () => {
-  test('null returns unknown', () => {
+  it('returns unknown for null', () => {
     expect(getMessageType(null)).toBe('unknown')
   })
 
-  test('undefined returns unknown', () => {
+  it('returns unknown for undefined input', () => {
     expect(getMessageType(undefined)).toBe('unknown')
   })
 
-  test('conversation returns text', () => {
+  it('returns text for conversation message', () => {
     expect(getMessageType({ conversation: 'hi' })).toBe('text')
   })
 
-  test('extendedTextMessage returns text', () => {
+  it('returns text for extendedTextMessage', () => {
     expect(getMessageType({ extendedTextMessage: { text: 'hi' } })).toBe('text')
   })
 
-  test('imageMessage returns image', () => {
+  it('returns image for imageMessage', () => {
     expect(getMessageType({ imageMessage: {} })).toBe('image')
   })
 
-  test('videoMessage returns video', () => {
+  it('returns video for videoMessage', () => {
     expect(getMessageType({ videoMessage: {} })).toBe('video')
   })
 
-  test('videoMessage with gifPlayback returns gif', () => {
+  it('returns gif for videoMessage with gifPlayback', () => {
     expect(getMessageType({ videoMessage: { gifPlayback: true } })).toBe('gif')
   })
 
-  test('audioMessage returns audio', () => {
+  it('returns audio for audioMessage', () => {
     expect(getMessageType({ audioMessage: {} })).toBe('audio')
   })
 
-  test('documentMessage returns document', () => {
+  it('returns document for documentMessage', () => {
     expect(getMessageType({ documentMessage: {} })).toBe('document')
   })
 
-  test('stickerMessage returns sticker', () => {
+  it('returns sticker for stickerMessage', () => {
     expect(getMessageType({ stickerMessage: {} })).toBe('sticker')
   })
 
-  test('locationMessage returns location', () => {
+  it('returns location for locationMessage', () => {
     expect(getMessageType({ locationMessage: {} })).toBe('location')
   })
 
-  test('contactMessage returns contact', () => {
+  it('returns contact for contactMessage', () => {
     expect(getMessageType({ contactMessage: {} })).toBe('contact')
   })
 
-  test('reactionMessage returns reaction', () => {
+  it('returns reaction for reactionMessage', () => {
     expect(getMessageType({ reactionMessage: {} })).toBe('reaction')
   })
 
-  test('pollCreationMessage returns poll', () => {
+  it('returns poll for pollCreationMessage', () => {
     expect(getMessageType({ pollCreationMessage: {} })).toBe('poll')
   })
 
-  test('empty object returns unknown', () => {
+  it('returns unknown for empty object', () => {
     expect(getMessageType({})).toBe('unknown')
   })
 })

package/src/platforms/whatsappbot/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 { WhatsAppBotClient } from '@/platforms/whatsappbot/client'
 import { WhatsAppBotError } from '@/platforms/whatsappbot/types'
@@ -50,7 +50,7 @@ describe('WhatsAppBotClient', () => {
   }
 
   describe('login', () => {
-    test('throws on empty phoneNumberId', async () => {
+    it('throws on empty phoneNumberId', async () => {
       await expect(new WhatsAppBotClient().login({ phoneNumberId: '', accessToken: 'access-token' })).rejects.toThrow(
         WhatsAppBotError,
       )
@@ -59,7 +59,7 @@ describe('WhatsAppBotClient', () => {
       )
     })
 
-    test('throws on empty accessToken', async () => {
+    it('throws on empty accessToken', async () => {
       await expect(new WhatsAppBotClient().login({ phoneNumberId: 'phone-123', accessToken: '' })).rejects.toThrow(
         WhatsAppBotError,
       )
@@ -68,14 +68,14 @@ describe('WhatsAppBotClient', () => {
       )
     })
 
-    test('accepts valid phoneNumberId and accessToken', async () => {
+    it('accepts valid phoneNumberId and accessToken', async () => {
       const client = await new WhatsAppBotClient().login({ phoneNumberId: 'phone-123', accessToken: 'access-token' })
       expect(client).toBeInstanceOf(WhatsAppBotClient)
     })
   })
 
   describe('verifyToken', () => {
-    test('sends GET request with correct URL and auth header', async () => {
+    it('sends GET request with correct URL and auth header', async () => {
       mockResponse({ verified_name: 'Test Business' })
 
       const client = await new WhatsAppBotClient().login({ phoneNumberId: 'phone-123', accessToken: 'my-token' })
@@ -89,7 +89,7 @@ describe('WhatsAppBotClient', () => {
       })
     })
 
-    test('throws WhatsAppBotError on API error', async () => {
+    it('throws WhatsAppBotError on API error', async () => {
       mockResponse({ error: { message: 'Invalid token', code: 190 } }, 401)
 
       const client = await new WhatsAppBotClient().login({ phoneNumberId: 'phone-123', accessToken: 'bad-token' })
@@ -98,7 +98,7 @@ describe('WhatsAppBotClient', () => {
   })
 
   describe('sendTextMessage', () => {
-    test('sends POST request with correct body shape', async () => {
+    it('sends POST request with correct body shape', async () => {
       mockResponse({
         messaging_product: 'whatsapp',
         contacts: [{ input: '+15551234567', wa_id: '15551234567' }],
@@ -121,7 +121,7 @@ describe('WhatsAppBotClient', () => {
       })
     })
 
-    test('sends correct Authorization header', async () => {
+    it('sends correct Authorization header', async () => {
       mockResponse({
         messaging_product: 'whatsapp',
         contacts: [],
@@ -136,7 +136,7 @@ describe('WhatsAppBotClient', () => {
       })
     })
 
-    test('returns WhatsAppBotMessageResponse', async () => {
+    it('returns WhatsAppBotMessageResponse', async () => {
       mockResponse({
         messaging_product: 'whatsapp',
         contacts: [{ input: '+15551234567', wa_id: '15551234567' }],
@@ -153,7 +153,7 @@ describe('WhatsAppBotClient', () => {
   })
 
   describe('sendTemplateMessage', () => {
-    test('sends POST with template payload', async () => {
+    it('sends POST with template payload', async () => {
       mockResponse({
         messaging_product: 'whatsapp',
         contacts: [{ input: '+15551234567', wa_id: '15551234567' }],
@@ -172,7 +172,7 @@ describe('WhatsAppBotClient', () => {
       })
     })
 
-    test('includes components when provided', async () => {
+    it('includes components when provided', async () => {
       mockResponse({
         messaging_product: 'whatsapp',
         contacts: [],
@@ -189,7 +189,7 @@ describe('WhatsAppBotClient', () => {
   })
 
   describe('sendReaction', () => {
-    test('sends POST with reaction payload', async () => {
+    it('sends POST with reaction payload', async () => {
       mockResponse({
         messaging_product: 'whatsapp',
         contacts: [],
@@ -210,7 +210,7 @@ describe('WhatsAppBotClient', () => {
   })
 
   describe('sendImageMessage', () => {
-    test('sends POST with image payload', async () => {
+    it('sends POST with image payload', async () => {
       mockResponse({
         messaging_product: 'whatsapp',
         contacts: [],
@@ -231,7 +231,7 @@ describe('WhatsAppBotClient', () => {
   })
 
   describe('sendDocumentMessage', () => {
-    test('sends POST with document payload', async () => {
+    it('sends POST with document payload', async () => {
       mockResponse({
         messaging_product: 'whatsapp',
         contacts: [],
@@ -252,7 +252,7 @@ describe('WhatsAppBotClient', () => {
   })
 
   describe('listTemplates', () => {
-    test('sends GET and unwraps data key from response', async () => {
+    it('sends GET and unwraps data key from response', async () => {
       mockResponse({
         data: [
           { name: 'hello_world', status: 'APPROVED', category: 'UTILITY', language: 'en_US', components: [] },
@@ -270,7 +270,7 @@ describe('WhatsAppBotClient', () => {
       expect(fetchCalls[0].options?.method).toBe('GET')
     })
 
-    test('passes limit parameter in URL', async () => {
+    it('passes limit parameter in URL', async () => {
       mockResponse({ data: [] })
 
       const client = await new WhatsAppBotClient().login({ phoneNumberId: 'phone-123', accessToken: 'my-token' })
@@ -281,7 +281,7 @@ describe('WhatsAppBotClient', () => {
   })
 
   describe('getTemplate', () => {
-    test('sends GET with name filter and returns first match', async () => {
+    it('sends GET with name filter and returns first match', async () => {
       mockResponse({
         data: [{ name: 'hello_world', status: 'APPROVED', category: 'UTILITY', language: 'en_US', components: [] }],
       })
@@ -293,7 +293,7 @@ describe('WhatsAppBotClient', () => {
       expect(fetchCalls[0].url).toContain('name=hello_world')
     })
 
-    test('throws WhatsAppBotError with not_found code when template not found', async () => {
+    it('throws WhatsAppBotError with not_found code when template not found', async () => {
       mockResponse({ data: [] })
 
       const client = await new WhatsAppBotClient().login({ phoneNumberId: 'phone-123', accessToken: 'my-token' })
@@ -309,7 +309,7 @@ describe('WhatsAppBotClient', () => {
   })
 
   describe('rate limiting', () => {
-    test('waits when x-business-use-case-usage indicates throttle', async () => {
+    it('waits when x-business-use-case-usage indicates throttle', async () => {
       const usageHeader = JSON.stringify({
         '123456789': [
           {
@@ -336,7 +336,7 @@ describe('WhatsAppBotClient', () => {
       expect(fetchCalls.length).toBe(2)
     })
 
-    test('retries on 429 with Retry-After header', async () => {
+    it('retries on 429 with Retry-After header', async () => {
       mockResponse({ error: { message: 'Rate limited', code: 613 } }, 429, { 'Retry-After': '0.1' })
       mockResponse({ verified_name: 'Test Business' })
 
@@ -347,7 +347,7 @@ describe('WhatsAppBotClient', () => {
       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({ error: { message: 'Rate limited', code: 613 } }, 429, { 'Retry-After': '0.01' })
       }
@@ -359,7 +359,7 @@ describe('WhatsAppBotClient', () => {
   })
 
   describe('retry logic', () => {
-    test('retries on 500 server error for GET requests', async () => {
+    it('retries on 500 server error for GET requests', async () => {
       mockResponse({ error: { message: 'Internal Server Error' } }, 500)
       mockResponse({ verified_name: 'Test Business' })
 
@@ -370,7 +370,7 @@ describe('WhatsAppBotClient', () => {
       expect(fetchCalls.length).toBe(2)
     })
 
-    test('does not retry on 500 for POST requests', async () => {
+    it('does not retry on 500 for POST requests', async () => {
       mockResponse({ error: { message: 'Internal Server Error' } }, 500)
 
       const client = await new WhatsAppBotClient().login({ phoneNumberId: 'phone-123', accessToken: 'my-token' })
@@ -378,7 +378,7 @@ describe('WhatsAppBotClient', () => {
       expect(fetchCalls.length).toBe(1)
     })
 
-    test('does not retry on 4xx client errors (except 429)', async () => {
+    it('does not retry on 4xx client errors (except 429)', async () => {
       mockResponse({ error: { message: 'Not Found', code: 100 } }, 404)
 
       const client = await new WhatsAppBotClient().login({ phoneNumberId: 'phone-123', accessToken: 'my-token' })
@@ -386,7 +386,7 @@ describe('WhatsAppBotClient', () => {
       expect(fetchCalls.length).toBe(1)
     })
 
-    test('does not retry on 403 forbidden', async () => {
+    it('does not retry on 403 forbidden', async () => {
       mockResponse({ error: { message: 'Forbidden', code: 200 } }, 403)
 
       const client = await new WhatsAppBotClient().login({ phoneNumberId: 'phone-123', accessToken: 'my-token' })
@@ -396,7 +396,7 @@ describe('WhatsAppBotClient', () => {
   })
 
   describe('request URL construction', () => {
-    test('all requests go to https://graph.facebook.com/v23.0/...', async () => {
+    it('all requests go to https://graph.facebook.com/v23.0/...', async () => {
       mockResponse({ verified_name: 'Test' })
       const client = await new WhatsAppBotClient().login({ phoneNumberId: 'my-phone', accessToken: 'my-token' })
       await client.verifyToken()
@@ -404,7 +404,7 @@ describe('WhatsAppBotClient', () => {
       expect(fetchCalls[0].url.startsWith('https://graph.facebook.com/v23.0/')).toBe(true)
     })
 
-    test('Authorization header is Bearer <token>', async () => {
+    it('Authorization header is Bearer <token>', async () => {
       mockResponse({ verified_name: 'Test' })
       const client = await new WhatsAppBotClient().login({ phoneNumberId: 'phone-123', accessToken: 'my-secret-token' })
       await client.verifyToken()

package/src/platforms/whatsappbot/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'
@@ -38,7 +38,7 @@ describe('auth commands', () => {
   })
 
   describe('setAction', () => {
-    test('validates token and stores credentials', async () => {
+    it('validates token and stores credentials', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
 
       const result = await setAction('12345678901', 'EAAtest-token', { _credManager: manager })
@@ -52,7 +52,7 @@ describe('auth commands', () => {
       expect(creds?.access_token).toBe('EAAtest-token')
     })
 
-    test('returns error when client throws', async () => {
+    it('returns error when client throws', async () => {
       mockVerifyToken.mockImplementationOnce(() => Promise.reject(new Error('Invalid token')))
       const manager = new WhatsAppBotCredentialManager(tempDir)
 
@@ -64,7 +64,7 @@ describe('auth commands', () => {
   })
 
   describe('statusAction', () => {
-    test('returns invalid status when no credentials set', async () => {
+    it('returns invalid status when no credentials set', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
 
       const result = await statusAction({ _credManager: manager })
@@ -73,7 +73,7 @@ describe('auth commands', () => {
       expect(result.error).toBeDefined()
     })
 
-    test('returns valid status when credentials are set', async () => {
+    it('returns valid status when credentials are set', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
       await manager.setCredentials({
         phone_number_id: '12345678901',
@@ -88,7 +88,7 @@ describe('auth commands', () => {
       expect(result.account_name).toBe('Test Business')
     })
 
-    test('returns invalid status when verifyToken fails', async () => {
+    it('returns invalid status when verifyToken fails', async () => {
       mockVerifyToken.mockImplementationOnce(() => Promise.reject(new Error('Token expired')))
       const manager = new WhatsAppBotCredentialManager(tempDir)
       await manager.setCredentials({
@@ -103,7 +103,7 @@ describe('auth commands', () => {
       expect(result.phone_number_id).toBe('12345678901')
     })
 
-    test('returns error for unknown account', async () => {
+    it('returns error for unknown account', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
 
       const result = await statusAction({ account: 'nonexistent', _credManager: manager })
@@ -114,7 +114,7 @@ describe('auth commands', () => {
   })
 
   describe('clearAction', () => {
-    test('removes all stored credentials', async () => {
+    it('removes all stored credentials', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
       await manager.setCredentials({
         phone_number_id: '12345678901',
@@ -130,7 +130,7 @@ describe('auth commands', () => {
   })
 
   describe('listAction', () => {
-    test('returns empty accounts when none set', async () => {
+    it('returns empty accounts when none set', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
 
       const result = await listAction({ _credManager: manager })
@@ -138,7 +138,7 @@ describe('auth commands', () => {
       expect(result.accounts).toHaveLength(0)
     })
 
-    test('returns all stored accounts', async () => {
+    it('returns all stored accounts', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
       await manager.setCredentials({
         phone_number_id: '11111111111',
@@ -160,7 +160,7 @@ describe('auth commands', () => {
   })
 
   describe('useAction', () => {
-    test('switches current account', async () => {
+    it('switches current account', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
       await manager.setCredentials({
         phone_number_id: '11111111111',
@@ -180,7 +180,7 @@ describe('auth commands', () => {
       expect(result.account_name).toBe('Account A')
     })
 
-    test('returns error for unknown account', async () => {
+    it('returns error for unknown account', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
 
       const result = await useAction('nonexistent', { _credManager: manager })
@@ -191,7 +191,7 @@ describe('auth commands', () => {
   })
 
   describe('removeAction', () => {
-    test('removes a stored account', async () => {
+    it('removes a stored account', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
       await manager.setCredentials({
         phone_number_id: '12345678901',
@@ -205,7 +205,7 @@ describe('auth commands', () => {
       expect(await manager.getCredentials('12345678901')).toBeNull()
     })
 
-    test('returns error for unknown account', async () => {
+    it('returns error for unknown account', async () => {
       const manager = new WhatsAppBotCredentialManager(tempDir)
 
       const result = await removeAction('nonexistent', { _credManager: manager })