agent-messenger 2.10.1 → 2.11.0

package/src/platforms/webex/commands/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 childProcess from 'node:child_process'
 
 import { WebexClient } from '../client'
@@ -42,7 +42,7 @@ describe('auth commands', () => {
   })
 
   describe('loginAction with --token', () => {
-    test('authenticates with provided token (bot token flow)', async () => {
+    it('authenticates with provided token (bot token flow)', async () => {
       protoSpy(WebexClient.prototype, 'login').mockResolvedValue(new WebexClient())
       protoSpy(WebexClient.prototype, 'testAuth').mockResolvedValue(mockPerson)
       protoSpy(WebexCredentialManager.prototype, 'saveConfig').mockResolvedValue(undefined)
@@ -56,7 +56,7 @@ describe('auth commands', () => {
       expect(output.user.displayName).toBe('Test User')
     })
 
-    test('saves tokenType as manual with expiresAt 0', async () => {
+    it('saves tokenType as manual with expiresAt 0', async () => {
       protoSpy(WebexClient.prototype, 'login').mockResolvedValue(new WebexClient())
       protoSpy(WebexClient.prototype, 'testAuth').mockResolvedValue(mockPerson)
       const saveSpy = protoSpy(WebexCredentialManager.prototype, 'saveConfig').mockResolvedValue(undefined)
@@ -71,7 +71,7 @@ describe('auth commands', () => {
   })
 
   describe('loginAction with --client-id and --client-secret', () => {
-    test('uses provided credentials for Device Grant flow', async () => {
+    it('uses provided credentials for Device Grant flow', async () => {
       protoSpy(WebexCredentialManager.prototype, 'requestDeviceCode').mockResolvedValue({
         deviceCode: 'd',
         userCode: 'u',
@@ -101,7 +101,7 @@ describe('auth commands', () => {
       )
     })
 
-    test('saves tokenType as oauth in config', async () => {
+    it('saves tokenType as oauth in config', async () => {
       protoSpy(WebexCredentialManager.prototype, 'requestDeviceCode').mockResolvedValue({
         deviceCode: 'd',
         userCode: 'u',
@@ -125,7 +125,7 @@ describe('auth commands', () => {
       expect(savedConfig.tokenType).toBe('oauth')
     })
 
-    test('saves clientId and clientSecret in config', async () => {
+    it('saves clientId and clientSecret in config', async () => {
       protoSpy(WebexCredentialManager.prototype, 'requestDeviceCode').mockResolvedValue({
         deviceCode: 'd',
         userCode: 'u',
@@ -152,7 +152,7 @@ describe('auth commands', () => {
   })
 
   describe('statusAction', () => {
-    test('shows authenticated status when token is valid', async () => {
+    it('shows authenticated status when token is valid', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue(null)
       protoSpy(WebexCredentialManager.prototype, 'getToken').mockResolvedValue('valid-token')
       protoSpy(WebexClient.prototype, 'login').mockResolvedValue(new WebexClient())
@@ -166,7 +166,7 @@ describe('auth commands', () => {
       expect(output.user.displayName).toBe('Test User')
     })
 
-    test('shows not authenticated when no token', async () => {
+    it('shows not authenticated when no token', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue(null)
       protoSpy(WebexCredentialManager.prototype, 'getToken').mockResolvedValue(null)
       const exitSpy = protoSpy(process, 'exit').mockImplementation(() => undefined as never)
@@ -179,7 +179,7 @@ describe('auth commands', () => {
       expect(exitSpy).toHaveBeenCalledWith(1)
     })
 
-    test('shows not authenticated when token validation fails', async () => {
+    it('shows not authenticated when token validation fails', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue(null)
       protoSpy(WebexCredentialManager.prototype, 'getToken').mockResolvedValue('invalid-token')
       protoSpy(WebexClient.prototype, 'login').mockResolvedValue(new WebexClient())
@@ -192,7 +192,7 @@ describe('auth commands', () => {
       expect(output.authenticated).toBe(false)
     })
 
-    test('loads config for stored client credentials', async () => {
+    it('loads config for stored client credentials', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue({
         accessToken: 'at',
         refreshToken: 'rt',
@@ -211,7 +211,7 @@ describe('auth commands', () => {
   })
 
   describe('extractAction', () => {
-    test('passes deviceUrl and tokenType to client.login', async () => {
+    it('passes deviceUrl and tokenType to client.login', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue({
         accessToken: 'extracted-token-at-least-twenty-chars',
         refreshToken: 'refresh-token',
@@ -232,7 +232,7 @@ describe('auth commands', () => {
       })
     })
 
-    test('attempts refresh when token is expired', async () => {
+    it('attempts refresh when token is expired', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue({
         accessToken: 'expired-token-at-least-twenty-chars-',
         refreshToken: 'valid-refresh-token',
@@ -257,7 +257,7 @@ describe('auth commands', () => {
       expect(output.refreshed).toBe(true)
     })
 
-    test('reports expired token with actionable hint when refresh fails', async () => {
+    it('reports expired token with actionable hint when refresh fails', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue({
         accessToken: 'expired-token-at-least-twenty-chars-',
         refreshToken: 'bad-refresh-token',
@@ -278,7 +278,7 @@ describe('auth commands', () => {
       expect(exitSpy).toHaveBeenCalledWith(1)
     })
 
-    test('rethrows non-auth errors even when token is expired', async () => {
+    it('rethrows non-auth errors even when token is expired', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue({
         accessToken: 'expired-token-at-least-twenty-chars-',
         refreshToken: 'bad-refresh-token',
@@ -298,7 +298,7 @@ describe('auth commands', () => {
       }
     })
 
-    test('rethrows non-expiry auth errors', async () => {
+    it('rethrows non-expiry auth errors', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue({
         accessToken: 'valid-token-at-least-twenty-chars-xx',
         expiresAt: Date.now() + 3600000,
@@ -316,7 +316,7 @@ describe('auth commands', () => {
       }
     })
 
-    test('outputs no token found when extract returns null', async () => {
+    it('outputs no token found when extract returns null', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue(null)
       const exitSpy = protoSpy(process, 'exit').mockImplementation(() => undefined as never)
 
@@ -330,7 +330,7 @@ describe('auth commands', () => {
   })
 
   describe('logoutAction', () => {
-    test('clears credentials when authenticated', async () => {
+    it('clears credentials when authenticated', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue({
         accessToken: 'token',
         refreshToken: 'refresh',
@@ -346,7 +346,7 @@ describe('auth commands', () => {
       expect(output.success).toBe(true)
     })
 
-    test('shows error when not authenticated', async () => {
+    it('shows error when not authenticated', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue(null)
       const exitSpy = protoSpy(process, 'exit').mockImplementation(() => undefined as never)
 

package/src/platforms/webex/commands/auth.ts

@@ -1,5 +1,6 @@
 import { Command } from 'commander'
 
+import { collectBrowserProfileOption } from '@/shared/chromium'
 import { handleError } from '@/shared/utils/error-handler'
 import { formatOutput } from '@/shared/utils/output'
 import { info, debug } from '@/shared/utils/stderr'
@@ -141,10 +142,14 @@ export async function statusAction(options: { pretty?: boolean }): Promise<void>
   }
 }
 
-export async function extractAction(options: { pretty?: boolean; debug?: boolean }): Promise<void> {
+export async function extractAction(options: {
+  pretty?: boolean
+  debug?: boolean
+  browserProfile?: string[]
+}): Promise<void> {
   try {
     const debugLog = options.debug ? (msg: string) => debug(`[debug] ${msg}`) : undefined
-    const extractor = new WebexTokenExtractor(undefined, debugLog)
+    const extractor = new WebexTokenExtractor(undefined, debugLog, undefined, options.browserProfile)
 
     if (options.debug) {
       debug('[debug] Searching browser profiles for Webex tokens...')
@@ -283,6 +288,12 @@ export const authCommand = new Command('auth')
       .description('Extract Webex token from browser (Chrome, Edge, Arc, Brave)')
       .option('--pretty', 'Pretty print JSON output')
       .option('--debug', 'Show debug output')
+      .option(
+        '--browser-profile <path>',
+        'Additional Chromium profile/user-data directory to scan (repeatable, comma-separated supported)',
+        collectBrowserProfileOption,
+        [],
+      )
       .action(extractAction),
   )
   .addCommand(

package/src/platforms/webex/commands/member.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 import { WebexError } from '../types'
 
@@ -59,7 +59,7 @@ describe('member commands', () => {
     consoleSpy.mockRestore()
   })
 
-  test('listAction calls listMemberships with spaceId and outputs mapped members', async () => {
+  it('calls listMemberships with spaceId and outputs mapped members', async () => {
     await listAction('room-1', {})
 
     expect(mockListMemberships).toHaveBeenCalledWith('room-1', { max: undefined })
@@ -85,13 +85,13 @@ describe('member commands', () => {
     )
   })
 
-  test('listAction passes limit option to listMemberships', async () => {
+  it('passes limit option to listMemberships', async () => {
     await listAction('room-1', { limit: 25 })
 
     expect(mockListMemberships).toHaveBeenCalledWith('room-1', { max: 25 })
   })
 
-  test('listAction handles not-authenticated case', async () => {
+  it('throws when not authenticated', async () => {
     mockLogin.mockImplementation(async () => {
       throw new WebexError('No Webex credentials found.', 'no_credentials')
     })
@@ -101,7 +101,7 @@ describe('member commands', () => {
     expect(mockHandleError).toHaveBeenCalledWith(expect.any(WebexError))
   })
 
-  test('listAction handles API error', async () => {
+  it('throws on API error', async () => {
     mockListMemberships.mockImplementation(async () => {
       throw new Error('API failure')
     })

package/src/platforms/webex/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'
 
 import { WebexError } from '../types'
 
@@ -77,7 +77,7 @@ afterEach(() => {
   consoleLogSpy.mockRestore()
 })
 
-test('send: calls sendMessage with correct args and outputs result', async () => {
+it('calls sendMessage with correct args and outputs result', async () => {
   await sendAction('space_456', 'Hello world', { pretty: false })
 
   expect(mockSendMessage).toHaveBeenCalledWith('space_456', 'Hello world', {
@@ -90,13 +90,13 @@ test('send: calls sendMessage with correct args and outputs result', async () =>
   expect(output).toContain('user@example.com')
 })
 
-test('send: with --markdown passes markdown option', async () => {
+it('passes markdown option when --markdown flag is set on send', async () => {
   await sendAction('space_456', '**bold**', { markdown: true, pretty: false })
 
   expect(mockSendMessage).toHaveBeenCalledWith('space_456', '**bold**', { markdown: true })
 })
 
-test('send: not authenticated shows error', async () => {
+it('throws when not authenticated on send', async () => {
   mockLogin.mockImplementation(async () => {
     throw new WebexError('No Webex credentials found.', 'no_credentials')
   })
@@ -106,7 +106,7 @@ test('send: not authenticated shows error', async () => {
   expect(mockHandleError).toHaveBeenCalledWith(expect.any(WebexError))
 })
 
-test('dm: calls sendDirectMessage with email and text', async () => {
+it('calls sendDirectMessage with email and text', async () => {
   await dmAction('alice@example.com', 'Hello!', { pretty: false })
 
   expect(mockSendDirectMessage).toHaveBeenCalledWith('alice@example.com', 'Hello!', {
@@ -117,7 +117,7 @@ test('dm: calls sendDirectMessage with email and text', async () => {
   expect(output).toContain('msg_123')
 })
 
-test('dm: with --markdown passes markdown option', async () => {
+it('passes markdown option to sendDirectMessage when --markdown flag is set', async () => {
   await dmAction('alice@example.com', '**bold**', { markdown: true, pretty: false })
 
   expect(mockSendDirectMessage).toHaveBeenCalledWith('alice@example.com', '**bold**', {
@@ -125,7 +125,7 @@ test('dm: with --markdown passes markdown option', async () => {
   })
 })
 
-test('list: calls listMessages with limit and outputs array', async () => {
+it('calls listMessages with limit and outputs array', async () => {
   await listAction('space_456', { limit: 50, pretty: false })
 
   expect(mockListMessages).toHaveBeenCalledWith('space_456', { max: 50 })
@@ -135,7 +135,7 @@ test('list: calls listMessages with limit and outputs array', async () => {
   expect(output).toContain('msg_124')
 })
 
-test('get: calls getMessage with correct id and outputs result', async () => {
+it('calls getMessage with correct id and outputs result', async () => {
   await getAction('msg_123', { pretty: false })
 
   expect(mockGetMessage).toHaveBeenCalledWith('msg_123')
@@ -145,7 +145,7 @@ test('get: calls getMessage with correct id and outputs result', async () => {
   expect(output).toContain('user@example.com')
 })
 
-test('delete: with --force calls deleteMessage and outputs deleted id', async () => {
+it('calls deleteMessage and outputs deleted id when --force flag is set', async () => {
   await deleteAction('msg_123', { force: true, pretty: false })
 
   expect(mockDeleteMessage).toHaveBeenCalledWith('msg_123')
@@ -155,7 +155,7 @@ test('delete: with --force calls deleteMessage and outputs deleted id', async ()
   expect(output).toContain('msg_123')
 })
 
-test('delete: without --force shows warning and does not delete', async () => {
+it('shows warning and does not delete without --force flag', async () => {
   try {
     await deleteAction('msg_123', { force: false, pretty: false })
   } catch {}
@@ -167,7 +167,7 @@ test('delete: without --force shows warning and does not delete', async () => {
   expect(output).toContain('--force')
 })
 
-test('edit: calls editMessage with roomId in args and outputs result', async () => {
+it('calls editMessage with roomId in args and outputs result', async () => {
   await editAction('msg_123', 'space_456', 'Updated message', { pretty: false })
 
   expect(mockEditMessage).toHaveBeenCalledWith('msg_123', 'space_456', 'Updated message', {
@@ -179,7 +179,7 @@ test('edit: calls editMessage with roomId in args and outputs result', async ()
   expect(output).toContain('Updated message')
 })
 
-test('edit: with --markdown passes markdown option', async () => {
+it('passes markdown option to editMessage when --markdown flag is set', async () => {
   await editAction('msg_123', 'space_456', '**updated**', { markdown: true, pretty: false })
 
   expect(mockEditMessage).toHaveBeenCalledWith('msg_123', 'space_456', '**updated**', {

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 import { WebexError } from '../types'
 
@@ -79,7 +79,7 @@ describe('snapshot command', () => {
     consoleSpy.mockRestore()
   })
 
-  test('brief snapshot returns spaces with id and title only', async () => {
+  it('returns spaces with id and title only in brief mode', async () => {
     await snapshotAction({})
 
     expect(consoleSpy).toHaveBeenCalled()
@@ -91,7 +91,7 @@ describe('snapshot command', () => {
     expect(output.hint).toBeDefined()
   })
 
-  test('full snapshot returns spaces with id, title, type, lastActivity', async () => {
+  it('returns spaces with id, title, type, and lastActivity in full mode', async () => {
     await snapshotAction({ full: true })
 
     expect(consoleSpy).toHaveBeenCalled()
@@ -104,7 +104,7 @@ describe('snapshot command', () => {
     expect(output.hint).toBeUndefined()
   })
 
-  test('filters spaces to only those in my memberships', async () => {
+  it('filters spaces to only those in my memberships', async () => {
     await snapshotAction({})
 
     const output = JSON.parse(consoleSpy.mock.calls[consoleSpy.mock.calls.length - 1][0])
@@ -112,7 +112,7 @@ describe('snapshot command', () => {
     expect(output.spaces[0].id).toBe('space-1')
   })
 
-  test('not authenticated outputs error', async () => {
+  it('throws when not authenticated', async () => {
     mockLogin.mockImplementation(async () => {
       throw new WebexError('No Webex credentials found.', 'no_credentials')
     })

package/src/platforms/webex/commands/space.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 import { WebexError } from '../types'
 
@@ -74,7 +74,7 @@ afterEach(() => {
 })
 
 describe('listAction', () => {
-  test('calls listSpaces and outputs mapped array', async () => {
+  it('calls listSpaces and outputs mapped array', async () => {
     await listAction({})
 
     expect(mockListSpaces).toHaveBeenCalled()
@@ -98,19 +98,19 @@ describe('listAction', () => {
     )
   })
 
-  test('passes type and limit options to listSpaces', async () => {
+  it('passes type and limit options to listSpaces', async () => {
     await listAction({ type: 'group', limit: 10 })
 
     expect(mockListSpaces).toHaveBeenCalledWith({ type: 'group', max: 10 })
   })
 
-  test('passes undefined type and limit when not provided', async () => {
+  it('passes undefined type and limit when not provided', async () => {
     await listAction({})
 
     expect(mockListSpaces).toHaveBeenCalledWith({ type: undefined, max: undefined })
   })
 
-  test('outputs pretty-printed JSON when pretty is true', async () => {
+  it('outputs pretty-printed JSON when pretty is true', async () => {
     await listAction({ pretty: true })
 
     expect(consoleLogSpy).toHaveBeenCalledWith(
@@ -137,7 +137,7 @@ describe('listAction', () => {
     )
   })
 
-  test('not authenticated: outputs error and exits', async () => {
+  it('throws when not authenticated', async () => {
     mockLogin.mockImplementation(async () => {
       throw new WebexError('No Webex credentials found.', 'no_credentials')
     })
@@ -150,7 +150,7 @@ describe('listAction', () => {
 })
 
 describe('infoAction', () => {
-  test('calls getSpace with spaceId and outputs space details', async () => {
+  it('calls getSpace with spaceId and outputs space details', async () => {
     await infoAction('space-1', {})
 
     expect(mockGetSpace).toHaveBeenCalledWith('space-1')
@@ -168,7 +168,7 @@ describe('infoAction', () => {
     )
   })
 
-  test('outputs null for teamId when not present', async () => {
+  it('outputs null for teamId when not present', async () => {
     mockGetSpace.mockImplementation(() => Promise.resolve({ ...mockSpace, teamId: undefined }))
 
     await infoAction('space-1', {})
@@ -179,7 +179,7 @@ describe('infoAction', () => {
     expect(output.teamId).toBeNull()
   })
 
-  test('outputs pretty-printed JSON when pretty is true', async () => {
+  it('outputs pretty-printed JSON when pretty is true', async () => {
     await infoAction('space-1', { pretty: true })
 
     expect(consoleLogSpy).toHaveBeenCalledWith(
@@ -200,7 +200,7 @@ describe('infoAction', () => {
     )
   })
 
-  test('not authenticated: outputs error and exits', async () => {
+  it('throws when not authenticated', async () => {
     mockLogin.mockImplementation(async () => {
       throw new WebexError('No Webex credentials found.', 'no_credentials')
     })

package/src/platforms/webex/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 * as clientModule from '../client'
 import { WebexError } from '../types'
@@ -42,19 +42,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 calls testAuth and outputs user fields', async () => {
+it('whoami calls testAuth and outputs user fields', async () => {
   // given: authenticated webex user
   // when: running whoami
   await whoamiCommand.parseAsync([], { from: 'user' })
@@ -75,7 +75,7 @@ test('whoami calls testAuth and outputs user fields', async () => {
   )
 })
 
-test('whoami outputs pretty-printed JSON when --pretty flag is passed', async () => {
+it('whoami outputs pretty-printed JSON when --pretty flag is passed', async () => {
   // given: authenticated webex user
   // when: running whoami with --pretty
   await whoamiCommand.parseAsync(['--pretty'], { from: 'user' })
@@ -100,7 +100,7 @@ test('whoami outputs pretty-printed JSON when --pretty flag is passed', async ()
   )
 })
 
-test('whoami exits with code 1 when not authenticated', async () => {
+it('whoami exits with code 1 when not authenticated', async () => {
   // given: no credentials
   webexClientSpy.mockImplementation(
     () =>