agent-messenger 2.10.1 → 2.11.0

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

@@ -4,6 +4,8 @@ import { join } from 'node:path'
 
 import { ClassicLevel } from 'classic-level'
 
+import { getAgentBrowserProfileDirs } from '@/shared/chromium'
+
 export interface ExtractedWebexToken {
   accessToken: string
   refreshToken?: string
@@ -74,11 +76,18 @@ export class WebexTokenExtractor {
   private platform: NodeJS.Platform
   private baseDir: string | null
   private debugLog: ((message: string) => void) | null
-
-  constructor(platform?: NodeJS.Platform, debugLog?: (message: string) => void, baseDir?: string) {
+  private customBrowserProfileDirs: string[]
+
+  constructor(
+    platform?: NodeJS.Platform,
+    debugLog?: (message: string) => void,
+    baseDir?: string,
+    customBrowserProfileDirs?: string[],
+  ) {
     this.platform = platform ?? process.platform
     this.debugLog = debugLog ?? null
     this.baseDir = baseDir ?? null
+    this.customBrowserProfileDirs = customBrowserProfileDirs ?? []
   }
 
   private debug(message: string): void {
@@ -87,7 +96,7 @@ export class WebexTokenExtractor {
 
   getBrowserProfileDirs(): string[] {
     if (this.baseDir) {
-      return this.discoverProfileDirs(this.baseDir)
+      return [...this.discoverProfileDirs(this.baseDir), ...this.getCustomBrowserProfileLevelDBDirs()]
     }
 
     const dirs: string[] = []
@@ -100,6 +109,19 @@ export class WebexTokenExtractor {
       dirs.push(...profileDirs)
     }
 
+    dirs.push(...this.getCustomBrowserProfileLevelDBDirs())
+
+    return dirs
+  }
+
+  private getCustomBrowserProfileLevelDBDirs(): string[] {
+    const dirs: string[] = []
+    for (const profileDir of getAgentBrowserProfileDirs({ customProfileDirs: this.customBrowserProfileDirs })) {
+      const leveldb = join(profileDir, 'Local Storage', 'leveldb')
+      if (existsSync(leveldb)) {
+        dirs.push(leveldb)
+      }
+    }
     return dirs
   }
 

package/src/platforms/webex/types.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   WebexConfigSchema,
@@ -9,7 +9,7 @@ import {
   WebexSpaceSchema,
 } from './types'
 
-test('WebexSpaceSchema validates valid space', () => {
+it('WebexSpaceSchema validates valid space', () => {
   const result = WebexSpaceSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
     title: 'Project Alpha',
@@ -22,7 +22,7 @@ test('WebexSpaceSchema validates valid space', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexSpaceSchema validates space with optional teamId', () => {
+it('WebexSpaceSchema validates space with optional teamId', () => {
   const result = WebexSpaceSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
     title: 'Project Alpha',
@@ -36,7 +36,7 @@ test('WebexSpaceSchema validates space with optional teamId', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexSpaceSchema validates direct space type', () => {
+it('WebexSpaceSchema validates direct space type', () => {
   const result = WebexSpaceSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL1JPT00vZGlyZWN0',
     title: 'Direct Message',
@@ -49,7 +49,7 @@ test('WebexSpaceSchema validates direct space type', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexSpaceSchema rejects missing required fields', () => {
+it('WebexSpaceSchema rejects missing required fields', () => {
   const result = WebexSpaceSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
     title: 'Project Alpha',
@@ -57,7 +57,7 @@ test('WebexSpaceSchema rejects missing required fields', () => {
   expect(result.success).toBe(false)
 })
 
-test('WebexSpaceSchema rejects invalid type', () => {
+it('WebexSpaceSchema rejects invalid type', () => {
   const result = WebexSpaceSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
     title: 'Project Alpha',
@@ -70,7 +70,7 @@ test('WebexSpaceSchema rejects invalid type', () => {
   expect(result.success).toBe(false)
 })
 
-test('WebexMessageSchema validates valid message', () => {
+it('WebexMessageSchema validates valid message', () => {
   const result = WebexMessageSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL01FU1NBR0UvbXNn',
     roomId: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
@@ -83,7 +83,7 @@ test('WebexMessageSchema validates valid message', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexMessageSchema validates message with optional fields', () => {
+it('WebexMessageSchema validates message with optional fields', () => {
   const result = WebexMessageSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL01FU1NBR0UvbXNn',
     roomId: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
@@ -101,7 +101,7 @@ test('WebexMessageSchema validates message with optional fields', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexMessageSchema rejects missing required fields', () => {
+it('WebexMessageSchema rejects missing required fields', () => {
   const result = WebexMessageSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL01FU1NBR0UvbXNn',
     roomId: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
@@ -110,7 +110,7 @@ test('WebexMessageSchema rejects missing required fields', () => {
   expect(result.success).toBe(false)
 })
 
-test('WebexMessageSchema rejects invalid roomType', () => {
+it('WebexMessageSchema rejects invalid roomType', () => {
   const result = WebexMessageSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL01FU1NBR0UvbXNn',
     roomId: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
@@ -122,7 +122,7 @@ test('WebexMessageSchema rejects invalid roomType', () => {
   expect(result.success).toBe(false)
 })
 
-test('WebexPersonSchema validates valid person', () => {
+it('WebexPersonSchema validates valid person', () => {
   const result = WebexPersonSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL1BFT1BMRS9hYmM',
     emails: ['user@example.com'],
@@ -134,7 +134,7 @@ test('WebexPersonSchema validates valid person', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexPersonSchema validates person with optional fields', () => {
+it('WebexPersonSchema validates person with optional fields', () => {
   const result = WebexPersonSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL1BFT1BMRS9hYmM',
     emails: ['user@example.com', 'user@work.com'],
@@ -150,7 +150,7 @@ test('WebexPersonSchema validates person with optional fields', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexPersonSchema validates bot type', () => {
+it('WebexPersonSchema validates bot type', () => {
   const result = WebexPersonSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL1BFT1BMRS9ib3Q',
     emails: ['bot@webex.bot'],
@@ -162,7 +162,7 @@ test('WebexPersonSchema validates bot type', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexPersonSchema rejects missing required fields', () => {
+it('WebexPersonSchema rejects missing required fields', () => {
   const result = WebexPersonSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL1BFT1BMRS9hYmM',
     displayName: 'Test User',
@@ -170,7 +170,7 @@ test('WebexPersonSchema rejects missing required fields', () => {
   expect(result.success).toBe(false)
 })
 
-test('WebexPersonSchema rejects invalid type', () => {
+it('WebexPersonSchema rejects invalid type', () => {
   const result = WebexPersonSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL1BFT1BMRS9hYmM',
     emails: ['user@example.com'],
@@ -182,7 +182,7 @@ test('WebexPersonSchema rejects invalid type', () => {
   expect(result.success).toBe(false)
 })
 
-test('WebexMembershipSchema validates valid membership', () => {
+it('WebexMembershipSchema validates valid membership', () => {
   const result = WebexMembershipSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL01FTUJFUlNISVAvbWVt',
     roomId: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
@@ -195,7 +195,7 @@ test('WebexMembershipSchema validates valid membership', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexMembershipSchema validates moderator membership', () => {
+it('WebexMembershipSchema validates moderator membership', () => {
   const result = WebexMembershipSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL01FTUJFUlNISVAvbWVt',
     roomId: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
@@ -208,7 +208,7 @@ test('WebexMembershipSchema validates moderator membership', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexMembershipSchema rejects missing required fields', () => {
+it('WebexMembershipSchema rejects missing required fields', () => {
   const result = WebexMembershipSchema.safeParse({
     id: 'Y2lzY29zcGFyazovL3VzL01FTUJFUlNISVAvbWVt',
     roomId: 'Y2lzY29zcGFyazovL3VzL1JPT00vYWJj',
@@ -216,7 +216,7 @@ test('WebexMembershipSchema rejects missing required fields', () => {
   expect(result.success).toBe(false)
 })
 
-test('WebexConfigSchema validates valid OAuth config', () => {
+it('WebexConfigSchema validates valid OAuth config', () => {
   const result = WebexConfigSchema.safeParse({
     accessToken: 'test',
     refreshToken: 'test',
@@ -225,7 +225,7 @@ test('WebexConfigSchema validates valid OAuth config', () => {
   expect(result.success).toBe(true)
 })
 
-test('WebexConfigSchema validates config with clientId and clientSecret', () => {
+it('WebexConfigSchema validates config with clientId and clientSecret', () => {
   const result = WebexConfigSchema.safeParse({
     accessToken: 'test',
     refreshToken: 'test',
@@ -240,7 +240,7 @@ test('WebexConfigSchema validates config with clientId and clientSecret', () =>
   }
 })
 
-test('WebexConfigSchema validates config with tokenType oauth', () => {
+it('WebexConfigSchema validates config with tokenType oauth', () => {
   const result = WebexConfigSchema.safeParse({
     accessToken: 'test',
     refreshToken: 'test',
@@ -253,7 +253,7 @@ test('WebexConfigSchema validates config with tokenType oauth', () => {
   }
 })
 
-test('WebexConfigSchema validates config with tokenType manual', () => {
+it('WebexConfigSchema validates config with tokenType manual', () => {
   const result = WebexConfigSchema.safeParse({
     accessToken: 'test',
     refreshToken: '',
@@ -266,7 +266,7 @@ test('WebexConfigSchema validates config with tokenType manual', () => {
   }
 })
 
-test('WebexConfigSchema rejects invalid tokenType', () => {
+it('WebexConfigSchema rejects invalid tokenType', () => {
   const result = WebexConfigSchema.safeParse({
     accessToken: 'test',
     refreshToken: 'test',
@@ -276,7 +276,7 @@ test('WebexConfigSchema rejects invalid tokenType', () => {
   expect(result.success).toBe(false)
 })
 
-test('WebexConfigSchema accepts config without clientId/clientSecret (backward compat)', () => {
+it('WebexConfigSchema accepts config without clientId/clientSecret (backward compat)', () => {
   const result = WebexConfigSchema.safeParse({
     accessToken: 'test',
     refreshToken: 'test',
@@ -289,19 +289,19 @@ test('WebexConfigSchema accepts config without clientId/clientSecret (backward c
   }
 })
 
-test('WebexConfigSchema rejects missing fields', () => {
+it('WebexConfigSchema rejects missing fields', () => {
   const result = WebexConfigSchema.safeParse({})
   expect(result.success).toBe(false)
 })
 
-test('WebexError has correct name and code', () => {
+it('WebexError has correct name and code', () => {
   const error = new WebexError('Not found', 'NOT_FOUND')
   expect(error.name).toBe('WebexError')
   expect(error.message).toBe('Not found')
   expect(error.code).toBe('NOT_FOUND')
 })
 
-test('WebexError is instance of Error', () => {
+it('WebexError is instance of Error', () => {
   const error = new WebexError('Unauthorized', 'UNAUTHORIZED')
   expect(error instanceof Error).toBe(true)
 })

package/src/platforms/wechatbot/cli.ts

@@ -13,6 +13,15 @@ program
   .version(pkg.version)
   .option('--pretty', 'Pretty-print JSON output')
   .option('--account <id>', 'Account ID to use')
+  .hook('preAction', (thisCmd, actionCmd) => {
+    for (const [key, value] of Object.entries(thisCmd.opts())) {
+      if (value === undefined) continue
+      const source = actionCmd.getOptionValueSource(key)
+      if (source === undefined || source === 'default') {
+        actionCmd.setOptionValue(key, value)
+      }
+    }
+  })
 
 program.addCommand(authCommand)
 program.addCommand(whoamiCommand)

package/src/platforms/wechatbot/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 { WeChatBotClient } from '@/platforms/wechatbot/client'
 import { WeChatBotError } from '@/platforms/wechatbot/types'
@@ -48,28 +48,28 @@ describe('WeChatBotClient', () => {
   const tokenResponse = () => mockResponse({ access_token: 'test-token', expires_in: 7200 })
 
   describe('login', () => {
-    test('throws on empty appId', async () => {
+    it('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',
       )
     })
 
-    test('throws on empty appSecret', async () => {
+    it('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',
       )
     })
 
-    test('accepts valid credentials and returns client', async () => {
+    it('accepts valid credentials and returns client', async () => {
       const client = await new WeChatBotClient().login({ appId: 'wx123', appSecret: 'secret123' })
       expect(client).toBeInstanceOf(WeChatBotClient)
     })
   })
 
   describe('verifyCredentials', () => {
-    test('calls token endpoint and returns true on success', async () => {
+    it('calls token endpoint and returns true on success', async () => {
       tokenResponse()
 
       const client = await new WeChatBotClient().login({ appId: 'wx123', appSecret: 'secret123' })
@@ -82,7 +82,7 @@ describe('WeChatBotClient', () => {
       expect(fetchCalls[0].url).toContain('secret=secret123')
     })
 
-    test('returns false on token error', async () => {
+    it('returns false on token error', async () => {
       mockResponse({ errcode: 40125, errmsg: 'invalid appsecret' })
 
       const client = await new WeChatBotClient().login({ appId: 'wx123', appSecret: 'bad-secret' })
@@ -91,7 +91,7 @@ describe('WeChatBotClient', () => {
       expect(result).toBe(false)
     })
 
-    test('returns false on network error', async () => {
+    it('returns false on network error', async () => {
       fetchResponses = []
       Object.defineProperty(globalThis, 'fetch', {
         value: async (): Promise<Response> => {
@@ -109,7 +109,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('sendTextMessage', () => {
-    test('sends POST to /cgi-bin/message/custom/send with token in query', async () => {
+    it('sends POST to /cgi-bin/message/custom/send with token in query', async () => {
       tokenResponse()
       mockResponse({ errcode: 0, errmsg: 'ok' })
 
@@ -123,7 +123,7 @@ describe('WeChatBotClient', () => {
       expect(call.options?.method).toBe('POST')
     })
 
-    test('sends correct body shape', async () => {
+    it('sends correct body shape', async () => {
       tokenResponse()
       mockResponse({ errcode: 0, errmsg: 'ok' })
 
@@ -140,7 +140,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('sendImageMessage', () => {
-    test('sends POST with image payload', async () => {
+    it('sends POST with image payload', async () => {
       tokenResponse()
       mockResponse({ errcode: 0, errmsg: 'ok' })
 
@@ -157,7 +157,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('sendNewsMessage', () => {
-    test('sends POST with news/articles payload', async () => {
+    it('sends POST with news/articles payload', async () => {
       tokenResponse()
       mockResponse({ errcode: 0, errmsg: 'ok' })
 
@@ -182,7 +182,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('sendTemplateMessage', () => {
-    test('sends POST to /cgi-bin/message/template/send and returns msgid', async () => {
+    it('sends POST to /cgi-bin/message/template/send and returns msgid', async () => {
       tokenResponse()
       mockResponse({ errcode: 0, errmsg: 'ok', msgid: 12345 })
 
@@ -208,7 +208,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('listTemplates', () => {
-    test('sends GET and unwraps template_list from response', async () => {
+    it('sends GET and unwraps template_list from response', async () => {
       tokenResponse()
       mockResponse({
         errcode: 0,
@@ -236,7 +236,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('deleteTemplate', () => {
-    test('sends POST with template_id body', async () => {
+    it('sends POST with template_id body', async () => {
       tokenResponse()
       mockResponse({ errcode: 0, errmsg: 'ok' })
 
@@ -250,7 +250,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('getFollowers', () => {
-    test('sends GET to /cgi-bin/user/get and returns openids array', async () => {
+    it('sends GET to /cgi-bin/user/get and returns openids array', async () => {
       tokenResponse()
       mockResponse({
         errcode: 0,
@@ -270,7 +270,7 @@ describe('WeChatBotClient', () => {
       expect(fetchCalls[1].url).toContain('/cgi-bin/user/get')
     })
 
-    test('passes next_openid parameter when provided', async () => {
+    it('passes next_openid parameter when provided', async () => {
       tokenResponse()
       mockResponse({
         total: 1,
@@ -285,7 +285,7 @@ describe('WeChatBotClient', () => {
       expect(fetchCalls[1].url).toContain('next_openid=openid-2')
     })
 
-    test('returns empty openids when data is missing', async () => {
+    it('returns empty openids when data is missing', async () => {
       tokenResponse()
       mockResponse({
         total: 0,
@@ -301,7 +301,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('getUserInfo', () => {
-    test('sends GET with openid and lang params', async () => {
+    it('sends GET with openid and lang params', async () => {
       tokenResponse()
       mockResponse({
         subscribe: 1,
@@ -325,7 +325,7 @@ describe('WeChatBotClient', () => {
       expect(fetchCalls[1].url).toContain('lang=zh_CN')
     })
 
-    test('defaults to zh_CN lang', async () => {
+    it('defaults to zh_CN lang', async () => {
       tokenResponse()
       mockResponse({
         subscribe: 1,
@@ -347,7 +347,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('token caching', () => {
-    test('second call does not re-fetch token if not expired', async () => {
+    it('second call does not re-fetch token if not expired', async () => {
       tokenResponse()
       mockResponse({ errcode: 0, errmsg: 'ok' })
       mockResponse({ errcode: 0, errmsg: 'ok' })
@@ -364,7 +364,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('token auto-refresh on 40001', () => {
-    test('fetches new token and retries on errcode 40001', async () => {
+    it('fetches new token and retries on errcode 40001', async () => {
       tokenResponse()
       mockResponse({ errcode: 40001, errmsg: 'invalid credential' })
       mockResponse({ access_token: 'new-token', expires_in: 7200 })
@@ -381,7 +381,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('retry on system busy (errcode -1)', () => {
-    test('retries with backoff on errcode -1', async () => {
+    it('retries with backoff on errcode -1', async () => {
       tokenResponse()
       mockResponse({ errcode: -1, errmsg: 'system busy' })
       mockResponse({ errcode: -1, errmsg: 'system busy' })
@@ -401,7 +401,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('network error retry', () => {
-    test('GET retries on fetch throw', async () => {
+    it('GET retries on fetch throw', async () => {
       let callCount = 0
       Object.defineProperty(globalThis, 'fetch', {
         value: async (url: string | URL | Request, options?: RequestInit): Promise<Response> => {
@@ -434,7 +434,7 @@ describe('WeChatBotClient', () => {
       expect(callCount).toBe(4)
     })
 
-    test('POST does not retry on fetch throw', async () => {
+    it('POST does not retry on fetch throw', async () => {
       let callCount = 0
       Object.defineProperty(globalThis, 'fetch', {
         value: async (url: string | URL | Request, options?: RequestInit): Promise<Response> => {
@@ -456,7 +456,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('errcode handling', () => {
-    test('non-zero errcode throws WeChatBotError with correct code', async () => {
+    it('non-zero errcode throws WeChatBotError with correct code', async () => {
       tokenResponse()
       mockResponse({ errcode: 48001, errmsg: 'api unauthorized' })
 
@@ -472,7 +472,7 @@ describe('WeChatBotClient', () => {
       }
     })
 
-    test('errcode 0 does not throw', async () => {
+    it('errcode 0 does not throw', async () => {
       tokenResponse()
       mockResponse({ errcode: 0, errmsg: 'ok' })
 
@@ -482,7 +482,7 @@ describe('WeChatBotClient', () => {
   })
 
   describe('rate limit (errcode 45009)', () => {
-    test('throws WeChatBotError with code 45009 and appropriate message', async () => {
+    it('throws WeChatBotError with code 45009 and appropriate message', async () => {
       tokenResponse()
       mockResponse({ errcode: 45009, errmsg: 'reach max api daily quota limit' })
 

package/src/platforms/wechatbot/commands/auth.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'
 
@@ -31,7 +31,7 @@ afterAll(() => {
 })
 
 describe('setAction', () => {
-  test('returns success with app_id when credentials are valid', async () => {
+  it('returns success with app_id when credentials are valid', async () => {
     const credManager = makeCredManager()
     const result = await setAction('wx123', 'secret123', { _credManager: credManager })
 
@@ -40,7 +40,7 @@ describe('setAction', () => {
     expect(result.account_name).toBe('wx123')
   })
 
-  test('saves credentials to credManager', async () => {
+  it('saves credentials to credManager', async () => {
     const credManager = makeCredManager()
     await setAction('wx456', 'secret456', { _credManager: credManager })
 
@@ -48,7 +48,7 @@ describe('setAction', () => {
     expect(creds?.app_id).toBe('wx456')
   })
 
-  test('returns error when verifyCredentials returns false', async () => {
+  it('returns error when verifyCredentials returns false', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -68,7 +68,7 @@ describe('setAction', () => {
 })
 
 describe('statusAction', () => {
-  test('returns valid: false when no credentials configured', async () => {
+  it('returns valid: false when no credentials configured', async () => {
     const credManager = makeCredManager()
     const result = await statusAction({ _credManager: credManager })
 
@@ -76,7 +76,7 @@ describe('statusAction', () => {
     expect(result.error).toBeDefined()
   })
 
-  test('returns valid: true when credentials exist and are valid', async () => {
+  it('returns valid: true when credentials exist and are valid', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -96,7 +96,7 @@ describe('statusAction', () => {
     expect(result.app_id).toBe('wx123')
   })
 
-  test('returns valid: false when credentials exist but are invalid', async () => {
+  it('returns valid: false when credentials exist but are invalid', async () => {
     mock.module('../client', () => ({
       WeChatBotClient: class MockWeChatBotClient {
         async login() {
@@ -115,7 +115,7 @@ describe('statusAction', () => {
     expect(result.valid).toBe(false)
   })
 
-  test('returns error message when account not found', async () => {
+  it('returns error message when account not found', async () => {
     const credManager = makeCredManager()
     const result = await statusAction({ account: 'nonexistent', _credManager: credManager })
 
@@ -125,7 +125,7 @@ describe('statusAction', () => {
 })
 
 describe('clearAction', () => {
-  test('clears all credentials and returns success', async () => {
+  it('clears all credentials and returns success', async () => {
     const credManager = makeCredManager()
     await credManager.setCredentials({ app_id: 'wx123', app_secret: 'secret123', account_name: 'My Account' })
 
@@ -139,14 +139,14 @@ describe('clearAction', () => {
 })
 
 describe('listAction', () => {
-  test('returns empty accounts array when none configured', async () => {
+  it('returns empty accounts array when none configured', async () => {
     const credManager = makeCredManager()
     const result = await listAction({ _credManager: credManager })
 
     expect(result.accounts).toEqual([])
   })
 
-  test('returns all accounts with is_current flag', async () => {
+  it('returns all accounts with is_current flag', async () => {
     const credManager = makeCredManager()
     await credManager.setCredentials({ app_id: 'wx-a', app_secret: 'secret-a', account_name: 'Account A' })
     await credManager.setCredentials({ app_id: 'wx-b', app_secret: 'secret-b', account_name: 'Account B' })
@@ -162,7 +162,7 @@ describe('listAction', () => {
 })
 
 describe('useAction', () => {
-  test('switches to specified account and returns success', async () => {
+  it('switches to specified account and returns success', async () => {
     const credManager = makeCredManager()
     await credManager.setCredentials({ app_id: 'wx-a', app_secret: 'secret-a', account_name: 'Account A' })
     await credManager.setCredentials({ app_id: 'wx-b', app_secret: 'secret-b', account_name: 'Account B' })
@@ -173,7 +173,7 @@ describe('useAction', () => {
     expect(result.app_id).toBe('wx-a')
   })
 
-  test('returns error when account not found', async () => {
+  it('returns error when account not found', async () => {
     const credManager = makeCredManager()
     const result = await useAction('nonexistent', { _credManager: credManager })
 
@@ -184,7 +184,7 @@ describe('useAction', () => {
 })
 
 describe('removeAction', () => {
-  test('removes specified account and returns success', async () => {
+  it('removes specified account and returns success', async () => {
     const credManager = makeCredManager()
     await credManager.setCredentials({ app_id: 'wx123', app_secret: 'secret123', account_name: 'My Account' })
 
@@ -196,7 +196,7 @@ describe('removeAction', () => {
     expect(creds).toBeNull()
   })
 
-  test('returns error when account not found', async () => {
+  it('returns error when account not found', async () => {
     const credManager = makeCredManager()
     const result = await removeAction('nonexistent', { _credManager: credManager })
 

package/src/platforms/wechatbot/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 { WeChatBotClient } from '../client'
@@ -139,11 +140,6 @@ export async function removeAction(accountId: string, options: ActionOptions): P
   }
 }
 
-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(