agent-messenger 2.6.3 → 2.8.0

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

@@ -41,7 +41,12 @@ async function resolveClientCredentials(options: {
   return getWebexAppCredentials()
 }
 
-export async function loginAction(options: { token?: string; clientId?: string; clientSecret?: string; pretty?: boolean }): Promise<void> {
+export async function loginAction(options: {
+  token?: string
+  clientId?: string
+  clientSecret?: string
+  pretty?: boolean
+}): Promise<void> {
   try {
     const credManager = new WebexCredentialManager()
 
@@ -110,9 +115,7 @@ export async function statusAction(options: { pretty?: boolean }): Promise<void>
     const token = await credManager.getToken(config?.clientId, config?.clientSecret)
 
     if (!token) {
-      console.log(
-        formatOutput({ error: 'Not authenticated. Run "auth login" first.' }, options.pretty),
-      )
+      console.log(formatOutput({ error: 'Not authenticated. Run "auth login" first.' }, options.pretty))
       process.exit(1)
       return
     }
@@ -139,10 +142,7 @@ export async function statusAction(options: { pretty?: boolean }): Promise<void>
 
 export async function extractAction(options: { pretty?: boolean; debug?: boolean }): Promise<void> {
   try {
-    const extractor = new WebexTokenExtractor(
-      undefined,
-      options.debug ? (msg) => debug(`[debug] ${msg}`) : undefined,
-    )
+    const extractor = new WebexTokenExtractor(undefined, options.debug ? (msg) => debug(`[debug] ${msg}`) : undefined)
 
     if (options.debug) {
       debug('[debug] Searching browser profiles for Webex tokens...')
@@ -154,7 +154,8 @@ export async function extractAction(options: { pretty?: boolean; debug?: boolean
       console.log(
         formatOutput(
           {
-            error: 'No Webex token found in any browser. Make sure you are logged in to web.webex.com in Chrome, Edge, Arc, or Brave.',
+            error:
+              'No Webex token found in any browser. Make sure you are logged in to web.webex.com in Chrome, Edge, Arc, or Brave.',
             hint: 'Run "auth login" for OAuth Device Grant flow, or --debug for more info.',
           },
           options.pretty,
@@ -175,9 +176,7 @@ export async function extractAction(options: { pretty?: boolean; debug?: boolean
       tokenType: 'extracted',
       deviceUrl: extracted.deviceUrl,
       userId: extracted.userId,
-      encryptionKeys: extracted.encryptionKeys
-        ? Object.fromEntries(extracted.encryptionKeys)
-        : undefined,
+      encryptionKeys: extracted.encryptionKeys ? Object.fromEntries(extracted.encryptionKeys) : undefined,
     })
 
     console.log(
@@ -201,9 +200,7 @@ export async function logoutAction(options: { pretty?: boolean }): Promise<void>
     const config = await credManager.loadConfig()
 
     if (!config) {
-      console.log(
-        formatOutput({ error: 'Not authenticated. Run "auth login" first.' }, options.pretty),
-      )
+      console.log(formatOutput({ error: 'Not authenticated. Run "auth login" first.' }, options.pretty))
       process.exit(1)
       return
     }

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

@@ -47,9 +47,7 @@ describe('member commands', () => {
 
   beforeEach(() => {
     mockListMemberships.mockReset().mockImplementation(() => Promise.resolve(mockMembers))
-    mockLogin.mockReset().mockImplementation(() =>
-      Promise.resolve({ listMemberships: mockListMemberships }),
-    )
+    mockLogin.mockReset().mockImplementation(() => Promise.resolve({ listMemberships: mockListMemberships }))
     mockHandleError.mockReset().mockImplementation((err: Error) => {
       throw err
     })

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

@@ -5,10 +5,7 @@ import { formatOutput } from '@/shared/utils/output'
 
 import { WebexClient } from '../client'
 
-export async function listAction(
-  spaceId: string,
-  options: { limit?: number; pretty?: boolean },
-): Promise<void> {
+export async function listAction(spaceId: string, options: { limit?: number; pretty?: boolean }): Promise<void> {
   try {
     const client = await new WebexClient().login()
     const members = await client.listMemberships(spaceId, { max: options.limit })
@@ -28,18 +25,16 @@ export async function listAction(
   }
 }
 
-export const memberCommand = new Command('member')
-  .description('Member commands')
-  .addCommand(
-    new Command('list')
-      .description('List members of a space')
-      .argument('<space-id>', 'Space ID')
-      .option('--limit <n>', 'Number of members to retrieve', '100')
-      .option('--pretty', 'Pretty print JSON output')
-      .action((spaceId, options) =>
-        listAction(spaceId, {
-          limit: parseInt(options.limit, 10),
-          pretty: options.pretty,
-        }),
-      ),
-  )
+export const memberCommand = new Command('member').description('Member commands').addCommand(
+  new Command('list')
+    .description('List members of a space')
+    .argument('<space-id>', 'Space ID')
+    .option('--limit <n>', 'Number of members to retrieve', '100')
+    .option('--pretty', 'Pretty print JSON output')
+    .action((spaceId, options) =>
+      listAction(spaceId, {
+        limit: parseInt(options.limit, 10),
+        pretty: options.pretty,
+      }),
+    ),
+)

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

@@ -29,10 +29,7 @@ export async function sendAction(
   }
 }
 
-export async function listAction(
-  spaceId: string,
-  options: { limit?: number; pretty?: boolean },
-): Promise<void> {
+export async function listAction(spaceId: string, options: { limit?: number; pretty?: boolean }): Promise<void> {
   try {
     const client = await new WebexClient().login()
     const limit = options.limit ?? 50
@@ -52,10 +49,7 @@ export async function listAction(
   }
 }
 
-export async function getAction(
-  messageId: string,
-  options: { pretty?: boolean },
-): Promise<void> {
+export async function getAction(messageId: string, options: { pretty?: boolean }): Promise<void> {
   try {
     const client = await new WebexClient().login()
     const message = await client.getMessage(messageId)
@@ -74,15 +68,10 @@ export async function getAction(
   }
 }
 
-export async function deleteAction(
-  messageId: string,
-  options: { force?: boolean; pretty?: boolean },
-): Promise<void> {
+export async function deleteAction(messageId: string, options: { force?: boolean; pretty?: boolean }): Promise<void> {
   try {
     if (!options.force) {
-      console.log(
-        formatOutput({ warning: 'Use --force to confirm deletion', messageId }, options.pretty),
-      )
+      console.log(formatOutput({ warning: 'Use --force to confirm deletion', messageId }, options.pretty))
       return process.exit(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))

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: { pretty?: boolean }): Promise<void> {
   try {
     const client = await new WebexClient().login()
 

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

@@ -0,0 +1,141 @@
+import { describe, expect, test } from 'bun:test'
+
+import { markdownToHtml, stripMarkdown } from './markdown-to-html'
+
+describe('markdownToHtml', () => {
+  test('converts bold text', () => {
+    expect(markdownToHtml('**bold**')).toBe('<strong>bold</strong>')
+  })
+
+  test('converts italic text', () => {
+    expect(markdownToHtml('_italic_')).toBe('<em>italic</em>')
+  })
+
+  test('does not italicize mid-word underscores', () => {
+    expect(markdownToHtml('some_variable_name')).toBe('some_variable_name')
+  })
+
+  test('converts bold and italic text', () => {
+    expect(markdownToHtml('***both***')).toBe('<strong><em>both</em></strong>')
+  })
+
+  test('converts inline code', () => {
+    expect(markdownToHtml('Use `code` here')).toBe('Use <code>code</code> here')
+  })
+
+  test('converts code blocks with language', () => {
+    expect(markdownToHtml('```ts\nconst x = 1 < 2\n```')).toBe(
+      '<pre><code class="language-ts">const x = 1 &lt; 2</code></pre>',
+    )
+  })
+
+  test('converts code blocks without language', () => {
+    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>')
+  })
+
+  test('converts links', () => {
+    expect(markdownToHtml('[Webex](https://example.com?a=1&b=2)')).toBe(
+      '<a href="https://example.com?a=1&amp;b=2">Webex</a>',
+    )
+  })
+
+  test('strips unsafe javascript: URLs to plain text', () => {
+    expect(markdownToHtml('[click](javascript:void)')).toBe('click')
+  })
+
+  test('strips unsafe data: URLs to plain text', () => {
+    expect(markdownToHtml('[x](data:text/html,payload)')).toBe('x')
+  })
+
+  test('allows mailto: links', () => {
+    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>')
+  })
+
+  test('converts unordered lists', () => {
+    expect(markdownToHtml('- one\n- two')).toBe('<ul><li>one</li><li>two</li></ul>')
+  })
+
+  test('converts ordered lists', () => {
+    expect(markdownToHtml('1. one\n2. two')).toBe('<ol><li>one</li><li>two</li></ol>')
+  })
+
+  test('converts blockquotes', () => {
+    expect(markdownToHtml('> one\n> two')).toBe('<blockquote>one<br/>two</blockquote>')
+  })
+
+  test('converts headings', () => {
+    expect(markdownToHtml('# one\n###### six')).toBe('<h1>one</h1><br/><br/><h6>six</h6>')
+  })
+
+  test('converts horizontal rules', () => {
+    expect(markdownToHtml('before\n---\nafter')).toBe('before<br/><br/><hr><br/><br/>after')
+  })
+
+  test('converts paragraph newlines to br', () => {
+    expect(markdownToHtml('one\ntwo')).toBe('one<br/>two')
+  })
+
+  test('supports nested formatting', () => {
+    expect(markdownToHtml('**bold _and italic_**')).toBe('<strong>bold <em>and italic</em></strong>')
+  })
+
+  test('escapes html special characters in text', () => {
+    expect(markdownToHtml('5 < 7 & 8 > 3')).toBe('5 &lt; 7 &amp; 8 &gt; 3')
+  })
+
+  test('separates multiple paragraphs with br', () => {
+    expect(markdownToHtml('first\n\nsecond')).toBe('first<br/><br/>second')
+  })
+
+  test('renders mixed content', () => {
+    expect(markdownToHtml('Hello **team**\n\n- one\n- two\n\n```js\nconst x = 1\n```')).toBe(
+      'Hello <strong>team</strong><br/><br/><ul><li>one</li><li>two</li></ul><br/><br/><pre><code class="language-js">const x = 1</code></pre>',
+    )
+  })
+
+  test('returns empty string for empty input', () => {
+    expect(markdownToHtml('')).toBe('')
+  })
+
+  test('returns plain text when there is no markdown', () => {
+    expect(markdownToHtml('hello world')).toBe('hello world')
+  })
+
+  test('preserves whitespace-only input', () => {
+    expect(markdownToHtml('   ')).toBe('   ')
+  })
+})
+
+describe('stripMarkdown', () => {
+  test('strips inline markdown syntax', () => {
+    expect(stripMarkdown('**bold** _italic_ `code`')).toBe('bold italic code')
+  })
+
+  test('strips links to their labels', () => {
+    expect(stripMarkdown('[Webex](https://example.com)')).toBe('Webex')
+  })
+
+  test('strips block markdown syntax', () => {
+    expect(stripMarkdown('# Title\n> quote\n- item\n1. first\n---')).toBe('Title\nquote\nitem\nfirst\n')
+  })
+
+  test('keeps code block content', () => {
+    expect(stripMarkdown('```ts\nconst x = 1\n```')).toBe('const x = 1')
+  })
+
+  test('handles nested formatting', () => {
+    expect(stripMarkdown('**bold _and italic_**')).toBe('bold and italic')
+  })
+
+  test('returns plain text unchanged', () => {
+    expect(stripMarkdown('hello world')).toBe('hello world')
+  })
+})