agent-messenger 2.10.1 → 2.11.0

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

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

@@ -13,6 +13,15 @@ program
   .version(pkg.version)
   .option('--pretty', 'Pretty-print JSON output')
   .option('--account <id>', 'Account ID to use')
+  .hook('preAction', (thisCmd, actionCmd) => {
+    for (const [key, value] of Object.entries(thisCmd.opts())) {
+      if (value === undefined) continue
+      const source = actionCmd.getOptionValueSource(key)
+      if (source === undefined || source === 'default') {
+        actionCmd.setOptionValue(key, value)
+      }
+    }
+  })
 
 program.addCommand(authCommand)
 program.addCommand(whoamiCommand)