Revert "🤖 debug: remove hook test to check CI"

This reverts commit cfab8bb.

Signed-off-by: Thomas Kosiewski <tk@coder.com>

Author: ThomasK33

src/browser/hooks/useAvailableScripts.test.ts

@@ -0,0 +1,100 @@
+import { GlobalWindow } from "happy-dom";
+
+const win = new GlobalWindow();
+// @ts-expect-error -- Patching global for test environment
+global.window = win;
+// @ts-expect-error -- Patching global for test environment
+global.document = win.document;
+// @ts-expect-error -- Patching global for test environment
+global.navigator = win.navigator;
+
+import { describe, it, expect, beforeEach, afterEach, mock, type Mock } from "bun:test";
+import { renderHook, waitFor } from "@testing-library/react";
+import { useAvailableScripts } from "./useAvailableScripts";
+
+// Define types for our mock
+type ListScriptsMock = Mock<() => Promise<{ success: boolean; data?: unknown[]; error?: string }>>;
+
+describe("useAvailableScripts", () => {
+  const mockListScripts = mock() as unknown as ListScriptsMock;
+
+  beforeEach(() => {
+    // Mock window.api attached to the global window
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access
+    (global.window as any).api = {
+      workspace: {
+        listScripts: mockListScripts,
+      },
+    };
+  });
+
+  afterEach(() => {
+    mock.restore();
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access
+    delete (global.window as any).api;
+  });
+
+  it("should fetch and return executable scripts", async () => {
+    mockListScripts.mockResolvedValue({
+      success: true,
+      data: [
+        { name: "script1", isExecutable: true, description: "desc1" },
+        { name: "script2", isExecutable: false, description: "desc2" },
+      ],
+    });
+
+    const { result } = renderHook(() => useAvailableScripts("ws-1"));
+
+    // Initial state
+    expect(result.current).toEqual([]);
+
+    await waitFor(() => {
+      expect(result.current).toHaveLength(1);
+    });
+
+    expect(result.current).toEqual([{ name: "script1", description: "desc1" }]);
+  });
+
+  it("should clear scripts immediately when workspaceId changes", async () => {
+    // First load
+    mockListScripts.mockResolvedValue({
+      success: true,
+      data: [{ name: "script1", isExecutable: true }],
+    });
+
+    const { result, rerender } = renderHook(({ wsId }) => useAvailableScripts(wsId), {
+      initialProps: { wsId: "ws-1" as string | null },
+    });
+
+    await waitFor(() => {
+      expect(result.current).toHaveLength(1);
+    });
+
+    // Prepare for second load (hangs to prove clearance)
+    mockListScripts.mockReturnValue(
+      new Promise(() => {
+        // Never resolve
+      })
+    );
+
+    rerender({ wsId: "ws-2" });
+
+    // Should be cleared immediately
+    expect(result.current).toEqual([]);
+  });
+
+  it("should clear scripts if listing fails", async () => {
+    mockListScripts.mockResolvedValue({
+      success: false,
+      error: "Failed",
+    });
+
+    const { result } = renderHook(() => useAvailableScripts("ws-1"));
+
+    await waitFor(() => {
+      expect(mockListScripts).toHaveBeenCalled();
+    });
+
+    expect(result.current).toEqual([]);
+  });
+});