agent-messenger 2.7.0 → 2.9.0

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

@@ -1,4 +1,5 @@
 import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+
 import { WebexError } from '../types'
 
 const mockHandleError = mock((err: Error) => {
@@ -10,12 +11,36 @@ mock.module('@/shared/utils/error-handler', () => ({
 }))
 
 const mockSpaces = [
-  { id: 'space-1', title: 'General', type: 'group', isLocked: false, lastActivity: '2024-01-15T00:00:00.000Z', created: '2024-01-01T00:00:00.000Z', creatorId: 'person-1' },
-  { id: 'space-2', title: 'Random', type: 'group', isLocked: false, lastActivity: '2024-01-14T00:00:00.000Z', created: '2024-01-01T00:00:00.000Z', creatorId: 'person-1' },
+  {
+    id: 'space-1',
+    title: 'General',
+    type: 'group',
+    isLocked: false,
+    lastActivity: '2024-01-15T00:00:00.000Z',
+    created: '2024-01-01T00:00:00.000Z',
+    creatorId: 'person-1',
+  },
+  {
+    id: 'space-2',
+    title: 'Random',
+    type: 'group',
+    isLocked: false,
+    lastActivity: '2024-01-14T00:00:00.000Z',
+    created: '2024-01-01T00:00:00.000Z',
+    creatorId: 'person-1',
+  },
 ]
 
 const mockMyMemberships = [
-  { id: 'mem-1', roomId: 'space-1', personId: 'person-1', personEmail: 'alice@example.com', personDisplayName: 'Alice', isModerator: true, created: '2024-01-01T00:00:00.000Z' },
+  {
+    id: 'mem-1',
+    roomId: 'space-1',
+    personId: 'person-1',
+    personEmail: 'alice@example.com',
+    personDisplayName: 'Alice',
+    isModerator: true,
+    created: '2024-01-01T00:00:00.000Z',
+  },
 ]
 
 const mockListSpaces = mock(() => Promise.resolve(mockSpaces as any))
@@ -54,9 +79,21 @@ describe('snapshot command', () => {
     consoleSpy.mockRestore()
   })
 
-  test('returns spaces with id, title, type, lastActivity', async () => {
+  test('brief snapshot returns spaces with id and title only', async () => {
     await snapshotAction({})
 
+    expect(consoleSpy).toHaveBeenCalled()
+    const output = JSON.parse(consoleSpy.mock.calls[consoleSpy.mock.calls.length - 1][0])
+    expect(output.spaces).toHaveLength(1)
+    expect(output.spaces[0].id).toBe('space-1')
+    expect(output.spaces[0].title).toBe('General')
+    expect(output.spaces[0].type).toBeUndefined()
+    expect(output.hint).toBeDefined()
+  })
+
+  test('full snapshot returns spaces with id, title, type, lastActivity', async () => {
+    await snapshotAction({ full: true })
+
     expect(consoleSpy).toHaveBeenCalled()
     const output = JSON.parse(consoleSpy.mock.calls[consoleSpy.mock.calls.length - 1][0])
     expect(output.spaces).toHaveLength(1)
@@ -64,6 +101,7 @@ describe('snapshot command', () => {
     expect(output.spaces[0].title).toBe('General')
     expect(output.spaces[0].type).toBe('group')
     expect(output.spaces[0].lastActivity).toBe('2024-01-15T00:00:00.000Z')
+    expect(output.hint).toBeUndefined()
   })
 
   test('filters spaces to only those in my memberships', async () => {

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

@@ -1,11 +1,11 @@
 import { Command } from 'commander'
+
 import { handleError } from '@/shared/utils/error-handler'
 import { formatOutput } from '@/shared/utils/output'
+
 import { WebexClient } from '../client'
 
-export async function snapshotAction(options: {
-  pretty?: boolean
-}): Promise<void> {
+export async function snapshotAction(options: { full?: boolean; pretty?: boolean }): Promise<void> {
   try {
     const client = await new WebexClient().login()
 
@@ -15,13 +15,19 @@ export async function snapshotAction(options: {
     const allSpaces = await client.listSpaces({ max: 100 })
     const spaces = allSpaces.filter((s) => myRoomIds.has(s.id))
 
-    const snapshot = {
-      spaces: spaces.map((s) => ({
-        id: s.id,
-        title: s.title,
-        type: s.type,
-        lastActivity: s.lastActivity,
-      })),
+    const snapshot: Record<string, any> = {
+      spaces: options.full
+        ? spaces.map((s) => ({
+            id: s.id,
+            title: s.title,
+            type: s.type,
+            lastActivity: s.lastActivity,
+          }))
+        : spaces.map((s) => ({ id: s.id, title: s.title })),
+    }
+
+    if (!options.full) {
+      snapshot.hint = "Use 'message list <space>' for messages, 'space info <space>' for space details."
     }
 
     console.log(formatOutput(snapshot, options.pretty))
@@ -31,10 +37,12 @@ export async function snapshotAction(options: {
 }
 
 export const snapshotCommand = new Command('snapshot')
-  .description('Get workspace spaces overview for AI agents')
+  .description('Get workspace overview for AI agents (brief by default, use --full for comprehensive data)')
+  .option('--full', 'Include full space details (verbose)')
   .option('--pretty', 'Pretty print JSON output')
   .action(async (options) => {
     await snapshotAction({
+      full: options.full,
       pretty: options.pretty,
     })
   })

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

@@ -59,9 +59,9 @@ let consoleLogSpy: ReturnType<typeof spyOn>
 beforeEach(() => {
   mockListSpaces.mockReset().mockImplementation(() => Promise.resolve(mockSpaces))
   mockGetSpace.mockReset().mockImplementation(() => Promise.resolve(mockSpace))
-  mockLogin.mockReset().mockImplementation(() =>
-    Promise.resolve({ listSpaces: mockListSpaces, getSpace: mockGetSpace }),
-  )
+  mockLogin
+    .mockReset()
+    .mockImplementation(() => Promise.resolve({ listSpaces: mockListSpaces, getSpace: mockGetSpace }))
   mockHandleError.mockReset().mockImplementation((err: Error) => {
     throw err
   })

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

@@ -5,11 +5,7 @@ import { formatOutput } from '@/shared/utils/output'
 
 import { WebexClient } from '../client'
 
-export async function listAction(options: {
-  type?: string
-  limit?: number
-  pretty?: boolean
-}): Promise<void> {
+export async function listAction(options: { type?: string; limit?: number; pretty?: boolean }): Promise<void> {
   try {
     const client = await new WebexClient().login()
     const spaces = await client.listSpaces({ type: options.type, max: options.limit })
@@ -26,10 +22,7 @@ export async function listAction(options: {
   }
 }
 
-export async function infoAction(
-  spaceId: string,
-  options: { pretty?: boolean },
-): Promise<void> {
+export async function infoAction(spaceId: string, options: { pretty?: boolean }): Promise<void> {
   try {
     const client = await new WebexClient().login()
     const space = await client.getSpace(spaceId)

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

@@ -18,7 +18,9 @@ const mockUser = {
 }
 
 const makeFakeClient = () => ({
-  login: async function (this: unknown) { return this },
+  login: async function (this: unknown) {
+    return this
+  },
   testAuth: async () => mockUser,
 })
 
@@ -100,10 +102,15 @@ test('whoami outputs pretty-printed JSON when --pretty flag is passed', async ()
 
 test('whoami exits with code 1 when not authenticated', async () => {
   // given: no credentials
-  webexClientSpy.mockImplementation(() => ({
-    login: async () => { throw new WebexError('No Webex credentials found.', 'no_credentials') },
-    testAuth: async () => mockUser,
-  }) as unknown as clientModule.WebexClient)
+  webexClientSpy.mockImplementation(
+    () =>
+      ({
+        login: async () => {
+          throw new WebexError('No Webex credentials found.', 'no_credentials')
+        },
+        testAuth: async () => mockUser,
+      }) as unknown as clientModule.WebexClient,
+  )
 
   // when: running whoami
   await whoamiCommand.parseAsync([], { from: 'user' })

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

@@ -1,6 +1,8 @@
 import { Command } from 'commander'
+
 import { handleError } from '@/shared/utils/error-handler'
 import { formatOutput } from '@/shared/utils/output'
+
 import { WebexClient } from '../client'
 
 export async function whoamiAction(options: { pretty?: boolean }): Promise<void> {

package/src/platforms/webex/credential-manager.ts

@@ -95,7 +95,10 @@ export class WebexCredentialManager {
     }
   }
 
-  async requestDeviceCode(clientId: string, scopes?: string): Promise<{
+  async requestDeviceCode(
+    clientId: string,
+    scopes?: string,
+  ): Promise<{
     deviceCode: string
     userCode: string
     verificationUri: string
@@ -136,7 +139,13 @@ export class WebexCredentialManager {
     }
   }
 
-  async pollDeviceToken(deviceCode: string, interval: number, expiresIn: number, clientId: string, clientSecret?: string): Promise<WebexConfig> {
+  async pollDeviceToken(
+    deviceCode: string,
+    interval: number,
+    expiresIn: number,
+    clientId: string,
+    clientSecret?: string,
+  ): Promise<WebexConfig> {
     const basicAuth = btoa(`${clientId}:${clientSecret ?? ''}`)
     const deadline = Date.now() + expiresIn * 1000
 

package/src/platforms/webex/ensure-auth.ts

@@ -32,9 +32,7 @@ export async function ensureWebexAuth(): Promise<void> {
       tokenType: 'extracted',
       deviceUrl: extracted.deviceUrl,
       userId: extracted.userId,
-      encryptionKeys: extracted.encryptionKeys
-        ? Object.fromEntries(extracted.encryptionKeys)
-        : undefined,
+      encryptionKeys: extracted.encryptionKeys ? Object.fromEntries(extracted.encryptionKeys) : undefined,
     })
   } catch {
     // Intentionally silent — best-effort preflight that should not block commands

package/src/platforms/webex/index.ts

@@ -3,13 +3,7 @@ export { WebexCredentialManager } from './credential-manager'
 export { WebexTokenExtractor } from './token-extractor'
 export type { ExtractedWebexToken } from './token-extractor'
 export { WebexError } from './types'
-export type {
-  WebexConfig,
-  WebexMembership,
-  WebexMessage,
-  WebexPerson,
-  WebexSpace,
-} from './types'
+export type { WebexConfig, WebexMembership, WebexMessage, WebexPerson, WebexSpace } from './types'
 export {
   WebexConfigSchema,
   WebexMembershipSchema,

package/src/platforms/webex/markdown-to-html.test.ts

@@ -30,15 +30,11 @@ describe('markdownToHtml', () => {
   })
 
   test('converts code blocks without language', () => {
-    expect(markdownToHtml('```\nconst x = 1 & 2\n```')).toBe(
-      '<pre><code>const x = 1 &amp; 2</code></pre>',
-    )
+    expect(markdownToHtml('```\nconst x = 1 & 2\n```')).toBe('<pre><code>const x = 1 &amp; 2</code></pre>')
   })
 
   test('does not process markdown inside code blocks', () => {
-    expect(markdownToHtml('```\n**bold** _italic_\n```')).toBe(
-      '<pre><code>**bold** _italic_</code></pre>',
-    )
+    expect(markdownToHtml('```\n**bold** _italic_\n```')).toBe('<pre><code>**bold** _italic_</code></pre>')
   })
 
   test('converts links', () => {
@@ -56,15 +52,11 @@ describe('markdownToHtml', () => {
   })
 
   test('allows mailto: links', () => {
-    expect(markdownToHtml('[email](mailto:a@b.com)')).toBe(
-      '<a href="mailto:a@b.com">email</a>',
-    )
+    expect(markdownToHtml('[email](mailto:a@b.com)')).toBe('<a href="mailto:a@b.com">email</a>')
   })
 
   test('escapes quotes in URLs to prevent attribute breakout', () => {
-    expect(markdownToHtml('[x](https://a.com?q="test")')).toBe(
-      '<a href="https://a.com?q=&quot;test&quot;">x</a>',
-    )
+    expect(markdownToHtml('[x](https://a.com?q="test")')).toBe('<a href="https://a.com?q=&quot;test&quot;">x</a>')
   })
 
   test('converts unordered lists', () => {
@@ -92,9 +84,7 @@ describe('markdownToHtml', () => {
   })
 
   test('supports nested formatting', () => {
-    expect(markdownToHtml('**bold _and italic_**')).toBe(
-      '<strong>bold <em>and italic</em></strong>',
-    )
+    expect(markdownToHtml('**bold _and italic_**')).toBe('<strong>bold <em>and italic</em></strong>')
   })
 
   test('escapes html special characters in text', () => {
@@ -134,9 +124,7 @@ describe('stripMarkdown', () => {
   })
 
   test('strips block markdown syntax', () => {
-    expect(stripMarkdown('# Title\n> quote\n- item\n1. first\n---')).toBe(
-      'Title\nquote\nitem\nfirst\n',
-    )
+    expect(stripMarkdown('# Title\n> quote\n- item\n1. first\n---')).toBe('Title\nquote\nitem\nfirst\n')
   })
 
   test('keeps code block content', () => {

package/src/platforms/webex/markdown-to-html.ts

@@ -129,10 +129,7 @@ function processInline(markdown: string): string {
     if (!isSafeUrl(url)) {
       return processInline(label)
     }
-    return createInlinePlaceholder(
-      placeholders,
-      `<a href="${escapeHtml(url)}">${processInline(label)}</a>`,
-    )
+    return createInlinePlaceholder(placeholders, `<a href="${escapeHtml(url)}">${processInline(label)}</a>`)
   })
 
   text = escapeHtml(text)
@@ -145,10 +142,13 @@ function processInline(markdown: string): string {
 
 function extractCodeBlocks(markdown: string): { text: string; blocks: string[] } {
   const blocks: string[] = []
-  const text = markdown.replace(/```([a-zA-Z0-9_-]+)?\n([\s\S]*?)\n```/g, (_, language: string | undefined, code: string) => {
-    const className = language ? ` class="language-${escapeHtml(language)}"` : ''
-    return createBlockPlaceholder(blocks, `<pre><code${className}>${escapeHtml(code)}</code></pre>`)
-  })
+  const text = markdown.replace(
+    /```([a-zA-Z0-9_-]+)?\n([\s\S]*?)\n```/g,
+    (_, language: string | undefined, code: string) => {
+      const className = language ? ` class="language-${escapeHtml(language)}"` : ''
+      return createBlockPlaceholder(blocks, `<pre><code${className}>${escapeHtml(code)}</code></pre>`)
+    },
+  )
 
   return { text, blocks }
 }

package/src/platforms/webex/token-extractor.ts

@@ -1,12 +1,4 @@
-import {
-  copyFileSync,
-  existsSync,
-  mkdirSync,
-  readdirSync,
-  readFileSync,
-  rmSync,
-  statSync,
-} from 'node:fs'
+import { copyFileSync, existsSync, mkdirSync, readdirSync, readFileSync, rmSync, statSync } from 'node:fs'
 import { homedir, tmpdir } from 'node:os'
 import { join } from 'node:path'
 
@@ -73,8 +65,6 @@ const BROWSERS: BrowserConfig[] = [
   },
 ]
 
-const WEBEX_STORAGE_KEY = '_https://web.webex.com\x00\x01webex-web-client-bounded'
-
 interface ScanResult {
   token: ExtractedWebexToken | null
   encryptionKeys: Map<string, string>
@@ -128,10 +118,7 @@ export class WebexTokenExtractor {
       case 'win32':
         relative = browser.win32
         if (!relative) return null
-        return join(
-          process.env.LOCALAPPDATA || join(homedir(), 'AppData', 'Local'),
-          relative,
-        )
+        return join(process.env.LOCALAPPDATA || join(homedir(), 'AppData', 'Local'), relative)
       default:
         return null
     }
@@ -176,8 +163,7 @@ export class WebexTokenExtractor {
     for (const leveldbDir of profileDirs) {
       this.debug(`Scanning: ${leveldbDir}`)
 
-      const result = await this.scanViaClassicLevelCopy(leveldbDir)
-        ?? this.scanRawFiles(leveldbDir)
+      const result = (await this.scanViaClassicLevelCopy(leveldbDir)) ?? this.scanRawFiles(leveldbDir)
 
       if (result?.token) {
         this.debug(`Found token in: ${leveldbDir}`)
@@ -339,9 +325,7 @@ export class WebexTokenExtractor {
     const innermostMarkerIdx = content.indexOf('"supertoken"')
 
     const markerIdx =
-      outerObjectMarkerIdx !== -1 ? outerObjectMarkerIdx
-      : innermostMarkerIdx !== -1 ? innermostMarkerIdx
-      : -1
+      outerObjectMarkerIdx !== -1 ? outerObjectMarkerIdx : innermostMarkerIdx !== -1 ? innermostMarkerIdx : -1
 
     if (markerIdx === -1) return null
 
@@ -387,11 +371,7 @@ export class WebexTokenExtractor {
     try {
       const data = JSON.parse(jsonStr)
 
-      const supertoken =
-        data?.Credentials?.['@']?.supertoken ??
-        data?.['@']?.supertoken ??
-        data?.supertoken ??
-        null
+      const supertoken = data?.Credentials?.['@']?.supertoken ?? data?.['@']?.supertoken ?? data?.supertoken ?? null
 
       if (!supertoken?.access_token) return null
 
@@ -410,10 +390,7 @@ export class WebexTokenExtractor {
         result.expiresAt = Date.now() + supertoken.expires_in * 1000
       }
 
-      const deviceUrl =
-        data?.Device?.['@']?.url ??
-        data?.['@']?.deviceUrl ??
-        null
+      const deviceUrl = data?.Device?.['@']?.url ?? data?.['@']?.deviceUrl ?? null
       if (deviceUrl && typeof deviceUrl === 'string') {
         result.deviceUrl = deviceUrl
       }

package/src/platforms/wechatbot/client.test.ts

@@ -50,12 +50,16 @@ describe('WeChatBotClient', () => {
   describe('login', () => {
     test('throws on empty appId', async () => {
       await expect(new WeChatBotClient().login({ appId: '', appSecret: 'secret' })).rejects.toThrow(WeChatBotError)
-      await expect(new WeChatBotClient().login({ appId: '', appSecret: 'secret' })).rejects.toThrow('App ID is required')
+      await expect(new WeChatBotClient().login({ appId: '', appSecret: 'secret' })).rejects.toThrow(
+        'App ID is required',
+      )
     })
 
     test('throws on empty appSecret', async () => {
       await expect(new WeChatBotClient().login({ appId: 'wx123', appSecret: '' })).rejects.toThrow(WeChatBotError)
-      await expect(new WeChatBotClient().login({ appId: 'wx123', appSecret: '' })).rejects.toThrow('App Secret is required')
+      await expect(new WeChatBotClient().login({ appId: 'wx123', appSecret: '' })).rejects.toThrow(
+        'App Secret is required',
+      )
     })
 
     test('accepts valid credentials and returns client', async () => {

package/src/platforms/wechatbot/client.ts

@@ -62,7 +62,12 @@ export class WeChatBotClient {
       throw new WeChatBotError(`Network error fetching access token: ${msg}`, 'network_error')
     }
 
-    const data = (await response.json()) as { access_token?: string; expires_in?: number; errcode?: number; errmsg?: string }
+    const data = (await response.json()) as {
+      access_token?: string
+      expires_in?: number
+      errcode?: number
+      errmsg?: string
+    }
 
     if (data.errcode) {
       throw new WeChatBotError(data.errmsg ?? `Error ${data.errcode}`, String(data.errcode))

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

@@ -13,17 +13,13 @@ mock.module('../client', () => ({
   },
 }))
 
-const { setAction, statusAction, clearAction, listAction, useAction, removeAction } = await import(
-  '@/platforms/wechatbot/commands/auth'
-)
+const { setAction, statusAction, clearAction, listAction, useAction, removeAction } =
+  await import('@/platforms/wechatbot/commands/auth')
 
 const testDirs: string[] = []
 
 function makeCredManager(): WeChatBotCredentialManager {
-  const dir = join(
-    import.meta.dir,
-    `.test-auth-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
-  )
+  const dir = join(import.meta.dir, `.test-auth-config-${Date.now()}-${Math.random().toString(36).slice(2)}`)
   testDirs.push(dir)
   return new WeChatBotCredentialManager(dir)
 }

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

@@ -24,10 +24,7 @@ const { sendAction, sendNewsAction } = await import('@/platforms/wechatbot/comma
 const testDirs: string[] = []
 
 function makeCredManager(): WeChatBotCredentialManager {
-  const dir = join(
-    import.meta.dir,
-    `.test-message-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
-  )
+  const dir = join(import.meta.dir, `.test-message-config-${Date.now()}-${Math.random().toString(36).slice(2)}`)
   testDirs.push(dir)
   const manager = new WeChatBotCredentialManager(dir)
   return manager

package/src/platforms/wechatbot/commands/message.ts

@@ -28,7 +28,11 @@ export async function sendAction(openId: string, text: string, options: MessageO
   }
 }
 
-export async function sendImageAction(openId: string, mediaId: string, options: MessageOptions): Promise<MessageResult> {
+export async function sendImageAction(
+  openId: string,
+  mediaId: string,
+  options: MessageOptions,
+): Promise<MessageResult> {
   try {
     const client = await getClient(options)
     await client.sendImageMessage(openId, mediaId)

package/src/platforms/wechatbot/commands/template.test.ts

@@ -31,10 +31,7 @@ const { listAction } = await import('@/platforms/wechatbot/commands/template')
 const testDirs: string[] = []
 
 function makeCredManager(): WeChatBotCredentialManager {
-  const dir = join(
-    import.meta.dir,
-    `.test-template-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
-  )
+  const dir = join(import.meta.dir, `.test-template-config-${Date.now()}-${Math.random().toString(36).slice(2)}`)
   testDirs.push(dir)
   return new WeChatBotCredentialManager(dir)
 }

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

@@ -38,10 +38,7 @@ const { listAction } = await import('@/platforms/wechatbot/commands/user')
 const testDirs: string[] = []
 
 function makeCredManager(): WeChatBotCredentialManager {
-  const dir = join(
-    import.meta.dir,
-    `.test-user-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
-  )
+  const dir = join(import.meta.dir, `.test-user-config-${Date.now()}-${Math.random().toString(36).slice(2)}`)
   testDirs.push(dir)
   return new WeChatBotCredentialManager(dir)
 }
@@ -88,9 +85,7 @@ describe('listAction (user)', () => {
   })
 
   test('passes nextOpenid option to client', async () => {
-    const getFollowersMock = mock(() =>
-      Promise.resolve({ total: 1, count: 1, openids: ['openid-3'], next_openid: '' }),
-    )
+    const getFollowersMock = mock(() => Promise.resolve({ total: 1, count: 1, openids: ['openid-3'], next_openid: '' }))
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {

package/src/platforms/whatsapp/cli.ts

@@ -19,10 +19,7 @@ function isAuthCommand(command: CommandType): boolean {
 
 const program = new Command()
 
-program
-  .name('agent-whatsapp')
-  .description('CLI tool for WhatsApp via Baileys (linked device)')
-  .version(pkg.version)
+program.name('agent-whatsapp').description('CLI tool for WhatsApp via Baileys (linked device)').version(pkg.version)
 
 program.hook('preAction', async (_thisCommand, actionCommand) => {
   if (isAuthCommand(actionCommand)) return