agent-messenger 2.10.1 → 2.10.2

package/src/platforms/slack/commands/snapshot.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, spyOn, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import { snapshotCommand } from '@/platforms/slack/commands/snapshot'
@@ -6,35 +6,35 @@ import { CredentialManager } from '@/platforms/slack/credential-manager'
 import type { SlackChannel, SlackMessage, SlackUser } from '@/platforms/slack/types'
 
 // Test the command structure (no mocks needed)
-test('snapshot command exports correctly', () => {
+it('snapshot command exports correctly', () => {
   expect(snapshotCommand).toBeDefined()
   expect(typeof snapshotCommand).toBe('object')
 })
 
-test('snapshot command has correct structure', () => {
+it('snapshot command has correct structure', () => {
   expect(snapshotCommand.name()).toBe('snapshot')
   expect(snapshotCommand.description()).toContain('workspace')
 })
 
-test('snapshot command has --channels-only option', () => {
+it('snapshot command has --channels-only option', () => {
   const options = snapshotCommand.options
   const hasChannelsOnly = options.some((opt: any) => opt.long === '--channels-only')
   expect(hasChannelsOnly).toBe(true)
 })
 
-test('snapshot command has --users-only option', () => {
+it('snapshot command has --users-only option', () => {
   const options = snapshotCommand.options
   const hasUsersOnly = options.some((opt: any) => opt.long === '--users-only')
   expect(hasUsersOnly).toBe(true)
 })
 
-test('snapshot command has --limit option', () => {
+it('snapshot command has --limit option', () => {
   const options = snapshotCommand.options
   const hasLimit = options.some((opt: any) => opt.long === '--limit')
   expect(hasLimit).toBe(true)
 })
 
-test('snapshot command has --full option', () => {
+it('snapshot command has --full option', () => {
   const options = snapshotCommand.options
   const hasFull = options.some((opt: any) => opt.long === '--full')
   expect(hasFull).toBe(true)
@@ -153,7 +153,7 @@ afterEach(() => {
   clientGetMessagesSpy?.mockRestore()
 })
 
-test('brief snapshot (default) returns workspace, channels as {id, name}, and hint', async () => {
+it('brief snapshot (default) returns workspace, channels as {id, name}, and hint', async () => {
   const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
 
   const auth = await client.testAuth()
@@ -185,7 +185,7 @@ test('brief snapshot (default) returns workspace, channels as {id, name}, and hi
   expect(snapshot.hint).toContain('message list')
 })
 
-test('brief snapshot excludes archived channels', async () => {
+it('brief snapshot excludes archived channels', async () => {
   const channelsWithArchived: SlackChannel[] = [
     ...mockChannels,
     {
@@ -210,7 +210,7 @@ test('brief snapshot excludes archived channels', async () => {
   expect(active.every((ch) => !ch.is_archived)).toBe(true)
 })
 
-test('full snapshot returns workspace, channels, messages, and users', async () => {
+it('full snapshot returns workspace, channels, messages, and users', async () => {
   const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
 
   const auth = await client.testAuth()
@@ -278,7 +278,7 @@ test('full snapshot returns workspace, channels, messages, and users', async ()
   expect(snapshot.users[0].name).toBe('alice')
 })
 
-test('snapshot with --channels-only excludes messages and users', async () => {
+it('snapshot with --channels-only excludes messages and users', async () => {
   const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
 
   const auth = await client.testAuth()
@@ -307,7 +307,7 @@ test('snapshot with --channels-only excludes messages and users', async () => {
   expect((snapshot as any).users).toBeUndefined()
 })
 
-test('snapshot with --users-only excludes channels and messages', async () => {
+it('snapshot with --users-only excludes channels and messages', async () => {
   const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
 
   const auth = await client.testAuth()
@@ -335,7 +335,7 @@ test('snapshot with --users-only excludes channels and messages', async () => {
   expect((snapshot as any).recent_messages).toBeUndefined()
 })
 
-test('snapshot respects --limit option for messages', async () => {
+it('snapshot respects --limit option for messages', async () => {
   const client = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
 
   const auth = await client.testAuth()

package/src/platforms/slack/commands/unread.test.ts

@@ -43,7 +43,7 @@ afterEach(() => {
 
 describe('unread commands', () => {
   describe('getUnreadCounts', () => {
-    it('should return unread counts', async () => {
+    it('returns unread counts', async () => {
       const mockCounts = {
         channels: [
           { id: 'C123', name: 'general', unread_count: 5, mention_count: 2 },
@@ -66,7 +66,7 @@ describe('unread commands', () => {
   })
 
   describe('getThreadView', () => {
-    it('should return thread subscription details', async () => {
+    it('returns thread subscription details', async () => {
       const mockThreadView = {
         channel_id: 'C123',
         thread_ts: '1234567890.123456',
@@ -87,7 +87,7 @@ describe('unread commands', () => {
   })
 
   describe('markRead', () => {
-    it('should mark channel as read', async () => {
+    it('marks channel as read', async () => {
       const mockClient = await new SlackClient().login({ token: 'xoxc-test', cookie: 'test-cookie' })
       await mockClient.markRead('C123', '1234567890.123456')
 
@@ -96,7 +96,7 @@ describe('unread commands', () => {
   })
 
   describe('CLI commands', () => {
-    it('should handle unread counts command', async () => {
+    it('handles unread counts command', async () => {
       const mockConsoleLog = mock(() => {})
       const originalLog = console.log
       console.log = mockConsoleLog
@@ -109,7 +109,7 @@ describe('unread commands', () => {
       console.log = originalLog
     })
 
-    it('should handle unread threads command', async () => {
+    it('handles unread threads command', async () => {
       const mockConsoleLog = mock(() => {})
       const originalLog = console.log
       console.log = mockConsoleLog
@@ -122,7 +122,7 @@ describe('unread commands', () => {
       console.log = originalLog
     })
 
-    it('should handle mark read command', async () => {
+    it('handles mark read command', async () => {
       const mockConsoleLog = mock(() => {})
       const originalLog = console.log
       console.log = mockConsoleLog

package/src/platforms/slack/commands/user.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from 'bun:test'
+import { describe, expect, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import { userCommand } from '@/platforms/slack/commands/user'
@@ -44,7 +44,7 @@ const mockUsers: SlackUser[] = [
 
 describe('User Commands', () => {
   describe('user list', () => {
-    test('lists all users', async () => {
+    it('lists all users', async () => {
       // Given: SlackClient with users
       const _mockClient = {
         listUsers: async () => mockUsers,
@@ -57,14 +57,14 @@ describe('User Commands', () => {
       expect(result).toBeDefined()
     })
 
-    test('filters out bots by default', async () => {
+    it('filters out bots by default', async () => {
       // Given: Users including bots
       // When: Listing without --include-bots flag
       // Then: Should exclude bots (is_bot: true)
       expect(mockUsers.filter((u) => !u.is_bot)).toHaveLength(2)
     })
 
-    test('includes bots with --include-bots flag', async () => {
+    it('includes bots with --include-bots flag', async () => {
       // Given: Users including bots
       // When: Listing with --include-bots flag
       // Then: Should include all users
@@ -73,7 +73,7 @@ describe('User Commands', () => {
   })
 
   describe('user info', () => {
-    test('shows user details by ID', async () => {
+    it('shows user details by ID', async () => {
       // Given: User ID
       // When: Getting user info
       // Then: Should return user details
@@ -82,7 +82,7 @@ describe('User Commands', () => {
       expect(user.name).toBe('alice')
     })
 
-    test('returns error for invalid user ID', async () => {
+    it('returns error for invalid user ID', async () => {
       // Given: Invalid user ID
       // When: Getting user info
       // Then: Should return error
@@ -91,14 +91,14 @@ describe('User Commands', () => {
   })
 
   describe('user me', () => {
-    test('shows current authenticated user', async () => {
+    it('shows current authenticated user', async () => {
       // Given: Authenticated client
       // When: Running user me
       // Then: Should return current user info
       expect(true).toBe(true)
     })
 
-    test('includes user profile details', async () => {
+    it('includes user profile details', async () => {
       // Given: Current user
       const user = mockUsers[0]
 
@@ -110,7 +110,7 @@ describe('User Commands', () => {
   })
 
   describe('output format', () => {
-    test('returns JSON with user data', async () => {
+    it('returns JSON with user data', async () => {
       // Given: Users
       const user = mockUsers[0]
 
@@ -122,7 +122,7 @@ describe('User Commands', () => {
       expect(json).toContain('alice')
     })
 
-    test('supports --pretty flag for formatting', async () => {
+    it('supports --pretty flag for formatting', async () => {
       // Given: Output data
       const data = { users: mockUsers }
 

package/src/platforms/slack/commands/usergroup.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import type { SlackUsergroup } from '@/platforms/slack/types'
@@ -39,14 +39,14 @@ describe('Usergroup Commands', () => {
   })
 
   describe('usergroup list', () => {
-    test('lists all user groups', async () => {
+    it('lists all user groups', async () => {
       const usergroups = await (mockClient as SlackClient).listUsergroups()
       expect(usergroups).toHaveLength(1)
       expect(usergroups[0].name).toBe('Marketing Team')
       expect(usergroups[0].handle).toBe('marketing-team')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.listUsergroups = mock(async () => {
         throw new Error('invalid_auth')
       })
@@ -55,7 +55,7 @@ describe('Usergroup Commands', () => {
   })
 
   describe('usergroup create', () => {
-    test('creates a user group successfully', async () => {
+    it('creates a user group successfully', async () => {
       const result = await (mockClient as SlackClient).createUsergroup('Marketing Team', {
         handle: 'marketing-team',
         description: 'Marketing gurus, PR experts and product advocates.',
@@ -64,7 +64,7 @@ describe('Usergroup Commands', () => {
       expect(result.name).toBe('Marketing Team')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.createUsergroup = mock(async () => {
         throw new Error('name_already_exists')
       })
@@ -73,7 +73,7 @@ describe('Usergroup Commands', () => {
   })
 
   describe('usergroup update', () => {
-    test('updates a user group successfully', async () => {
+    it('updates a user group successfully', async () => {
       const result = await (mockClient as SlackClient).updateUsergroup('S0616NG6M', {
         name: 'Marketing Team',
         handle: 'marketing-team',
@@ -82,7 +82,7 @@ describe('Usergroup Commands', () => {
       expect(result.name).toBe('Marketing Team')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.updateUsergroup = mock(async () => {
         throw new Error('no_such_subteam')
       })
@@ -93,13 +93,13 @@ describe('Usergroup Commands', () => {
   })
 
   describe('usergroup enable', () => {
-    test('enables a user group successfully', async () => {
+    it('enables a user group successfully', async () => {
       const result = await (mockClient as SlackClient).enableUsergroup('S0616NG6M')
       expect(result.id).toBe('S0616NG6M')
       expect(result.date_delete).toBe(0)
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.enableUsergroup = mock(async () => {
         throw new Error('no_such_subteam')
       })
@@ -108,13 +108,13 @@ describe('Usergroup Commands', () => {
   })
 
   describe('usergroup disable', () => {
-    test('disables a user group successfully', async () => {
+    it('disables a user group successfully', async () => {
       const result = await (mockClient as SlackClient).disableUsergroup('S0616NG6M')
       expect(result.id).toBe('S0616NG6M')
       expect(result.date_delete).toBeGreaterThan(0)
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.disableUsergroup = mock(async () => {
         throw new Error('no_such_subteam')
       })
@@ -123,14 +123,14 @@ describe('Usergroup Commands', () => {
   })
 
   describe('usergroup members', () => {
-    test('lists members of a user group', async () => {
+    it('lists members of a user group', async () => {
       const users = await (mockClient as SlackClient).listUsergroupMembers('S0616NG6M')
       expect(users).toHaveLength(2)
       expect(users).toContain('U060R4BJ4')
       expect(users).toContain('U060RNRCZ')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.listUsergroupMembers = mock(async () => {
         throw new Error('no_such_subteam')
       })
@@ -139,13 +139,13 @@ describe('Usergroup Commands', () => {
   })
 
   describe('usergroup members-update', () => {
-    test('updates members of a user group', async () => {
+    it('updates members of a user group', async () => {
       const result = await (mockClient as SlackClient).updateUsergroupMembers('S0616NG6M', ['U060R4BJ4', 'U060RNRCZ'])
       expect(result.id).toBe('S0616NG6M')
       expect(result.users).toHaveLength(2)
     })
 
-    test('throws error when no users provided', async () => {
+    it('throws error when no users provided', async () => {
       mockClient.updateUsergroupMembers = mock(async () => {
         throw new Error('no_users_provided')
       })

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, spyOn, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import { whoamiAction, whoamiCommand } from '@/platforms/slack/commands/whoami'
@@ -54,19 +54,19 @@ afterEach(() => {
   processExitSpy?.mockRestore()
 })
 
-test('whoami command is defined with correct name and description', () => {
+it('whoami command is defined with correct name and description', () => {
   expect(whoamiCommand).toBeDefined()
   expect(whoamiCommand.name()).toBe('whoami')
   expect(whoamiCommand.description()).toBe('Show current authenticated user')
 })
 
-test('whoami command has --pretty option', () => {
+it('whoami command has --pretty option', () => {
   const options = whoamiCommand.options
   const hasPretty = options.some((opt: { long?: string }) => opt.long === '--pretty')
   expect(hasPretty).toBe(true)
 })
 
-test('whoami outputs expected fields', async () => {
+it('whoami outputs expected fields', async () => {
   await whoamiAction({})
 
   expect(consoleLogSpy).toHaveBeenCalledWith(
@@ -88,7 +88,7 @@ test('whoami outputs expected fields', async () => {
   )
 })
 
-test('whoami outputs pretty-printed JSON when pretty is true', async () => {
+it('whoami outputs pretty-printed JSON when pretty is true', async () => {
   await whoamiAction({ pretty: true })
 
   expect(consoleLogSpy).toHaveBeenCalledWith(
@@ -114,7 +114,7 @@ test('whoami outputs pretty-printed JSON when pretty is true', async () => {
   )
 })
 
-test('whoami exits with error when no workspace is set', async () => {
+it('whoami exits with error when no workspace is set', async () => {
   credManagerSpy.mockResolvedValue(null)
 
   await whoamiAction({})

package/src/platforms/slack/commands/workspace.test.ts

@@ -1,4 +1,4 @@
-import { afterAll, describe, expect, test } from 'bun:test'
+import { afterAll, describe, expect, it } from 'bun:test'
 import { rmSync } from 'node:fs'
 import { join } from 'node:path'
 
@@ -24,13 +24,13 @@ afterAll(() => {
 
 describe('Workspace Commands', () => {
   describe('workspace list', () => {
-    test('returns empty list when no workspaces exist', async () => {
+    it('returns empty list when no workspaces exist', async () => {
       const credManager = setup()
       const config = await credManager.load()
       expect(Object.keys(config.workspaces)).toHaveLength(0)
     })
 
-    test('lists all workspaces with current marker', async () => {
+    it('lists all workspaces with current marker', async () => {
       const credManager = setup()
       const ws1: WorkspaceCredentials = {
         workspace_id: 'T123',
@@ -58,7 +58,7 @@ describe('Workspace Commands', () => {
       expect(config.current_workspace).toBe('T123')
     })
 
-    test('shows current marker for active workspace', async () => {
+    it('shows current marker for active workspace', async () => {
       const credManager = setup()
       const ws: WorkspaceCredentials = {
         workspace_id: 'T789',
@@ -75,7 +75,7 @@ describe('Workspace Commands', () => {
       expect(config.workspaces.T789).toBeDefined()
     })
 
-    test('handles list with no current workspace', async () => {
+    it('handles list with no current workspace', async () => {
       const credManager = setup()
       const ws: WorkspaceCredentials = {
         workspace_id: 'T999',
@@ -93,7 +93,7 @@ describe('Workspace Commands', () => {
   })
 
   describe('workspace switch', () => {
-    test('switches to existing workspace', async () => {
+    it('switches to existing workspace', async () => {
       const credManager = setup()
       const ws1: WorkspaceCredentials = {
         workspace_id: 'T111',
@@ -117,13 +117,13 @@ describe('Workspace Commands', () => {
       expect(config.current_workspace).toBe('T222')
     })
 
-    test('fails when workspace does not exist', async () => {
+    it('fails when workspace does not exist', async () => {
       const credManager = setup()
       const ws = await credManager.getWorkspace('nonexistent')
       expect(ws).toBeNull()
     })
 
-    test('validates workspace exists before switching', async () => {
+    it('validates workspace exists before switching', async () => {
       const credManager = setup()
       const ws: WorkspaceCredentials = {
         workspace_id: 'T333',
@@ -140,7 +140,7 @@ describe('Workspace Commands', () => {
       expect(targetExists).toBe(false)
     })
 
-    test('preserves workspace credentials when switching', async () => {
+    it('preserves workspace credentials when switching', async () => {
       const credManager = setup()
       const ws1: WorkspaceCredentials = {
         workspace_id: 'T444',
@@ -171,7 +171,7 @@ describe('Workspace Commands', () => {
   })
 
   describe('workspace current', () => {
-    test('returns current workspace details', async () => {
+    it('returns current workspace details', async () => {
       const credManager = setup()
       const ws: WorkspaceCredentials = {
         workspace_id: 'T666',
@@ -191,13 +191,13 @@ describe('Workspace Commands', () => {
       expect(current?.cookie).toBe('xoxd-666')
     })
 
-    test('returns null when no current workspace set', async () => {
+    it('returns null when no current workspace set', async () => {
       const credManager = setup()
       const current = await credManager.getWorkspace()
       expect(current).toBeNull()
     })
 
-    test('returns null when current workspace is deleted', async () => {
+    it('returns null when current workspace is deleted', async () => {
       const credManager = setup()
       const ws: WorkspaceCredentials = {
         workspace_id: 'T777',
@@ -214,7 +214,7 @@ describe('Workspace Commands', () => {
       expect(current).toBeNull()
     })
 
-    test('shows correct workspace after switching', async () => {
+    it('shows correct workspace after switching', async () => {
       const credManager = setup()
       const ws1: WorkspaceCredentials = {
         workspace_id: 'T888',
@@ -241,7 +241,7 @@ describe('Workspace Commands', () => {
   })
 
   describe('workspace remove', () => {
-    test('removes workspace by id', async () => {
+    it('removes workspace by id', async () => {
       const credManager = setup()
       const ws: WorkspaceCredentials = {
         workspace_id: 'T-remove',
@@ -257,7 +257,7 @@ describe('Workspace Commands', () => {
       expect(retrieved).toBeNull()
     })
 
-    test('removes current workspace and clears current', async () => {
+    it('removes current workspace and clears current', async () => {
       const credManager = setup()
       const ws: WorkspaceCredentials = {
         workspace_id: 'T-current-remove',
@@ -274,7 +274,7 @@ describe('Workspace Commands', () => {
       expect(config.current_workspace).toBeNull()
     })
 
-    test('removes workspace without affecting others', async () => {
+    it('removes workspace without affecting others', async () => {
       const credManager = setup()
       const ws1: WorkspaceCredentials = {
         workspace_id: 'T-keep',
@@ -300,7 +300,7 @@ describe('Workspace Commands', () => {
       expect(config.current_workspace).toBe('T-keep')
     })
 
-    test('handles removing non-existent workspace gracefully', async () => {
+    it('handles removing non-existent workspace gracefully', async () => {
       const credManager = setup()
       await credManager.removeWorkspace('nonexistent')
 
@@ -308,7 +308,7 @@ describe('Workspace Commands', () => {
       expect(config.workspaces).toEqual({})
     })
 
-    test('clears current only if removed workspace was current', async () => {
+    it('clears current only if removed workspace was current', async () => {
       const credManager = setup()
       const ws1: WorkspaceCredentials = {
         workspace_id: 'T-current-1',
@@ -334,7 +334,7 @@ describe('Workspace Commands', () => {
   })
 
   describe('Output Formatting', () => {
-    test('formats list output correctly', async () => {
+    it('formats list output correctly', async () => {
       const credManager = setup()
       const ws1: WorkspaceCredentials = {
         workspace_id: 'T-format-1',
@@ -366,7 +366,7 @@ describe('Workspace Commands', () => {
       expect(parsed[1].current).toBe(false)
     })
 
-    test('formats switch output correctly', async () => {
+    it('formats switch output correctly', async () => {
       const credManager = setup()
       const ws: WorkspaceCredentials = {
         workspace_id: 'T-switch-format',
@@ -384,7 +384,7 @@ describe('Workspace Commands', () => {
       expect(parsed.current).toBe('T-switch-format')
     })
 
-    test('formats current output correctly', async () => {
+    it('formats current output correctly', async () => {
       const credManager = setup()
       const ws: WorkspaceCredentials = {
         workspace_id: 'T-current-format',
@@ -407,7 +407,7 @@ describe('Workspace Commands', () => {
       expect(parsed.workspace_name).toBe('current-format')
     })
 
-    test('formats remove output correctly', async () => {
+    it('formats remove output correctly', async () => {
       const credManager = setup()
       const ws: WorkspaceCredentials = {
         workspace_id: 'T-remove-format',
@@ -424,7 +424,7 @@ describe('Workspace Commands', () => {
       expect(parsed.removed).toBe('T-remove-format')
     })
 
-    test('pretty prints JSON correctly', async () => {
+    it('pretty prints JSON correctly', async () => {
       const output = {
         id: 'T123',
         name: 'test',
@@ -439,13 +439,13 @@ describe('Workspace Commands', () => {
   })
 
   describe('Error Handling', () => {
-    test('handles missing workspace gracefully', async () => {
+    it('handles missing workspace gracefully', async () => {
       const credManager = setup()
       const ws = await credManager.getWorkspace('missing')
       expect(ws).toBeNull()
     })
 
-    test('handles corrupted config gracefully', async () => {
+    it('handles corrupted config gracefully', async () => {
       const credManager = setup()
       const config = await credManager.load()
 
@@ -453,7 +453,7 @@ describe('Workspace Commands', () => {
       expect(config.workspaces).toEqual({})
     })
 
-    test('handles concurrent operations', async () => {
+    it('handles concurrent operations', async () => {
       const credManager = setup()
       const ws1: WorkspaceCredentials = {
         workspace_id: 'T-concurrent-1',