test: add tab-lock-manager tests and panel webhook coverage

Author: hydro13

src/agents/tests/tab-lock-manager.test.ts

@@ -0,0 +1,72 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { TabLockManager } from '../tab-lock-manager';
+
+describe('TabLockManager', () => {
+  let lm: TabLockManager;
+
+  beforeEach(() => {
+    lm = new TabLockManager();
+  });
+
+  describe('acquire()', () => {
+    it('user always acquires, even when locked by agent', () => {
+      lm.acquire('tab-1', 'claude');
+      const result = lm.acquire('tab-1', 'user');
+      expect(result.acquired).toBe(true);
+      expect(lm.getOwner('tab-1')).toBe('user');
+    });
+
+    it('emits lock-overridden when user overrides agent lock', () => {
+      const handler = vi.fn();
+      lm.on('lock-overridden', handler);
+
+      lm.acquire('tab-1', 'claude');
+      lm.acquire('tab-1', 'user');
+
+      expect(handler).toHaveBeenCalledWith({
+        tabId: 'tab-1',
+        previousOwner: 'claude',
+        newOwner: 'user',
+      });
+    });
+
+    it('does not emit lock-overridden when user acquires unlocked tab', () => {
+      const handler = vi.fn();
+      lm.on('lock-overridden', handler);
+
+      lm.acquire('tab-1', 'user');
+      expect(handler).not.toHaveBeenCalled();
+    });
+
+    it('agent cannot acquire tab locked by another agent', () => {
+      lm.acquire('tab-1', 'claude');
+      const result = lm.acquire('tab-1', 'other-agent');
+      expect(result.acquired).toBe(false);
+      expect(result.owner).toBe('claude');
+    });
+
+    it('agent can renew own lock', () => {
+      lm.acquire('tab-1', 'claude');
+      const result = lm.acquire('tab-1', 'claude');
+      expect(result.acquired).toBe(true);
+    });
+  });
+
+  describe('release()', () => {
+    it('user can release any lock', () => {
+      lm.acquire('tab-1', 'claude');
+      expect(lm.release('tab-1', 'user')).toBe(true);
+      expect(lm.isLocked('tab-1')).toBe(false);
+    });
+
+    it('agent cannot release another agent lock', () => {
+      lm.acquire('tab-1', 'claude');
+      expect(lm.release('tab-1', 'other-agent')).toBe(false);
+      expect(lm.isLocked('tab-1')).toBe(true);
+    });
+
+    it('returns true for unlocked tab', () => {
+      expect(lm.release('tab-1', 'claude')).toBe(true);
+    });
+  });
+});

src/panel/tests/manager.test.ts

@@ -89,6 +89,56 @@ describe('PanelManager reply notifications', () => {
     expect(lastTyping[1]).toEqual({ typing: false });
   });
 
+  it('does not fire webhook for wingman messages', async () => {
+    const win = createWindowStub();
+    const manager = new PanelManager(win as never);
+
+    const mockConfigManager = {
+      getConfig: vi.fn().mockReturnValue({
+        webhook: { enabled: true, url: 'http://localhost:9999', notifyOnRobinChat: true },
+      }),
+    };
+    (manager as any).configManager = mockConfigManager;
+
+    const fetchSpy = vi.spyOn(globalThis, 'fetch').mockResolvedValue({ ok: true } as Response);
+
+    manager.addChatMessage('wingman', 'AI response');
+
+    // Give the async fireWebhook a tick to run
+    await new Promise(r => setTimeout(r, 10));
+
+    expect(fetchSpy).not.toHaveBeenCalled();
+    fetchSpy.mockRestore();
+  });
+
+  it('fires webhook for user messages when configured', async () => {
+    const win = createWindowStub();
+    const manager = new PanelManager(win as never);
+
+    const mockConfigManager = {
+      getConfig: vi.fn().mockReturnValue({
+        webhook: { enabled: true, url: 'http://localhost:9999', notifyOnRobinChat: true },
+      }),
+    };
+    (manager as any).configManager = mockConfigManager;
+
+    const fetchSpy = vi.spyOn(globalThis, 'fetch').mockResolvedValue({ ok: true } as Response);
+
+    manager.addChatMessage('user', 'Hello from user');
+
+    await new Promise(r => setTimeout(r, 10));
+
+    expect(fetchSpy).toHaveBeenCalledTimes(1);
+    expect(fetchSpy).toHaveBeenCalledWith(
+      'http://localhost:9999/hooks/wake',
+      expect.objectContaining({
+        method: 'POST',
+        body: expect.stringContaining('Hello from user'),
+      }),
+    );
+    fetchSpy.mockRestore();
+  });
+
   it('falls back to an image message when there is no text', () => {
     const win = createWindowStub();
     const manager = new PanelManager(win as never);