agent-messenger 2.10.1 → 2.10.2

package/src/platforms/instagram/token-extractor.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, spyOn, test } from 'bun:test'
+import { beforeEach, describe, expect, spyOn, it } from 'bun:test'
 import { homedir } from 'node:os'
 import { join } from 'node:path'
 
@@ -12,7 +12,7 @@ describe('InstagramTokenExtractor', () => {
   })
 
   describe('getBrowserCookiesPaths', () => {
-    test('returns darwin paths for all browsers on macOS', () => {
+    it('returns darwin paths for all browsers on macOS', () => {
       const darwinExtractor = new InstagramTokenExtractor('darwin')
       const paths = darwinExtractor.getBrowserCookiesPaths()
 
@@ -26,7 +26,7 @@ describe('InstagramTokenExtractor', () => {
       expect(paths).toContain(join(base, 'Chromium', 'Default', 'Cookies'))
     })
 
-    test('returns linux paths for supported browsers', () => {
+    it('returns linux paths for supported browsers', () => {
       const linuxExtractor = new InstagramTokenExtractor('linux')
       const paths = linuxExtractor.getBrowserCookiesPaths()
 
@@ -38,7 +38,7 @@ describe('InstagramTokenExtractor', () => {
       expect(paths).toContain(join(base, 'chromium', 'Default', 'Cookies'))
     })
 
-    test('returns win32 paths for all browsers on Windows', () => {
+    it('returns win32 paths for all browsers on Windows', () => {
       const winExtractor = new InstagramTokenExtractor('win32')
       const paths = winExtractor.getBrowserCookiesPaths()
 
@@ -48,13 +48,13 @@ describe('InstagramTokenExtractor', () => {
       expect(paths).toContain(join(localAppData, 'BraveSoftware', 'Brave-Browser', 'User Data', 'Default', 'Cookies'))
     })
 
-    test('returns empty array for unsupported platform', () => {
+    it('returns empty array for unsupported platform', () => {
       const unsupportedExtractor = new InstagramTokenExtractor('freebsd' as NodeJS.Platform)
       const paths = unsupportedExtractor.getBrowserCookiesPaths()
       expect(paths).toEqual([])
     })
 
-    test('includes Network/Cookies variant paths', () => {
+    it('includes Network/Cookies variant paths', () => {
       const darwinExtractor = new InstagramTokenExtractor('darwin')
       const paths = darwinExtractor.getBrowserCookiesPaths()
 
@@ -64,7 +64,7 @@ describe('InstagramTokenExtractor', () => {
   })
 
   describe('getLocalStatePaths', () => {
-    test('returns darwin Local State paths for all browsers', () => {
+    it('returns darwin Local State paths for all browsers', () => {
       const darwinExtractor = new InstagramTokenExtractor('darwin')
       const paths = darwinExtractor.getLocalStatePaths()
 
@@ -74,7 +74,7 @@ describe('InstagramTokenExtractor', () => {
       expect(paths).toContain(join(base, 'BraveSoftware', 'Brave-Browser', 'Local State'))
     })
 
-    test('returns linux Local State paths for supported browsers', () => {
+    it('returns linux Local State paths for supported browsers', () => {
       const linuxExtractor = new InstagramTokenExtractor('linux')
       const paths = linuxExtractor.getLocalStatePaths()
 
@@ -83,7 +83,7 @@ describe('InstagramTokenExtractor', () => {
       expect(paths).toContain(join(base, 'microsoft-edge', 'Local State'))
     })
 
-    test('returns win32 Local State paths for all browsers', () => {
+    it('returns win32 Local State paths for all browsers', () => {
       const winExtractor = new InstagramTokenExtractor('win32')
       const paths = winExtractor.getLocalStatePaths()
 
@@ -94,7 +94,7 @@ describe('InstagramTokenExtractor', () => {
   })
 
   describe('getKeychainVariants', () => {
-    test('returns all Chromium browser keychain variants', () => {
+    it('returns all Chromium browser keychain variants', () => {
       expect(extractor.getKeychainVariants()).toEqual([
         { service: 'Chrome Safe Storage', account: 'Chrome' },
         { service: 'Chrome Canary Safe Storage', account: 'Chrome Canary' },
@@ -108,59 +108,59 @@ describe('InstagramTokenExtractor', () => {
   })
 
   describe('isEncryptedValue', () => {
-    test('detects v10 encrypted values', () => {
+    it('detects v10 encrypted values', () => {
       const encrypted = Buffer.from('v10encrypted_data')
       expect(extractor.isEncryptedValue(encrypted)).toBe(true)
     })
 
-    test('detects v11 encrypted values', () => {
+    it('detects v11 encrypted values', () => {
       const encrypted = Buffer.from('v11encrypted_data')
       expect(extractor.isEncryptedValue(encrypted)).toBe(true)
     })
 
-    test('rejects non-encrypted values', () => {
+    it('rejects non-encrypted values', () => {
       const plain = Buffer.from('plain_text')
       expect(extractor.isEncryptedValue(plain)).toBe(false)
     })
 
-    test('rejects empty buffers', () => {
+    it('rejects empty buffers', () => {
       const empty = Buffer.alloc(0)
       expect(extractor.isEncryptedValue(empty)).toBe(false)
     })
 
-    test('rejects short buffers under 4 bytes', () => {
+    it('rejects short buffers under 4 bytes', () => {
       const short = Buffer.from('v1')
       expect(extractor.isEncryptedValue(short)).toBe(false)
     })
   })
 
   describe('isValidSessionId', () => {
-    test('accepts session IDs of 20 or more chars', () => {
+    it('accepts session IDs of 20 or more chars', () => {
       const valid = 'abcdefghijklmnopqrstu'
       expect(extractor.isValidSessionId(valid)).toBe(true)
     })
 
-    test('accepts long session IDs', () => {
+    it('accepts long session IDs', () => {
       const valid = 'a'.repeat(100)
       expect(extractor.isValidSessionId(valid)).toBe(true)
     })
 
-    test('rejects session IDs shorter than 20 chars', () => {
+    it('rejects session IDs shorter than 20 chars', () => {
       expect(extractor.isValidSessionId('short')).toBe(false)
     })
 
-    test('rejects empty string', () => {
+    it('rejects empty string', () => {
       expect(extractor.isValidSessionId('')).toBe(false)
     })
 
-    test('rejects null and undefined', () => {
+    it('rejects null and undefined', () => {
       expect(extractor.isValidSessionId(null as unknown as string)).toBe(false)
       expect(extractor.isValidSessionId(undefined as unknown as string)).toBe(false)
     })
   })
 
   describe('extract', () => {
-    test('returns empty array when no cookie paths exist', async () => {
+    it('returns empty array when no cookie paths exist', async () => {
       const linuxExtractor = new InstagramTokenExtractor('linux')
       const spy = spyOn(linuxExtractor as any, 'copyAndExtract').mockResolvedValue(null)
 
@@ -170,7 +170,7 @@ describe('InstagramTokenExtractor', () => {
       spy.mockRestore()
     })
 
-    test('returns extracted cookies when found', async () => {
+    it('returns extracted cookies when found', async () => {
       const mockCookies = {
         sessionid: 'a'.repeat(25),
         ds_user_id: '12345678',
@@ -193,7 +193,7 @@ describe('InstagramTokenExtractor', () => {
       copyAndExtractSpy.mockRestore()
     })
 
-    test('tries next path when first fails', async () => {
+    it('tries next path when first fails', async () => {
       const mockCookies = {
         sessionid: 'a'.repeat(25),
         ds_user_id: '12345678',
@@ -219,7 +219,7 @@ describe('InstagramTokenExtractor', () => {
       copyAndExtractSpy.mockRestore()
     })
 
-    test('returns multiple entries when different ds_user_id values found across profiles', async () => {
+    it('returns multiple entries when different ds_user_id values found across profiles', async () => {
       const mockCookies1 = {
         sessionid: 'a'.repeat(25),
         ds_user_id: '11111111',
@@ -251,7 +251,7 @@ describe('InstagramTokenExtractor', () => {
       copyAndExtractSpy.mockRestore()
     })
 
-    test('deduplicates entries with the same ds_user_id across profiles', async () => {
+    it('deduplicates entries with the same ds_user_id across profiles', async () => {
       const mockCookies = {
         sessionid: 'a'.repeat(25),
         ds_user_id: '12345678',
@@ -277,7 +277,7 @@ describe('InstagramTokenExtractor', () => {
   })
 
   describe('copyAndExtract', () => {
-    test('returns null when copy fails due to locked database', async () => {
+    it('returns null when copy fails due to locked database', async () => {
       const darwinExtractor = new InstagramTokenExtractor('darwin')
       const copyFileSpy = spyOn(darwinExtractor as any, 'copyDatabaseToTemp').mockImplementation(() => {
         throw new Error('EBUSY: resource busy or locked')
@@ -289,7 +289,7 @@ describe('InstagramTokenExtractor', () => {
       copyFileSpy.mockRestore()
     })
 
-    test('cleans up temp file after extraction', async () => {
+    it('cleans up temp file after extraction', async () => {
       const darwinExtractor = new InstagramTokenExtractor('darwin')
       const copyFileSpy = spyOn(darwinExtractor as any, 'copyDatabaseToTemp').mockReturnValue('/tmp/test-cookies')
       const extractSpy = spyOn(darwinExtractor as any, 'extractFromSQLite').mockResolvedValue(null)
@@ -308,7 +308,7 @@ describe('InstagramTokenExtractor', () => {
 
   describe('decryption', () => {
     describe('decryptAESGCM', () => {
-      test('returns null for data too short for AES-GCM', () => {
+      it('returns null for data too short for AES-GCM', () => {
         const invalidData = Buffer.from('too_short')
         const key = Buffer.alloc(32, 0)
 
@@ -316,7 +316,7 @@ describe('InstagramTokenExtractor', () => {
         expect(result).toBeNull()
       })
 
-      test('returns null when AES-GCM decryption fails with wrong key', () => {
+      it('returns null when AES-GCM decryption fails with wrong key', () => {
         const fakeEncrypted = Buffer.concat([
           Buffer.from('v10'),
           Buffer.alloc(12, 1),
@@ -331,7 +331,7 @@ describe('InstagramTokenExtractor', () => {
     })
 
     describe('decryptAESCBC', () => {
-      test('strips 32-byte non-printable integrity hash prefix (Chromium v130+)', () => {
+      it('strips 32-byte non-printable integrity hash prefix (Chromium v130+)', () => {
         // given — AES-128-CBC encrypted buffer where decrypted result has a 32-byte non-printable prefix
         const { createCipheriv, pbkdf2Sync } = require('node:crypto')
         const key = pbkdf2Sync('peanuts', 'saltysalt', 1, 16, 'sha1')
@@ -351,7 +351,7 @@ describe('InstagramTokenExtractor', () => {
         expect(result).toBe(actualValue)
       })
 
-      test('returns plaintext as-is when no non-printable prefix detected', () => {
+      it('returns plaintext as-is when no non-printable prefix detected', () => {
         // given — AES-128-CBC encrypted buffer without integrity hash prefix
         const { createCipheriv, pbkdf2Sync } = require('node:crypto')
         const key = pbkdf2Sync('peanuts', 'saltysalt', 1, 16, 'sha1')
@@ -371,7 +371,7 @@ describe('InstagramTokenExtractor', () => {
     })
 
     describe('getKeychainPassword on macOS', () => {
-      test('tries multiple keychain variants until one succeeds', () => {
+      it('tries multiple keychain variants until one succeeds', () => {
         const darwinExtractor = new InstagramTokenExtractor('darwin')
         const execSyncSpy = spyOn(darwinExtractor as any, 'execSecurityCommand')
           .mockReturnValueOnce(null)
@@ -385,7 +385,7 @@ describe('InstagramTokenExtractor', () => {
         execSyncSpy.mockRestore()
       })
 
-      test('returns null when all keychain variants fail', () => {
+      it('returns null when all keychain variants fail', () => {
         const darwinExtractor = new InstagramTokenExtractor('darwin')
         const execSyncSpy = spyOn(darwinExtractor as any, 'execSecurityCommand').mockReturnValue(null)
 
@@ -399,12 +399,12 @@ describe('InstagramTokenExtractor', () => {
   })
 
   describe('SQLite extraction', () => {
-    test('returns null when database path does not exist', async () => {
+    it('returns null when database path does not exist', async () => {
       const result = await (extractor as any).extractFromSQLite('/nonexistent/path', '/nonexistent/path')
       expect(result).toBeNull()
     })
 
-    test('returns null when extraction throws', async () => {
+    it('returns null when extraction throws', async () => {
       const result = await (extractor as any).extractFromSQLite('/dev/null', '/dev/null')
       expect(result).toBeNull()
     })

package/src/platforms/kakaotalk/client.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 { KakaoTalkClient, KakaoTalkError } from './client'
 
@@ -82,19 +82,19 @@ describe('KakaoTalkClient', () => {
   })
 
   describe('constructor', () => {
-    test('creates client with required params', async () => {
+    it('creates client with required params', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1' })
       expect(client).toBeInstanceOf(KakaoTalkClient)
       client.close()
     })
 
-    test('defaults deviceUuid when not provided', async () => {
+    it('defaults deviceUuid when not provided', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1' })
       expect(client).toBeInstanceOf(KakaoTalkClient)
       client.close()
     })
 
-    test('throws KakaoTalkError with code missing_token when oauthToken is empty', async () => {
+    it('throws KakaoTalkError with code missing_token when oauthToken is empty', async () => {
       await expect(new KakaoTalkClient().login({ oauthToken: '', userId: 'user1' })).rejects.toThrow(KakaoTalkError)
       try {
         await new KakaoTalkClient().login({ oauthToken: '', userId: 'user1' })
@@ -103,7 +103,7 @@ describe('KakaoTalkClient', () => {
       }
     })
 
-    test('throws KakaoTalkError with code missing_user_id when userId is empty', async () => {
+    it('throws KakaoTalkError with code missing_user_id when userId is empty', async () => {
       await expect(new KakaoTalkClient().login({ oauthToken: 'token', userId: '' })).rejects.toThrow(KakaoTalkError)
       try {
         await new KakaoTalkClient().login({ oauthToken: 'token', userId: '' })
@@ -114,7 +114,7 @@ describe('KakaoTalkClient', () => {
   })
 
   describe('getChats', () => {
-    test('returns formatted chats from login snapshot', async () => {
+    it('returns formatted chats from login snapshot', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
       const chats = await client.getChats()
 
@@ -133,7 +133,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('sorts chats by recency (o field descending)', async () => {
+    it('sorts chats by recency (o field descending)', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
       const chats = await client.getChats()
 
@@ -144,7 +144,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('filters by search term', async () => {
+    it('filters by search term', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
       const chats = await client.getChats({ search: 'alice' })
 
@@ -154,7 +154,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('falls back to LCHATLIST when login snapshot is empty (new device)', async () => {
+    it('falls back to LCHATLIST when login snapshot is empty (new device)', async () => {
       // given — LOGINLIST returns empty chatDatas with eof:true (new device scenario)
       const emptyLoginResult = {
         chatDatas: [],
@@ -207,7 +207,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('does not call LCHATLIST when login snapshot has chats', async () => {
+    it('does not call LCHATLIST when login snapshot has chats', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
       await client.getChats()
 
@@ -216,7 +216,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('paginates when all=true and not eof', async () => {
+    it('paginates when all=true and not eof', async () => {
       const loginResult = {
         ...DEFAULT_LOGIN_RESULT,
         eof: false,
@@ -252,7 +252,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('deduplicates chats by id', async () => {
+    it('deduplicates chats by id', async () => {
       const loginResult = {
         ...DEFAULT_LOGIN_RESULT,
         eof: false,
@@ -287,7 +287,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('wraps errors as KakaoTalkError', async () => {
+    it('wraps errors as KakaoTalkError', async () => {
       mockLogin.mockRejectedValue(new Error('Connection refused'))
 
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
@@ -304,7 +304,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('wraps getChatList failure as KakaoTalkError with code get_chats_failed', async () => {
+    it('wraps getChatList failure as KakaoTalkError with code get_chats_failed', async () => {
       const loginResult = { ...DEFAULT_LOGIN_RESULT, eof: false }
       mockLogin.mockResolvedValue(loginResult)
       mockGetChatList.mockRejectedValue(new Error('Network error'))
@@ -323,7 +323,7 @@ describe('KakaoTalkClient', () => {
   })
 
   describe('getMessages', () => {
-    test('returns formatted messages', async () => {
+    it('returns formatted messages', async () => {
       mockGetChatLogs.mockResolvedValueOnce({
         body: {
           status: 0,
@@ -357,7 +357,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('respects count option', async () => {
+    it('respects count option', async () => {
       const logs = Array.from({ length: 50 }, (_, i) => ({
         logId: makeLong(i + 1),
         chatId: 100,
@@ -382,7 +382,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('sorts messages by sent_at ascending', async () => {
+    it('sorts messages by sent_at ascending', async () => {
       mockGetChatLogs.mockResolvedValueOnce({
         body: {
           status: 0,
@@ -405,7 +405,7 @@ describe('KakaoTalkClient', () => {
   })
 
   describe('sendMessage', () => {
-    test('returns send result on success', async () => {
+    it('returns send result on success', async () => {
       mockSendMessage.mockResolvedValueOnce({
         statusCode: 0,
         body: { logId: makeLong(42), sendAt: 1700000099 },
@@ -425,7 +425,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('reports failure when statusCode is non-zero', async () => {
+    it('reports failure when statusCode is non-zero', async () => {
       mockSendMessage.mockResolvedValueOnce({
         statusCode: -500,
         body: { logId: makeLong(0), sendAt: 0 },
@@ -440,7 +440,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('wraps transport errors as KakaoTalkError', async () => {
+    it('wraps transport errors as KakaoTalkError', async () => {
       mockSendMessage.mockRejectedValue(new Error('Socket closed'))
 
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
@@ -476,7 +476,7 @@ describe('KakaoTalkClient', () => {
       })
     }
 
-    test('returns profile data on success', async () => {
+    it('returns profile data on success', async () => {
       mockFetch
         .mockResolvedValueOnce(
           makeJsonResponse({
@@ -507,7 +507,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('throws not_authenticated when not logged in', async () => {
+    it('throws not_authenticated when not logged in', async () => {
       const client = new KakaoTalkClient()
       try {
         await client.getProfile()
@@ -518,7 +518,7 @@ describe('KakaoTalkClient', () => {
       }
     })
 
-    test('throws profile_request_failed when profile HTTP request fails', async () => {
+    it('throws profile_request_failed when profile HTTP request fails', async () => {
       mockFetch
         .mockResolvedValueOnce(makeJsonResponse({}, 401))
         .mockResolvedValueOnce(makeJsonResponse({ accountDisplayId: null }))
@@ -539,7 +539,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('returns null account_display_id when more_settings request fails', async () => {
+    it('returns null account_display_id when more_settings request fails', async () => {
       mockFetch
         .mockResolvedValueOnce(
           makeJsonResponse({
@@ -569,20 +569,20 @@ describe('KakaoTalkClient', () => {
   })
 
   describe('session lifecycle', () => {
-    test('lazy init: does not call login until first method call', async () => {
+    it('lazy init: does not call login until first method call', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
       expect(mockLogin).not.toHaveBeenCalled()
       client.close()
     })
 
-    test('calls login on first method call', async () => {
+    it('calls login on first method call', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
       await client.getChats()
       expect(mockLogin).toHaveBeenCalledTimes(1)
       client.close()
     })
 
-    test('reuses session across multiple calls', async () => {
+    it('reuses session across multiple calls', async () => {
       mockGetChatLogs.mockResolvedValue({
         body: { status: 0, chatLogs: [], eof: true },
       })
@@ -596,7 +596,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('concurrent calls share a single login', async () => {
+    it('concurrent calls share a single login', async () => {
       // Make login take some time
       mockLogin.mockImplementation(() => new Promise((resolve) => setTimeout(() => resolve(DEFAULT_LOGIN_RESULT), 50)))
       mockGetChatLogs.mockResolvedValue({
@@ -611,7 +611,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('retries login after failure', async () => {
+    it('retries login after failure', async () => {
       mockLogin.mockRejectedValueOnce(new Error('Connection refused')).mockResolvedValueOnce(DEFAULT_LOGIN_RESULT)
 
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
@@ -627,7 +627,7 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
-    test('close cleans up session state', async () => {
+    it('close cleans up session state', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
       await client.getChats()
 
@@ -635,7 +635,7 @@ describe('KakaoTalkClient', () => {
       expect(mockClose).toHaveBeenCalledTimes(1)
     })
 
-    test('methods throw client_closed after close', async () => {
+    it('methods throw client_closed after close', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
       await client.getChats()
       client.close()
@@ -649,13 +649,13 @@ describe('KakaoTalkClient', () => {
       }
     })
 
-    test('close is idempotent', async () => {
+    it('close is idempotent', async () => {
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
       client.close()
       expect(() => client.close()).not.toThrow()
     })
 
-    test('login failure closes the session to prevent socket leak', async () => {
+    it('login failure closes the session to prevent socket leak', async () => {
       mockLogin.mockRejectedValue(new Error('Auth failed'))
 
       const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 const originalConsoleLog = console.log
 
@@ -67,7 +67,7 @@ describe('auth commands', () => {
   })
 
   describe('list', () => {
-    test('outputs empty array when no accounts', async () => {
+    it('outputs empty array when no accounts', async () => {
       mockListAccounts.mockImplementation(() => Promise.resolve([]))
 
       await authCommand.parseAsync(['list'], { from: 'user' })
@@ -76,7 +76,7 @@ describe('auth commands', () => {
       expect(output).toEqual([])
     })
 
-    test('outputs accounts list with is_current flag', async () => {
+    it('outputs accounts list with is_current flag', async () => {
       mockListAccounts.mockImplementation(() =>
         Promise.resolve([
           {
@@ -116,7 +116,7 @@ describe('auth commands', () => {
   })
 
   describe('use', () => {
-    test('switches to specified account', async () => {
+    it('switches to specified account', async () => {
       mockGetAccount.mockImplementation(() =>
         Promise.resolve({
           account_id: 'user-1',
@@ -138,7 +138,7 @@ describe('auth commands', () => {
       expect(output.account_id).toBe('user-1')
     })
 
-    test('outputs error and exits when account not found', async () => {
+    it('outputs error and exits when account not found', async () => {
       mockGetAccount.mockImplementation(() => Promise.resolve(null))
 
       await expect(authCommand.parseAsync(['use', 'nonexistent'], { from: 'user' })).rejects.toThrow('process.exit')
@@ -150,7 +150,7 @@ describe('auth commands', () => {
   })
 
   describe('status', () => {
-    test('outputs error and exits when no account configured', async () => {
+    it('outputs error and exits when no account configured', async () => {
       mockGetAccount.mockImplementation(() => Promise.resolve(null))
 
       await expect(authCommand.parseAsync(['status'], { from: 'user' })).rejects.toThrow('process.exit')
@@ -160,7 +160,7 @@ describe('auth commands', () => {
       expect(output.error).toBeDefined()
     })
 
-    test('outputs account info when account exists', async () => {
+    it('outputs account info when account exists', async () => {
       mockGetAccount.mockImplementation(() =>
         Promise.resolve({
           account_id: 'user-1',
@@ -184,7 +184,7 @@ describe('auth commands', () => {
       expect(output.has_device_uuid).toBe(true)
     })
 
-    test('outputs status for specific --account', async () => {
+    it('outputs status for specific --account', async () => {
       mockGetAccount.mockImplementation((id?: string) => {
         if (id === 'user-2') {
           return Promise.resolve({
@@ -210,7 +210,7 @@ describe('auth commands', () => {
   })
 
   describe('logout', () => {
-    test('removes current account and outputs success', async () => {
+    it('removes current account and outputs success', async () => {
       mockLoad.mockImplementation(() =>
         Promise.resolve({
           current_account: 'user-1',
@@ -237,7 +237,7 @@ describe('auth commands', () => {
       expect(output.removed).toBe('user-1')
     })
 
-    test('removes specific account with --account flag', async () => {
+    it('removes specific account with --account flag', async () => {
       mockLoad.mockImplementation(() =>
         Promise.resolve({
           current_account: 'user-1',
@@ -274,7 +274,7 @@ describe('auth commands', () => {
       expect(output.removed).toBe('user-2')
     })
 
-    test('outputs error and exits when no account configured', async () => {
+    it('outputs error and exits when no account configured', async () => {
       mockLoad.mockImplementation(() => Promise.resolve({ current_account: null, accounts: {} }))
 
       await expect(authCommand.parseAsync(['logout'], { from: 'user' })).rejects.toThrow('process.exit')