agentbnb 8.4.2 → 8.4.3

package/skills/agentbnb/bootstrap.test.ts

@@ -1,323 +0,0 @@
-/**
- * Unit tests for bootstrap.ts thin OpenClaw adapter.
- *
- * bootstrap.ts is a thin adapter — it delegates all lifecycle logic to
- * ServiceCoordinator via AgentBnBService. Tests verify the adapter's own
- * responsibilities:
- *   - CONFIG_NOT_FOUND when no config exists
- *   - BootstrapContext shape (service, status, startDisposition)
- *   - Signal handler registration and removal
- *   - deactivate() only stops node when startDisposition === 'started'
- *   - deactivate() is idempotent
- */
-
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { homedir } from 'node:os';
-import { join } from 'node:path';
-
-// ---------------------------------------------------------------------------
-// Mocks — must be hoisted before imports
-// ---------------------------------------------------------------------------
-
-const mockEnsureRunning = vi.fn<() => Promise<'started' | 'already_running'>>();
-const mockGetNodeStatus = vi.fn();
-const mockStop = vi.fn<() => Promise<void>>();
-
-vi.mock('../../src/app/agentbnb-service.js', () => ({
-  AgentBnBService: vi.fn().mockImplementation(() => ({
-    ensureRunning: mockEnsureRunning,
-    getNodeStatus: mockGetNodeStatus,
-    stop: mockStop,
-  })),
-}));
-
-vi.mock('../../src/runtime/service-coordinator.js', () => ({
-  ServiceCoordinator: vi.fn().mockImplementation(() => ({})),
-}));
-
-vi.mock('../../src/runtime/process-guard.js', () => ({
-  ProcessGuard: vi.fn().mockImplementation(() => ({})),
-}));
-
-vi.mock('../../src/cli/config.js', () => ({
-  loadConfig: vi.fn(),
-  getConfigDir: vi.fn(() => join(homedir(), '.agentbnb')),
-}));
-
-vi.mock('../../src/registry/store.js', () => ({
-  openDatabase: vi.fn(() => ({
-    prepare: vi.fn(() => ({
-      get: vi.fn(() => ({ id: 'existing-card' })),
-      run: vi.fn(),
-    })),
-  })),
-}));
-
-import { loadConfig } from '../../src/cli/config.js';
-import { activate, deactivate } from './bootstrap.js';
-import bootstrapDefault from './bootstrap.js';
-import type { BootstrapContext, OnboardDeps } from './bootstrap.js';
-
-const mockLoadConfig = vi.mocked(loadConfig);
-
-const MINIMAL_CONFIG = {
-  owner: 'test-agent',
-  gateway_url: 'http://localhost:7700',
-  gateway_port: 7700,
-  db_path: ':memory:',
-  credit_db_path: ':memory:',
-  token: 'test-token',
-  api_key: 'test-api-key',
-  registry: 'https://agentbnb.fly.dev',
-};
-
-const MOCK_STATUS = {
-  state: 'running' as const,
-  pid: 1234,
-  port: 7700,
-  owner: 'test-agent',
-  relayConnected: false,
-  uptime_ms: 100,
-};
-
-// ---------------------------------------------------------------------------
-// Test suite
-// ---------------------------------------------------------------------------
-
-describe('bootstrap activate/deactivate lifecycle', () => {
-  let ctx: BootstrapContext | undefined;
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockEnsureRunning.mockResolvedValue('started');
-    mockGetNodeStatus.mockResolvedValue(MOCK_STATUS);
-    mockStop.mockResolvedValue(undefined);
-    mockLoadConfig.mockReturnValue(MINIMAL_CONFIG as ReturnType<typeof loadConfig>);
-  });
-
-  afterEach(async () => {
-    if (ctx) {
-      await deactivate(ctx).catch(() => undefined);
-      ctx = undefined;
-    }
-    process.removeAllListeners('SIGTERM');
-    process.removeAllListeners('SIGINT');
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 1: INIT_FAILED when CLI not found and config missing
-  // ---------------------------------------------------------------------------
-  it('activate() throws INIT_FAILED when CLI not found and config missing', async () => {
-    mockLoadConfig.mockReturnValue(null);
-    const deps: OnboardDeps = {
-      resolveSelfCli: () => {
-        throw new Error('not found');
-      },
-      runCommand: vi.fn(),
-    };
-
-    await expect(activate({}, deps)).rejects.toMatchObject({
-      code: 'INIT_FAILED',
-    });
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 1b: Auto-onboard runs init + openclaw sync when config missing
-  // ---------------------------------------------------------------------------
-  it('activate() auto-onboards when config missing and CLI available', async () => {
-    mockLoadConfig.mockReturnValueOnce(null).mockReturnValue(MINIMAL_CONFIG as ReturnType<typeof loadConfig>);
-    const mockRun = vi.fn().mockResolvedValue({ stdout: '', stderr: '' });
-    const deps: OnboardDeps = {
-      resolveSelfCli: () => '/usr/local/bin/agentbnb',
-      runCommand: mockRun,
-    };
-
-    ctx = await activate({}, deps);
-
-    expect(mockRun).toHaveBeenCalledTimes(2);
-    expect(mockRun.mock.calls[0][0]).toMatch(/^'\/usr\/local\/bin\/agentbnb' init --owner .* --yes --no-detect$/);
-    expect(mockRun.mock.calls[1][0]).toBe('\'/usr/local/bin/agentbnb\' openclaw sync');
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 1c: Auto-onboard continues if openclaw sync fails
-  // ---------------------------------------------------------------------------
-  it('activate() continues if openclaw sync fails during auto-onboard', async () => {
-    mockLoadConfig.mockReturnValueOnce(null).mockReturnValue(MINIMAL_CONFIG as ReturnType<typeof loadConfig>);
-    const mockRun = vi.fn()
-      .mockResolvedValueOnce({ stdout: '', stderr: '' })
-      .mockRejectedValueOnce(new Error('SOUL.md not found'));
-    const deps: OnboardDeps = {
-      resolveSelfCli: () => '/usr/local/bin/agentbnb',
-      runCommand: mockRun,
-    };
-
-    ctx = await activate({}, deps);
-
-    expect(ctx.startDisposition).toBe('started');
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 1d: Skips auto-onboard when config already exists
-  // ---------------------------------------------------------------------------
-  it('activate() skips auto-onboard when config already exists', async () => {
-    const mockRun = vi.fn();
-    const deps: OnboardDeps = {
-      resolveSelfCli: () => '/usr/local/bin/agentbnb',
-      runCommand: mockRun,
-    };
-
-    ctx = await activate({}, deps);
-
-    expect(mockRun).not.toHaveBeenCalled();
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 2: BootstrapContext shape
-  // ---------------------------------------------------------------------------
-  it('activate() returns BootstrapContext with correct shape', async () => {
-    ctx = await activate();
-
-    expect(ctx).toHaveProperty('service');
-    expect(ctx).toHaveProperty('status');
-    expect(ctx).toHaveProperty('startDisposition');
-    expect(ctx).toHaveProperty('_removeSignalHandlers');
-    expect(typeof ctx._removeSignalHandlers).toBe('function');
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 3: startDisposition reflects ensureRunning result
-  // ---------------------------------------------------------------------------
-  it('startDisposition is "started" when ensureRunning returns "started"', async () => {
-    mockEnsureRunning.mockResolvedValue('started');
-    ctx = await activate();
-    expect(ctx.startDisposition).toBe('started');
-  });
-
-  it('startDisposition is "already_running" when node was already up', async () => {
-    mockEnsureRunning.mockResolvedValue('already_running');
-    ctx = await activate();
-    expect(ctx.startDisposition).toBe('already_running');
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 4: status snapshot from getNodeStatus
-  // ---------------------------------------------------------------------------
-  it('activate() status reflects getNodeStatus() snapshot', async () => {
-    ctx = await activate();
-    expect(ctx.status).toEqual(MOCK_STATUS);
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 5: signal handlers registered
-  // ---------------------------------------------------------------------------
-  it('activate() registers SIGTERM and SIGINT handlers', async () => {
-    const sigtermBefore = process.listenerCount('SIGTERM');
-    const sigintBefore = process.listenerCount('SIGINT');
-
-    ctx = await activate();
-
-    expect(process.listenerCount('SIGTERM')).toBe(sigtermBefore + 1);
-    expect(process.listenerCount('SIGINT')).toBe(sigintBefore + 1);
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 6: _removeSignalHandlers removes them
-  // ---------------------------------------------------------------------------
-  it('_removeSignalHandlers() removes SIGTERM and SIGINT handlers', async () => {
-    ctx = await activate();
-    const sigtermAfterActivate = process.listenerCount('SIGTERM');
-    const sigintAfterActivate = process.listenerCount('SIGINT');
-
-    ctx._removeSignalHandlers();
-
-    expect(process.listenerCount('SIGTERM')).toBe(sigtermAfterActivate - 1);
-    expect(process.listenerCount('SIGINT')).toBe(sigintAfterActivate - 1);
-
-    ctx = undefined;
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 7: deactivate() stops node when startDisposition === 'started'
-  // ---------------------------------------------------------------------------
-  it('deactivate() calls service.stop() when startDisposition is "started"', async () => {
-    mockEnsureRunning.mockResolvedValue('started');
-    ctx = await activate();
-
-    await deactivate(ctx);
-    ctx = undefined;
-
-    expect(mockStop).toHaveBeenCalledTimes(1);
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 8: deactivate() does NOT stop node when already_running
-  // ---------------------------------------------------------------------------
-  it('deactivate() does NOT call service.stop() when startDisposition is "already_running"', async () => {
-    mockEnsureRunning.mockResolvedValue('already_running');
-    ctx = await activate();
-
-    await deactivate(ctx);
-    ctx = undefined;
-
-    expect(mockStop).not.toHaveBeenCalled();
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 9: deactivate() is idempotent
-  // ---------------------------------------------------------------------------
-  it('deactivate() is idempotent — second call does not throw', async () => {
-    ctx = await activate();
-
-    await deactivate(ctx);
-    await expect(deactivate(ctx)).resolves.not.toThrow();
-
-    ctx = undefined;
-  });
-
-  // ---------------------------------------------------------------------------
-  // Test 10: deactivate() removes signal handlers
-  // ---------------------------------------------------------------------------
-  it('deactivate() removes signal handlers', async () => {
-    ctx = await activate();
-    const sigtermAfterActivate = process.listenerCount('SIGTERM');
-
-    await deactivate(ctx);
-    ctx = undefined;
-
-    expect(process.listenerCount('SIGTERM')).toBeLessThan(sigtermAfterActivate);
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Default export (OpenClaw plugin definition)
-// ---------------------------------------------------------------------------
-
-describe('bootstrap default export (OpenClaw plugin definition)', () => {
-  it('has id, name, description, and register', () => {
-    expect(bootstrapDefault).toHaveProperty('id', 'agentbnb');
-    expect(bootstrapDefault).toHaveProperty('name', 'AgentBnB');
-    expect(bootstrapDefault).toHaveProperty('description');
-    expect(typeof bootstrapDefault.description).toBe('string');
-    expect(bootstrapDefault).toHaveProperty('register');
-    expect(typeof bootstrapDefault.register).toBe('function');
-  });
-
-  it('register() calls api.registerTool with a factory', () => {
-    const mockRegisterTool = vi.fn();
-    bootstrapDefault.register({ registerTool: mockRegisterTool });
-    expect(mockRegisterTool).toHaveBeenCalledTimes(1);
-    expect(typeof mockRegisterTool.mock.calls[0][0]).toBe('function');
-  });
-
-  it('factory returns 5 tools', () => {
-    let factory: ((ctx: { workspaceDir?: string; agentDir?: string }) => unknown[]) | undefined;
-    const mockRegisterTool = vi.fn((fn: typeof factory) => { factory = fn; });
-    bootstrapDefault.register({ registerTool: mockRegisterTool });
-
-    // Call the factory with a mock context — it will fail on config read,
-    // so we mock createAllTools indirectly by checking the factory is callable
-    expect(factory).toBeDefined();
-    expect(typeof factory).toBe('function');
-  });
-});

package/skills/agentbnb/openclaw-tools.test.ts

@@ -1,328 +0,0 @@
-/**
- * Unit tests for openclaw-tools.ts — OpenClaw tool factory.
- *
- * Tests verify:
- *   - Tool factory returns 5 tools with correct shape
- *   - Tool names are kebab-case with agentbnb- prefix
- *   - Parameter schemas are valid JSON Schema
- *   - Result conversion from MCP format to AgentToolResult
- *   - Context caching and multi-agent isolation
- */
-
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { mkdtempSync, writeFileSync, mkdirSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-
-import {
-  createAllTools,
-  createDiscoverTool,
-  createRequestTool,
-  createConductTool,
-  createStatusTool,
-  createPublishTool,
-  toAgentToolResult,
-  buildMcpContext,
-  resolveConfigDir,
-  resetContextCache,
-} from './openclaw-tools.js';
-import type { AgentTool, OpenClawToolContext } from './openclaw-tools.js';
-
-// ---------------------------------------------------------------------------
-// Mocks
-// ---------------------------------------------------------------------------
-
-vi.mock('../../src/identity/identity.js', () => ({
-  ensureIdentity: vi.fn((_dir: string, owner: string) => ({
-    agent_id: 'mock-agent-id',
-    owner,
-    public_key: 'deadbeef'.repeat(8),
-    created_at: '2026-01-01T00:00:00.000Z',
-  })),
-}));
-
-vi.mock('../../src/mcp/tools/discover.js', () => ({
-  handleDiscover: vi.fn(async () => ({
-    content: [{ type: 'text', text: JSON.stringify({ results: [], count: 0 }) }],
-  })),
-}));
-
-vi.mock('../../src/mcp/tools/request.js', () => ({
-  handleRequest: vi.fn(async () => ({
-    content: [{ type: 'text', text: JSON.stringify({ success: true }) }],
-  })),
-}));
-
-vi.mock('../../src/mcp/tools/conduct.js', () => ({
-  handleConduct: vi.fn(async () => ({
-    content: [{ type: 'text', text: JSON.stringify({ success: true, plan: [] }) }],
-  })),
-}));
-
-vi.mock('../../src/mcp/tools/status.js', () => ({
-  handleStatus: vi.fn(async () => ({
-    content: [{ type: 'text', text: JSON.stringify({ agent_id: 'mock', balance: 50 }) }],
-  })),
-}));
-
-vi.mock('../../src/mcp/tools/publish.js', () => ({
-  handlePublish: vi.fn(async () => ({
-    content: [{ type: 'text', text: JSON.stringify({ success: true, card_id: 'abc' }) }],
-  })),
-}));
-
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-
-/**
- * Creates a temp dir ending with `.agentbnb` containing config.json.
- * resolveConfigDir() uses the dir as-is when it ends with `.agentbnb`.
- */
-function createTempConfigDir(): string {
-  const parent = mkdtempSync(join(tmpdir(), 'agentbnb-test-'));
-  const configDir = join(parent, '.agentbnb');
-  mkdirSync(configDir, { recursive: true });
-  writeFileSync(
-    join(configDir, 'config.json'),
-    JSON.stringify({
-      owner: 'test-agent',
-      gateway_url: 'http://localhost:7700',
-      gateway_port: 7700,
-      db_path: ':memory:',
-      credit_db_path: ':memory:',
-      token: 'test-token',
-      registry: 'https://hub.agentbnb.dev',
-    }),
-  );
-  return configDir;
-}
-
-// ---------------------------------------------------------------------------
-// Tests
-// ---------------------------------------------------------------------------
-
-describe('openclaw-tools', () => {
-  beforeEach(() => {
-    resetContextCache();
-  });
-
-  afterEach(() => {
-    resetContextCache();
-  });
-
-  // -------------------------------------------------------------------------
-  // 1. Tool factory shape
-  // -------------------------------------------------------------------------
-  describe('createAllTools()', () => {
-    it('returns array of 5 tools', () => {
-      const configDir = createTempConfigDir();
-      const tools = createAllTools({ agentDir: configDir });
-      expect(tools).toHaveLength(5);
-    });
-
-    it('each tool has required properties', () => {
-      const configDir = createTempConfigDir();
-      const tools = createAllTools({ agentDir: configDir });
-      for (const tool of tools) {
-        expect(tool).toHaveProperty('name');
-        expect(tool).toHaveProperty('label');
-        expect(tool).toHaveProperty('description');
-        expect(tool).toHaveProperty('parameters');
-        expect(tool).toHaveProperty('execute');
-        expect(typeof tool.name).toBe('string');
-        expect(typeof tool.label).toBe('string');
-        expect(typeof tool.description).toBe('string');
-        expect(typeof tool.parameters).toBe('object');
-        expect(typeof tool.execute).toBe('function');
-      }
-    });
-  });
-
-  // -------------------------------------------------------------------------
-  // 2. Tool names — kebab-case with agentbnb- prefix
-  // -------------------------------------------------------------------------
-  describe('tool names', () => {
-    it('all names follow agentbnb-<action> pattern', () => {
-      const configDir = createTempConfigDir();
-      const tools = createAllTools({ agentDir: configDir });
-      const names = tools.map((t) => t.name);
-
-      expect(names).toEqual([
-        'agentbnb-discover',
-        'agentbnb-request',
-        'agentbnb-conduct',
-        'agentbnb-status',
-        'agentbnb-publish',
-      ]);
-
-      for (const name of names) {
-        expect(name).toMatch(/^agentbnb-[a-z]+$/);
-      }
-    });
-  });
-
-  // -------------------------------------------------------------------------
-  // 3. Parameter schemas
-  // -------------------------------------------------------------------------
-  describe('parameter schemas', () => {
-    it('discover requires query', () => {
-      const configDir = createTempConfigDir();
-      const tool = createDiscoverTool({ agentDir: configDir });
-      const schema = tool.parameters as { required: string[]; properties: Record<string, unknown> };
-      expect(schema.required).toContain('query');
-      expect(schema.properties).toHaveProperty('query');
-      expect(schema.properties).toHaveProperty('level');
-      expect(schema.properties).toHaveProperty('online_only');
-    });
-
-    it('request has no required fields', () => {
-      const configDir = createTempConfigDir();
-      const tool = createRequestTool({ agentDir: configDir });
-      const schema = tool.parameters as { required: string[] };
-      expect(schema.required).toEqual([]);
-    });
-
-    it('conduct requires task', () => {
-      const configDir = createTempConfigDir();
-      const tool = createConductTool({ agentDir: configDir });
-      const schema = tool.parameters as { required: string[] };
-      expect(schema.required).toContain('task');
-    });
-
-    it('status has no required fields', () => {
-      const configDir = createTempConfigDir();
-      const tool = createStatusTool({ agentDir: configDir });
-      const schema = tool.parameters as { required: string[]; properties: Record<string, unknown> };
-      expect(schema.required).toEqual([]);
-      expect(Object.keys(schema.properties)).toHaveLength(0);
-    });
-
-    it('publish requires card_json', () => {
-      const configDir = createTempConfigDir();
-      const tool = createPublishTool({ agentDir: configDir });
-      const schema = tool.parameters as { required: string[] };
-      expect(schema.required).toContain('card_json');
-    });
-  });
-
-  // -------------------------------------------------------------------------
-  // 4. Result conversion
-  // -------------------------------------------------------------------------
-  describe('toAgentToolResult()', () => {
-    it('converts MCP format to AgentToolResult with parsed details', () => {
-      const mcpResult = {
-        content: [{ type: 'text' as const, text: '{"results":[],"count":0}' }],
-      };
-      const result = toAgentToolResult(mcpResult);
-      expect(result.content).toBe('{"results":[],"count":0}');
-      expect(result.details).toEqual({ results: [], count: 0 });
-    });
-
-    it('handles non-JSON text gracefully', () => {
-      const mcpResult = {
-        content: [{ type: 'text' as const, text: 'not json' }],
-      };
-      const result = toAgentToolResult(mcpResult);
-      expect(result.content).toBe('not json');
-      expect(result.details).toBeUndefined();
-    });
-
-    it('handles empty content array', () => {
-      const mcpResult = { content: [] as Array<{ type: string; text: string }> };
-      const result = toAgentToolResult(mcpResult);
-      expect(result.content).toBe('{}');
-    });
-  });
-
-  // -------------------------------------------------------------------------
-  // 5. Context caching
-  // -------------------------------------------------------------------------
-  describe('context caching', () => {
-    it('same configDir reuses cached context', () => {
-      const configDir = createTempConfigDir();
-      const ctx1 = buildMcpContext({ agentDir: configDir });
-      const ctx2 = buildMcpContext({ agentDir: configDir });
-      expect(ctx1).toBe(ctx2); // same reference
-    });
-
-    it('resetContextCache() clears the cache', () => {
-      const configDir = createTempConfigDir();
-      const ctx1 = buildMcpContext({ agentDir: configDir });
-      resetContextCache();
-      const ctx2 = buildMcpContext({ agentDir: configDir });
-      expect(ctx1).not.toBe(ctx2); // different reference
-    });
-  });
-
-  // -------------------------------------------------------------------------
-  // 6. Multi-agent isolation
-  // -------------------------------------------------------------------------
-  describe('multi-agent isolation', () => {
-    it('different workspaceDir values produce different contexts', () => {
-      const dir1 = createTempConfigDir();
-      const dir2 = createTempConfigDir();
-
-      const ctx1 = buildMcpContext({ agentDir: dir1 });
-      const ctx2 = buildMcpContext({ agentDir: dir2 });
-
-      expect(ctx1).not.toBe(ctx2);
-      expect(ctx1.configDir).not.toBe(ctx2.configDir);
-    });
-  });
-
-  // -------------------------------------------------------------------------
-  // 7. resolveConfigDir
-  // -------------------------------------------------------------------------
-  describe('resolveConfigDir()', () => {
-    it('uses agentDir directly if it ends with .agentbnb', () => {
-      const dir = resolveConfigDir({ agentDir: '/tmp/test/.agentbnb' });
-      expect(dir).toBe('/tmp/test/.agentbnb');
-    });
-
-    it('appends .agentbnb to agentDir if it does not end with .agentbnb', () => {
-      const dir = resolveConfigDir({ agentDir: '/tmp/test-agent' });
-      expect(dir).toBe('/tmp/test-agent/.agentbnb');
-    });
-
-    it('derives from workspaceDir when agentDir is not set', () => {
-      const dir = resolveConfigDir({ workspaceDir: '/tmp/workspace' });
-      expect(dir).toBe('/tmp/workspace/.agentbnb');
-    });
-
-    it('falls back to ~/.agentbnb when neither is set', () => {
-      const dir = resolveConfigDir({});
-      expect(dir).toMatch(/\.agentbnb$/);
-    });
-  });
-
-  // -------------------------------------------------------------------------
-  // 8. buildMcpContext error on missing config
-  // -------------------------------------------------------------------------
-  describe('buildMcpContext()', () => {
-    it('throws when config.json does not exist', () => {
-      const dir = mkdtempSync(join(tmpdir(), 'agentbnb-noconfig-'));
-      expect(() => buildMcpContext({ agentDir: dir })).toThrow(/not initialized/);
-    });
-  });
-
-  // -------------------------------------------------------------------------
-  // 9. Tool execution delegates to MCP handlers
-  // -------------------------------------------------------------------------
-  describe('tool execution', () => {
-    it('discover tool delegates to handleDiscover', async () => {
-      const configDir = createTempConfigDir();
-      const tool = createDiscoverTool({ agentDir: configDir });
-      const result = await tool.execute('call-1', { query: 'stock analysis' });
-      expect(result.content).toBeTruthy();
-      expect(result.details).toEqual({ results: [], count: 0 });
-    });
-
-    it('status tool delegates to handleStatus', async () => {
-      const configDir = createTempConfigDir();
-      const tool = createStatusTool({ agentDir: configDir });
-      const result = await tool.execute('call-2', {});
-      expect(result.details).toEqual({ agent_id: 'mock', balance: 50 });
-    });
-  });
-});