agent-messenger 2.10.1 → 2.11.0

package/src/platforms/line/client.test.ts

@@ -1,43 +1,43 @@
-import { describe, expect, test } from 'bun:test'
+import { describe, expect, it } from 'bun:test'
 
 import { LineClient } from './client'
 import { LineError } from './types'
 import type { LineChat, LineDevice, LineLoginResult, LineMessage, LineSendResult } from './types'
 
 describe('LineClient', () => {
-  test('constructor creates instance without errors', () => {
+  it('constructor creates instance without errors', () => {
     const client = new LineClient()
     expect(client).toBeInstanceOf(LineClient)
   })
 
-  test('constructor accepts a custom credential manager', () => {
+  it('constructor accepts a custom credential manager', () => {
     const { LineCredentialManager } = require('./credential-manager')
     const manager = new LineCredentialManager()
     const client = new LineClient(manager)
     expect(client).toBeInstanceOf(LineClient)
   })
 
-  test('close() is idempotent - can be called multiple times without error', () => {
+  it('close() is idempotent - can be called multiple times without error', () => {
     const client = new LineClient()
     expect(() => client.close()).not.toThrow()
     expect(() => client.close()).not.toThrow()
     expect(() => client.close()).not.toThrow()
   })
 
-  test('close() is idempotent after login attempt fails', async () => {
+  it('close() is idempotent after login attempt fails', async () => {
     const client = new LineClient()
     client.close()
     client.close()
   })
 
   describe('ensureClient throws when not logged in', () => {
-    test('getChats() throws LineError with code not_connected', async () => {
+    it('getChats() throws LineError with code not_connected', async () => {
       const client = new LineClient()
       await expect(client.getChats()).rejects.toThrow(LineError)
       await expect(client.getChats()).rejects.toMatchObject({ code: 'not_connected' })
     })
 
-    test('getMessages() throws LineError when not logged in', async () => {
+    it('getMessages() throws LineError when not logged in', async () => {
       const client = new LineClient()
       await expect(client.getMessages('chat123')).rejects.toThrow(LineError)
       await expect(client.getMessages('chat123')).rejects.toMatchObject({
@@ -45,7 +45,7 @@ describe('LineClient', () => {
       })
     })
 
-    test('sendMessage() throws LineError when not logged in', async () => {
+    it('sendMessage() throws LineError when not logged in', async () => {
       const client = new LineClient()
       await expect(client.sendMessage('chat123', 'hello')).rejects.toThrow(LineError)
       await expect(client.sendMessage('chat123', 'hello')).rejects.toMatchObject({
@@ -55,7 +55,7 @@ describe('LineClient', () => {
   })
 
   describe('login() without credentials', () => {
-    test('throws LineError when no saved credentials exist', async () => {
+    it('throws LineError when no saved credentials exist', async () => {
       const { LineCredentialManager } = require('./credential-manager')
       const { mkdtemp } = require('node:fs/promises')
       const { tmpdir } = require('node:os')
@@ -71,7 +71,7 @@ describe('LineClient', () => {
   })
 
   describe('LineError', () => {
-    test('LineError has correct name, code, and message', () => {
+    it('LineError has correct name, code, and message', () => {
       const err = new LineError('test_code', 'test message')
       expect(err).toBeInstanceOf(Error)
       expect(err).toBeInstanceOf(LineError)
@@ -80,7 +80,7 @@ describe('LineClient', () => {
       expect(err.message).toBe('test message')
     })
 
-    test('LineError is thrown by getChats and wraps the not_connected error', async () => {
+    it('LineError is thrown by getChats and wraps the not_connected error', async () => {
       const client = new LineClient()
       try {
         await client.getChats()
@@ -95,13 +95,13 @@ describe('LineClient', () => {
   })
 
   describe('default device detection', () => {
-    test('LineClient can be instantiated (device detection does not throw)', () => {
+    it('LineClient can be instantiated (device detection does not throw)', () => {
       expect(() => new LineClient()).not.toThrow()
     })
   })
 
   describe('type exports', () => {
-    test('LineChat type is correctly shaped', () => {
+    it('LineChat type is correctly shaped', () => {
       const chat: LineChat = {
         chat_id: 'c1234567890abcdef1234567890abcdef',
         type: 'group',
@@ -114,7 +114,7 @@ describe('LineClient', () => {
       expect(chat.member_count).toBe(5)
     })
 
-    test('LineMessage type is correctly shaped', () => {
+    it('LineMessage type is correctly shaped', () => {
       const msg: LineMessage = {
         message_id: 'msg123',
         chat_id: 'chat456',
@@ -127,7 +127,7 @@ describe('LineClient', () => {
       expect(msg.text).toBe('Hello')
     })
 
-    test('LineMessage text can be null', () => {
+    it('LineMessage text can be null', () => {
       const msg: LineMessage = {
         message_id: 'msg123',
         chat_id: 'chat456',
@@ -139,7 +139,7 @@ describe('LineClient', () => {
       expect(msg.text).toBeNull()
     })
 
-    test('LineSendResult type is correctly shaped', () => {
+    it('LineSendResult type is correctly shaped', () => {
       const result: LineSendResult = {
         success: true,
         chat_id: 'chat456',
@@ -149,7 +149,7 @@ describe('LineClient', () => {
       expect(result.success).toBe(true)
     })
 
-    test('LineLoginResult type is correctly shaped', () => {
+    it('LineLoginResult type is correctly shaped', () => {
       const result: LineLoginResult = {
         authenticated: true,
         account_id: 'u1234567890abcdef1234567890abcdef',

package/src/platforms/line/commands/auth.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, mock, spyOn, it } from 'bun:test'
 
 const originalConsoleLog = console.log
 
@@ -55,7 +55,7 @@ afterEach(() => {
   console.log = originalConsoleLog
 })
 
-test('status: outputs account info when account exists', async () => {
+it('status: outputs account info when account exists', async () => {
   // when
   await authCommand.parseAsync(['node', 'auth', 'status'])
 
@@ -68,7 +68,7 @@ test('status: outputs account info when account exists', async () => {
   expect(output.device).toBe('ANDROIDSECONDARY')
 })
 
-test('status: outputs error when no account configured', async () => {
+it('status: outputs error when no account configured', async () => {
   // given
   getAccountSpy.mockResolvedValue(null)
 
@@ -81,7 +81,7 @@ test('status: outputs error when no account configured', async () => {
   expect(output.error).toBe('No LINE account configured')
 })
 
-test('list: outputs all accounts', async () => {
+it('list: outputs all accounts', async () => {
   // when
   await authCommand.parseAsync(['node', 'auth', 'list'])
 
@@ -94,7 +94,7 @@ test('list: outputs all accounts', async () => {
   expect(output[1].account_id).toBe('u456')
 })
 
-test('list: outputs empty array when no accounts', async () => {
+it('list: outputs empty array when no accounts', async () => {
   // given
   listAccountsSpy.mockResolvedValue([])
 
@@ -106,7 +106,7 @@ test('list: outputs empty array when no accounts', async () => {
   expect(output).toHaveLength(0)
 })
 
-test('use: sets current account and outputs result', async () => {
+it('use: sets current account and outputs result', async () => {
   // when
   await authCommand.parseAsync(['node', 'auth', 'use', 'u456'])
 
@@ -117,7 +117,7 @@ test('use: sets current account and outputs result', async () => {
   expect(output.current_account).toBe('u456')
 })
 
-test('logout: removes specific account when account-id provided', async () => {
+it('logout: removes specific account when account-id provided', async () => {
   // when
   await authCommand.parseAsync(['node', 'auth', 'logout', 'u123'])
 
@@ -129,7 +129,7 @@ test('logout: removes specific account when account-id provided', async () => {
   expect(output.message).toContain('u123')
 })
 
-test('logout: clears all accounts when no account-id provided', async () => {
+it('logout: clears all accounts when no account-id provided', async () => {
   // when
   await authCommand.parseAsync(['node', 'auth', 'logout'])
 

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, mock, spyOn, it } from 'bun:test'
 
 const originalConsoleLog = console.log
 
@@ -48,7 +48,7 @@ afterEach(() => {
   console.log = originalConsoleLog
 })
 
-test('list: fetches and outputs chats', async () => {
+it('list: fetches and outputs chats', async () => {
   // when
   await chatCommand.parseAsync(['node', 'chat', 'list'])
 
@@ -62,7 +62,7 @@ test('list: fetches and outputs chats', async () => {
   expect(output[1].chat_id).toBe('c222')
 })
 
-test('list: uses default limit of 50 when no limit provided', async () => {
+it('list: uses default limit of 50 when no limit provided', async () => {
   // when
   await chatCommand.parseAsync(['node', 'chat', 'list'])
 
@@ -70,7 +70,7 @@ test('list: uses default limit of 50 when no limit provided', async () => {
   expect(getChatsSpy).toHaveBeenCalledWith({ limit: 50 })
 })
 
-test('list: uses custom limit when --limit option provided', async () => {
+it('list: uses custom limit when --limit option provided', async () => {
   // when
   await chatCommand.parseAsync(['node', 'chat', 'list', '--limit', '10'])
 
@@ -78,7 +78,7 @@ test('list: uses custom limit when --limit option provided', async () => {
   expect(getChatsSpy).toHaveBeenCalledWith({ limit: 10 })
 })
 
-test('list: closes client after fetching chats', async () => {
+it('list: closes client after fetching chats', async () => {
   // when
   await chatCommand.parseAsync(['node', 'chat', 'list'])
 
@@ -86,7 +86,7 @@ test('list: closes client after fetching chats', async () => {
   expect(closeSpy).toHaveBeenCalledTimes(1)
 })
 
-test('list: outputs chat metadata', async () => {
+it('list: outputs chat metadata', async () => {
   // when
   await chatCommand.parseAsync(['node', 'chat', 'list'])
 
@@ -98,7 +98,7 @@ test('list: outputs chat metadata', async () => {
   expect(chat.display_name).toBeDefined()
 })
 
-test('list: includes different chat types', async () => {
+it('list: includes different chat types', async () => {
   // when
   await chatCommand.parseAsync(['node', 'chat', 'list'])
 

package/src/platforms/line/commands/friend.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, mock, spyOn, it } from 'bun:test'
 
 const originalConsoleLog = console.log
 
@@ -42,7 +42,7 @@ afterEach(() => {
   console.log = originalConsoleLog
 })
 
-test('list: fetches and outputs friends', async () => {
+it('list: fetches and outputs friends', async () => {
   // when
   await friendCommand.parseAsync(['node', 'friend', 'list'])
 
@@ -56,7 +56,7 @@ test('list: fetches and outputs friends', async () => {
   expect(output[0].display_name).toBe('Alice')
 })
 
-test('list: outputs friends with metadata', async () => {
+it('list: outputs friends with metadata', async () => {
   // when
   await friendCommand.parseAsync(['node', 'friend', 'list'])
 
@@ -67,7 +67,7 @@ test('list: outputs friends with metadata', async () => {
   expect(friend.display_name).toBeDefined()
 })
 
-test('list: closes client after fetching friends', async () => {
+it('list: closes client after fetching friends', async () => {
   // when
   await friendCommand.parseAsync(['node', 'friend', 'list'])
 
@@ -75,7 +75,7 @@ test('list: closes client after fetching friends', async () => {
   expect(closeSpy).toHaveBeenCalledTimes(1)
 })
 
-test('list: outputs empty array when no friends', async () => {
+it('list: outputs empty array when no friends', async () => {
   // given
   getFriendsSpy.mockResolvedValue([])
 
@@ -87,7 +87,7 @@ test('list: outputs empty array when no friends', async () => {
   expect(output).toHaveLength(0)
 })
 
-test('list: includes all friend fields', async () => {
+it('list: includes all friend fields', async () => {
   // when
   await friendCommand.parseAsync(['node', 'friend', 'list'])
 

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, mock, spyOn, it } from 'bun:test'
 
 const originalConsoleLog = console.log
 
@@ -55,7 +55,7 @@ afterEach(() => {
   console.log = originalConsoleLog
 })
 
-test('list: fetches and outputs messages for a chat', async () => {
+it('list: fetches and outputs messages for a chat', async () => {
   // when
   await messageCommand.parseAsync(['node', 'message', 'list', 'chat-1'])
 
@@ -69,7 +69,7 @@ test('list: fetches and outputs messages for a chat', async () => {
   expect(output[0].text).toBe('Hello world')
 })
 
-test('list: uses custom count when --count option provided', async () => {
+it('list: uses custom count when --count option provided', async () => {
   // when
   await messageCommand.parseAsync(['node', 'message', 'list', 'chat-1', '--count', '5'])
 
@@ -77,7 +77,7 @@ test('list: uses custom count when --count option provided', async () => {
   expect(getMessagesSpy).toHaveBeenCalledWith('chat-1', { count: 5 })
 })
 
-test('list: closes client after fetching messages', async () => {
+it('list: closes client after fetching messages', async () => {
   // when
   await messageCommand.parseAsync(['node', 'message', 'list', 'chat-1'])
 
@@ -85,7 +85,7 @@ test('list: closes client after fetching messages', async () => {
   expect(closeSpy).toHaveBeenCalledTimes(1)
 })
 
-test('send: sends message and outputs result', async () => {
+it('send: sends message and outputs result', async () => {
   // when
   await messageCommand.parseAsync(['node', 'message', 'send', 'chat-1', 'Hello!'])
 
@@ -98,7 +98,7 @@ test('send: sends message and outputs result', async () => {
   expect(output.message_id).toBe('msg-new')
 })
 
-test('send: closes client after sending message', async () => {
+it('send: closes client after sending message', async () => {
   // when
   await messageCommand.parseAsync(['node', 'message', 'send', 'chat-1', 'Hello!'])
 
@@ -106,7 +106,7 @@ test('send: closes client after sending message', async () => {
   expect(closeSpy).toHaveBeenCalledTimes(1)
 })
 
-test('list: outputs messages with metadata', async () => {
+it('list: outputs messages with metadata', async () => {
   // when
   await messageCommand.parseAsync(['node', 'message', 'list', 'chat-1'])
 

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, mock, spyOn, it } from 'bun:test'
 
 const originalConsoleLog = console.log
 
@@ -34,7 +34,7 @@ afterEach(() => {
   console.log = originalConsoleLog
 })
 
-test('whoami: fetches and outputs profile', async () => {
+it('whoami: fetches and outputs profile', async () => {
   // when
   await whoamiCommand.parseAsync(['node', 'whoami'])
 
@@ -47,7 +47,7 @@ test('whoami: fetches and outputs profile', async () => {
   expect(output.display_name).toBe('Test User')
 })
 
-test('whoami: outputs profile with all fields', async () => {
+it('whoami: outputs profile with all fields', async () => {
   // when
   await whoamiCommand.parseAsync(['node', 'whoami'])
 
@@ -59,7 +59,7 @@ test('whoami: outputs profile with all fields', async () => {
   expect(output.status_message).toBeDefined()
 })
 
-test('whoami: closes client after fetching profile', async () => {
+it('whoami: closes client after fetching profile', async () => {
   // when
   await whoamiCommand.parseAsync(['node', 'whoami'])
 
@@ -67,7 +67,7 @@ test('whoami: closes client after fetching profile', async () => {
   expect(closeSpy).toHaveBeenCalledTimes(1)
 })
 
-test('whoami: outputs profile with picture_url', async () => {
+it('whoami: outputs profile with picture_url', async () => {
   // when
   await whoamiCommand.parseAsync(['node', 'whoami'])
 
@@ -76,7 +76,7 @@ test('whoami: outputs profile with picture_url', async () => {
   expect(output.picture_url).toBe('https://example.com/pic.jpg')
 })
 
-test('whoami: outputs profile with status_message', async () => {
+it('whoami: outputs profile with status_message', async () => {
   // when
   await whoamiCommand.parseAsync(['node', 'whoami'])
 

package/src/platforms/line/credential-manager.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 { mkdir, rm, stat } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
 import { join } from 'node:path'
@@ -29,12 +29,12 @@ afterEach(async () => {
 })
 
 describe('LineCredentialManager', () => {
-  test('load returns default config when file does not exist', async () => {
+  it('load returns default config when file does not exist', async () => {
     const config = await manager.load()
     expect(config).toEqual({ current_account: null, accounts: {} })
   })
 
-  test('save and load round trip', async () => {
+  it('saves and loads in a round trip', async () => {
     const account = makeAccount('alice')
     const config = { current_account: 'alice', accounts: { alice: account } }
 
@@ -44,14 +44,14 @@ describe('LineCredentialManager', () => {
     expect(loaded).toEqual(config)
   })
 
-  test('file is created with 0o600 permissions', async () => {
+  it('file is created with 0o600 permissions', async () => {
     await manager.save({ current_account: null, accounts: {} })
     const fileStat = await stat(join(tempDir, 'line-credentials.json'))
     // Check owner read/write only (0o600)
     expect(fileStat.mode & 0o777).toBe(0o600)
   })
 
-  test('setAccount adds account and sets as current if first', async () => {
+  it('setAccount adds account and sets as current if first', async () => {
     const account = makeAccount('alice')
     await manager.setAccount(account)
 
@@ -60,7 +60,7 @@ describe('LineCredentialManager', () => {
     expect(loaded.current_account).toBe('alice')
   })
 
-  test('setAccount does not change current if already set', async () => {
+  it('setAccount does not change current if already set', async () => {
     await manager.setAccount(makeAccount('alice'))
     await manager.setAccount(makeAccount('bob'))
 
@@ -68,12 +68,12 @@ describe('LineCredentialManager', () => {
     expect(loaded.current_account).toBe('alice')
   })
 
-  test('getAccount returns null when no accounts', async () => {
+  it('getAccount returns null when no accounts', async () => {
     const result = await manager.getAccount()
     expect(result).toBeNull()
   })
 
-  test('getAccount returns current account when no id given', async () => {
+  it('getAccount returns current account when no id given', async () => {
     const account = makeAccount('alice')
     await manager.setAccount(account)
 
@@ -81,7 +81,7 @@ describe('LineCredentialManager', () => {
     expect(result).toEqual(account)
   })
 
-  test('getAccount returns account by id', async () => {
+  it('getAccount returns account by id', async () => {
     await manager.setAccount(makeAccount('alice'))
     const bob = makeAccount('bob')
     await manager.setAccount(bob)
@@ -90,13 +90,13 @@ describe('LineCredentialManager', () => {
     expect(result).toEqual(bob)
   })
 
-  test('getAccount returns null for unknown id', async () => {
+  it('getAccount returns null for unknown id', async () => {
     await manager.setAccount(makeAccount('alice'))
     const result = await manager.getAccount('unknown')
     expect(result).toBeNull()
   })
 
-  test('removeAccount removes the account', async () => {
+  it('removeAccount removes the account', async () => {
     await manager.setAccount(makeAccount('alice'))
     await manager.removeAccount('alice')
 
@@ -104,7 +104,7 @@ describe('LineCredentialManager', () => {
     expect(loaded.accounts['alice']).toBeUndefined()
   })
 
-  test('removeAccount clears current if it was active', async () => {
+  it('removeAccount clears current if it was active', async () => {
     await manager.setAccount(makeAccount('alice'))
     await manager.removeAccount('alice')
 
@@ -112,7 +112,7 @@ describe('LineCredentialManager', () => {
     expect(loaded.current_account).toBeNull()
   })
 
-  test('removeAccount sets next account as current when active is removed', async () => {
+  it('removeAccount sets next account as current when active is removed', async () => {
     await manager.setAccount(makeAccount('alice'))
     await manager.setAccount(makeAccount('bob'))
     await manager.removeAccount('alice')
@@ -121,7 +121,7 @@ describe('LineCredentialManager', () => {
     expect(loaded.current_account).toBe('bob')
   })
 
-  test('setCurrentAccount updates current account', async () => {
+  it('setCurrentAccount updates current account', async () => {
     await manager.setAccount(makeAccount('alice'))
     await manager.setAccount(makeAccount('bob'))
     await manager.setCurrentAccount('bob')
@@ -130,7 +130,7 @@ describe('LineCredentialManager', () => {
     expect(loaded.current_account).toBe('bob')
   })
 
-  test('listAccounts returns correct format', async () => {
+  it('listAccounts returns correct format', async () => {
     await manager.setAccount(makeAccount('alice'))
     await manager.setAccount(makeAccount('bob'))
 
@@ -148,7 +148,7 @@ describe('LineCredentialManager', () => {
     expect(bob?.is_current).toBe(false)
   })
 
-  test('clearAll removes the credentials file', async () => {
+  it('clearAll removes the credentials file', async () => {
     await manager.setAccount(makeAccount('alice'))
     await manager.clearAll()
 
@@ -156,7 +156,7 @@ describe('LineCredentialManager', () => {
     expect(config).toEqual({ current_account: null, accounts: {} })
   })
 
-  test('clearAll does not throw if file does not exist', async () => {
+  it('clearAll does not throw if file does not exist', async () => {
     await expect(manager.clearAll()).resolves.toBeUndefined()
   })
 })

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

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   CredentialManager,
@@ -12,38 +12,38 @@ import {
   LineSendResultSchema,
 } from '@/platforms/line/index'
 
-test('LineClient is exported from barrel', () => {
+it('LineClient is exported from barrel', () => {
   expect(typeof LineClient).toBe('function')
 })
 
-test('LineError is exported from barrel', () => {
+it('LineError is exported from barrel', () => {
   expect(typeof LineError).toBe('function')
 })
 
-test('CredentialManager is exported from barrel', () => {
+it('CredentialManager is exported from barrel', () => {
   expect(typeof CredentialManager).toBe('function')
 })
 
-test('LineCredentialManager is exported from barrel', () => {
+it('LineCredentialManager is exported from barrel', () => {
   expect(typeof LineCredentialManager).toBe('function')
 })
 
-test('LineChatSchema is exported from barrel', () => {
+it('LineChatSchema is exported from barrel', () => {
   expect(typeof LineChatSchema.parse).toBe('function')
 })
 
-test('LineMessageSchema is exported from barrel', () => {
+it('LineMessageSchema is exported from barrel', () => {
   expect(typeof LineMessageSchema.parse).toBe('function')
 })
 
-test('LineSendResultSchema is exported from barrel', () => {
+it('LineSendResultSchema is exported from barrel', () => {
   expect(typeof LineSendResultSchema.parse).toBe('function')
 })
 
-test('LineAccountCredentialsSchema is exported from barrel', () => {
+it('LineAccountCredentialsSchema is exported from barrel', () => {
   expect(typeof LineAccountCredentialsSchema.parse).toBe('function')
 })
 
-test('LineConfigSchema is exported from barrel', () => {
+it('LineConfigSchema is exported from barrel', () => {
   expect(typeof LineConfigSchema.parse).toBe('function')
 })