agent-messenger 2.10.1 → 2.10.2

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

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

@@ -1,4 +1,4 @@
-import { afterEach, describe, expect, mock, test } from 'bun:test'
+import { afterEach, describe, expect, mock, it } from 'bun:test'
 
 import { LineListener } from '@/platforms/line/listener'
 import type { LinePushGenericEvent, LinePushMessageEvent } from '@/platforms/line/types'
@@ -73,7 +73,7 @@ describe('LineListener', () => {
   })
 
   describe('start', () => {
-    test('calls login on LineClient', async () => {
+    it('calls login on LineClient', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -82,7 +82,7 @@ describe('LineListener', () => {
       expect(mockLogin).toHaveBeenCalledTimes(1)
     })
 
-    test('is idempotent', async () => {
+    it('is idempotent', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -94,7 +94,7 @@ describe('LineListener', () => {
   })
 
   describe('connected event', () => {
-    test('emits connected with account_id after successful login', async () => {
+    it('emits connected with account_id after successful login', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -109,7 +109,7 @@ describe('LineListener', () => {
   })
 
   describe('message events', () => {
-    test('emits message with parsed fields on incoming message', async () => {
+    it('emits message with parsed fields on incoming message', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -138,7 +138,7 @@ describe('LineListener', () => {
       expect(messages[0].content_type).toBe('NONE')
     })
 
-    test('uses to.id as chat_id for own messages', async () => {
+    it('uses to.id as chat_id for own messages', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -160,7 +160,7 @@ describe('LineListener', () => {
   })
 
   describe('line_event catch-all', () => {
-    test('emits line_event for every message', async () => {
+    it('emits line_event for every message', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -180,7 +180,7 @@ describe('LineListener', () => {
       expect(events[0].type).toBe('message')
     })
 
-    test('emits line_event for raw operation events', async () => {
+    it('emits line_event for raw operation events', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -197,7 +197,7 @@ describe('LineListener', () => {
   })
 
   describe('stop', () => {
-    test('aborts and prevents reconnection', async () => {
+    it('aborts and prevents reconnection', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -210,7 +210,7 @@ describe('LineListener', () => {
   })
 
   describe('reconnection', () => {
-    test('reconnects on listen error when still running', async () => {
+    it('reconnects on listen error when still running', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -228,7 +228,7 @@ describe('LineListener', () => {
       expect(mockLogin.mock.calls.length).toBeGreaterThanOrEqual(2)
     })
 
-    test('emits error and reconnects on login failure', async () => {
+    it('emits error and reconnects on login failure', async () => {
       let callCount = 0
       mockLogin.mockImplementation((): Promise<void> => {
         callCount++
@@ -252,7 +252,7 @@ describe('LineListener', () => {
       expect(mockLogin.mock.calls.length).toBeGreaterThanOrEqual(2)
     })
 
-    test('does not reconnect after stop', async () => {
+    it('does not reconnect after stop', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -267,7 +267,7 @@ describe('LineListener', () => {
   })
 
   describe('on/off/once', () => {
-    test('off removes listener', async () => {
+    it('off removes listener', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -297,7 +297,7 @@ describe('LineListener', () => {
       expect(messages[0].text).toBe('first')
     })
 
-    test('once fires only once', async () => {
+    it('once fires only once', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)
 
@@ -326,7 +326,7 @@ describe('LineListener', () => {
   })
 
   describe('start after stop', () => {
-    test('resets reconnect attempts on fresh start', async () => {
+    it('resets reconnect attempts on fresh start', async () => {
       const client = createMockLineClient()
       listener = new LineListener(client)