agent-messenger 2.10.1 → 2.10.2

package/src/platforms/slack/credential-manager.test.ts

@@ -1,4 +1,4 @@
-import { afterAll, beforeEach, describe, expect, test } from 'bun:test'
+import { afterAll, beforeEach, describe, expect, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { join } from 'node:path'
 
@@ -19,7 +19,7 @@ describe('CredentialManager', () => {
     rmSync(testConfigDir, { recursive: true, force: true })
   })
 
-  test('load() returns empty config when no file exists', async () => {
+  it('load() returns empty config when no file exists', async () => {
     // Given: No credentials file exists
     // When: load() is called
     const config = await manager.load()
@@ -31,7 +31,7 @@ describe('CredentialManager', () => {
     })
   })
 
-  test('save() and load() round-trip', async () => {
+  it('save() and load() round-trip', async () => {
     // Given: A config with workspace credentials
     const testConfig: Config = {
       current_workspace: 'workspace-1',
@@ -55,7 +55,7 @@ describe('CredentialManager', () => {
     expect(loaded).toEqual(testConfig)
   })
 
-  test('save() creates credentials file with 0600 permissions', async () => {
+  it('save() creates credentials file with 0600 permissions', async () => {
     // Given: A config to save
     const testConfig: Config = {
       current_workspace: null,
@@ -76,7 +76,7 @@ describe('CredentialManager', () => {
     expect(permissions).toBe(0o600)
   })
 
-  test('setWorkspace() adds or updates workspace', async () => {
+  it('setWorkspace() adds or updates workspace', async () => {
     // Given: A credential manager with empty config
     // When: setWorkspace() is called
     const creds: WorkspaceCredentials = {
@@ -92,7 +92,7 @@ describe('CredentialManager', () => {
     expect(loaded.workspaces['ws-123']).toEqual(creds)
   })
 
-  test('getWorkspace() returns workspace by id', async () => {
+  it('getWorkspace() returns workspace by id', async () => {
     // Given: A workspace is saved
     const creds: WorkspaceCredentials = {
       workspace_id: 'ws-456',
@@ -109,7 +109,7 @@ describe('CredentialManager', () => {
     expect(retrieved).toEqual(creds)
   })
 
-  test('getWorkspace() returns null for non-existent workspace', async () => {
+  it('getWorkspace() returns null for non-existent workspace', async () => {
     // Given: No workspace with this id exists
     // When: getWorkspace() is called
     const retrieved = await manager.getWorkspace('non-existent')
@@ -118,7 +118,7 @@ describe('CredentialManager', () => {
     expect(retrieved).toBeNull()
   })
 
-  test('getWorkspace() returns current workspace when no id provided', async () => {
+  it('getWorkspace() returns current workspace when no id provided', async () => {
     // Given: A current workspace is set
     const creds: WorkspaceCredentials = {
       workspace_id: 'ws-789',
@@ -136,7 +136,7 @@ describe('CredentialManager', () => {
     expect(retrieved).toEqual(creds)
   })
 
-  test('getWorkspace() returns null when no current workspace set', async () => {
+  it('getWorkspace() returns null when no current workspace set', async () => {
     // Given: No current workspace is set
     // When: getWorkspace() is called without id
     const retrieved = await manager.getWorkspace()
@@ -145,7 +145,7 @@ describe('CredentialManager', () => {
     expect(retrieved).toBeNull()
   })
 
-  test('removeWorkspace() deletes workspace', async () => {
+  it('removeWorkspace() deletes workspace', async () => {
     // Given: A workspace is saved
     const creds: WorkspaceCredentials = {
       workspace_id: 'ws-delete',
@@ -163,7 +163,7 @@ describe('CredentialManager', () => {
     expect(retrieved).toBeNull()
   })
 
-  test('removeWorkspace() clears current workspace if it matches', async () => {
+  it('removeWorkspace() clears current workspace if it matches', async () => {
     // Given: A workspace is set as current
     const creds: WorkspaceCredentials = {
       workspace_id: 'ws-current',
@@ -182,7 +182,7 @@ describe('CredentialManager', () => {
     expect(config.current_workspace).toBeNull()
   })
 
-  test('setCurrentWorkspace() sets current workspace', async () => {
+  it('setCurrentWorkspace() sets current workspace', async () => {
     // Given: A workspace exists
     const creds: WorkspaceCredentials = {
       workspace_id: 'ws-set-current',
@@ -200,7 +200,7 @@ describe('CredentialManager', () => {
     expect(config.current_workspace).toBe('ws-set-current')
   })
 
-  test('creates config directory if it does not exist', async () => {
+  it('creates config directory if it does not exist', async () => {
     // Given: Config directory does not exist
     expect(existsSync(testConfigDir)).toBe(false)
 
@@ -215,7 +215,7 @@ describe('CredentialManager', () => {
     expect(existsSync(testConfigDir)).toBe(true)
   })
 
-  test('multiple workspaces can coexist', async () => {
+  it('multiple workspaces can coexist', async () => {
     // Given: Multiple workspaces are added
     const creds1: WorkspaceCredentials = {
       workspace_id: 'ws-1',

package/src/platforms/slack/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 { SlackClient } from './client'
 import { CredentialManager } from './credential-manager'
@@ -63,7 +63,7 @@ afterEach(() => {
 })
 
 describe('ensureSlackAuth', () => {
-  test('skips extraction when stored credentials are valid', async () => {
+  it('skips extraction when stored credentials are valid', async () => {
     // given
     getWorkspaceSpy.mockResolvedValue({
       workspace_id: 'T123',
@@ -80,7 +80,7 @@ describe('ensureSlackAuth', () => {
     expect(extractSpy).not.toHaveBeenCalled()
   })
 
-  test('refreshes cookie when stored credentials are stale', async () => {
+  it('refreshes cookie when stored credentials are stale', async () => {
     // given
     getWorkspaceSpy.mockResolvedValue({
       workspace_id: 'T123',
@@ -110,7 +110,7 @@ describe('ensureSlackAuth', () => {
     expect(setWorkspaceSpy).toHaveBeenCalledWith(expect.objectContaining({ cookie: 'xoxd-fresh-cookie' }))
   })
 
-  test('falls through to full extraction when cookie refresh fails', async () => {
+  it('falls through to full extraction when cookie refresh fails', async () => {
     // given
     getWorkspaceSpy.mockResolvedValue({
       workspace_id: 'T123',
@@ -128,7 +128,7 @@ describe('ensureSlackAuth', () => {
     expect(extractSpy).toHaveBeenCalled()
   })
 
-  test('extracts and saves credentials when none exist', async () => {
+  it('extracts and saves credentials when none exist', async () => {
     // when
     await ensureSlackAuth()
 
@@ -145,7 +145,7 @@ describe('ensureSlackAuth', () => {
     )
   })
 
-  test('sets first workspace as current when none set', async () => {
+  it('sets first workspace as current when none set', async () => {
     // when
     await ensureSlackAuth()
 
@@ -153,7 +153,7 @@ describe('ensureSlackAuth', () => {
     expect(setCurrentWorkspaceSpy).toHaveBeenCalledWith('T123')
   })
 
-  test('does not override existing current workspace', async () => {
+  it('does not override existing current workspace', async () => {
     // given
     loadSpy.mockResolvedValue({
       current_workspace: 'T999',
@@ -167,7 +167,7 @@ describe('ensureSlackAuth', () => {
     expect(setCurrentWorkspaceSpy).not.toHaveBeenCalled()
   })
 
-  test('handles multiple workspaces', async () => {
+  it('handles multiple workspaces', async () => {
     // given
     extractSpy.mockResolvedValue([
       { workspace_id: 'T1', workspace_name: 'ws1', token: 'xoxc-1', cookie: 'xoxd-1' },
@@ -191,7 +191,7 @@ describe('ensureSlackAuth', () => {
     expect(setCurrentWorkspaceSpy).toHaveBeenCalledWith('T1')
   })
 
-  test('skips invalid workspaces during validation', async () => {
+  it('skips invalid workspaces during validation', async () => {
     // given
     extractSpy.mockResolvedValue([
       { workspace_id: 'T-bad', workspace_name: 'bad', token: 'xoxc-bad', cookie: 'xoxd-bad' },
@@ -213,7 +213,7 @@ describe('ensureSlackAuth', () => {
     expect(setCurrentWorkspaceSpy).toHaveBeenCalledWith('T-good')
   })
 
-  test('silently handles extraction failure', async () => {
+  it('silently handles extraction failure', async () => {
     // given
     extractSpy.mockRejectedValue(new Error('Slack directory not found'))
 
@@ -224,7 +224,7 @@ describe('ensureSlackAuth', () => {
     expect(setWorkspaceSpy).not.toHaveBeenCalled()
   })
 
-  test('propagates cookie lock error when Slack app is running', async () => {
+  it('propagates cookie lock error when Slack app is running', async () => {
     // given
     const error = new Error(
       'Failed to read Slack cookies. The Slack app is currently running and locking the cookie database. ' +
@@ -237,7 +237,7 @@ describe('ensureSlackAuth', () => {
     await expect(ensureSlackAuth()).rejects.toThrow('locking the cookie')
   })
 
-  test('does not save when no workspaces extracted', async () => {
+  it('does not save when no workspaces extracted', async () => {
     // given
     extractSpy.mockResolvedValue([])
 
@@ -249,7 +249,7 @@ describe('ensureSlackAuth', () => {
     expect(setCurrentWorkspaceSpy).not.toHaveBeenCalled()
   })
 
-  test('refreshes token from web when local token is invalid', async () => {
+  it('refreshes token from web when local token is invalid', async () => {
     // given — local token is stale, but cookie is valid and domain is known
     extractSpy.mockResolvedValue([
       { workspace_id: 'T-stale', workspace_name: 'stale-ws', token: 'xoxc-stale', cookie: 'xoxd-valid' },
@@ -278,7 +278,7 @@ describe('ensureSlackAuth', () => {
     )
   })
 
-  test('skips web refresh when no domain is known for workspace', async () => {
+  it('skips web refresh when no domain is known for workspace', async () => {
     // given — domain mapping is empty
     extractSpy.mockResolvedValue([
       { workspace_id: 'T-stale', workspace_name: 'stale-ws', token: 'xoxc-stale', cookie: 'xoxd-valid' },
@@ -294,7 +294,7 @@ describe('ensureSlackAuth', () => {
     expect(setWorkspaceSpy).not.toHaveBeenCalled()
   })
 
-  test('skips web refresh when workspace has no cookie', async () => {
+  it('skips web refresh when workspace has no cookie', async () => {
     // given — cookie is empty
     extractSpy.mockResolvedValue([
       { workspace_id: 'T-stale', workspace_name: 'stale-ws', token: 'xoxc-stale', cookie: '' },
@@ -310,7 +310,7 @@ describe('ensureSlackAuth', () => {
     expect(setWorkspaceSpy).not.toHaveBeenCalled()
   })
 
-  test('updates workspace_name from auth response', async () => {
+  it('updates workspace_name from auth response', async () => {
     // given
     extractSpy.mockResolvedValue([
       { workspace_id: 'T1', workspace_name: 'old-name', token: 'xoxc-1', cookie: 'xoxd-1' },
@@ -329,7 +329,7 @@ describe('ensureSlackAuth', () => {
     expect(setWorkspaceSpy).toHaveBeenCalledWith(expect.objectContaining({ workspace_name: 'New Team Name' }))
   })
 
-  test('resolves unknown workspace_id from testAuth before saving', async () => {
+  it('resolves unknown workspace_id from testAuth before saving', async () => {
     // given — extractor returns unknown workspace_id
     extractSpy.mockResolvedValue([
       { workspace_id: 'unknown', workspace_name: 'unknown', token: 'xoxc-1', cookie: 'xoxd-1' },
@@ -352,7 +352,7 @@ describe('ensureSlackAuth', () => {
 })
 
 describe('refreshTokenFromWeb', () => {
-  test('extracts token from ssb/redirect HTML response', async () => {
+  it('extracts token from ssb/redirect HTML response', async () => {
     // given
     fetchSpy.mockResolvedValue(
       new Response(
@@ -372,7 +372,7 @@ describe('refreshTokenFromWeb', () => {
     })
   })
 
-  test('returns null when response has no token', async () => {
+  it('returns null when response has no token', async () => {
     // given
     fetchSpy.mockResolvedValue(new Response('<html>no token here</html>', { status: 200 }))
 
@@ -383,7 +383,7 @@ describe('refreshTokenFromWeb', () => {
     expect(token).toBeNull()
   })
 
-  test('returns null on HTTP error', async () => {
+  it('returns null on HTTP error', async () => {
     // given
     fetchSpy.mockResolvedValue(new Response('', { status: 403 }))
 
@@ -394,7 +394,7 @@ describe('refreshTokenFromWeb', () => {
     expect(token).toBeNull()
   })
 
-  test('returns null on network error', async () => {
+  it('returns null on network error', async () => {
     // given
     fetchSpy.mockRejectedValue(new Error('network timeout'))
 

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

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   CredentialManager,
@@ -14,46 +14,46 @@ import {
   ConfigSchema,
 } from '@/platforms/slack/index'
 
-test('SlackClient is exported from barrel', () => {
+it('SlackClient is exported from barrel', () => {
   expect(typeof SlackClient).toBe('function')
 })
 
-test('SlackError is exported from barrel', () => {
+it('SlackError is exported from barrel', () => {
   expect(typeof SlackError).toBe('function')
 })
 
-test('CredentialManager is exported from barrel', () => {
+it('CredentialManager is exported from barrel', () => {
   expect(typeof CredentialManager).toBe('function')
 })
 
-test('SlackCredentialManager is exported from barrel', () => {
+it('SlackCredentialManager is exported from barrel', () => {
   expect(typeof SlackCredentialManager).toBe('function')
 })
 
-test('SlackChannelSchema is exported from barrel', () => {
+it('SlackChannelSchema is exported from barrel', () => {
   expect(typeof SlackChannelSchema.parse).toBe('function')
 })
 
-test('SlackReactionSchema is exported from barrel', () => {
+it('SlackReactionSchema is exported from barrel', () => {
   expect(typeof SlackReactionSchema.parse).toBe('function')
 })
 
-test('SlackFileSchema is exported from barrel', () => {
+it('SlackFileSchema is exported from barrel', () => {
   expect(typeof SlackFileSchema.parse).toBe('function')
 })
 
-test('SlackMessageSchema is exported from barrel', () => {
+it('SlackMessageSchema is exported from barrel', () => {
   expect(typeof SlackMessageSchema.parse).toBe('function')
 })
 
-test('SlackUserSchema is exported from barrel', () => {
+it('SlackUserSchema is exported from barrel', () => {
   expect(typeof SlackUserSchema.parse).toBe('function')
 })
 
-test('WorkspaceCredentialsSchema is exported from barrel', () => {
+it('WorkspaceCredentialsSchema is exported from barrel', () => {
   expect(typeof WorkspaceCredentialsSchema.parse).toBe('function')
 })
 
-test('ConfigSchema is exported from barrel', () => {
+it('ConfigSchema is exported from barrel', () => {
   expect(typeof ConfigSchema.parse).toBe('function')
 })

package/src/platforms/slack/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 { SlackListener } from '@/platforms/slack/listener'
 import type { SlackRTMMessageEvent, SlackRTMReactionEvent } from '@/platforms/slack/types'
@@ -79,7 +79,7 @@ describe('SlackListener', () => {
   })
 
   describe('start', () => {
-    test('calls rtmConnect and opens WebSocket', async () => {
+    it('calls rtmConnect and opens WebSocket', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -89,7 +89,7 @@ describe('SlackListener', () => {
       expect(client.rtmConnect).toHaveBeenCalledTimes(1)
     })
 
-    test('is idempotent', async () => {
+    it('is idempotent', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -101,7 +101,7 @@ describe('SlackListener', () => {
   })
 
   describe('connected event', () => {
-    test('emits connected with self/team on hello', async () => {
+    it('emits connected with self/team on hello', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -119,7 +119,7 @@ describe('SlackListener', () => {
   })
 
   describe('message events', () => {
-    test('emits message events', async () => {
+    it('emits message events', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -145,7 +145,7 @@ describe('SlackListener', () => {
   })
 
   describe('reaction events', () => {
-    test('emits reaction_added events', async () => {
+    it('emits reaction_added events', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -168,7 +168,7 @@ describe('SlackListener', () => {
   })
 
   describe('slack_event catch-all', () => {
-    test('emits slack_event for every non-hello event', async () => {
+    it('emits slack_event for every non-hello event', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -188,7 +188,7 @@ describe('SlackListener', () => {
   })
 
   describe('ping/pong', () => {
-    test('does not treat pong reply as an event', async () => {
+    it('does not treat pong reply as an event', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -205,7 +205,7 @@ describe('SlackListener', () => {
   })
 
   describe('stop', () => {
-    test('closes WebSocket and prevents reconnection', async () => {
+    it('closes WebSocket and prevents reconnection', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -220,7 +220,7 @@ describe('SlackListener', () => {
   })
 
   describe('reconnection', () => {
-    test('reconnects on WebSocket close when still running', async () => {
+    it('reconnects on WebSocket close when still running', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -237,7 +237,7 @@ describe('SlackListener', () => {
       expect(client.rtmConnect.mock.calls.length).toBeGreaterThanOrEqual(2)
     })
 
-    test('emits error and reconnects on rtmConnect failure', async () => {
+    it('emits error and reconnects on rtmConnect failure', async () => {
       let callCount = 0
       const client = createMockClient({
         rtmConnect: mock(() => {
@@ -268,7 +268,7 @@ describe('SlackListener', () => {
   })
 
   describe('on/off/once', () => {
-    test('off removes listener', async () => {
+    it('off removes listener', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -287,7 +287,7 @@ describe('SlackListener', () => {
       expect(messages[0].text).toBe('a')
     })
 
-    test('once fires only once', async () => {
+    it('once fires only once', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -305,7 +305,7 @@ describe('SlackListener', () => {
   })
 
   describe('goodbye / team_migration_started', () => {
-    test('goodbye triggers immediate reconnect without backoff', async () => {
+    it('goodbye triggers immediate reconnect without backoff', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -322,7 +322,7 @@ describe('SlackListener', () => {
       expect((listener as any).reconnectAttempts).toBe(0)
     })
 
-    test('team_migration_started triggers immediate reconnect', async () => {
+    it('team_migration_started triggers immediate reconnect', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -336,7 +336,7 @@ describe('SlackListener', () => {
       expect((listener as any).reconnectAttempts).toBe(0)
     })
 
-    test('goodbye and team_migration_started are not emitted as events', async () => {
+    it('goodbye and team_migration_started are not emitted as events', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 
@@ -356,7 +356,7 @@ describe('SlackListener', () => {
   })
 
   describe('start after stop', () => {
-    test('resets reconnect attempts on fresh start', async () => {
+    it('resets reconnect attempts on fresh start', async () => {
       const client = createMockClient()
       listener = new SlackListener(client)
 

package/src/platforms/slack/token-extractor-node.test.ts

@@ -1,7 +1,7 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 import { execSync } from 'node:child_process'
 
-test('TokenExtractor works in Node.js', () => {
+it('TokenExtractor works in Node.js', () => {
   const result = execSync('bun tsx src/platforms/slack/token-extractor-node-test.ts', {
     cwd: process.cwd(),
     encoding: 'utf-8',