agent-messenger 2.10.1 → 2.11.0

package/src/platforms/telegram/commands/auth.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 { getNonInteractiveLoginMessage, promptNextLoginInput } from './auth'
 
@@ -19,37 +19,37 @@ describe('promptNextLoginInput non-interactive', () => {
     Object.defineProperty(process.stdout, 'isTTY', { value: originalStdoutTTY, writable: true, configurable: true })
   })
 
-  test('returns null for provide_phone_number', async () => {
+  it('returns null for provide_phone_number', async () => {
     const result = await promptNextLoginInput({ next_action: 'provide_phone_number' }, {})
     expect(result).toBeNull()
   })
 
-  test('returns null for provide_code', async () => {
+  it('returns null for provide_code', async () => {
     const result = await promptNextLoginInput({ next_action: 'provide_code' }, {})
     expect(result).toBeNull()
   })
 
-  test('returns null for provide_password', async () => {
+  it('returns null for provide_password', async () => {
     const result = await promptNextLoginInput({ next_action: 'provide_password' }, {})
     expect(result).toBeNull()
   })
 
-  test('returns null for provide_email', async () => {
+  it('returns null for provide_email', async () => {
     const result = await promptNextLoginInput({ next_action: 'provide_email' }, {})
     expect(result).toBeNull()
   })
 
-  test('returns null for provide_email_code', async () => {
+  it('returns null for provide_email_code', async () => {
     const result = await promptNextLoginInput({ next_action: 'provide_email_code' }, {})
     expect(result).toBeNull()
   })
 
-  test('returns null for provide_registration', async () => {
+  it('returns null for provide_registration', async () => {
     const result = await promptNextLoginInput({ next_action: 'provide_registration' }, {})
     expect(result).toBeNull()
   })
 
-  test('returns options unchanged for unknown next_action', async () => {
+  it('returns options unchanged for unknown next_action', async () => {
     const options = { phone: '+14155551234' }
     const result = await promptNextLoginInput({ next_action: undefined }, options)
 
@@ -59,56 +59,56 @@ describe('promptNextLoginInput non-interactive', () => {
 })
 
 describe('getNonInteractiveLoginMessage', () => {
-  test('maps provide_phone_number to provide_phone with --phone hint', () => {
+  it('maps provide_phone_number to provide_phone with --phone hint', () => {
     const msg = getNonInteractiveLoginMessage('provide_phone_number')
     expect(msg).not.toBeNull()
     expect(msg!.next_action).toBe('provide_phone')
     expect(msg!.message).toContain('--phone')
   })
 
-  test('maps provide_code with --code hint', () => {
+  it('maps provide_code with --code hint', () => {
     const msg = getNonInteractiveLoginMessage('provide_code')
     expect(msg).not.toBeNull()
     expect(msg!.next_action).toBe('provide_code')
     expect(msg!.message).toContain('--code')
   })
 
-  test('maps provide_password with --password hint', () => {
+  it('maps provide_password with --password hint', () => {
     const msg = getNonInteractiveLoginMessage('provide_password')
     expect(msg).not.toBeNull()
     expect(msg!.next_action).toBe('provide_password')
     expect(msg!.message).toContain('--password')
   })
 
-  test('maps provide_email with --email hint', () => {
+  it('maps provide_email with --email hint', () => {
     const msg = getNonInteractiveLoginMessage('provide_email')
     expect(msg).not.toBeNull()
     expect(msg!.next_action).toBe('provide_email')
     expect(msg!.message).toContain('--email')
   })
 
-  test('maps provide_email_code with --email-code hint', () => {
+  it('maps provide_email_code with --email-code hint', () => {
     const msg = getNonInteractiveLoginMessage('provide_email_code')
     expect(msg).not.toBeNull()
     expect(msg!.next_action).toBe('provide_email_code')
     expect(msg!.message).toContain('--email-code')
   })
 
-  test('maps provide_registration with --first-name hint', () => {
+  it('maps provide_registration with --first-name hint', () => {
     const msg = getNonInteractiveLoginMessage('provide_registration')
     expect(msg).not.toBeNull()
     expect(msg!.next_action).toBe('provide_registration')
     expect(msg!.message).toContain('--first-name')
   })
 
-  test('maps provide_provisioning_code with --provisioning-code hint', () => {
+  it('maps provide_provisioning_code with --provisioning-code hint', () => {
     const msg = getNonInteractiveLoginMessage('provide_provisioning_code')
     expect(msg).not.toBeNull()
     expect(msg!.next_action).toBe('provide_provisioning_code')
     expect(msg!.message).toContain('--provisioning-code')
   })
 
-  test('returns null for unknown action', () => {
+  it('returns null for unknown action', () => {
     expect(getNonInteractiveLoginMessage('unknown_action')).toBeNull()
   })
 })

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 const mockListChats = mock(() =>
   Promise.resolve([
@@ -55,19 +55,19 @@ describe('chat commands', () => {
   })
 
   describe('list subcommand', () => {
-    test('calls listChats with default limit', async () => {
+    it('calls listChats with default limit', async () => {
       await chatCommand.parseAsync(['list'], { from: 'user' })
 
       expect(mockListChats).toHaveBeenCalledWith(20)
     })
 
-    test('calls listChats with custom limit', async () => {
+    it('calls listChats with custom limit', async () => {
       await chatCommand.parseAsync(['list', '--limit', '5'], { from: 'user' })
 
       expect(mockListChats).toHaveBeenCalledWith(5)
     })
 
-    test('outputs JSON array to console', async () => {
+    it('outputs JSON array to console', async () => {
       await chatCommand.parseAsync(['list'], { from: 'user' })
 
       expect(consoleSpy).toHaveBeenCalled()
@@ -79,19 +79,19 @@ describe('chat commands', () => {
   })
 
   describe('search subcommand', () => {
-    test('calls searchChats with query and default limit', async () => {
+    it('calls searchChats with query and default limit', async () => {
       await chatCommand.parseAsync(['search', 'General'], { from: 'user' })
 
       expect(mockSearchChats).toHaveBeenCalledWith('General', 20)
     })
 
-    test('calls searchChats with query and custom limit', async () => {
+    it('calls searchChats with query and custom limit', async () => {
       await chatCommand.parseAsync(['search', 'General', '--limit', '3'], { from: 'user' })
 
       expect(mockSearchChats).toHaveBeenCalledWith('General', 3)
     })
 
-    test('outputs JSON array to console', async () => {
+    it('outputs JSON array to console', async () => {
       await chatCommand.parseAsync(['search', 'General'], { from: 'user' })
 
       expect(consoleSpy).toHaveBeenCalled()
@@ -103,13 +103,13 @@ describe('chat commands', () => {
   })
 
   describe('get subcommand', () => {
-    test('calls getChat with chat reference', async () => {
+    it('calls getChat with chat reference', async () => {
       await chatCommand.parseAsync(['get', 'chat-1'], { from: 'user' })
 
       expect(mockGetChat).toHaveBeenCalledWith('chat-1')
     })
 
-    test('outputs JSON object to console', async () => {
+    it('outputs JSON object to console', async () => {
       await chatCommand.parseAsync(['get', 'chat-1'], { from: 'user' })
 
       expect(consoleSpy).toHaveBeenCalled()

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 const mockListMessages = mock(() =>
   Promise.resolve([
@@ -46,19 +46,19 @@ describe('message commands', () => {
   })
 
   describe('list subcommand', () => {
-    test('calls listMessages with chat reference and default limit', async () => {
+    it('calls listMessages with chat reference and default limit', async () => {
       await messageCommand.parseAsync(['list', 'chat-123'], { from: 'user' })
 
       expect(mockListMessages).toHaveBeenCalledWith('chat-123', 20)
     })
 
-    test('calls listMessages with custom limit', async () => {
+    it('calls listMessages with custom limit', async () => {
       await messageCommand.parseAsync(['list', 'chat-123', '--limit', '10'], { from: 'user' })
 
       expect(mockListMessages).toHaveBeenCalledWith('chat-123', 10)
     })
 
-    test('outputs JSON to console', async () => {
+    it('outputs JSON to console', async () => {
       await messageCommand.parseAsync(['list', 'chat-123'], { from: 'user' })
 
       expect(consoleSpy).toHaveBeenCalled()
@@ -70,13 +70,13 @@ describe('message commands', () => {
   })
 
   describe('send subcommand', () => {
-    test('calls sendMessage with chat reference and text', async () => {
+    it('calls sendMessage with chat reference and text', async () => {
       await messageCommand.parseAsync(['send', 'chat-123', 'Hello there'], { from: 'user' })
 
       expect(mockSendMessage).toHaveBeenCalledWith('chat-123', 'Hello there')
     })
 
-    test('outputs JSON to console', async () => {
+    it('outputs JSON to console', async () => {
       await messageCommand.parseAsync(['send', 'chat-123', 'Hello there'], { from: 'user' })
 
       expect(consoleSpy).toHaveBeenCalled()

package/src/platforms/telegram/commands/shared.test.ts

@@ -1,14 +1,14 @@
-import { describe, expect, test } from 'bun:test'
+import { describe, expect, it } from 'bun:test'
 
 import { TelegramError } from '../types'
 import { parseLimitOption } from './shared'
 
 describe('telegram shared command helpers', () => {
-  test('parseLimitOption accepts normal integer strings', () => {
+  it('parseLimitOption accepts normal integer strings', () => {
     expect(parseLimitOption('10', 20)).toBe(10)
   })
 
-  test('parseLimitOption rejects malformed numeric prefixes', () => {
+  it('parseLimitOption rejects malformed numeric prefixes', () => {
     expect(() => parseLimitOption('10abc', 20)).toThrow(TelegramError)
     expect(() => parseLimitOption('1.5', 20)).toThrow(TelegramError)
     expect(() => parseLimitOption('2e3', 20)).toThrow(TelegramError)

package/src/platforms/telegram/commands/whoami.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, spyOn, it } from 'bun:test'
 
 import { TelegramTdlibClient } from '../client'
 import { TelegramCredentialManager } from '../credential-manager'
@@ -41,7 +41,7 @@ describe('whoami command', () => {
     consoleLogSpy?.mockRestore()
   })
 
-  test('outputs account info with user when authenticated', async () => {
+  it('outputs account info with user when authenticated', async () => {
     await whoamiAction({})
 
     expect(consoleLogSpy).toHaveBeenCalledTimes(1)
@@ -54,7 +54,7 @@ describe('whoami command', () => {
     expect(output.user.username).toBe('testuser')
   })
 
-  test('omits user field when not present', async () => {
+  it('omits user field when not present', async () => {
     getAuthStatusSpy.mockResolvedValue({
       account_id: 'plus-12025551234',
       phone_number: '+12025551234',

package/src/platforms/telegram/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('TelegramCredentialManager', () => {
-  test('stores and retrieves accounts', async () => {
+  it('stores and retrieves accounts', async () => {
     const manager = setup()
 
     await manager.setAccount({
@@ -39,7 +39,7 @@ describe('TelegramCredentialManager', () => {
     expect(account?.phone_number).toBe('+821012345678')
   })
 
-  test('switches current account', async () => {
+  it('switches current account', async () => {
     const manager = setup()
 
     await manager.setAccount({
@@ -64,7 +64,7 @@ describe('TelegramCredentialManager', () => {
     expect(current?.account_id).toBe('second')
   })
 
-  test('creates account directories', async () => {
+  it('creates account directories', async () => {
     const manager = setup()
     const paths = await manager.ensureAccountPaths('+82 10 1234 5678')
 
@@ -74,7 +74,7 @@ describe('TelegramCredentialManager', () => {
     expect(paths.account_dir.endsWith('plus-82-10-1234-5678')).toBe(true)
   })
 
-  test('saves and loads provisioning state', async () => {
+  it('saves and loads provisioning state', async () => {
     const manager = setup()
     const state = {
       phone: '+14155551234',
@@ -90,7 +90,7 @@ describe('TelegramCredentialManager', () => {
     expect(loaded!.random_hash).toBe('abc123')
   })
 
-  test('returns null for expired provisioning state', async () => {
+  it('returns null for expired provisioning state', async () => {
     const manager = setup()
     const expiredDate = new Date(Date.now() - 11 * 60 * 1000).toISOString()
 
@@ -104,7 +104,7 @@ describe('TelegramCredentialManager', () => {
     expect(loaded).toBeNull()
   })
 
-  test('clears provisioning state', async () => {
+  it('clears provisioning state', async () => {
     const manager = setup()
 
     await manager.saveProvisioningState({
@@ -118,13 +118,13 @@ describe('TelegramCredentialManager', () => {
     expect(loaded).toBeNull()
   })
 
-  test('returns null when no provisioning state exists', async () => {
+  it('returns null when no provisioning state exists', async () => {
     const manager = setup()
     const loaded = await manager.loadProvisioningState()
     expect(loaded).toBeNull()
   })
 
-  test('returns null for corrupted created_at in provisioning state', async () => {
+  it('returns null for corrupted created_at in provisioning state', async () => {
     const manager = setup()
 
     await manager.saveProvisioningState({
@@ -137,7 +137,7 @@ describe('TelegramCredentialManager', () => {
     expect(loaded).toBeNull()
   })
 
-  test('clearCredentials also clears provisioning state', async () => {
+  it('clearCredentials also clears provisioning state', async () => {
     const manager = setup()
 
     await manager.saveProvisioningState({

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

@@ -1,14 +1,14 @@
-import { describe, expect, test } from 'bun:test'
+import { describe, expect, it } from 'bun:test'
 
 import { findFuzzyChats, normalizeChatSearchText } from './chat-utils'
 import { createAccountId, simplifyChat, simplifyMessage, summarizeAuthorizationState } from './types'
 
 describe('telegram type helpers', () => {
-  test('createAccountId normalizes phone numbers', () => {
+  it('createAccountId normalizes phone numbers', () => {
     expect(createAccountId('+82 10 1234 5678')).toBe('plus-82-10-1234-5678')
   })
 
-  test('summarizeAuthorizationState returns next action for wait code', () => {
+  it('summarizeAuthorizationState returns next action for wait code', () => {
     const summary = summarizeAuthorizationState({
       '@type': 'authorizationStateWaitCode',
       code_info: {
@@ -23,7 +23,7 @@ describe('telegram type helpers', () => {
     expect(summary.code_info?.type).toBe('authenticationCodeTypeSms')
   })
 
-  test('simplifyMessage extracts message text', () => {
+  it('simplifyMessage extracts message text', () => {
     const message = simplifyMessage({
       id: 42,
       chat_id: 1001,
@@ -43,7 +43,7 @@ describe('telegram type helpers', () => {
     expect(message.sender.type).toBe('user')
   })
 
-  test('simplifyChat normalizes chat type', () => {
+  it('simplifyChat normalizes chat type', () => {
     const chat = simplifyChat({
       id: 1001,
       title: 'General',
@@ -66,7 +66,7 @@ describe('telegram type helpers', () => {
     expect(chat.last_message?.text).toBe('latest')
   })
 
-  test('normalize and fuzzy search logic can match titles despite spacing differences', () => {
+  it('normalizes and fuzzy-matches titles despite spacing differences', () => {
     expect(normalizeChatSearchText('Project Room')).toBe('projectroom')
     expect(
       findFuzzyChats([{ id: 1, title: 'ProjectRoom', type: 'private', unread_count: 0 }], 'Project Room', 10),

package/src/platforms/webex/app-config.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 { getWebexAppCredentials } from './app-config'
 
@@ -30,7 +30,7 @@ describe('webex app config', () => {
     }
   })
 
-  test('returns env credentials when primary env vars are set', () => {
+  it('returns env credentials when primary env vars are set', () => {
     process.env.AGENT_WEBEX_CLIENT_ID = 'my-client-id'
     process.env.AGENT_WEBEX_CLIENT_SECRET = 'my-client-secret'
 
@@ -41,7 +41,7 @@ describe('webex app config', () => {
     })
   })
 
-  test('returns env credentials when legacy env vars are set', () => {
+  it('returns env credentials when legacy env vars are set', () => {
     process.env.AGENT_MESSENGER_WEBEX_CLIENT_ID = 'legacy-client-id'
     process.env.AGENT_MESSENGER_WEBEX_CLIENT_SECRET = 'legacy-client-secret'
 
@@ -52,28 +52,28 @@ describe('webex app config', () => {
     })
   })
 
-  test('returns builtin credentials when nothing is configured', () => {
+  it('returns builtin credentials when nothing is configured', () => {
     const result = getWebexAppCredentials()
     expect(result.source).toBe('builtin')
     expect(result.clientId).toBeTruthy()
     expect(result.clientSecret).toBeTruthy()
   })
 
-  test('returns builtin when only clientId is set', () => {
+  it('returns builtin when only clientId is set', () => {
     process.env.AGENT_WEBEX_CLIENT_ID = 'my-client-id'
 
     const result = getWebexAppCredentials()
     expect(result.source).toBe('builtin')
   })
 
-  test('returns builtin when only clientSecret is set', () => {
+  it('returns builtin when only clientSecret is set', () => {
     process.env.AGENT_WEBEX_CLIENT_SECRET = 'my-client-secret'
 
     const result = getWebexAppCredentials()
     expect(result.source).toBe('builtin')
   })
 
-  test('trims whitespace from clientId and clientSecret', () => {
+  it('trims whitespace from clientId and clientSecret', () => {
     process.env.AGENT_WEBEX_CLIENT_ID = '  my-client-id  '
     process.env.AGENT_WEBEX_CLIENT_SECRET = '  my-client-secret  '
 
@@ -84,7 +84,7 @@ describe('webex app config', () => {
     })
   })
 
-  test('primary env takes precedence over legacy env', () => {
+  it('primary env takes precedence over legacy env', () => {
     process.env.AGENT_WEBEX_CLIENT_ID = 'primary-id'
     process.env.AGENT_WEBEX_CLIENT_SECRET = 'primary-secret'
     process.env.AGENT_MESSENGER_WEBEX_CLIENT_ID = 'legacy-id'

package/src/platforms/webex/cli.test.ts

@@ -1,11 +1,11 @@
-import { describe, expect, test } from 'bun:test'
+import { describe, expect, it } from 'bun:test'
 
 import { spawn } from 'bun'
 
 import pkg from '../../../package.json' with { type: 'json' }
 
 describe('Webex CLI program structure', () => {
-  test('--help shows all commands', async () => {
+  it('--help shows all commands', async () => {
     const proc = spawn(['bun', 'run', './src/platforms/webex/cli.ts', '--help'], {
       cwd: process.cwd(),
       stdio: ['pipe', 'pipe', 'pipe'],
@@ -20,7 +20,7 @@ describe('Webex CLI program structure', () => {
     expect(output).toContain('space')
   })
 
-  test('--version shows package version', async () => {
+  it('--version shows package version', async () => {
     const proc = spawn(['bun', 'run', './src/platforms/webex/cli.ts', '--version'], {
       cwd: process.cwd(),
       stdio: ['pipe', 'pipe', 'pipe'],
@@ -30,7 +30,7 @@ describe('Webex CLI program structure', () => {
     expect(output.trim()).toBe(pkg.version)
   })
 
-  test('auth login --help shows Device Grant options', async () => {
+  it('auth login --help shows Device Grant options', async () => {
     const proc = spawn(['bun', 'run', './src/platforms/webex/cli.ts', 'auth', 'login', '--help'], {
       cwd: process.cwd(),
       stdio: ['pipe', 'pipe', 'pipe'],
@@ -44,7 +44,7 @@ describe('Webex CLI program structure', () => {
     expect(output).toContain('--pretty')
   })
 
-  test('message dm --help shows email argument', async () => {
+  it('message dm --help shows email argument', async () => {
     const proc = spawn(['bun', 'run', './src/platforms/webex/cli.ts', 'message', 'dm', '--help'], {
       cwd: process.cwd(),
       stdio: ['pipe', 'pipe', 'pipe'],