agent-messenger 2.10.1 → 2.11.0

package/src/platforms/whatsappbot/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 { WhatsAppBotClient } from '../client'
 import { WhatsAppBotCredentialManager } from '../credential-manager'
@@ -43,11 +43,6 @@ export async function whoamiAction(options: WhoamiOptions): 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/whatsappbot/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('WhatsAppBotCredentialManager', () => {
-  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('WhatsAppBotCredentialManager', () => {
     })
   })
 
-  test('save creates file with correct permissions', async () => {
+  it('save creates file with correct permissions', async () => {
     const testConfigDir = join(
       import.meta.dir,
       `.test-whatsappbot-config-${Date.now()}-${Math.random().toString(36).slice(2)}`,
@@ -61,13 +61,13 @@ describe('WhatsAppBotCredentialManager', () => {
     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 originalAccessToken = process.env.E2E_WHATSAPPBOT_ACCESS_TOKEN
     const originalPhoneNumberId = process.env.E2E_WHATSAPPBOT_PHONE_NUMBER_ID
 
@@ -96,7 +96,7 @@ describe('WhatsAppBotCredentialManager', () => {
     }
   })
 
-  test('getCredentials ignores env vars when accountId is provided', async () => {
+  it('getCredentials ignores env vars when accountId is provided', async () => {
     const originalAccessToken = process.env.E2E_WHATSAPPBOT_ACCESS_TOKEN
     const originalPhoneNumberId = process.env.E2E_WHATSAPPBOT_PHONE_NUMBER_ID
 
@@ -121,7 +121,7 @@ describe('WhatsAppBotCredentialManager', () => {
     }
   })
 
-  test('getCredentials returns specific account by accountId', async () => {
+  it('getCredentials returns specific account by accountId', async () => {
     const manager = setup()
     await manager.setCredentials({
       phone_number_id: 'phone-1',
@@ -142,13 +142,13 @@ describe('WhatsAppBotCredentialManager', () => {
     })
   })
 
-  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({
       phone_number_id: '123456789',
@@ -167,7 +167,7 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(config.current).toEqual({ account_id: '123456789' })
   })
 
-  test('removeAccount deletes account and adjusts current', async () => {
+  it('removeAccount deletes account and adjusts current', async () => {
     const manager = setup()
     await manager.setCredentials({
       phone_number_id: '123456789',
@@ -186,13 +186,13 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(config.accounts['123456789']).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({
       phone_number_id: 'phone-1',
@@ -212,7 +212,7 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(config.current).toEqual({ account_id: 'phone-2' })
   })
 
-  test('setCurrent switches active account', async () => {
+  it('setCurrent switches active account', async () => {
     const manager = setup()
     await manager.setCredentials({
       phone_number_id: 'phone-1',
@@ -232,13 +232,13 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(config.current).toEqual({ account_id: 'phone-1' })
   })
 
-  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({
       phone_number_id: 'phone-1',
@@ -261,13 +261,13 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(phone2?.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({
       phone_number_id: '123456789',
@@ -282,7 +282,7 @@ describe('WhatsAppBotCredentialManager', () => {
     expect(config.accounts).toEqual({})
   })
 
-  test('round-trip: set → get → remove → get null', async () => {
+  it('completes round-trip: set → get → remove → get null', async () => {
     const manager = setup()
 
     await manager.setCredentials({

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

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   WhatsAppBotAccountEntrySchema,
@@ -9,26 +9,26 @@ import {
   WhatsAppBotError,
 } from '@/platforms/whatsappbot/index'
 
-test('WhatsAppBotClient is exported from barrel', () => {
+it('WhatsAppBotClient is exported from barrel', () => {
   expect(typeof WhatsAppBotClient).toBe('function')
 })
 
-test('WhatsAppBotCredentialManager is exported from barrel', () => {
+it('WhatsAppBotCredentialManager is exported from barrel', () => {
   expect(typeof WhatsAppBotCredentialManager).toBe('function')
 })
 
-test('WhatsAppBotError is exported from barrel', () => {
+it('WhatsAppBotError is exported from barrel', () => {
   expect(typeof WhatsAppBotError).toBe('function')
 })
 
-test('WhatsAppBotAccountEntrySchema is exported from barrel', () => {
+it('WhatsAppBotAccountEntrySchema is exported from barrel', () => {
   expect(typeof WhatsAppBotAccountEntrySchema.parse).toBe('function')
 })
 
-test('WhatsAppBotConfigSchema is exported from barrel', () => {
+it('WhatsAppBotConfigSchema is exported from barrel', () => {
   expect(typeof WhatsAppBotConfigSchema.parse).toBe('function')
 })
 
-test('WhatsAppBotCredentialsSchema is exported from barrel', () => {
+it('WhatsAppBotCredentialsSchema is exported from barrel', () => {
   expect(typeof WhatsAppBotCredentialsSchema.parse).toBe('function')
 })

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

@@ -1,4 +1,4 @@
-import { expect, test } from 'bun:test'
+import { expect, it } from 'bun:test'
 
 import {
   WhatsAppBotAccountEntrySchema,
@@ -7,7 +7,7 @@ import {
   WhatsAppBotError,
 } from '@/platforms/whatsappbot/types'
 
-test('WhatsAppBotAccountEntrySchema validates correct data', () => {
+it('WhatsAppBotAccountEntrySchema validates correct data', () => {
   const result = WhatsAppBotAccountEntrySchema.safeParse({
     phone_number_id: '123456789',
     account_name: 'Test Business',
@@ -16,7 +16,7 @@ test('WhatsAppBotAccountEntrySchema validates correct data', () => {
   expect(result.success).toBe(true)
 })
 
-test('WhatsAppBotAccountEntrySchema rejects missing phone_number_id', () => {
+it('WhatsAppBotAccountEntrySchema rejects missing phone_number_id', () => {
   const result = WhatsAppBotAccountEntrySchema.safeParse({
     account_name: 'Test Business',
     access_token: 'EAAtest123',
@@ -24,7 +24,7 @@ test('WhatsAppBotAccountEntrySchema rejects missing phone_number_id', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotAccountEntrySchema rejects missing account_name', () => {
+it('WhatsAppBotAccountEntrySchema rejects missing account_name', () => {
   const result = WhatsAppBotAccountEntrySchema.safeParse({
     phone_number_id: '123456789',
     access_token: 'EAAtest123',
@@ -32,7 +32,7 @@ test('WhatsAppBotAccountEntrySchema rejects missing account_name', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotAccountEntrySchema rejects missing access_token', () => {
+it('WhatsAppBotAccountEntrySchema rejects missing access_token', () => {
   const result = WhatsAppBotAccountEntrySchema.safeParse({
     phone_number_id: '123456789',
     account_name: 'Test Business',
@@ -40,7 +40,7 @@ test('WhatsAppBotAccountEntrySchema rejects missing access_token', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotConfigSchema validates with current account', () => {
+it('WhatsAppBotConfigSchema validates with current account', () => {
   const result = WhatsAppBotConfigSchema.safeParse({
     current: { account_id: '123456789' },
     accounts: {
@@ -54,7 +54,7 @@ test('WhatsAppBotConfigSchema validates with current account', () => {
   expect(result.success).toBe(true)
 })
 
-test('WhatsAppBotConfigSchema validates with current null and empty accounts', () => {
+it('WhatsAppBotConfigSchema validates with current null and empty accounts', () => {
   const result = WhatsAppBotConfigSchema.safeParse({
     current: null,
     accounts: {},
@@ -62,7 +62,7 @@ test('WhatsAppBotConfigSchema validates with current null and empty accounts', (
   expect(result.success).toBe(true)
 })
 
-test('WhatsAppBotConfigSchema validates with multiple accounts', () => {
+it('WhatsAppBotConfigSchema validates with multiple accounts', () => {
   const result = WhatsAppBotConfigSchema.safeParse({
     current: { account_id: 'phone-1' },
     accounts: {
@@ -73,21 +73,21 @@ test('WhatsAppBotConfigSchema validates with multiple accounts', () => {
   expect(result.success).toBe(true)
 })
 
-test('WhatsAppBotConfigSchema rejects missing accounts field', () => {
+it('WhatsAppBotConfigSchema rejects missing accounts field', () => {
   const result = WhatsAppBotConfigSchema.safeParse({
     current: null,
   })
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotConfigSchema rejects missing current field', () => {
+it('WhatsAppBotConfigSchema rejects missing current field', () => {
   const result = WhatsAppBotConfigSchema.safeParse({
     accounts: {},
   })
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotCredentialsSchema validates correct data', () => {
+it('WhatsAppBotCredentialsSchema validates correct data', () => {
   const result = WhatsAppBotCredentialsSchema.safeParse({
     phone_number_id: '123456789',
     account_name: 'Test Business',
@@ -96,7 +96,7 @@ test('WhatsAppBotCredentialsSchema validates correct data', () => {
   expect(result.success).toBe(true)
 })
 
-test('WhatsAppBotCredentialsSchema rejects missing phone_number_id', () => {
+it('WhatsAppBotCredentialsSchema rejects missing phone_number_id', () => {
   const result = WhatsAppBotCredentialsSchema.safeParse({
     account_name: 'Test Business',
     access_token: 'EAAtest123',
@@ -104,7 +104,7 @@ test('WhatsAppBotCredentialsSchema rejects missing phone_number_id', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotCredentialsSchema rejects missing account_name', () => {
+it('WhatsAppBotCredentialsSchema rejects missing account_name', () => {
   const result = WhatsAppBotCredentialsSchema.safeParse({
     phone_number_id: '123456789',
     access_token: 'EAAtest123',
@@ -112,7 +112,7 @@ test('WhatsAppBotCredentialsSchema rejects missing account_name', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotCredentialsSchema rejects missing access_token', () => {
+it('WhatsAppBotCredentialsSchema rejects missing access_token', () => {
   const result = WhatsAppBotCredentialsSchema.safeParse({
     phone_number_id: '123456789',
     account_name: 'Test Business',
@@ -120,22 +120,22 @@ test('WhatsAppBotCredentialsSchema rejects missing access_token', () => {
   expect(result.success).toBe(false)
 })
 
-test('WhatsAppBotError has correct name', () => {
+it('WhatsAppBotError has correct name', () => {
   const error = new WhatsAppBotError('Test error', 'TEST_CODE')
   expect(error.name).toBe('WhatsAppBotError')
 })
 
-test('WhatsAppBotError has correct message', () => {
+it('WhatsAppBotError has correct message', () => {
   const error = new WhatsAppBotError('Test error', 'TEST_CODE')
   expect(error.message).toBe('Test error')
 })
 
-test('WhatsAppBotError has correct code', () => {
+it('WhatsAppBotError has correct code', () => {
   const error = new WhatsAppBotError('Test error', 'TEST_CODE')
   expect(error.code).toBe('TEST_CODE')
 })
 
-test('WhatsAppBotError is instance of Error', () => {
+it('WhatsAppBotError is instance of Error', () => {
   const error = new WhatsAppBotError('Test error', 'TEST_CODE')
   expect(error instanceof Error).toBe(true)
 })

package/src/shared/chromium/browsers.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, describe, expect, test } from 'bun:test'
+import { afterEach, describe, expect, it } from 'bun:test'
 import { existsSync, mkdirSync, mkdtempSync, rmSync, writeFileSync } from 'node:fs'
 import { tmpdir } from 'node:os'
 import { join } from 'node:path'
@@ -8,6 +8,7 @@ import {
   CHROMIUM_BROWSERS,
   discoverBrowserProfileDirs,
   findLocalStatePath,
+  getAgentBrowserProfileDirs,
   getBrowserBasePath,
 } from './browsers'
 
@@ -26,34 +27,34 @@ describe('browsers', () => {
   })
 
   describe('CHROMIUM_BROWSERS', () => {
-    test('has 7 browsers', () => {
+    it('has 7 browsers', () => {
       expect(CHROMIUM_BROWSERS).toHaveLength(7)
     })
 
-    test('includes major supported browsers', () => {
+    it('includes major supported browsers', () => {
       const browserNames = CHROMIUM_BROWSERS.map((browser) => browser.name)
 
       expect(browserNames).toEqual(expect.arrayContaining(['Chrome', 'Edge', 'Arc', 'Brave', 'Vivaldi', 'Chromium']))
     })
 
-    test('Arc has empty linux path', () => {
+    it('Arc has empty linux path', () => {
       expect(CHROMIUM_BROWSERS.find((browser) => browser.name === 'Arc')?.linux).toBe('')
     })
   })
 
   describe('BROWSER_KEYCHAIN_VARIANTS', () => {
-    test('has 7 keychain variants', () => {
+    it('has 7 keychain variants', () => {
       expect(BROWSER_KEYCHAIN_VARIANTS).toHaveLength(7)
     })
 
-    test('each variant has service and account properties', () => {
+    it('each variant has service and account properties', () => {
       for (const variant of BROWSER_KEYCHAIN_VARIANTS) {
         expect(variant.service).toBeString()
         expect(variant.account).toBeString()
       }
     })
 
-    test('includes known safe storage services', () => {
+    it('includes known safe storage services', () => {
       const services = BROWSER_KEYCHAIN_VARIANTS.map((variant) => variant.service)
 
       expect(services).toEqual(expect.arrayContaining(['Chrome Safe Storage', 'Microsoft Edge Safe Storage']))
@@ -61,7 +62,7 @@ describe('browsers', () => {
   })
 
   describe('getBrowserBasePath', () => {
-    test('returns darwin path with Library/Application Support prefix', () => {
+    it('returns darwin path with Library/Application Support prefix', () => {
       // given
       const chrome = CHROMIUM_BROWSERS.find((browser) => browser.name === 'Chrome')!
 
@@ -74,7 +75,7 @@ describe('browsers', () => {
       expect(path).toEndWith(join('Google', 'Chrome'))
     })
 
-    test('returns linux path with .config prefix', () => {
+    it('returns linux path with .config prefix', () => {
       // given
       const chrome = CHROMIUM_BROWSERS.find((browser) => browser.name === 'Chrome')!
 
@@ -86,7 +87,7 @@ describe('browsers', () => {
       expect(path).toContain(join('.config', 'google-chrome'))
     })
 
-    test('returns win32 path with LOCALAPPDATA prefix', () => {
+    it('returns win32 path with LOCALAPPDATA prefix', () => {
       // given
       const localAppData = mkdtempSync(join(tmpdir(), 'browser-localappdata-'))
       tempDirs.push(localAppData)
@@ -100,7 +101,7 @@ describe('browsers', () => {
       expect(path).toBe(join(localAppData, 'Google', 'Chrome', 'User Data'))
     })
 
-    test('returns null for unsupported platform', () => {
+    it('returns null for unsupported platform', () => {
       // given
       const chrome = CHROMIUM_BROWSERS.find((browser) => browser.name === 'Chrome')!
 
@@ -111,7 +112,7 @@ describe('browsers', () => {
       expect(path).toBeNull()
     })
 
-    test('returns null when browser has empty path for platform', () => {
+    it('returns null when browser has empty path for platform', () => {
       // given
       const arc = CHROMIUM_BROWSERS.find((browser) => browser.name === 'Arc')!
 
@@ -124,7 +125,7 @@ describe('browsers', () => {
   })
 
   describe('discoverBrowserProfileDirs', () => {
-    test('always includes Default dir even when base does not exist', () => {
+    it('always includes Default dir even when base does not exist', () => {
       // given
       const browserBase = join(tmpdir(), `missing-browser-base-${Date.now()}-${Math.random().toString(36).slice(2)}`)
 
@@ -135,7 +136,7 @@ describe('browsers', () => {
       expect(dirs).toEqual([join(browserBase, 'Default')])
     })
 
-    test('discovers Profile 1 and Profile 2 dirs when they exist', () => {
+    it('discovers Profile 1 and Profile 2 dirs when they exist', () => {
       // given
       const browserBase = mkdtempSync(join(tmpdir(), 'browser-profiles-'))
       tempDirs.push(browserBase)
@@ -153,7 +154,7 @@ describe('browsers', () => {
       ])
     })
 
-    test('ignores non-profile directories', () => {
+    it('ignores non-profile directories', () => {
       // given
       const browserBase = mkdtempSync(join(tmpdir(), 'browser-non-profile-'))
       tempDirs.push(browserBase)
@@ -168,7 +169,7 @@ describe('browsers', () => {
       expect(dirs).toEqual([join(browserBase, 'Default'), join(browserBase, 'Profile 1')])
     })
 
-    test('ignores files that match profile pattern', () => {
+    it('ignores files that match profile pattern', () => {
       // given
       const browserBase = mkdtempSync(join(tmpdir(), 'browser-profile-files-'))
       tempDirs.push(browserBase)
@@ -182,7 +183,7 @@ describe('browsers', () => {
       expect(dirs).toEqual([join(browserBase, 'Default'), join(browserBase, 'Profile 2')])
     })
 
-    test('returns only Default when base directory is empty', () => {
+    it('returns only Default when base directory is empty', () => {
       // given
       const browserBase = mkdtempSync(join(tmpdir(), 'browser-empty-'))
       tempDirs.push(browserBase)
@@ -195,8 +196,87 @@ describe('browsers', () => {
     })
   })
 
+  describe('getAgentBrowserProfileDirs', () => {
+    it('includes profile dirs from env and config files', () => {
+      // given
+      const homeDir = mkdtempSync(join(tmpdir(), 'agent-browser-home-'))
+      const projectDir = mkdtempSync(join(tmpdir(), 'agent-browser-project-'))
+      const customConfigDir = mkdtempSync(join(tmpdir(), 'agent-browser-custom-'))
+      tempDirs.push(homeDir, projectDir, customConfigDir)
+
+      mkdirSync(join(homeDir, '.agent-browser'))
+      writeFileSync(join(homeDir, '.agent-browser', 'config.json'), JSON.stringify({ profile: './global-profile' }))
+      writeFileSync(join(projectDir, 'agent-browser.json'), JSON.stringify({ profile: './project-profile' }))
+      const customConfigPath = join(customConfigDir, 'custom-agent-browser.json')
+      writeFileSync(customConfigPath, JSON.stringify({ profile: './custom-profile' }))
+
+      // when
+      const dirs = getAgentBrowserProfileDirs({
+        cwd: projectDir,
+        env: {
+          AGENT_BROWSER_CONFIG: customConfigPath,
+          AGENT_BROWSER_PROFILE: join(projectDir, 'env-profile'),
+        },
+        homeDir,
+        customProfileDirs: ['./cli-profile', join(projectDir, 'second-cli-profile')],
+      })
+
+      // then
+      expect(dirs).toEqual([
+        join(projectDir, 'cli-profile'),
+        join(projectDir, 'cli-profile', 'Default'),
+        join(projectDir, 'second-cli-profile'),
+        join(projectDir, 'second-cli-profile', 'Default'),
+        join(projectDir, 'env-profile'),
+        join(projectDir, 'env-profile', 'Default'),
+        join(homeDir, '.agent-browser', 'global-profile'),
+        join(homeDir, '.agent-browser', 'global-profile', 'Default'),
+        join(projectDir, 'project-profile'),
+        join(projectDir, 'project-profile', 'Default'),
+        join(customConfigDir, 'custom-profile'),
+        join(customConfigDir, 'custom-profile', 'Default'),
+      ])
+    })
+
+    it('expands existing agent-browser profile bases with numbered Chromium profiles', () => {
+      // given
+      const homeDir = mkdtempSync(join(tmpdir(), 'agent-browser-expand-home-'))
+      const projectDir = mkdtempSync(join(tmpdir(), 'agent-browser-expand-project-'))
+      const profileBase = join(projectDir, 'browser-data')
+      tempDirs.push(homeDir, projectDir)
+      mkdirSync(join(profileBase, 'Profile 1'), { recursive: true })
+      mkdirSync(join(profileBase, 'Profile 2'))
+      writeFileSync(join(projectDir, 'agent-browser.json'), JSON.stringify({ profile: './browser-data' }))
+
+      // when
+      const dirs = getAgentBrowserProfileDirs({ cwd: projectDir, env: {}, homeDir })
+
+      // then
+      expect(dirs).toEqual([
+        profileBase,
+        join(profileBase, 'Default'),
+        join(profileBase, 'Profile 1'),
+        join(profileBase, 'Profile 2'),
+      ])
+    })
+
+    it('ignores malformed agent-browser configs', () => {
+      // given
+      const homeDir = mkdtempSync(join(tmpdir(), 'agent-browser-malformed-home-'))
+      const projectDir = mkdtempSync(join(tmpdir(), 'agent-browser-malformed-project-'))
+      tempDirs.push(homeDir, projectDir)
+      writeFileSync(join(projectDir, 'agent-browser.json'), '{')
+
+      // when
+      const dirs = getAgentBrowserProfileDirs({ cwd: projectDir, env: {}, homeDir })
+
+      // then
+      expect(dirs).toEqual([])
+    })
+  })
+
   describe('findLocalStatePath', () => {
-    test('returns null when no Local State exists at any level', () => {
+    it('returns null when no Local State exists at any level', () => {
       // given
       const rootDir = mkdtempSync(join(tmpdir(), 'browser-local-state-missing-'))
       tempDirs.push(rootDir)
@@ -210,7 +290,7 @@ describe('browsers', () => {
       expect(path).toBeNull()
     })
 
-    test('finds Local State 2 levels up from cookie path', () => {
+    it('finds Local State 2 levels up from cookie path', () => {
       // given
       const rootDir = mkdtempSync(join(tmpdir(), 'browser-local-state-two-'))
       tempDirs.push(rootDir)
@@ -226,7 +306,7 @@ describe('browsers', () => {
       expect(path).toBe(join(profileDir, 'Local State'))
     })
 
-    test('finds Local State 3 levels up from cookie path', () => {
+    it('finds Local State 3 levels up from cookie path', () => {
       // given
       const rootDir = mkdtempSync(join(tmpdir(), 'browser-local-state-three-'))
       tempDirs.push(rootDir)
@@ -242,7 +322,7 @@ describe('browsers', () => {
       expect(path).toBe(join(browserDir, 'Local State'))
     })
 
-    test('returns null for path with too few segments', () => {
+    it('returns null for path with too few segments', () => {
       // given
       const cookiePath = 'Cookies'
 
@@ -253,7 +333,7 @@ describe('browsers', () => {
       expect(path).toBeNull()
     })
 
-    test('finds the first match when multiple levels could match', () => {
+    it('finds the first match when multiple levels could match', () => {
       // given
       const rootDir = mkdtempSync(join(tmpdir(), 'browser-local-state-first-'))
       tempDirs.push(rootDir)