agent-messenger 2.10.1 → 2.11.0

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, it } from 'bun:test'
 
 import { ChannelBotClient } from './client'
 import { ChannelBotError } from './types'
@@ -46,7 +46,7 @@ describe('ChannelBotClient', () => {
     )
   }
 
-  test('successful GET request returns unwrapped JSON', async () => {
+  it('successful GET request returns unwrapped JSON', async () => {
     mockResponse({ channel: { id: 'ch-1', name: 'My Channel' } })
 
     const client = await new ChannelBotClient().login({ accessKey: 'key-1', accessSecret: 'secret-1' })
@@ -56,7 +56,7 @@ describe('ChannelBotClient', () => {
     expect(fetchCalls[0].url).toBe('https://api.channel.io/open/v5/channel')
   })
 
-  test('auth headers are set on every request', async () => {
+  it('auth headers are set on every request', async () => {
     mockResponse({ channel: { id: 'ch-1', name: 'My Channel' } })
     mockResponse({ user: { id: 'u-1', channelId: 'ch-1' } })
 
@@ -73,7 +73,7 @@ describe('ChannelBotClient', () => {
     }
   })
 
-  test('429 response triggers retry with Retry-After wait', async () => {
+  it('429 response triggers retry with Retry-After wait', async () => {
     mockResponse({ message: 'Rate limited' }, 429, { 'Retry-After': '0.05' })
     mockResponse({ channel: { id: 'ch-1', name: 'My Channel' } })
 
@@ -87,7 +87,7 @@ describe('ChannelBotClient', () => {
     expect(elapsed).toBeGreaterThanOrEqual(40)
   })
 
-  test('500 response triggers retry with exponential backoff', async () => {
+  it('500 response triggers retry with exponential backoff', async () => {
     mockResponse({ message: 'Server error' }, 500)
     mockResponse({ message: 'Server error' }, 500)
     mockResponse({ channel: { id: 'ch-1', name: 'My Channel' } })
@@ -102,7 +102,7 @@ describe('ChannelBotClient', () => {
     expect(elapsed).toBeGreaterThanOrEqual(280)
   })
 
-  test('4xx non-429 throws immediately without retry', async () => {
+  it('4xx non-429 throws immediately without retry', async () => {
     mockResponse({ message: 'Forbidden' }, 403)
 
     const client = await new ChannelBotClient().login({ accessKey: 'key-1', accessSecret: 'secret-1' })
@@ -110,7 +110,7 @@ describe('ChannelBotClient', () => {
     expect(fetchCalls).toHaveLength(1)
   })
 
-  test('network error retries then throws ChannelBotError with code network_error', async () => {
+  it('network error retries then throws ChannelBotError with code network_error', async () => {
     ;(globalThis as Record<string, unknown>).fetch = async (
       url: string | URL | Request,
       options?: RequestInit,
@@ -131,7 +131,7 @@ describe('ChannelBotClient', () => {
     }
   })
 
-  test('204 response returns undefined', async () => {
+  it('204 response returns undefined', async () => {
     mockResponse(null, 204)
 
     const client = await new ChannelBotClient().login({ accessKey: 'key-1', accessSecret: 'secret-1' })
@@ -140,11 +140,11 @@ describe('ChannelBotClient', () => {
     expect(result).toBeUndefined()
   })
 
-  test('wrapTextInBlocks returns a single text block with value', () => {
+  it('wrapTextInBlocks returns a single text block with value', () => {
     expect(ChannelBotClient.wrapTextInBlocks('Hello world')).toEqual([{ type: 'text', value: 'Hello world' }])
   })
 
-  test('sendUserChatMessage includes botName in query string', async () => {
+  it('sendUserChatMessage includes botName in query string', async () => {
     mockResponse({ message: { id: 'm-1' } })
 
     const client = await new ChannelBotClient().login({ accessKey: 'key-1', accessSecret: 'secret-1' })
@@ -153,7 +153,7 @@ describe('ChannelBotClient', () => {
     expect(fetchCalls[0].url).toBe('https://api.channel.io/open/v5/user-chats/chat-1/messages?botName=SupportBot')
   })
 
-  test('sendGroupMessage includes botName in query string', async () => {
+  it('sendGroupMessage includes botName in query string', async () => {
     mockResponse({ message: { id: 'm-1' } })
 
     const client = await new ChannelBotClient().login({ accessKey: 'key-1', accessSecret: 'secret-1' })
@@ -162,7 +162,7 @@ describe('ChannelBotClient', () => {
     expect(fetchCalls[0].url).toBe('https://api.channel.io/open/v5/groups/grp-1/messages?botName=OpsBot')
   })
 
-  test('resolveGroup("@team-name") calls groups by name endpoint', async () => {
+  it('resolveGroup("@team-name") calls groups by name endpoint', async () => {
     mockResponse({ group: { id: 'grp-1', channelId: 'ch-1', name: 'team-name' } })
 
     const client = await new ChannelBotClient().login({ accessKey: 'key-1', accessSecret: 'secret-1' })
@@ -171,7 +171,7 @@ describe('ChannelBotClient', () => {
     expect(fetchCalls[0].url).toBe('https://api.channel.io/open/v5/groups/@team-name')
   })
 
-  test('resolveGroup("grp123") calls groups by id endpoint', async () => {
+  it('resolveGroup("grp123") calls groups by id endpoint', async () => {
     mockResponse({ group: { id: 'grp123', channelId: 'ch-1', name: 'team-name' } })
 
     const client = await new ChannelBotClient().login({ accessKey: 'key-1', accessSecret: 'secret-1' })
@@ -180,7 +180,7 @@ describe('ChannelBotClient', () => {
     expect(fetchCalls[0].url).toBe('https://api.channel.io/open/v5/groups/grp123')
   })
 
-  test('pagination params are included in query string', async () => {
+  it('pagination params are included in query string', async () => {
     mockResponse({ messages: [] })
 
     const client = await new ChannelBotClient().login({ accessKey: 'key-1', accessSecret: 'secret-1' })

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { mkdir } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
@@ -39,7 +39,7 @@ describe('auth commands', () => {
   })
 
   describe('setAction', () => {
-    test('validates and stores credentials with workspace info from API', async () => {
+    it('validates and stores credentials with workspace info from API', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
 
       const result = await setAction('key123', 'secret123', { _credManager: manager })
@@ -53,7 +53,7 @@ describe('auth commands', () => {
       expect(creds?.workspace_id).toBe('ch123')
     })
 
-    test('uses --workspace option as workspace name', async () => {
+    it('uses --workspace option as workspace name', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
 
       const result = await setAction('key123', 'secret123', {
@@ -68,7 +68,7 @@ describe('auth commands', () => {
       expect(creds?.workspace_name).toBe('My Custom Name')
     })
 
-    test('handles client errors gracefully', async () => {
+    it('handles client errors gracefully', async () => {
       mockGetChannel.mockImplementationOnce(() => Promise.reject(new Error('Invalid credentials')))
 
       const manager = new ChannelBotCredentialManager(tempDir)
@@ -81,7 +81,7 @@ describe('auth commands', () => {
   })
 
   describe('statusAction', () => {
-    test('returns no credentials when none set', async () => {
+    it('returns no credentials when none set', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
 
       const result = await statusAction({ _credManager: manager })
@@ -90,7 +90,7 @@ describe('auth commands', () => {
       expect(result.error).toBeDefined()
     })
 
-    test('returns valid status for current workspace', async () => {
+    it('returns valid status for current workspace', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await manager.setCredentials({
         workspace_id: 'ch123',
@@ -105,7 +105,7 @@ describe('auth commands', () => {
       expect(result.workspace_id).toBe('ch123')
     })
 
-    test('returns invalid when API call fails', async () => {
+    it('returns invalid when API call fails', async () => {
       mockGetChannel.mockImplementationOnce(() => Promise.reject(new Error('Unauthorized')))
 
       const manager = new ChannelBotCredentialManager(tempDir)
@@ -123,7 +123,7 @@ describe('auth commands', () => {
   })
 
   describe('clearAction', () => {
-    test('removes all stored credentials', async () => {
+    it('removes all stored credentials', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await manager.setCredentials({
         workspace_id: 'ch123',
@@ -140,7 +140,7 @@ describe('auth commands', () => {
   })
 
   describe('listAction', () => {
-    test('returns all stored workspaces', async () => {
+    it('returns all stored workspaces', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await manager.setCredentials({
         workspace_id: 'ch1',
@@ -162,7 +162,7 @@ describe('auth commands', () => {
       expect(result.workspaces?.find((w) => w.workspace_id === 'ch1')?.is_current).toBe(false)
     })
 
-    test('returns empty list when no workspaces stored', async () => {
+    it('returns empty list when no workspaces stored', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
 
       const result = await listAction({ _credManager: manager })
@@ -172,7 +172,7 @@ describe('auth commands', () => {
   })
 
   describe('useAction', () => {
-    test('switches current workspace', async () => {
+    it('switches current workspace', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await manager.setCredentials({
         workspace_id: 'ch1',
@@ -193,7 +193,7 @@ describe('auth commands', () => {
       expect(result.workspace_id).toBe('ch1')
     })
 
-    test('returns error for unknown workspace', async () => {
+    it('returns error for unknown workspace', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
 
       const result = await useAction('nonexistent', { _credManager: manager })
@@ -203,7 +203,7 @@ describe('auth commands', () => {
   })
 
   describe('removeAction', () => {
-    test('removes a stored workspace', async () => {
+    it('removes a stored workspace', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await manager.setCredentials({
         workspace_id: 'ch1',
@@ -218,7 +218,7 @@ describe('auth commands', () => {
       expect(await manager.getCredentials('ch1')).toBeNull()
     })
 
-    test('returns error for unknown workspace', async () => {
+    it('returns error for unknown workspace', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
 
       const result = await removeAction('nonexistent', { _credManager: manager })
@@ -228,7 +228,7 @@ describe('auth commands', () => {
   })
 
   describe('botAction', () => {
-    test('sets default bot name', async () => {
+    it('sets default bot name', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
 
       const result = await botAction('my-bot', { _credManager: manager })
@@ -238,7 +238,7 @@ describe('auth commands', () => {
       expect(await manager.getDefaultBot()).toBe('my-bot')
     })
 
-    test('updates existing default bot', async () => {
+    it('updates existing default bot', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await manager.setDefaultBot('old-bot')
 

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

@@ -1,5 +1,6 @@
 import { Command } from 'commander'
 
+import { cliOutput } from '@/shared/utils/cli-output'
 import { formatOutput } from '@/shared/utils/output'
 
 import { ChannelBotClient } from '../client'
@@ -158,11 +159,6 @@ export async function botAction(name: string, options: ActionOptions): Promise<A
   }
 }
 
-function cliOutput(result: ActionResult, pretty?: boolean, exitOnError = true): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error && exitOnError) process.exit(1)
-}
-
 export const authCommand = new Command('auth')
   .description('Authentication commands')
   .addCommand(

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { mkdir } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
@@ -67,7 +67,7 @@ describe('bot commands', () => {
   })
 
   describe('listAction', () => {
-    test('returns all bots', async () => {
+    it('returns all bots', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await listAction({ _credManager: manager })
 
@@ -78,7 +78,7 @@ describe('bot commands', () => {
   })
 
   describe('createAction', () => {
-    test('creates bot with name', async () => {
+    it('creates bot with name', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await createAction('New Bot', { _credManager: manager })
 
@@ -88,7 +88,7 @@ describe('bot commands', () => {
       expect(capturedCreateArgs[0]).toBe('New Bot')
     })
 
-    test('creates bot with optional color and avatar', async () => {
+    it('creates bot with optional color and avatar', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await createAction('New Bot', {
         color: '#00FF00',
@@ -101,7 +101,7 @@ describe('bot commands', () => {
   })
 
   describe('deleteAction', () => {
-    test('deletes bot with --force flag', async () => {
+    it('deletes bot with --force flag', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await deleteAction('bot1', { force: true, _credManager: manager })
 
@@ -111,7 +111,7 @@ describe('bot commands', () => {
       expect(capturedDeleteArg).toBe('bot1')
     })
 
-    test('returns error without --force flag', async () => {
+    it('returns error without --force flag', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await deleteAction('bot1', { _credManager: manager })
 

package/src/platforms/channeltalkbot/commands/bot.ts

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 
-import { formatOutput } from '@/shared/utils/output'
+import { cliOutput } from '@/shared/utils/cli-output'
 
 import type { WorkspaceOption } from './shared'
 import { getClient } from './shared'
@@ -89,11 +89,6 @@ export async function deleteAction(botId: string, options: BotOptions): Promise<
   }
 }
 
-function cliOutput(result: BotResult, pretty?: boolean): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error) process.exit(1)
-}
-
 export const botCommand = new Command('bot')
   .description('Bot management commands')
   .addCommand(

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { mkdir } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
@@ -92,7 +92,7 @@ describe('chat commands', () => {
   })
 
   describe('listAction', () => {
-    test('lists opened chats by default', async () => {
+    it('lists opened chats by default', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await listAction({ _credManager: manager })
 
@@ -102,14 +102,14 @@ describe('chat commands', () => {
       expect(capturedListArgs[0]).toMatchObject({ state: 'opened' })
     })
 
-    test('passes state filter to API', async () => {
+    it('passes state filter to API', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await listAction({ state: 'closed', _credManager: manager })
 
       expect(capturedListArgs[0]).toMatchObject({ state: 'closed' })
     })
 
-    test('passes pagination params', async () => {
+    it('passes pagination params', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await listAction({ limit: '10', sort: 'asc', since: 'cursor123', _credManager: manager })
 
@@ -118,7 +118,7 @@ describe('chat commands', () => {
   })
 
   describe('getAction', () => {
-    test('returns specific chat', async () => {
+    it('returns specific chat', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await getAction('chat1', { _credManager: manager })
 
@@ -129,7 +129,7 @@ describe('chat commands', () => {
   })
 
   describe('closeAction', () => {
-    test('closes chat with bot name', async () => {
+    it('closes chat with bot name', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await closeAction('chat1', { bot: 'my-bot', _credManager: manager })
 
@@ -138,7 +138,7 @@ describe('chat commands', () => {
       expect(capturedCloseArgs[1]).toBe('my-bot')
     })
 
-    test('returns error when no bot name provided', async () => {
+    it('returns error when no bot name provided', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await closeAction('chat1', { _credManager: manager })
 
@@ -148,7 +148,7 @@ describe('chat commands', () => {
   })
 
   describe('deleteAction', () => {
-    test('deletes chat with --force flag', async () => {
+    it('deletes chat with --force flag', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await deleteAction('chat1', { force: true, _credManager: manager })
 
@@ -157,7 +157,7 @@ describe('chat commands', () => {
       expect(result.deleted).toBe('chat1')
     })
 
-    test('returns error without --force flag', async () => {
+    it('returns error without --force flag', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await deleteAction('chat1', { _credManager: manager })
 

package/src/platforms/channeltalkbot/commands/chat.ts

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 
-import { formatOutput } from '@/shared/utils/output'
+import { cliOutput } from '@/shared/utils/cli-output'
 
 import type { WorkspaceOption } from './shared'
 import { getClient, getDefaultBotName } from './shared'
@@ -120,11 +120,6 @@ export async function deleteAction(chatId: string, options: ChatOptions): Promis
   }
 }
 
-function cliOutput(result: ChatResult, pretty?: boolean): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error) process.exit(1)
-}
-
 export const chatCommand = new Command('chat')
   .description('UserChat management commands')
   .addCommand(

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { mkdir } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
@@ -86,7 +86,7 @@ describe('group commands', () => {
   })
 
   describe('listAction', () => {
-    test('lists all groups', async () => {
+    it('lists all groups', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await listAction({ _credManager: manager })
 
@@ -97,7 +97,7 @@ describe('group commands', () => {
   })
 
   describe('getAction', () => {
-    test('resolves group by ID', async () => {
+    it('resolves group by ID', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await getAction('grp1', { _credManager: manager })
 
@@ -106,7 +106,7 @@ describe('group commands', () => {
       expect(capturedResolveArg).toBe('grp1')
     })
 
-    test('resolves group by @name', async () => {
+    it('resolves group by @name', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await getAction('@team-alpha', { _credManager: manager })
 
@@ -116,7 +116,7 @@ describe('group commands', () => {
   })
 
   describe('messagesAction', () => {
-    test('gets messages from group', async () => {
+    it('gets messages from group', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await messagesAction('grp1', { _credManager: manager })
 
@@ -125,7 +125,7 @@ describe('group commands', () => {
       expect(result.messages?.[0].id).toBe('msg1')
     })
 
-    test('resolves group by @name before fetching messages', async () => {
+    it('resolves group by @name before fetching messages', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await messagesAction('@team-alpha', { _credManager: manager })
 

package/src/platforms/channeltalkbot/commands/group.ts

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 
-import { formatOutput } from '@/shared/utils/output'
+import { cliOutput } from '@/shared/utils/cli-output'
 
 import type { WorkspaceOption } from './shared'
 import { getClient } from './shared'
@@ -93,11 +93,6 @@ export async function messagesAction(group: string, options: GroupOptions): Prom
   }
 }
 
-function cliOutput(result: GroupResult, pretty?: boolean): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error) process.exit(1)
-}
-
 export const groupCommand = new Command('group')
   .description('Group management commands')
   .addCommand(

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { mkdir } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
@@ -48,7 +48,7 @@ describe('manager commands', () => {
   })
 
   describe('listAction', () => {
-    test('returns all managers with id, name, description', async () => {
+    it('returns all managers with id, name, description', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await listAction({ _credManager: manager })
 
@@ -60,7 +60,7 @@ describe('manager commands', () => {
   })
 
   describe('getAction', () => {
-    test('returns specific manager', async () => {
+    it('returns specific manager', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await getAction('mgr1', { _credManager: manager })
 

package/src/platforms/channeltalkbot/commands/manager.ts

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 
-import { formatOutput } from '@/shared/utils/output'
+import { cliOutput } from '@/shared/utils/cli-output'
 
 import type { WorkspaceOption } from './shared'
 import { getClient } from './shared'
@@ -68,11 +68,6 @@ export async function getAction(managerId: string, options: ManagerOptions): Pro
   }
 }
 
-function cliOutput(result: ManagerResult, pretty?: boolean): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error) process.exit(1)
-}
-
 export const managerCommand = new Command('manager')
   .description('Manager commands')
   .addCommand(

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { mkdir } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
@@ -113,7 +113,7 @@ describe('message commands', () => {
   })
 
   describe('sendAction', () => {
-    test('sends to userchat and wraps text in blocks', async () => {
+    it('sends to userchat and wraps text in blocks', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await sendAction('chat1', 'Hello world', { type: 'userchat', _credManager: manager })
 
@@ -122,7 +122,7 @@ describe('message commands', () => {
       expect(capturedSendUserChatArgs[1]).toEqual([{ type: 'text', value: 'Hello world' }])
     })
 
-    test('sends to group when type=group', async () => {
+    it('sends to group when type=group', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await sendAction('grp1', 'Hello group', { type: 'group', _credManager: manager })
 
@@ -131,14 +131,14 @@ describe('message commands', () => {
       expect(mockSendGroupMessage).toHaveBeenCalledTimes(1)
     })
 
-    test('includes botName in request when --bot provided', async () => {
+    it('includes botName in request when --bot provided', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await sendAction('chat1', 'Hello', { type: 'userchat', bot: 'my-bot', _credManager: manager })
 
       expect(capturedSendUserChatArgs[2]).toBe('my-bot')
     })
 
-    test('auto-detects group target from @ prefix', async () => {
+    it('auto-detects group target from @ prefix', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await sendAction('@team', 'Hello', { _credManager: manager })
 
@@ -147,7 +147,7 @@ describe('message commands', () => {
   })
 
   describe('listAction', () => {
-    test('lists userchat messages', async () => {
+    it('lists userchat messages', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await listAction('chat1', { type: 'userchat', _credManager: manager })
 
@@ -156,7 +156,7 @@ describe('message commands', () => {
       expect(result.messages?.[0].id).toBe('msg1')
     })
 
-    test('lists group messages when type=group', async () => {
+    it('lists group messages when type=group', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await listAction('grp1', { type: 'group', _credManager: manager })
 
@@ -165,7 +165,7 @@ describe('message commands', () => {
       expect(result.messages?.[0].id).toBe('msg2')
     })
 
-    test('passes pagination params', async () => {
+    it('passes pagination params', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       await listAction('chat1', {
         type: 'userchat',
@@ -181,7 +181,7 @@ describe('message commands', () => {
   })
 
   describe('getAction', () => {
-    test('returns specific message by ID', async () => {
+    it('returns specific message by ID', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await getAction('chat1', 'msg1', { type: 'userchat', _credManager: manager })
 
@@ -189,7 +189,7 @@ describe('message commands', () => {
       expect(result.id).toBe('msg1')
     })
 
-    test('returns error when message not found', async () => {
+    it('returns error when message not found', async () => {
       const manager = new ChannelBotCredentialManager(tempDir)
       const result = await getAction('chat1', 'nonexistent', { type: 'userchat', _credManager: manager })
 

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

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 
-import { formatOutput } from '@/shared/utils/output'
+import { cliOutput } from '@/shared/utils/cli-output'
 
 import { wrapTextInBlocks } from '../message-utils'
 import type { WorkspaceOption } from './shared'
@@ -140,11 +140,6 @@ function detectTargetType(target: string): 'userchat' | 'group' {
   return 'userchat'
 }
 
-function cliOutput(result: MessageResult, pretty?: boolean): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error) process.exit(1)
-}
-
 export const messageCommand = new Command('message')
   .description('Message commands')
   .addCommand(