agent-messenger 2.10.1 → 2.10.2

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

@@ -1,51 +1,51 @@
-import { describe, expect, test } from 'bun:test'
+import { describe, expect, it } from 'bun:test'
 
 import { parseLimitOption } from '@/platforms/instagram/commands/shared'
 import { InstagramError } from '@/platforms/instagram/types'
 
 describe('parseLimitOption', () => {
-  test('returns parsed integer for valid input', () => {
+  it('returns parsed integer for valid input', () => {
     expect(parseLimitOption('10', 20)).toBe(10)
   })
 
-  test('returns default when undefined', () => {
+  it('returns default when undefined', () => {
     expect(parseLimitOption(undefined, 20)).toBe(20)
   })
 
-  test('trims whitespace', () => {
+  it('trims whitespace', () => {
     expect(parseLimitOption('  15  ', 20)).toBe(15)
   })
 
-  test('throws InstagramError for non-numeric input', () => {
+  it('throws InstagramError for non-numeric input', () => {
     expect(() => parseLimitOption('abc', 20)).toThrow(InstagramError)
   })
 
-  test('throws for zero', () => {
+  it('throws for zero', () => {
     expect(() => parseLimitOption('0', 20)).toThrow(InstagramError)
   })
 
-  test('throws for negative numbers', () => {
+  it('throws for negative numbers', () => {
     expect(() => parseLimitOption('-5', 20)).toThrow(InstagramError)
   })
 
-  test('throws for values exceeding default maxValue', () => {
+  it('throws for values exceeding default maxValue', () => {
     expect(() => parseLimitOption('101', 20)).toThrow(InstagramError)
   })
 
-  test('respects custom maxValue', () => {
+  it('respects custom maxValue', () => {
     expect(parseLimitOption('50', 20, 50)).toBe(50)
     expect(() => parseLimitOption('51', 20, 50)).toThrow(InstagramError)
   })
 
-  test('throws for decimal numbers', () => {
+  it('throws for decimal numbers', () => {
     expect(() => parseLimitOption('10.5', 20)).toThrow(InstagramError)
   })
 
-  test('accepts boundary value 1', () => {
+  it('accepts boundary value 1', () => {
     expect(parseLimitOption('1', 20)).toBe(1)
   })
 
-  test('accepts boundary value 100', () => {
+  it('accepts boundary value 100', () => {
     expect(parseLimitOption('100', 20)).toBe(100)
   })
 })

package/src/platforms/instagram/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 { InstagramClient } from '../client'
 import * as sharedModule from './shared'
@@ -29,7 +29,7 @@ describe('whoami command', () => {
     consoleLogSpy?.mockRestore()
   })
 
-  test('outputs profile information', async () => {
+  it('outputs profile information', async () => {
     await whoamiAction({})
 
     expect(consoleLogSpy).toHaveBeenCalledTimes(1)
@@ -40,7 +40,7 @@ describe('whoami command', () => {
     expect(output.profile_pic_url).toBe('https://example.com/pic.jpg')
   })
 
-  test('outputs profile with null optional fields', async () => {
+  it('outputs profile with null optional fields', async () => {
     getProfileSpy.mockResolvedValue({
       user_id: '987654321',
       username: 'testuser',

package/src/platforms/instagram/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'
 
@@ -34,7 +34,7 @@ afterAll(() => {
 
 describe('InstagramCredentialManager', () => {
   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()
 
@@ -43,7 +43,7 @@ describe('InstagramCredentialManager', () => {
   })
 
   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',
@@ -60,14 +60,14 @@ describe('InstagramCredentialManager', () => {
   })
 
   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')
 
@@ -76,7 +76,7 @@ describe('InstagramCredentialManager', () => {
   })
 
   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()
 
@@ -86,7 +86,7 @@ describe('InstagramCredentialManager', () => {
       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()
 
@@ -96,7 +96,7 @@ describe('InstagramCredentialManager', () => {
       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' })
@@ -108,7 +108,7 @@ describe('InstagramCredentialManager', () => {
       expect(current?.account_id).toBe('first-account')
     })
 
-    test('getAccount with normalized ID lookup via createAccountId', async () => {
+    it('getAccount looks up account by normalized ID via createAccountId', async () => {
       const manager = setup()
       const account = makeAccount({ account_id: 'my-username' })
 
@@ -120,14 +120,14 @@ describe('InstagramCredentialManager', () => {
   })
 
   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' })
@@ -146,7 +146,7 @@ describe('InstagramCredentialManager', () => {
   })
 
   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' })
@@ -161,7 +161,7 @@ describe('InstagramCredentialManager', () => {
       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')
@@ -171,7 +171,7 @@ describe('InstagramCredentialManager', () => {
   })
 
   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' })
@@ -190,7 +190,7 @@ describe('InstagramCredentialManager', () => {
       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')
@@ -200,7 +200,7 @@ describe('InstagramCredentialManager', () => {
   })
 
   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)
@@ -214,7 +214,7 @@ describe('InstagramCredentialManager', () => {
       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()
@@ -222,7 +222,7 @@ describe('InstagramCredentialManager', () => {
   })
 
   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-instagram-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
@@ -236,7 +236,7 @@ describe('InstagramCredentialManager', () => {
       expect(paths.session_path).toBe(join(testConfigDir, 'instagram', 'test-account', 'session.json'))
     })
 
-    test('normalizes account ID via createAccountId', async () => {
+    it('normalizes account ID via createAccountId', async () => {
       const testConfigDir = join(
         import.meta.dir,
         `.test-instagram-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
@@ -252,7 +252,7 @@ describe('InstagramCredentialManager', () => {
   })
 
   describe('ensureAccountPaths', () => {
-    test('creates directories', async () => {
+    it('creates directories', async () => {
       const manager = setup()
 
       const paths = await manager.ensureAccountPaths('test-account')
@@ -260,7 +260,7 @@ describe('InstagramCredentialManager', () => {
       expect(existsSync(paths.account_dir)).toBe(true)
     })
 
-    test('returns paths structure', async () => {
+    it('returns paths structure', async () => {
       const testConfigDir = join(
         import.meta.dir,
         `.test-instagram-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,

package/src/platforms/instagram/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 { InstagramCredentialManager } from '@/platforms/instagram/credential-manager'
@@ -46,7 +46,7 @@ afterEach(() => {
 })
 
 describe('ensureInstagramAuth', () => {
-  test('exits with error when no account configured', async () => {
+  it('exits with error when no account configured', async () => {
     // given
     getAccountSpy.mockResolvedValue(null)
 
@@ -56,7 +56,7 @@ describe('ensureInstagramAuth', () => {
     expect(consoleSpy).toHaveBeenCalled()
   })
 
-  test('exits with error when session file missing', async () => {
+  it('exits with error when session file missing', async () => {
     // given
     getAccountSpy.mockResolvedValue(validAccount)
     existsSyncSpy.mockReturnValue(false)
@@ -67,7 +67,7 @@ describe('ensureInstagramAuth', () => {
     expect(consoleSpy).toHaveBeenCalled()
   })
 
-  test('succeeds when account and session file exist', async () => {
+  it('succeeds when account and session file exist', async () => {
     // given
     getAccountSpy.mockResolvedValue(validAccount)
     existsSyncSpy.mockReturnValue(true)

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

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   createAccountId,
@@ -11,34 +11,34 @@ import {
   InstagramListener,
 } from '@/platforms/instagram/index'
 
-test('InstagramClient is exported from barrel', () => {
+it('InstagramClient is exported from barrel', () => {
   expect(typeof InstagramClient).toBe('function')
 })
 
-test('InstagramCredentialManager is exported from barrel', () => {
+it('InstagramCredentialManager is exported from barrel', () => {
   expect(typeof InstagramCredentialManager).toBe('function')
 })
 
-test('InstagramListener is exported from barrel', () => {
+it('InstagramListener is exported from barrel', () => {
   expect(typeof InstagramListener).toBe('function')
 })
 
-test('InstagramError is exported from barrel', () => {
+it('InstagramError is exported from barrel', () => {
   expect(typeof InstagramError).toBe('function')
 })
 
-test('createAccountId is exported from barrel', () => {
+it('createAccountId is exported from barrel', () => {
   expect(typeof createAccountId).toBe('function')
 })
 
-test('extractMediaUrl is exported from barrel', () => {
+it('extractMediaUrl is exported from barrel', () => {
   expect(typeof extractMediaUrl).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/instagram/listener.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, describe, expect, spyOn, test } from 'bun:test'
+import { afterEach, describe, expect, spyOn, it } from 'bun:test'
 
 import { InstagramListener } from '@/platforms/instagram/listener'
 import type { InstagramChatSummary } from '@/platforms/instagram/types'
@@ -38,7 +38,7 @@ describe('InstagramListener', () => {
     listener?.stop()
   })
 
-  test('emits connected on start', async () => {
+  it('emits connected on start', async () => {
     const client = makeMockClient([makeChat()])
     listener = new InstagramListener(client, { pollInterval: 60_000 })
 
@@ -51,7 +51,7 @@ describe('InstagramListener', () => {
     expect(result.userId).toBe('123')
   })
 
-  test('stop emits disconnected', async () => {
+  it('emits disconnected on stop', async () => {
     const client = makeMockClient([])
     listener = new InstagramListener(client, { pollInterval: 60_000 })
 
@@ -64,7 +64,7 @@ describe('InstagramListener', () => {
     await disconnected
   })
 
-  test('does not emit message for initial poll', async () => {
+  it('does not emit message for initial poll', async () => {
     const client = makeMockClient([makeChat()])
     listener = new InstagramListener(client, { pollInterval: 60_000 })
 
@@ -77,7 +77,7 @@ describe('InstagramListener', () => {
     expect(messages).toHaveLength(0)
   })
 
-  test('multiple start calls are idempotent', async () => {
+  it('multiple start calls are idempotent', async () => {
     const client = makeMockClient([])
     listener = new InstagramListener(client, { pollInterval: 60_000 })
 
@@ -87,7 +87,7 @@ describe('InstagramListener', () => {
     expect(client.listChats).toHaveBeenCalledTimes(1)
   })
 
-  test('emits error when listChats throws', async () => {
+  it('emits error when listChats throws', async () => {
     const client = makeMockClient()
     client.listChats.mockRejectedValueOnce(new Error('network error'))
     listener = new InstagramListener(client, { pollInterval: 60_000 })
@@ -101,7 +101,7 @@ describe('InstagramListener', () => {
     expect(err.message).toBe('network error')
   })
 
-  test('on/off registers and unregisters handlers', () => {
+  it('on/off registers and unregisters handlers', () => {
     const client = makeMockClient()
     listener = new InstagramListener(client, { pollInterval: 60_000 })
 
@@ -116,7 +116,7 @@ describe('InstagramListener', () => {
     expect(called).toBe(false)
   })
 
-  test('once fires handler only once', async () => {
+  it('once fires handler only once', async () => {
     const client = makeMockClient([])
     listener = new InstagramListener(client, { pollInterval: 60_000 })