@wopr-network/platform-ui-core 1.27.8 → 1.27.9

package/src/__tests__/notifications.test.tsx

@@ -32,8 +32,7 @@ const DEFAULT_PREFS: NotificationPreferences = {
 };
 
 const mockGetPrefs = vi.fn<() => Promise<NotificationPreferences>>();
-const mockUpdatePrefs =
-  vi.fn<(p: Partial<NotificationPreferences>) => Promise<NotificationPreferences>>();
+const mockUpdatePrefs = vi.fn<(p: Partial<NotificationPreferences>) => Promise<NotificationPreferences>>();
 
 vi.mock("@/lib/settings-api", async (importOriginal) => {
   const actual = await importOriginal<typeof import("@/lib/settings-api")>();
@@ -60,9 +59,7 @@ describe("Notifications page - toggle behavior", () => {
     mockUpdatePrefs.mockResolvedValue({ ...DEFAULT_PREFS, billing_low_balance: false });
 
     const user = userEvent.setup();
-    const { default: NotificationsPage } = await import(
-      "../app/(dashboard)/settings/notifications/page"
-    );
+    const { default: NotificationsPage } = await import("../app/(dashboard)/settings/notifications/page");
     render(<NotificationsPage />);
 
     const toggle = await screen.findByRole("switch", { name: "Low balance alerts" });
@@ -90,9 +87,7 @@ describe("Notifications page - toggle behavior", () => {
     mockUpdatePrefs.mockReturnValueOnce(pending);
 
     const user = userEvent.setup();
-    const { default: NotificationsPage } = await import(
-      "../app/(dashboard)/settings/notifications/page"
-    );
+    const { default: NotificationsPage } = await import("../app/(dashboard)/settings/notifications/page");
     render(<NotificationsPage />);
 
     const toggle = await screen.findByRole("switch", { name: "Low balance alerts" });
@@ -111,9 +106,7 @@ describe("Notifications page - toggle behavior", () => {
     mockUpdatePrefs.mockRejectedValueOnce(new Error("Network error"));
 
     const user = userEvent.setup();
-    const { default: NotificationsPage } = await import(
-      "../app/(dashboard)/settings/notifications/page"
-    );
+    const { default: NotificationsPage } = await import("../app/(dashboard)/settings/notifications/page");
     render(<NotificationsPage />);
 
     const toggle = await screen.findByRole("switch", { name: "Low balance alerts" });

package/src/__tests__/oauth-buttons.test.tsx

@@ -1,14 +1,8 @@
 import { render, screen } from "@testing-library/react";
-import { beforeEach, describe, expect, it, vi } from "vitest";
+import { afterEach, describe, expect, it, vi } from "vitest";
 
-vi.mock("@/lib/trpc", () => ({
-  trpc: {
-    authSocial: {
-      enabledSocialProviders: {
-        useQuery: vi.fn(),
-      },
-    },
-  },
+vi.mock("@/lib/api-config", () => ({
+  API_BASE_URL: "https://api.test/api",
 }));
 
 vi.mock("@/lib/auth-client", () => ({
@@ -18,84 +12,67 @@ vi.mock("@/lib/auth-client", () => ({
 }));
 
 import { OAuthButtons } from "@/components/oauth-buttons";
-import { trpc } from "@/lib/trpc";
 
 describe("OAuthButtons", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
+  afterEach(() => {
+    vi.unstubAllGlobals();
   });
 
   it("renders nothing while loading", () => {
-    vi.mocked(trpc.authSocial.enabledSocialProviders.useQuery).mockReturnValue({
-      data: undefined,
-      isLoading: true,
-    } as ReturnType<typeof trpc.authSocial.enabledSocialProviders.useQuery>);
-
+    // fetch never resolves — component stays in loading state
+    vi.stubGlobal("fetch", vi.fn().mockReturnValue(new Promise(() => {})));
     const { container } = render(<OAuthButtons />);
     expect(container.querySelector("button")).toBeNull();
   });
 
-  it("renders nothing when no providers are enabled", () => {
-    vi.mocked(trpc.authSocial.enabledSocialProviders.useQuery).mockReturnValue({
-      data: [],
-      isLoading: false,
-    } as ReturnType<typeof trpc.authSocial.enabledSocialProviders.useQuery>);
-
+  it("renders nothing when no providers are enabled", async () => {
+    vi.stubGlobal("fetch", vi.fn().mockResolvedValue({ ok: true, json: () => Promise.resolve([]) }));
     const { container } = render(<OAuthButtons />);
-    expect(container.querySelector("button")).toBeNull();
+    // Wait for useEffect to settle
+    await vi.waitFor(() => {
+      expect(container.querySelector("button")).toBeNull();
+    });
   });
 
-  it("renders nothing when data is undefined and not loading", () => {
-    vi.mocked(trpc.authSocial.enabledSocialProviders.useQuery).mockReturnValue({
-      data: undefined,
-      isLoading: false,
-    } as ReturnType<typeof trpc.authSocial.enabledSocialProviders.useQuery>);
-
+  it("renders nothing when fetch fails", async () => {
+    vi.stubGlobal("fetch", vi.fn().mockRejectedValue(new Error("Network error")));
     const { container } = render(<OAuthButtons />);
-    expect(container.querySelector("button")).toBeNull();
+    await vi.waitFor(() => {
+      expect(container.querySelector("button")).toBeNull();
+    });
   });
 
-  it("renders only enabled providers", () => {
-    vi.mocked(trpc.authSocial.enabledSocialProviders.useQuery).mockReturnValue({
-      data: ["github", "discord"],
-      isLoading: false,
-    } as ReturnType<typeof trpc.authSocial.enabledSocialProviders.useQuery>);
-
+  it("renders only enabled providers", async () => {
+    vi.stubGlobal("fetch", vi.fn().mockResolvedValue({ ok: true, json: () => Promise.resolve(["github", "discord"]) }));
     render(<OAuthButtons />);
-    expect(screen.getByRole("button", { name: "Continue with GitHub" })).toBeInTheDocument();
+    expect(await screen.findByRole("button", { name: "Continue with GitHub" })).toBeInTheDocument();
     expect(screen.getByRole("button", { name: "Continue with Discord" })).toBeInTheDocument();
     expect(screen.queryByRole("button", { name: /Google/ })).not.toBeInTheDocument();
   });
 
-  it("renders all three providers when all are enabled", () => {
-    vi.mocked(trpc.authSocial.enabledSocialProviders.useQuery).mockReturnValue({
-      data: ["github", "discord", "google"],
-      isLoading: false,
-    } as ReturnType<typeof trpc.authSocial.enabledSocialProviders.useQuery>);
-
+  it("renders all three providers when all are enabled", async () => {
+    vi.stubGlobal(
+      "fetch",
+      vi.fn().mockResolvedValue({ ok: true, json: () => Promise.resolve(["github", "discord", "google"]) }),
+    );
     render(<OAuthButtons />);
-    expect(screen.getByRole("button", { name: "Continue with GitHub" })).toBeInTheDocument();
+    expect(await screen.findByRole("button", { name: "Continue with GitHub" })).toBeInTheDocument();
     expect(screen.getByRole("button", { name: "Continue with Discord" })).toBeInTheDocument();
     expect(screen.getByRole("button", { name: "Continue with Google" })).toBeInTheDocument();
   });
 
-  it("renders the separator when providers are available", () => {
-    vi.mocked(trpc.authSocial.enabledSocialProviders.useQuery).mockReturnValue({
-      data: ["github"],
-      isLoading: false,
-    } as ReturnType<typeof trpc.authSocial.enabledSocialProviders.useQuery>);
-
+  it("renders the separator when providers are available", async () => {
+    vi.stubGlobal("fetch", vi.fn().mockResolvedValue({ ok: true, json: () => Promise.resolve(["github"]) }));
     render(<OAuthButtons />);
-    expect(screen.getByText(/or continue with/i)).toBeInTheDocument();
+    expect(await screen.findByText(/or continue with/i)).toBeInTheDocument();
   });
 
-  it("does not render the separator when no providers", () => {
-    vi.mocked(trpc.authSocial.enabledSocialProviders.useQuery).mockReturnValue({
-      data: [],
-      isLoading: false,
-    } as ReturnType<typeof trpc.authSocial.enabledSocialProviders.useQuery>);
-
+  it("does not render the separator when no providers", async () => {
+    vi.stubGlobal("fetch", vi.fn().mockResolvedValue({ ok: true, json: () => Promise.resolve([]) }));
     render(<OAuthButtons />);
-    expect(screen.queryByText(/or continue with/i)).not.toBeInTheDocument();
+    // Wait for the fetch to resolve and state to settle
+    await vi.waitFor(() => {
+      expect(screen.queryByText(/or continue with/i)).not.toBeInTheDocument();
+    });
   });
 });

package/src/__tests__/oauth-error-mapping.test.tsx

@@ -56,9 +56,7 @@ describe("getOAuthErrorMessage", () => {
   });
 
   it("returns generic fallback for unknown code", () => {
-    expect(getOAuthErrorMessage("something_random")).toBe(
-      "Authentication failed. Please try again.",
-    );
+    expect(getOAuthErrorMessage("something_random")).toBe("Authentication failed. Please try again.");
   });
 
   it("returns generic fallback for malicious input — never reflects raw value", () => {
@@ -76,9 +74,7 @@ const { default: OAuthCallbackPage } = await import("@/app/auth/callback/[provid
 describe("OAuthCallbackPage — error reflection", () => {
   it("never renders raw malicious error param in the DOM", () => {
     const malicious = "<img src=x onerror=alert(1)>";
-    vi.mocked(useSearchParams).mockReturnValue(
-      new URLSearchParams(`error=${encodeURIComponent(malicious)}`) as never,
-    );
+    vi.mocked(useSearchParams).mockReturnValue(new URLSearchParams(`error=${encodeURIComponent(malicious)}`) as never);
 
     render(<OAuthCallbackPage />);
 

package/src/__tests__/observability.test.tsx

@@ -8,12 +8,8 @@ vi.mock("recharts", () => ({
   ResponsiveContainer: ({ children }: { children: React.ReactNode }) => (
     <div data-testid="responsive-container">{children}</div>
   ),
-  LineChart: ({ children }: { children: React.ReactNode }) => (
-    <div data-testid="line-chart">{children}</div>
-  ),
-  BarChart: ({ children }: { children: React.ReactNode }) => (
-    <div data-testid="bar-chart">{children}</div>
-  ),
+  LineChart: ({ children }: { children: React.ReactNode }) => <div data-testid="line-chart">{children}</div>,
+  BarChart: ({ children }: { children: React.ReactNode }) => <div data-testid="bar-chart">{children}</div>,
   Line: () => <div data-testid="line" />,
   Bar: () => <div data-testid="bar" />,
   XAxis: () => <div data-testid="x-axis" />,
@@ -117,9 +113,7 @@ vi.mock("@/lib/api", () => ({
         memoryMb: 256,
       },
     ],
-    tokenUsage: [
-      { provider: "anthropic", inputTokens: 125000, outputTokens: 89000, totalCost: 4.28 },
-    ],
+    tokenUsage: [{ provider: "anthropic", inputTokens: 125000, outputTokens: 89000, totalCost: 4.28 }],
     pluginEvents: [{ plugin: "memory", count: 340 }],
   }),
   getImageStatus: vi.fn().mockResolvedValue({
@@ -128,19 +122,16 @@ vi.mock("@/lib/api", () => ({
     updateAvailable: false,
   }),
   pullImageUpdate: vi.fn().mockResolvedValue(undefined),
-  mapBotStatusToFleetInstance: vi.fn(
-    (bot: { id: string; name: string; state: string; health: string | null }) => ({
-      id: bot.id,
-      name: bot.name,
-      status: bot.state === "running" ? "running" : "stopped",
-      health:
-        bot.health === "healthy" ? "healthy" : bot.health === "degraded" ? "degraded" : "degraded",
-      uptime: 86400,
-      pluginCount: 2,
-      sessionCount: bot.health === "healthy" ? 2 : 0,
-      provider: bot.health === "healthy" ? "anthropic" : "openai",
-    }),
-  ),
+  mapBotStatusToFleetInstance: vi.fn((bot: { id: string; name: string; state: string; health: string | null }) => ({
+    id: bot.id,
+    name: bot.name,
+    status: bot.state === "running" ? "running" : "stopped",
+    health: bot.health === "healthy" ? "healthy" : bot.health === "degraded" ? "degraded" : "degraded",
+    uptime: 86400,
+    pluginCount: 2,
+    sessionCount: bot.health === "healthy" ? 2 : 0,
+    provider: bot.health === "healthy" ? "anthropic" : "openai",
+  })),
 }));
 
 vi.mock("framer-motion", () => ({
@@ -215,17 +206,15 @@ describe("HealthOverview", () => {
   it("recovers from error on retry", async () => {
     const user = userEvent.setup();
     const { getInstanceHealth } = await import("@/lib/api");
-    vi.mocked(getInstanceHealth)
-      .mockRejectedValueOnce(new Error("Network error"))
-      .mockResolvedValueOnce({
-        status: "healthy",
-        uptime: 86400,
-        activeSessions: 2,
-        totalSessions: 47,
-        plugins: [],
-        providers: [],
-        history: [],
-      });
+    vi.mocked(getInstanceHealth).mockRejectedValueOnce(new Error("Network error")).mockResolvedValueOnce({
+      status: "healthy",
+      uptime: 86400,
+      activeSessions: 2,
+      totalSessions: 47,
+      plugins: [],
+      providers: [],
+      history: [],
+    });
 
     render(<HealthOverview instanceId="inst-001" />);
     await waitFor(() => {
@@ -377,9 +366,7 @@ describe("MetricsDashboard", () => {
             memoryMb: 256,
           },
         ],
-        tokenUsage: [
-          { provider: "anthropic", inputTokens: 125000, outputTokens: 89000, totalCost: 4.28 },
-        ],
+        tokenUsage: [{ provider: "anthropic", inputTokens: 125000, outputTokens: 89000, totalCost: 4.28 }],
         pluginEvents: [{ plugin: "memory", count: 340 }],
       });
 

package/src/__tests__/onboarding-page.test.tsx

@@ -9,9 +9,7 @@ vi.mock("next/navigation", () => ({
 
 vi.mock("framer-motion", () => ({
   motion: {
-    div: ({ children, ...props }: React.HTMLAttributes<HTMLDivElement>) => (
-      <div {...props}>{children}</div>
-    ),
+    div: ({ children, ...props }: React.HTMLAttributes<HTMLDivElement>) => <div {...props}>{children}</div>,
   },
   AnimatePresence: ({ children }: { children: React.ReactNode }) => <>{children}</>,
 }));
@@ -74,11 +72,11 @@ describe("OnboardingPage", () => {
     expect(screen.getByRole("textbox")).toBeInTheDocument();
   });
 
-  it("redirects to /marketplace if onboarding already complete", async () => {
+  it("redirects to homePath if onboarding already complete", async () => {
     mockIsComplete = true;
     render(<OnboardingPage />);
     await waitFor(() => {
-      expect(mockPush).toHaveBeenCalledWith("/marketplace");
+      expect(mockPush).toHaveBeenCalledWith("/");
     });
   });
 
@@ -108,6 +106,6 @@ describe("OnboardingPage", () => {
 
     expect(mockMarkComplete).toHaveBeenCalled();
     expect(mockSaveState).toHaveBeenCalled();
-    expect(mockPush).toHaveBeenCalledWith("/marketplace");
+    expect(mockPush).toHaveBeenCalledWith("/");
   });
 });

package/src/__tests__/org-billing-api.test.tsx

@@ -56,12 +56,7 @@ describe("org-billing-api", () => {
   });
 
   it("createOrgTopupCheckout returns checkout URL", async () => {
-    const result = await createOrgTopupCheckout(
-      "org-1",
-      "price_1",
-      "https://ok.com",
-      "https://cancel.com",
-    );
+    const result = await createOrgTopupCheckout("org-1", "price_1", "https://ok.com", "https://cancel.com");
     expect(result.url).toContain("stripe.com");
   });
 });

package/src/__tests__/plugin-install-flow.test.tsx

@@ -43,33 +43,25 @@ vi.mock("framer-motion", () => {
 });
 
 // vi.hoisted runs before module imports so TEST_PLUGINS and mocks are available in vi.mock factories
-const {
-  TEST_PLUGINS,
-  ALL_PLUGINS,
-  mockInstallPlugin,
-  mockListBots,
-  mockListInstalledPlugins,
-  injectPathlessZodError,
-} = vi.hoisted(() => {
-  const mockInstallPlugin = vi.fn();
-  const mockListBots = vi
-    .fn()
-    .mockResolvedValue([{ id: "bot-001", name: "My Bot", state: "running" }]);
-  const mockListInstalledPlugins = vi.fn().mockResolvedValue([]);
-  // eslint-disable-next-line @typescript-eslint/no-require-imports
-  const { INSTALL_FLOW_TEST_PLUGINS, MARKETPLACE_TEST_PLUGINS } =
-    require("./fixtures/mock-manifests-data") as typeof import("./fixtures/mock-manifests");
-  // Flag to conditionally inject a path-less Zod error in the mocked z.object
-  const injectPathlessZodError = { value: false };
-  return {
-    TEST_PLUGINS: INSTALL_FLOW_TEST_PLUGINS,
-    ALL_PLUGINS: MARKETPLACE_TEST_PLUGINS,
-    mockInstallPlugin,
-    mockListBots,
-    mockListInstalledPlugins,
-    injectPathlessZodError,
-  };
-});
+const { TEST_PLUGINS, ALL_PLUGINS, mockInstallPlugin, mockListBots, mockListInstalledPlugins, injectPathlessZodError } =
+  vi.hoisted(() => {
+    const mockInstallPlugin = vi.fn();
+    const mockListBots = vi.fn().mockResolvedValue([{ id: "bot-001", name: "My Bot", state: "running" }]);
+    const mockListInstalledPlugins = vi.fn().mockResolvedValue([]);
+    // eslint-disable-next-line @typescript-eslint/no-require-imports
+    const { INSTALL_FLOW_TEST_PLUGINS, MARKETPLACE_TEST_PLUGINS } =
+      require("./fixtures/mock-manifests-data") as typeof import("./fixtures/mock-manifests");
+    // Flag to conditionally inject a path-less Zod error in the mocked z.object
+    const injectPathlessZodError = { value: false };
+    return {
+      TEST_PLUGINS: INSTALL_FLOW_TEST_PLUGINS,
+      ALL_PLUGINS: MARKETPLACE_TEST_PLUGINS,
+      mockInstallPlugin,
+      mockListBots,
+      mockListInstalledPlugins,
+      injectPathlessZodError,
+    };
+  });
 
 // Mock zod with a conditional wrapper: when injectPathlessZodError.value is true,
 // z.object() adds a superRefine that produces a path-less ZodIssue.
@@ -78,10 +70,7 @@ const {
 vi.mock("zod", async () => {
   const realZod = await vi.importActual<typeof import("zod")>("zod");
   const originalObject = realZod.z.object.bind(realZod.z);
-  const wrappedObject = (
-    shape?: Record<string, unknown>,
-    params?: string | Record<string, unknown>,
-  ) => {
+  const wrappedObject = (shape?: Record<string, unknown>, params?: string | Record<string, unknown>) => {
     const schema = originalObject(shape, params as undefined);
     if (!injectPathlessZodError.value || !shape || Object.keys(shape).length === 0) return schema;
     return schema.superRefine((_val, ctx) => {
@@ -105,11 +94,7 @@ vi.mock("next/navigation", () => ({
 }));
 
 vi.mock("next/link", () => ({
-  default: ({
-    children,
-    href,
-    ...props
-  }: { children: React.ReactNode; href: string } & Record<string, unknown>) => (
+  default: ({ children, href, ...props }: { children: React.ReactNode; href: string } & Record<string, unknown>) => (
     <a href={href} {...props}>
       {children}
     </a>
@@ -197,9 +182,7 @@ describe("Plugin Install Flow", () => {
 
   it("renders install button on plugin detail page for uninstalled plugin", async () => {
     mockParams.plugin = "webhooks";
-    const { default: PluginDetailPage } = await import(
-      "../app/(dashboard)/marketplace/[plugin]/page"
-    );
+    const { default: PluginDetailPage } = await import("../app/(dashboard)/marketplace/[plugin]/page");
     renderWithQueryClient(<PluginDetailPage />);
 
     expect(await screen.findByText("Webhooks")).toBeInTheDocument();
@@ -211,9 +194,7 @@ describe("Plugin Install Flow", () => {
   it("opens install wizard when install button is clicked", async () => {
     const user = userEvent.setup();
     mockParams.plugin = "webhooks";
-    const { default: PluginDetailPage } = await import(
-      "../app/(dashboard)/marketplace/[plugin]/page"
-    );
+    const { default: PluginDetailPage } = await import("../app/(dashboard)/marketplace/[plugin]/page");
     renderWithQueryClient(<PluginDetailPage />);
 
     await screen.findByText("Webhooks");
@@ -228,9 +209,7 @@ describe("Plugin Install Flow", () => {
 
     const user = userEvent.setup();
     mockParams.plugin = "webhooks";
-    const { default: PluginDetailPage } = await import(
-      "../app/(dashboard)/marketplace/[plugin]/page"
-    );
+    const { default: PluginDetailPage } = await import("../app/(dashboard)/marketplace/[plugin]/page");
     renderWithQueryClient(<PluginDetailPage />);
 
     await screen.findByText("Webhooks");
@@ -268,9 +247,7 @@ describe("Plugin Install Flow", () => {
 
     const user = userEvent.setup();
     mockParams.plugin = "webhooks";
-    const { default: PluginDetailPage } = await import(
-      "../app/(dashboard)/marketplace/[plugin]/page"
-    );
+    const { default: PluginDetailPage } = await import("../app/(dashboard)/marketplace/[plugin]/page");
     renderWithQueryClient(<PluginDetailPage />);
 
     await screen.findByText("Webhooks");
@@ -295,9 +272,7 @@ describe("Plugin Install Flow", () => {
   it("wizard cancel button returns to detail page", async () => {
     const user = userEvent.setup();
     mockParams.plugin = "webhooks";
-    const { default: PluginDetailPage } = await import(
-      "../app/(dashboard)/marketplace/[plugin]/page"
-    );
+    const { default: PluginDetailPage } = await import("../app/(dashboard)/marketplace/[plugin]/page");
     renderWithQueryClient(<PluginDetailPage />);
 
     await screen.findByText("Webhooks");
@@ -353,9 +328,7 @@ describe("Plugin Toggle (Enable/Disable)", () => {
 
     // Complete phase reached — "This plugin requires the following dependencies:" was never shown
     expect(screen.getByText("Plugin installed successfully")).toBeInTheDocument();
-    expect(
-      screen.queryByText("This plugin requires the following dependencies:"),
-    ).not.toBeInTheDocument();
+    expect(screen.queryByText("This plugin requires the following dependencies:")).not.toBeInTheDocument();
   });
 
   it("InstallWizard shows requirements phase for plugins with dependencies", async () => {
@@ -424,10 +397,7 @@ describe("Plugin Toggle (Enable/Disable)", () => {
     expect(screen.getByText("Plugin installed successfully")).toBeInTheDocument();
     await user.click(screen.getByText("Done"));
 
-    expect(onComplete).toHaveBeenCalledWith(
-      "bot-001",
-      expect.objectContaining({ _providerChoices: {} }),
-    );
+    expect(onComplete).toHaveBeenCalledWith("bot-001", expect.objectContaining({ _providerChoices: {} }));
   });
 
   it("InstallWizard shows loading skeleton while bots are loading", async () => {

package/src/__tests__/plugin-registry.test.tsx

@@ -192,10 +192,7 @@ describe("usePluginRegistry", () => {
   it("provides derived pluginOptions from all categories", async () => {
     const { result } = renderHook(() => usePluginRegistry());
     await waitFor(() => expect(result.current.categoriesLoaded).toBe(true));
-    const totalCategoryPlugins = result.current.categories.reduce(
-      (sum, cat) => sum + cat.plugins.length,
-      0,
-    );
+    const totalCategoryPlugins = result.current.categories.reduce((sum, cat) => sum + cat.plugins.length, 0);
     expect(result.current.pluginOptions.length).toBe(totalCategoryPlugins);
   });
 
@@ -414,13 +411,8 @@ describe("usePluginRegistry", () => {
     await waitFor(() => expect(result.current.loading).toBe(false));
 
     const all = result.current.getAllPlugins();
-    const categoryPluginCount = result.current.categories.reduce(
-      (sum, cat) => sum + cat.plugins.length,
-      0,
-    );
-    expect(all.length).toBe(
-      result.current.channels.length + result.current.providers.length + categoryPluginCount,
-    );
+    const categoryPluginCount = result.current.categories.reduce((sum, cat) => sum + cat.plugins.length, 0);
+    expect(all.length).toBe(result.current.channels.length + result.current.providers.length + categoryPluginCount);
   });
 
   it("collectConfigFields returns config fields for selected channels", async () => {

package/src/__tests__/plugin-tool-sync.test.ts

@@ -51,9 +51,7 @@ describe("tool definitions stay in sync with handlers", () => {
     const handlerNames = new Set(handlerTools.map((t) => t.name));
 
     for (const def of getPlatformUIToolDefinitions()) {
-      expect(handlerNames.has(def.name), `Definition "${def.name}" has no matching handler`).toBe(
-        true,
-      );
+      expect(handlerNames.has(def.name), `Definition "${def.name}" has no matching handler`).toBe(true);
     }
   });
 });

package/src/__tests__/plugins-catalog-error.test.tsx

@@ -118,9 +118,7 @@ describe("plugins catalog error state", () => {
     };
 
     // First call fails, second succeeds
-    mockListMarketplacePlugins
-      .mockRejectedValueOnce(new Error("Network error"))
-      .mockResolvedValueOnce([catalogPlugin]);
+    mockListMarketplacePlugins.mockRejectedValueOnce(new Error("Network error")).mockResolvedValueOnce([catalogPlugin]);
 
     render(<PluginsPage />);
 
@@ -184,9 +182,7 @@ describe("plugins catalog error state", () => {
     };
 
     // First call fails, second succeeds
-    mockListMarketplacePlugins
-      .mockRejectedValueOnce(new Error("Network error"))
-      .mockResolvedValueOnce([catalogPlugin]);
+    mockListMarketplacePlugins.mockRejectedValueOnce(new Error("Network error")).mockResolvedValueOnce([catalogPlugin]);
 
     render(<PluginsPage />);
 

package/src/__tests__/plugins-toggle-race.test.tsx

@@ -98,9 +98,7 @@ describe("togglePlugin race condition", () => {
     vi.clearAllMocks();
     mockListInstalledPlugins.mockResolvedValue([{ pluginId: "plugin-1", enabled: true }]);
     // Make togglePluginEnabled slow so we can test in-flight guard
-    mockTogglePluginEnabled.mockImplementation(
-      () => new Promise((resolve) => setTimeout(resolve, 500)),
-    );
+    mockTogglePluginEnabled.mockImplementation(() => new Promise((resolve) => setTimeout(resolve, 500)));
   });
 
   it("blocks a second toggle while the first is in flight (stale closure exposes race)", async () => {
@@ -113,8 +111,8 @@ describe("togglePlugin race condition", () => {
 
     render(<PluginsPage />);
 
-    // Wait for the page to fully load
-    const toggle = await screen.findByRole("switch", { name: /toggle test plugin/i });
+    // Wait for the page to fully load — extended timeout for CI runners
+    const toggle = await screen.findByRole("switch", { name: /toggle test plugin/i }, { timeout: 5000 });
     expect(toggle).toBeInTheDocument();
 
     // Simulate rapid double-click by firing two change events synchronously

package/src/__tests__/portfolio-chart.test.tsx

@@ -7,9 +7,7 @@ describe("PortfolioChart", () => {
   it("renders a canvas element", () => {
     const milestoneRef = createRef<((label: string) => void) | null>();
     const fadeStartRef = createRef<(() => void) | null>();
-    const { container } = render(
-      <PortfolioChart onMilestoneRef={milestoneRef} onFadeStartRef={fadeStartRef} />,
-    );
+    const { container } = render(<PortfolioChart onMilestoneRef={milestoneRef} onFadeStartRef={fadeStartRef} />);
     const canvas = container.querySelector("canvas");
     expect(canvas).toBeInTheDocument();
   });
@@ -17,8 +15,6 @@ describe("PortfolioChart", () => {
   it("renders without crashing when refs are provided", () => {
     const milestoneRef = createRef<((label: string) => void) | null>();
     const fadeStartRef = createRef<(() => void) | null>();
-    expect(() =>
-      render(<PortfolioChart onMilestoneRef={milestoneRef} onFadeStartRef={fadeStartRef} />),
-    ).not.toThrow();
+    expect(() => render(<PortfolioChart onMilestoneRef={milestoneRef} onFadeStartRef={fadeStartRef} />)).not.toThrow();
   });
 });

package/src/__tests__/promotion-form.test.tsx

@@ -26,9 +26,7 @@ vi.mock("@/lib/trpc", () => ({
 
 vi.mock("framer-motion", () => ({
   motion: {
-    div: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => (
-      <div {...props}>{children}</div>
-    ),
+    div: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => <div {...props}>{children}</div>,
   },
 }));
 
@@ -59,9 +57,7 @@ vi.mock("@/components/ui/select", () => ({
       {children}
     </select>
   ),
-  SelectTrigger: ({ children, id }: { children: React.ReactNode; id?: string }) => (
-    <span id={id}>{children}</span>
-  ),
+  SelectTrigger: ({ children, id }: { children: React.ReactNode; id?: string }) => <span id={id}>{children}</span>,
   SelectValue: () => null,
   SelectContent: ({ children }: { children: React.ReactNode }) => <>{children}</>,
   SelectItem: ({ children, value }: { children: React.ReactNode; value: string }) => (

package/src/__tests__/promotions-list.test.tsx

@@ -31,9 +31,7 @@ vi.mock("@/lib/trpc", () => ({
 
 vi.mock("framer-motion", () => ({
   motion: {
-    div: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => (
-      <div {...props}>{children}</div>
-    ),
+    div: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => <div {...props}>{children}</div>,
   },
 }));