agent-messenger 2.10.1 → 2.11.0

package/src/platforms/channeltalk/commands/auth.test.ts

@@ -1,4 +1,4 @@
-import { afterAll, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterAll, beforeEach, describe, expect, mock, it } from 'bun:test'
 
 type WorkspaceEntry = {
   workspace_id: string
@@ -20,6 +20,9 @@ const mockListChannels = mock(() =>
   ]),
 )
 const mockExtract = mock(() => Promise.resolve([{ accountCookie: 'fresh-account', sessionCookie: 'fresh-session' }]))
+const mockCreateTokenExtractor = mock((_browserProfile?: string[]) => ({
+  extract: mockExtract,
+}))
 
 import {
   clearAction,
@@ -89,9 +92,7 @@ setChannelAuthCommandDependenciesForTesting({
       return true
     },
   }),
-  createTokenExtractor: () => ({
-    extract: mockExtract,
-  }),
+  createTokenExtractor: mockCreateTokenExtractor,
 })
 
 describe('channel auth commands', () => {
@@ -105,6 +106,7 @@ describe('channel auth commands', () => {
     mockGetAccount.mockReset()
     mockListChannels.mockReset()
     mockExtract.mockReset()
+    mockCreateTokenExtractor.mockReset()
 
     mockGetAccount.mockImplementation(() => Promise.resolve({ id: 'acct-1', name: 'Alice' }))
     mockListChannels.mockImplementation(() =>
@@ -116,10 +118,13 @@ describe('channel auth commands', () => {
     mockExtract.mockImplementation(() =>
       Promise.resolve([{ accountCookie: 'fresh-account', sessionCookie: 'fresh-session' }]),
     )
+    mockCreateTokenExtractor.mockImplementation((_browserProfile?: string[]) => ({
+      extract: mockExtract,
+    }))
   })
 
   describe('extractAction', () => {
-    test('extracts fresh cookies and saves all workspaces', async () => {
+    it('extracts fresh cookies and saves all workspaces', async () => {
       const result = await extractAction()
 
       expect(mockExtract).toHaveBeenCalledTimes(1)
@@ -136,7 +141,14 @@ describe('channel auth commands', () => {
       expect(workspaceStore.get('ws-2')?.account_cookie).toBe('fresh-account')
     })
 
-    test('preserves current workspace if it still exists after re-extraction', async () => {
+    it('passes custom browser profile paths to the token extractor', async () => {
+      await extractAction({ browserProfile: ['/tmp/profile-a', '/tmp/profile-b'] })
+
+      expect(mockCreateTokenExtractor).toHaveBeenCalledWith(['/tmp/profile-a', '/tmp/profile-b'])
+      expect(mockExtract).toHaveBeenCalledTimes(1)
+    })
+
+    it('preserves current workspace if it still exists after re-extraction', async () => {
       workspaceStore.set('ws-2', {
         workspace_id: 'ws-2',
         workspace_name: 'Workspace 2',
@@ -157,7 +169,7 @@ describe('channel auth commands', () => {
       })
     })
 
-    test('switches to first workspace when previous current no longer exists', async () => {
+    it('switches to first workspace when previous current no longer exists', async () => {
       workspaceStore.set('ws-old', {
         workspace_id: 'ws-old',
         workspace_name: 'Old Workspace',
@@ -178,7 +190,7 @@ describe('channel auth commands', () => {
       })
     })
 
-    test('returns error when token extraction fails', async () => {
+    it('returns error when token extraction fails', async () => {
       mockExtract.mockImplementation(() => Promise.resolve([]))
 
       const result = await extractAction()
@@ -188,7 +200,7 @@ describe('channel auth commands', () => {
       })
     })
 
-    test('returns error when no workspaces found', async () => {
+    it('returns error when no workspaces found', async () => {
       mockListChannels.mockImplementation(() => Promise.resolve([]))
 
       const result = await extractAction()
@@ -200,14 +212,14 @@ describe('channel auth commands', () => {
   })
 
   describe('statusAction', () => {
-    test('returns an error when no credentials exist', async () => {
+    it('returns an error when no credentials exist', async () => {
       const result = await statusAction()
 
       expect(result.valid).toBe(false)
       expect(result.error).toBe('No credentials. Run "agent-channeltalk auth extract" first.')
     })
 
-    test('returns valid status for current workspace', async () => {
+    it('returns valid status for current workspace', async () => {
       workspaceStore.set('ws-1', {
         workspace_id: 'ws-1',
         workspace_name: 'Workspace 1',
@@ -228,7 +240,7 @@ describe('channel auth commands', () => {
       })
     })
 
-    test('returns invalid status with stored info when api validation fails', async () => {
+    it('returns invalid status with stored info when api validation fails', async () => {
       workspaceStore.set('ws-1', {
         workspace_id: 'ws-1',
         workspace_name: 'Workspace 1',
@@ -251,7 +263,7 @@ describe('channel auth commands', () => {
       })
     })
 
-    test('returns workspace-specific error for unknown workspace', async () => {
+    it('returns workspace-specific error for unknown workspace', async () => {
       const result = await statusAction({ workspace: 'missing' })
 
       expect(result).toEqual({
@@ -262,7 +274,7 @@ describe('channel auth commands', () => {
   })
 
   describe('clearAction', () => {
-    test('removes all stored credentials', async () => {
+    it('removes all stored credentials', async () => {
       workspaceStore.set('ws-1', {
         workspace_id: 'ws-1',
         workspace_name: 'Workspace 1',
@@ -280,7 +292,7 @@ describe('channel auth commands', () => {
   })
 
   describe('listAction', () => {
-    test('lists all workspaces with current flag', async () => {
+    it('lists all workspaces with current flag', async () => {
       workspaceStore.set('ws-1', {
         workspace_id: 'ws-1',
         workspace_name: 'Workspace 1',
@@ -305,7 +317,7 @@ describe('channel auth commands', () => {
   })
 
   describe('useAction', () => {
-    test('switches current workspace', async () => {
+    it('switches current workspace', async () => {
       workspaceStore.set('ws-1', {
         workspace_id: 'ws-1',
         workspace_name: 'Workspace 1',
@@ -326,7 +338,7 @@ describe('channel auth commands', () => {
       expect(currentWorkspaceId).toBe('ws-1')
     })
 
-    test('returns error for unknown workspace', async () => {
+    it('returns error for unknown workspace', async () => {
       const result = await useAction('missing')
 
       expect(result).toEqual({
@@ -336,7 +348,7 @@ describe('channel auth commands', () => {
   })
 
   describe('removeAction', () => {
-    test('removes a stored workspace', async () => {
+    it('removes a stored workspace', async () => {
       workspaceStore.set('ws-1', {
         workspace_id: 'ws-1',
         workspace_name: 'Workspace 1',
@@ -351,7 +363,7 @@ describe('channel auth commands', () => {
       expect(workspaceStore.has('ws-1')).toBe(false)
     })
 
-    test('returns error for unknown workspace', async () => {
+    it('returns error for unknown workspace', async () => {
       const result = await removeAction('missing')
 
       expect(result).toEqual({

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

@@ -1,5 +1,6 @@
 import { Command } from 'commander'
 
+import { collectBrowserProfileOption } from '@/shared/chromium'
 import { formatOutput } from '@/shared/utils/output'
 
 import { ChannelClient } from '../client'
@@ -9,6 +10,7 @@ import { ChannelTokenExtractor } from '../token-extractor'
 interface ActionOptions {
   workspace?: string
   pretty?: boolean
+  browserProfile?: string[]
   _credManager?: ChannelCredentialManager
 }
 
@@ -60,7 +62,8 @@ let createChannelClient: (
 
 let createCredentialManager = (): ChannelCredentialManagerLike => new ChannelCredentialManager()
 
-let createTokenExtractor = (): ChannelTokenExtractorLike => new ChannelTokenExtractor()
+let createTokenExtractor = (browserProfile?: string[]): ChannelTokenExtractorLike =>
+  new ChannelTokenExtractor(undefined, browserProfile)
 
 export function setChannelAuthCommandDependenciesForTesting(dependencies: {
   createChannelClient?: (
@@ -68,7 +71,7 @@ export function setChannelAuthCommandDependenciesForTesting(dependencies: {
     sessionCookie?: string,
   ) => ChannelClientLike | Promise<ChannelClientLike>
   createCredentialManager?: () => ChannelCredentialManagerLike
-  createTokenExtractor?: () => ChannelTokenExtractorLike
+  createTokenExtractor?: (browserProfile?: string[]) => ChannelTokenExtractorLike
 }): void {
   if (dependencies.createChannelClient) createChannelClient = dependencies.createChannelClient
   if (dependencies.createCredentialManager) createCredentialManager = dependencies.createCredentialManager
@@ -79,13 +82,14 @@ export function resetChannelAuthCommandDependenciesForTesting(): void {
   createChannelClient = async (accountCookie: string, sessionCookie?: string): Promise<ChannelClientLike> =>
     new ChannelClient().login({ accountCookie, sessionCookie })
   createCredentialManager = (): ChannelCredentialManagerLike => new ChannelCredentialManager()
-  createTokenExtractor = (): ChannelTokenExtractorLike => new ChannelTokenExtractor()
+  createTokenExtractor = (browserProfile?: string[]): ChannelTokenExtractorLike =>
+    new ChannelTokenExtractor(undefined, browserProfile)
 }
 
 export async function extractAction(options: ActionOptions = {}): Promise<ExtractResult | ErrorResult> {
   try {
     const credManager = options._credManager ?? createCredentialManager()
-    const extractor = createTokenExtractor()
+    const extractor = createTokenExtractor(options.browserProfile)
     const extracted = await extractor.extract()
 
     if (extracted.length === 0) {
@@ -270,7 +274,13 @@ export function createAuthCommand(): Command {
       new Command('extract')
         .description('Extract cookies from Channel Talk desktop app or a supported Chromium browser')
         .option('--pretty', 'Pretty print JSON output')
-        .action(async (opts: { pretty?: boolean }) => {
+        .option(
+          '--browser-profile <path>',
+          'Additional Chromium profile/user-data directory to scan (repeatable, comma-separated supported)',
+          collectBrowserProfileOption,
+          [],
+        )
+        .action(async (opts: { pretty?: boolean; browserProfile?: string[] }) => {
           cliOutput(await extractAction(opts), opts.pretty)
         }),
     )

package/src/platforms/channeltalk/commands/bot.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 const mockListBots = mock(() =>
   Promise.resolve([
@@ -35,7 +35,7 @@ describe('bot commands', () => {
     )
   })
 
-  test('listAction lists bots', async () => {
+  it('listAction lists bots', async () => {
     const result = await listAction({ limit: '8' })
 
     expect(mockListBots).toHaveBeenCalledWith('ws-1', { limit: 8 })

package/src/platforms/channeltalk/commands/chat.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 const mockListUserChats = mock(() =>
   Promise.resolve([
@@ -62,7 +62,7 @@ describe('chat commands', () => {
     )
   })
 
-  test('listAction lists user chats with the requested state filter', async () => {
+  it('listAction lists user chats with the requested state filter', async () => {
     const result = await listAction({ state: 'closed', limit: '5' })
 
     expect(mockListUserChats).toHaveBeenCalledWith('ws-1', { state: 'closed', limit: 5 })
@@ -78,7 +78,7 @@ describe('chat commands', () => {
     ])
   })
 
-  test('getAction gets a specific user chat', async () => {
+  it('getAction gets a specific user chat', async () => {
     const result = await getAction('chat-1')
 
     expect(mockGetUserChat).toHaveBeenCalledWith('ws-1', 'chat-1')

package/src/platforms/channeltalk/commands/group.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 const mockListGroups = mock(() =>
   Promise.resolve([
@@ -103,7 +103,7 @@ describe('group commands', () => {
     )
   })
 
-  test('listAction lists groups', async () => {
+  it('listAction lists groups', async () => {
     const result = await listAction({ limit: '3' })
 
     expect(mockListGroups).toHaveBeenCalledWith('ws-1', { limit: 3 })
@@ -121,7 +121,7 @@ describe('group commands', () => {
     ])
   })
 
-  test('getAction gets a specific group', async () => {
+  it('getAction gets a specific group', async () => {
     const result = await getAction('grp-1')
 
     expect(mockGetGroup).toHaveBeenCalledWith('ws-1', 'grp-1')
@@ -137,7 +137,7 @@ describe('group commands', () => {
     })
   })
 
-  test('messagesAction gets group messages with limit and sort', async () => {
+  it('messagesAction gets group messages with limit and sort', async () => {
     const result = await messagesAction('grp-1', { limit: '7', sort: 'asc' })
 
     expect(mockGetGroupMessages).toHaveBeenCalledWith('ws-1', 'grp-1', { limit: 7, sortOrder: 'asc' })

package/src/platforms/channeltalk/commands/manager.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 const mockListManagers = mock(() =>
   Promise.resolve([
@@ -43,7 +43,7 @@ describe('manager commands', () => {
     )
   })
 
-  test('listAction lists managers', async () => {
+  it('listAction lists managers', async () => {
     const result = await listAction({ limit: '12' })
 
     expect(mockListManagers).toHaveBeenCalledWith('ws-1', { limit: 12 })

package/src/platforms/channeltalk/commands/message.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 const mockSendGroupMessage = mock(() =>
   Promise.resolve({
@@ -306,7 +306,7 @@ describe('message commands', () => {
     )
   })
 
-  test('sendAction sends a group message with wrapped blocks', async () => {
+  it('sendAction sends a group message with wrapped blocks', async () => {
     const result = await sendAction('group', 'grp-1', 'Hello group')
 
     expect(mockSendGroupMessage).toHaveBeenCalledWith('ws-1', 'grp-1', [{ type: 'text', value: 'Hello group' }])
@@ -319,7 +319,7 @@ describe('message commands', () => {
     })
   })
 
-  test('sendAction sends a user chat message with wrapped blocks', async () => {
+  it('sendAction sends a user chat message with wrapped blocks', async () => {
     const result = await sendAction('user-chat', 'chat-1', 'Hello user chat')
 
     expect(mockSendUserChatMessage).toHaveBeenCalledWith('ws-1', 'chat-1', [{ type: 'text', value: 'Hello user chat' }])
@@ -332,7 +332,7 @@ describe('message commands', () => {
     })
   })
 
-  test('sendAction sends a direct chat message with wrapped blocks', async () => {
+  it('sendAction sends a direct chat message with wrapped blocks', async () => {
     const result = await sendAction('direct-chat', 'dm-1', 'Hello direct chat')
 
     expect(mockSendDirectChatMessage).toHaveBeenCalledWith('ws-1', 'dm-1', [
@@ -347,7 +347,7 @@ describe('message commands', () => {
     })
   })
 
-  test('listAction lists group messages with limit and sort options', async () => {
+  it('listAction lists group messages with limit and sort options', async () => {
     const result = await listAction('group', 'grp-1', { limit: '10', sort: 'asc' })
 
     expect(mockGetGroupMessages).toHaveBeenCalledWith('ws-1', 'grp-1', { limit: 10, sortOrder: 'asc' })
@@ -365,7 +365,7 @@ describe('message commands', () => {
     ])
   })
 
-  test('listAction lists user chat messages', async () => {
+  it('listAction lists user chat messages', async () => {
     const result = await listAction('user-chat', 'chat-1')
 
     expect(mockGetUserChatMessages).toHaveBeenCalledWith('ws-1', 'chat-1', { limit: 25, sortOrder: 'desc' })
@@ -376,7 +376,7 @@ describe('message commands', () => {
     })
   })
 
-  test('listAction lists direct chat messages', async () => {
+  it('listAction lists direct chat messages', async () => {
     const result = await listAction('direct-chat', 'dm-1')
 
     expect(mockGetDirectChatMessages).toHaveBeenCalledWith('ws-1', 'dm-1', { limit: 25, sortOrder: 'desc' })
@@ -387,21 +387,21 @@ describe('message commands', () => {
     })
   })
 
-  test('getAction returns specific message by ID', async () => {
+  it('getAction returns specific message by ID', async () => {
     const result = await getAction('group', 'grp-1', 'msg-group-list-1')
 
     expect(result.error).toBeUndefined()
     expect(result.id).toBe('msg-group-list-1')
   })
 
-  test('getAction returns error when message not found', async () => {
+  it('getAction returns error when message not found', async () => {
     const result = await getAction('group', 'grp-1', 'nonexistent')
 
     expect(result.error).toBeDefined()
     expect(result.error).toContain('not found')
   })
 
-  test('searchAction searches team chat messages by default', async () => {
+  it('searchAction searches team chat messages by default', async () => {
     const result = await searchAction('search')
 
     expect(mockSearchTeamChatMessages).toHaveBeenCalledWith('ws-1', 'search', { limit: undefined })
@@ -420,7 +420,7 @@ describe('message commands', () => {
     ])
   })
 
-  test('searchAction searches user chat messages with scope option', async () => {
+  it('searchAction searches user chat messages with scope option', async () => {
     const result = await searchAction('search', { scope: 'user-chat' })
 
     expect(mockSearchUserChatMessages).toHaveBeenCalledWith('ws-1', 'search', { limit: undefined })
@@ -439,13 +439,13 @@ describe('message commands', () => {
     ])
   })
 
-  test('searchAction passes limit parameter', async () => {
+  it('searchAction passes limit parameter', async () => {
     await searchAction('test', { limit: '5' })
 
     expect(mockSearchTeamChatMessages).toHaveBeenCalledWith('ws-1', 'test', { limit: 5 })
   })
 
-  test('searchAction returns error on failure', async () => {
+  it('searchAction returns error on failure', async () => {
     mockSearchTeamChatMessages.mockImplementation(() => Promise.reject(new Error('Network error')))
 
     const result = await searchAction('test')
@@ -454,7 +454,7 @@ describe('message commands', () => {
     expect(result.results).toBeUndefined()
   })
 
-  test('searchAction returns error for invalid scope', async () => {
+  it('searchAction returns error for invalid scope', async () => {
     const result = await searchAction('test', { scope: 'invalid' })
 
     expect(result.error).toContain('Invalid --scope value')
@@ -462,7 +462,7 @@ describe('message commands', () => {
     expect(result.results).toBeUndefined()
   })
 
-  test('searchAction handles zero hits', async () => {
+  it('searchAction handles zero hits', async () => {
     mockSearchTeamChatMessages.mockImplementation(() =>
       Promise.resolve({ hits: [], bots: [], sessions: [], groups: [] }),
     )
@@ -473,7 +473,7 @@ describe('message commands', () => {
     expect(result.error).toBeUndefined()
   })
 
-  test('searchAction handles missing highlight plainText key', async () => {
+  it('searchAction handles missing highlight plainText key', async () => {
     mockSearchTeamChatMessages.mockImplementation(() =>
       Promise.resolve({
         hits: [
@@ -506,7 +506,7 @@ describe('message commands', () => {
     expect(result.results?.[0]?.plain_text).toBe('some message')
   })
 
-  test('searchAction uses extractText for block-only messages', async () => {
+  it('searchAction uses extractText for block-only messages', async () => {
     mockSearchTeamChatMessages.mockImplementation(() =>
       Promise.resolve({
         hits: [

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

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 const mockGetChannel = mock(() => Promise.resolve({ id: 'ws-1', name: 'Workspace One' }))
 const mockListManagers = mock(() =>
@@ -81,7 +81,7 @@ describe('snapshot command', () => {
     mockListUserChats.mockClear()
   })
 
-  test('brief snapshot returns workspace, groups (name only), chat counts, and hint', async () => {
+  it('brief snapshot returns workspace, groups (name only), chat counts, and hint', async () => {
     const result = await snapshotAction()
 
     expect(result.error).toBeUndefined()
@@ -100,7 +100,7 @@ describe('snapshot command', () => {
     expect(mockGetGroupMessages).not.toHaveBeenCalled()
   })
 
-  test('brief groups-only returns group names without messages', async () => {
+  it('brief groups-only returns group names without messages', async () => {
     const result = await snapshotAction({ groupsOnly: true })
 
     expect(result.workspace).toEqual({ id: 'ws-1', name: 'Workspace One' })
@@ -112,7 +112,7 @@ describe('snapshot command', () => {
     expect(mockListUserChats).not.toHaveBeenCalled()
   })
 
-  test('brief chats-only returns counts without recent details', async () => {
+  it('brief chats-only returns counts without recent details', async () => {
     const result = await snapshotAction({ chatsOnly: true })
 
     expect(result.workspace).toEqual({ id: 'ws-1', name: 'Workspace One' })
@@ -126,7 +126,7 @@ describe('snapshot command', () => {
     expect(mockGetGroupMessages).not.toHaveBeenCalled()
   })
 
-  test('full snapshot returns workspace, managers, bots, groups, and user chats', async () => {
+  it('full snapshot returns workspace, managers, bots, groups, and user chats', async () => {
     const result = await snapshotAction({ full: true })
 
     expect(result.error).toBeUndefined()
@@ -201,7 +201,7 @@ describe('snapshot command', () => {
     })
   })
 
-  test('full groups-only includes messages', async () => {
+  it('full groups-only includes messages', async () => {
     const result = await snapshotAction({ full: true, groupsOnly: true, limit: 3 })
 
     expect(mockGetGroupMessages).toHaveBeenCalledWith('ws-1', 'grp-1', { limit: 3, sortOrder: 'desc' })
@@ -214,7 +214,7 @@ describe('snapshot command', () => {
     })
   })
 
-  test('full chats-only includes recent details', async () => {
+  it('full chats-only includes recent details', async () => {
     const result = await snapshotAction({ full: true, chatsOnly: true, limit: 1 })
 
     expect(result.workspace).toEqual({ id: 'ws-1', name: 'Workspace One' })

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

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 const mockGetAccount = mock(() =>
   Promise.resolve({
@@ -37,7 +37,7 @@ describe('whoami command', () => {
     )
   })
 
-  test('whoamiAction returns account info', async () => {
+  it('returns account info', async () => {
     const result = await whoamiAction({})
 
     expect(mockGetAccount).toHaveBeenCalled()
@@ -53,7 +53,7 @@ describe('whoami command', () => {
     expect(result.error).toBeUndefined()
   })
 
-  test('whoamiAction returns error when getAccount fails', async () => {
+  it('returns error when getAccount fails', async () => {
     mockGetAccount.mockImplementation(() => Promise.reject(new Error('Not authenticated')))
 
     const result = await whoamiAction({})