agent-messenger 2.10.1 → 2.11.0

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

@@ -1,4 +1,4 @@
-import { afterAll, describe, expect, mock, test } from 'bun:test'
+import { afterAll, describe, expect, mock, it } from 'bun:test'
 import { rmSync } from 'node:fs'
 import { join } from 'node:path'
 
@@ -43,7 +43,7 @@ afterAll(() => {
 })
 
 describe('sendAction', () => {
-  test('sends text message and returns success', async () => {
+  it('sends text message and returns success', async () => {
     const credManager = await makeCredManagerWithCreds()
     const result = await sendAction('openid-123', 'Hello world', { _credManager: credManager })
 
@@ -51,7 +51,7 @@ describe('sendAction', () => {
     expect(sendTextMessageMock).toHaveBeenCalledWith('openid-123', 'Hello world')
   })
 
-  test('returns error when client throws', async () => {
+  it('returns error when client throws', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -71,7 +71,7 @@ describe('sendAction', () => {
 })
 
 describe('sendImageAction', () => {
-  test('sends image message and returns success', async () => {
+  it('sends image message and returns success', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -88,7 +88,7 @@ describe('sendImageAction', () => {
     expect(result.success).toBe(true)
   })
 
-  test('returns error when client throws', async () => {
+  it('returns error when client throws', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -107,7 +107,7 @@ describe('sendImageAction', () => {
 })
 
 describe('sendNewsAction', () => {
-  test('returns error when required options are missing', async () => {
+  it('returns error when required options are missing', async () => {
     const credManager = await makeCredManagerWithCreds()
     const result = await sendNewsAction('openid-123', { _credManager: credManager })
 
@@ -115,7 +115,7 @@ describe('sendNewsAction', () => {
     expect(result.success).toBeUndefined()
   })
 
-  test('returns error when title is missing', async () => {
+  it('returns error when title is missing', async () => {
     const credManager = await makeCredManagerWithCreds()
     const result = await sendNewsAction('openid-123', {
       description: 'Test desc',
@@ -127,7 +127,7 @@ describe('sendNewsAction', () => {
     expect(result.error).toContain('--title')
   })
 
-  test('sends news message with all required options', async () => {
+  it('sends news message with all required options', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {

package/src/platforms/wechatbot/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 type { WeChatBotNewsArticle } from '../types'
 import type { AccountOption } from './shared'
@@ -63,11 +63,6 @@ export async function sendNewsAction(openId: string, options: MessageOptions): P
   }
 }
 
-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(

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

@@ -1,4 +1,4 @@
-import { afterAll, describe, expect, mock, test } from 'bun:test'
+import { afterAll, describe, expect, mock, it } from 'bun:test'
 import { rmSync } from 'node:fs'
 import { join } from 'node:path'
 
@@ -49,7 +49,7 @@ afterAll(() => {
 })
 
 describe('listAction', () => {
-  test('returns templates list', async () => {
+  it('returns templates list', async () => {
     const credManager = await makeCredManagerWithCreds()
     const result = await listAction({ _credManager: credManager })
 
@@ -58,7 +58,7 @@ describe('listAction', () => {
     expect(result.templates?.[0].title).toBe('Order Notification')
   })
 
-  test('returns error when client throws', async () => {
+  it('returns error when client throws', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -78,7 +78,7 @@ describe('listAction', () => {
 })
 
 describe('sendAction (template)', () => {
-  test('sends template message and returns msgid', async () => {
+  it('sends template message and returns msgid', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -100,7 +100,7 @@ describe('sendAction (template)', () => {
     expect(result.error).toBeUndefined()
   })
 
-  test('returns error when data is invalid JSON', async () => {
+  it('returns error when data is invalid JSON', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -121,7 +121,7 @@ describe('sendAction (template)', () => {
     expect(result.msgid).toBeUndefined()
   })
 
-  test('sends without data when not provided', async () => {
+  it('sends without data when not provided', async () => {
     const sendTemplateMsg = mock(() => Promise.resolve({ msgid: 777 }))
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
@@ -139,7 +139,7 @@ describe('sendAction (template)', () => {
     expect(result.msgid).toBe(777)
   })
 
-  test('returns error when client throws', async () => {
+  it('returns error when client throws', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -158,7 +158,7 @@ describe('sendAction (template)', () => {
 })
 
 describe('deleteAction', () => {
-  test('deletes template and returns success', async () => {
+  it('deletes template and returns success', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -176,7 +176,7 @@ describe('deleteAction', () => {
     expect(result.error).toBeUndefined()
   })
 
-  test('returns error when client throws', async () => {
+  it('returns error when client throws', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {

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

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 
-import { formatOutput } from '@/shared/utils/output'
+import { cliOutput } from '@/shared/utils/cli-output'
 
 import type { WeChatBotTemplate } from '../types'
 import type { AccountOption } from './shared'
@@ -61,11 +61,6 @@ export async function deleteAction(templateId: string, options: TemplateOptions)
   }
 }
 
-function cliOutput(result: TemplateResult, pretty?: boolean): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error) process.exit(1)
-}
-
 export const templateCommand = new Command('template')
   .description('Template message commands')
   .addCommand(

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

@@ -1,4 +1,4 @@
-import { afterAll, describe, expect, mock, test } from 'bun:test'
+import { afterAll, describe, expect, mock, it } from 'bun:test'
 import { rmSync } from 'node:fs'
 import { join } from 'node:path'
 
@@ -56,7 +56,7 @@ afterAll(() => {
 })
 
 describe('listAction (user)', () => {
-  test('returns followers list with total, count, openids, next_openid', async () => {
+  it('returns followers list with total, count, openids, next_openid', async () => {
     const credManager = await makeCredManagerWithCreds()
     const result = await listAction({ _credManager: credManager })
 
@@ -66,7 +66,7 @@ describe('listAction (user)', () => {
     expect(result.next_openid).toBe('')
   })
 
-  test('returns error when client throws', async () => {
+  it('returns error when client throws', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -84,7 +84,7 @@ describe('listAction (user)', () => {
     expect(result.openids).toBeUndefined()
   })
 
-  test('passes nextOpenid option to client', async () => {
+  it('passes nextOpenid option to client', async () => {
     const getFollowersMock = mock(() => Promise.resolve({ total: 1, count: 1, openids: ['openid-3'], next_openid: '' }))
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
@@ -104,7 +104,7 @@ describe('listAction (user)', () => {
 })
 
 describe('getAction', () => {
-  test('returns user info for given openId', async () => {
+  it('returns user info for given openId', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -122,7 +122,7 @@ describe('getAction', () => {
     expect(result.user?.openid).toBe('openid-123')
   })
 
-  test('returns error when client throws', async () => {
+  it('returns error when client throws', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -140,7 +140,7 @@ describe('getAction', () => {
     expect(result.user).toBeUndefined()
   })
 
-  test('passes lang option to client', async () => {
+  it('passes lang option to client', async () => {
     const getUserInfoMock = mock(() => Promise.resolve({ ...mockUserInfo, language: 'en' }))
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {

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

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 
-import { formatOutput } from '@/shared/utils/output'
+import { cliOutput } from '@/shared/utils/cli-output'
 
 import type { WeChatBotUserInfo } from '../types'
 import type { AccountOption } from './shared'
@@ -45,11 +45,6 @@ export async function getAction(openId: string, options: UserOptions): Promise<U
   }
 }
 
-function cliOutput(result: UserResult, pretty?: boolean): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error) process.exit(1)
-}
-
 export const userCommand = new Command('user')
   .description('User management commands')
   .addCommand(

package/src/platforms/wechatbot/commands/whoami.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'
@@ -38,7 +38,7 @@ describe('whoami command', () => {
     process.env = originalEnv
   })
 
-  test('returns app_id, account_name, and verified status', async () => {
+  it('returns app_id, account_name, and verified status', async () => {
     const manager = new WeChatBotCredentialManager(tempDir)
     await manager.setCredentials({
       app_id: 'wx1234567890',
@@ -54,7 +54,7 @@ describe('whoami command', () => {
     expect(result.error).toBeUndefined()
   })
 
-  test('returns info for specific --account', async () => {
+  it('returns info for specific --account', async () => {
     const manager = new WeChatBotCredentialManager(tempDir)
     await manager.setCredentials({
       app_id: 'wxAAA',
@@ -74,7 +74,7 @@ describe('whoami command', () => {
     expect(result.verified).toBe(true)
   })
 
-  test('returns verified false when credentials are invalid', async () => {
+  it('returns verified false when credentials are invalid', async () => {
     mockVerifyCredentials.mockImplementationOnce(() => Promise.resolve(false))
 
     const manager = new WeChatBotCredentialManager(tempDir)
@@ -91,7 +91,7 @@ describe('whoami command', () => {
     expect(result.error).toBeUndefined()
   })
 
-  test('returns error when client throws', async () => {
+  it('returns error when client throws', async () => {
     mockVerifyCredentials.mockImplementationOnce(() => Promise.reject(new Error('Network error')))
 
     const manager = new WeChatBotCredentialManager(tempDir)

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

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 
-import { formatOutput } from '@/shared/utils/output'
+import { cliOutput } from '@/shared/utils/cli-output'
 
 import { WeChatBotCredentialManager } from '../credential-manager'
 import type { AccountOption } from './shared'
@@ -29,11 +29,6 @@ export async function whoamiAction(options: AccountOption): Promise<WhoamiResult
   }
 }
 
-function cliOutput(result: WhoamiResult, pretty?: boolean): void {
-  console.log(formatOutput(result, pretty))
-  if (result.error) process.exit(1)
-}
-
 export const whoamiCommand = new Command('whoami')
   .description('Show current authenticated bot')
   .option('--account <id>', 'Account ID to use')

package/src/platforms/wechatbot/credential-manager.test.ts

@@ -1,4 +1,4 @@
-import { afterAll, describe, expect, test } from 'bun:test'
+import { afterAll, describe, expect, it } from 'bun:test'
 import { existsSync, rmSync } from 'node:fs'
 import { join } from 'node:path'
 
@@ -22,7 +22,7 @@ afterAll(() => {
 })
 
 describe('WeChatBotCredentialManager', () => {
-  test('load returns default config when file does not exist', async () => {
+  it('load returns default config when file does not exist', async () => {
     const manager = setup()
     const config = await manager.load()
 
@@ -32,7 +32,7 @@ describe('WeChatBotCredentialManager', () => {
     })
   })
 
-  test('save creates file with correct content', async () => {
+  it('save creates file with correct content', async () => {
     const testConfigDir = join(
       import.meta.dir,
       `.test-wechatbot-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
@@ -61,13 +61,13 @@ describe('WeChatBotCredentialManager', () => {
     expect(loaded).toEqual(config)
   })
 
-  test('getCredentials returns null when not configured', async () => {
+  it('getCredentials returns null when not configured', async () => {
     const manager = setup()
     const creds = await manager.getCredentials()
     expect(creds).toBeNull()
   })
 
-  test('getCredentials returns credentials from env vars when E2E env vars are set', async () => {
+  it('getCredentials returns credentials from env vars when E2E env vars are set', async () => {
     const originalAppId = process.env.E2E_WECHATBOT_APP_ID
     const originalAppSecret = process.env.E2E_WECHATBOT_APP_SECRET
 
@@ -96,7 +96,7 @@ describe('WeChatBotCredentialManager', () => {
     }
   })
 
-  test('getCredentials ignores env vars when accountId is provided', async () => {
+  it('getCredentials ignores env vars when accountId is provided', async () => {
     const originalAppId = process.env.E2E_WECHATBOT_APP_ID
     const originalAppSecret = process.env.E2E_WECHATBOT_APP_SECRET
 
@@ -121,7 +121,7 @@ describe('WeChatBotCredentialManager', () => {
     }
   })
 
-  test('getCredentials returns specific account by accountId', async () => {
+  it('getCredentials returns specific account by accountId', async () => {
     const manager = setup()
     await manager.setCredentials({ app_id: 'wx-a', app_secret: 'secret-a', account_name: 'Account A' })
     await manager.setCredentials({ app_id: 'wx-b', app_secret: 'secret-b', account_name: 'Account B' })
@@ -134,13 +134,13 @@ describe('WeChatBotCredentialManager', () => {
     })
   })
 
-  test('getCredentials returns null for nonexistent accountId', async () => {
+  it('getCredentials returns null for nonexistent accountId', async () => {
     const manager = setup()
     const creds = await manager.getCredentials('nonexistent')
     expect(creds).toBeNull()
   })
 
-  test('setCredentials saves and sets as current', async () => {
+  it('setCredentials saves and sets as current', async () => {
     const manager = setup()
     await manager.setCredentials({ app_id: 'wx123', app_secret: 'secret123', account_name: 'My Account' })
 
@@ -155,7 +155,7 @@ describe('WeChatBotCredentialManager', () => {
     expect(config.current).toEqual({ account_id: 'wx123' })
   })
 
-  test('removeAccount deletes account and adjusts current', async () => {
+  it('removeAccount deletes account and adjusts current', async () => {
     const manager = setup()
     await manager.setCredentials({ app_id: 'wx123', app_secret: 'secret123', account_name: 'My Account' })
 
@@ -170,13 +170,13 @@ describe('WeChatBotCredentialManager', () => {
     expect(config.accounts['wx123']).toBeUndefined()
   })
 
-  test('removeAccount returns false for non-existent account', async () => {
+  it('removeAccount returns false for non-existent account', async () => {
     const manager = setup()
     const removed = await manager.removeAccount('nonexistent')
     expect(removed).toBe(false)
   })
 
-  test('removeAccount does not clear current when a different account is removed', async () => {
+  it('removeAccount does not clear current when a different account is removed', async () => {
     const manager = setup()
     await manager.setCredentials({ app_id: 'wx-a', app_secret: 'secret-a', account_name: 'Account A' })
     await manager.setCredentials({ app_id: 'wx-b', app_secret: 'secret-b', account_name: 'Account B' })
@@ -188,7 +188,7 @@ describe('WeChatBotCredentialManager', () => {
     expect(config.current).toEqual({ account_id: 'wx-b' })
   })
 
-  test('setCurrent switches active account', async () => {
+  it('setCurrent switches active account', async () => {
     const manager = setup()
     await manager.setCredentials({ app_id: 'wx-a', app_secret: 'secret-a', account_name: 'Account A' })
     await manager.setCredentials({ app_id: 'wx-b', app_secret: 'secret-b', account_name: 'Account B' })
@@ -200,13 +200,13 @@ describe('WeChatBotCredentialManager', () => {
     expect(config.current).toEqual({ account_id: 'wx-a' })
   })
 
-  test('setCurrent returns false for non-existent account', async () => {
+  it('setCurrent returns false for non-existent account', async () => {
     const manager = setup()
     const result = await manager.setCurrent('nonexistent')
     expect(result).toBe(false)
   })
 
-  test('listAll returns all accounts with is_current flag', async () => {
+  it('listAll returns all accounts with is_current flag', async () => {
     const manager = setup()
     await manager.setCredentials({ app_id: 'wx-a', app_secret: 'secret-a', account_name: 'Account A' })
     await manager.setCredentials({ app_id: 'wx-b', app_secret: 'secret-b', account_name: 'Account B' })
@@ -221,13 +221,13 @@ describe('WeChatBotCredentialManager', () => {
     expect(acctB?.is_current).toBe(true)
   })
 
-  test('listAll returns empty array when no accounts', async () => {
+  it('listAll returns empty array when no accounts', async () => {
     const manager = setup()
     const accounts = await manager.listAll()
     expect(accounts).toHaveLength(0)
   })
 
-  test('clearCredentials resets everything', async () => {
+  it('clearCredentials resets everything', async () => {
     const manager = setup()
     await manager.setCredentials({ app_id: 'wx123', app_secret: 'secret123', account_name: 'My Account' })
 
@@ -238,7 +238,7 @@ describe('WeChatBotCredentialManager', () => {
     expect(config.accounts).toEqual({})
   })
 
-  test('round-trip: set → get → remove → get null', async () => {
+  it('supports round-trip: set, get, remove, then returns null', async () => {
     const manager = setup()
 
     await manager.setCredentials({ app_id: 'wx123', app_secret: 'secret123', account_name: 'My Account' })

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

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   WeChatBotAccountEntrySchema,
@@ -12,38 +12,38 @@ import {
   WeChatBotUserInfoSchema,
 } from '@/platforms/wechatbot/index'
 
-test('WeChatBotClient is exported from barrel', () => {
+it('WeChatBotClient is exported from barrel', () => {
   expect(typeof WeChatBotClient).toBe('function')
 })
 
-test('WeChatBotCredentialManager is exported from barrel', () => {
+it('WeChatBotCredentialManager is exported from barrel', () => {
   expect(typeof WeChatBotCredentialManager).toBe('function')
 })
 
-test('WeChatBotError is exported from barrel', () => {
+it('WeChatBotError is exported from barrel', () => {
   expect(typeof WeChatBotError).toBe('function')
 })
 
-test('WeChatBotAccountEntrySchema is exported from barrel', () => {
+it('WeChatBotAccountEntrySchema is exported from barrel', () => {
   expect(typeof WeChatBotAccountEntrySchema.parse).toBe('function')
 })
 
-test('WeChatBotConfigSchema is exported from barrel', () => {
+it('WeChatBotConfigSchema is exported from barrel', () => {
   expect(typeof WeChatBotConfigSchema.parse).toBe('function')
 })
 
-test('WeChatBotCredentialsSchema is exported from barrel', () => {
+it('WeChatBotCredentialsSchema is exported from barrel', () => {
   expect(typeof WeChatBotCredentialsSchema.parse).toBe('function')
 })
 
-test('WeChatBotNewsArticleSchema is exported from barrel', () => {
+it('WeChatBotNewsArticleSchema is exported from barrel', () => {
   expect(typeof WeChatBotNewsArticleSchema.parse).toBe('function')
 })
 
-test('WeChatBotTemplateSchema is exported from barrel', () => {
+it('WeChatBotTemplateSchema is exported from barrel', () => {
   expect(typeof WeChatBotTemplateSchema.parse).toBe('function')
 })
 
-test('WeChatBotUserInfoSchema is exported from barrel', () => {
+it('WeChatBotUserInfoSchema is exported from barrel', () => {
   expect(typeof WeChatBotUserInfoSchema.parse).toBe('function')
 })