agent-messenger 2.10.1 → 2.11.0

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 })

package/src/platforms/whatsappbot/commands/auth.ts

@@ -1,5 +1,6 @@
 import { Command } from 'commander'
 
+import { cliOutput } from '@/shared/utils/cli-output'
 import { formatOutput } from '@/shared/utils/output'
 
 import { WhatsAppBotClient } from '../client'
@@ -145,11 +146,6 @@ export async function removeAction(accountId: string, options: ActionOptions): P
   }
 }
 
-function cliOutput(result: ActionResult, pretty?: boolean, exitOnError = true): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error && exitOnError) process.exit(1)
-}
-
 export const authCommand = new Command('auth')
   .description('Authentication commands')
   .addCommand(

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/message.ts

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 
-import { formatOutput } from '@/shared/utils/output'
+import { cliOutput } from '@/shared/utils/cli-output'
 
 import type { AccountOption } from './shared'
 import { getClient } from './shared'
@@ -91,11 +91,6 @@ export async function sendDocumentAction(
   }
 }
 
-function cliOutput(result: MessageResult, pretty?: boolean): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error) process.exit(1)
-}
-
 export const messageCommand = new Command('message')
   .description('Message commands')
   .addCommand(

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/template.ts

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 
-import { formatOutput } from '@/shared/utils/output'
+import { cliOutput } from '@/shared/utils/cli-output'
 
 import type { WhatsAppBotTemplate } from '../types'
 import type { AccountOption } from './shared'
@@ -40,11 +40,6 @@ export async function getAction(templateName: string, options: TemplateOptions):
   }
 }
 
-function cliOutput(result: TemplateResult, pretty?: boolean): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error) process.exit(1)
-}
-
 export const templateCommand = new Command('template')
   .description('Template commands')
   .addCommand(

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 })