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

package/src/__tests__/api-client.test.ts

@@ -1,4 +1,5 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { trpcVanillaProxy } from "./setup.js";
 
 vi.mock("@/lib/api-config", () => ({
   API_BASE_URL: "https://test-api.local/api",
@@ -6,7 +7,7 @@ vi.mock("@/lib/api-config", () => ({
 }));
 
 vi.mock("@/lib/trpc", () => ({
-  trpcVanilla: {},
+  trpcVanilla: trpcVanillaProxy,
 }));
 
 const mockHandleUnauthorized = vi.fn(() => {
@@ -122,10 +123,7 @@ describe("apiFetch (via getProfile)", () => {
     const { getProfile } = await import("@/lib/api");
     await getProfile();
 
-    expect(mockFetch).toHaveBeenCalledWith(
-      "https://test-api.local/api/settings/profile",
-      expect.any(Object),
-    );
+    expect(mockFetch).toHaveBeenCalledWith("https://test-api.local/api/settings/profile", expect.any(Object));
   });
 
   it("throws on network failure", async () => {

package/src/__tests__/api-config.test.ts

@@ -1,15 +1,12 @@
 /**
- * Tests for api-config.ts — validateProductionApiUrl guard behaviour.
+ * Tests for api-config.ts — URL resolution behaviour.
  *
- * The module throws at import time if validation fails, so each test
+ * The module resolves API URL at import time, so each test
  * uses vi.resetModules() + a dynamic import to re-evaluate the module
  * with the desired environment variables.
  */
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 
-const VALID_PRODUCTION_URL = "https://api.example.com";
-const INTERNAL_URL = "http://localhost:3001";
-
 function setEnv(vars: Record<string, string | undefined>) {
   for (const [k, v] of Object.entries(vars)) {
     if (v === undefined) {
@@ -20,7 +17,7 @@ function setEnv(vars: Record<string, string | undefined>) {
   }
 }
 
-describe("validateProductionApiUrl", () => {
+describe("api-config URL resolution", () => {
   const originalEnv = { ...process.env };
 
   beforeEach(() => {
@@ -35,55 +32,38 @@ describe("validateProductionApiUrl", () => {
     Object.assign(process.env, originalEnv);
   });
 
-  it("does NOT throw in development regardless of URL", async () => {
+  it("uses NEXT_PUBLIC_API_URL when set", async () => {
     setEnv({
-      NODE_ENV: "development",
-      NEXT_RUNTIME: undefined,
-      NEXT_PHASE: undefined,
-      NEXT_PUBLIC_API_URL: INTERNAL_URL,
+      NEXT_PUBLIC_API_URL: "https://api.example.com",
     });
-    await expect(import("../lib/api-config")).resolves.toHaveProperty("API_BASE_URL");
+    const mod = await import("../lib/api-config");
+    expect(mod.PLATFORM_BASE_URL).toBe("https://api.example.com");
+    expect(mod.API_BASE_URL).toBe("https://api.example.com/api");
   });
 
-  it("does NOT throw during the Next.js build phase (NEXT_PHASE=phase-production-build)", async () => {
-    // This is the key regression test: CI sets NODE_ENV=production and
-    // NEXT_PUBLIC_API_URL=http://localhost:3001. The build must not crash.
+  it("falls back to localhost:3001 when no env var and no window", async () => {
     setEnv({
-      NODE_ENV: "production",
-      NEXT_PHASE: "phase-production-build",
-      NEXT_RUNTIME: "nodejs",
-      NEXT_PUBLIC_API_URL: INTERNAL_URL,
-    });
-    await expect(import("../lib/api-config")).resolves.toHaveProperty("API_BASE_URL");
-  });
-
-  it("throws at production runtime with an internal URL", async () => {
-    setEnv({
-      NODE_ENV: "production",
-      NEXT_PHASE: undefined,
-      NEXT_RUNTIME: "nodejs",
-      NEXT_PUBLIC_API_URL: INTERNAL_URL,
+      NEXT_PUBLIC_API_URL: undefined,
     });
-    await expect(import("../lib/api-config")).rejects.toThrow(/contains an internal hostname/);
+    const mod = await import("../lib/api-config");
+    // In test env (jsdom with localhost), should resolve to localhost
+    expect(mod.PLATFORM_BASE_URL).toContain("localhost");
   });
 
-  it("throws at production runtime when NEXT_PUBLIC_API_URL is not set", async () => {
+  it("exports SITE_URL from NEXT_PUBLIC_SITE_URL env var", async () => {
     setEnv({
-      NODE_ENV: "production",
-      NEXT_PHASE: undefined,
-      NEXT_RUNTIME: "nodejs",
-      NEXT_PUBLIC_API_URL: undefined,
+      NEXT_PUBLIC_SITE_URL: "https://mysite.com",
+      NEXT_PUBLIC_API_URL: "http://localhost:3001",
     });
-    await expect(import("../lib/api-config")).rejects.toThrow(/NEXT_PUBLIC_API_URL is not set/);
+    const mod = await import("../lib/api-config");
+    expect(mod.SITE_URL).toBe("https://mysite.com");
   });
 
-  it("does NOT throw at production runtime with a valid public HTTPS URL", async () => {
+  it("exports API_BASE_URL as PLATFORM_BASE_URL + /api", async () => {
     setEnv({
-      NODE_ENV: "production",
-      NEXT_PHASE: undefined,
-      NEXT_RUNTIME: "nodejs",
-      NEXT_PUBLIC_API_URL: VALID_PRODUCTION_URL,
+      NEXT_PUBLIC_API_URL: "https://api.test.com",
     });
-    await expect(import("../lib/api-config")).resolves.toHaveProperty("API_BASE_URL");
+    const mod = await import("../lib/api-config");
+    expect(mod.API_BASE_URL).toBe(`${mod.PLATFORM_BASE_URL}/api`);
   });
 });

package/src/__tests__/api-fleet-resources.test.ts

@@ -18,8 +18,7 @@ describe("getFleetResources", () => {
   it("calls /api/fleet/resources with credentials", async () => {
     mockFetch.mockResolvedValue({
       ok: true,
-      json: () =>
-        Promise.resolve({ totalCpuPercent: 42, totalMemoryMb: 512, memoryCapacityMb: 1024 }),
+      json: () => Promise.resolve({ totalCpuPercent: 42, totalMemoryMb: 512, memoryCapacityMb: 1024 }),
     });
 
     const { getFleetResources } = await import("@/lib/api");

package/src/__tests__/api-fleet-trpc.test.ts

@@ -61,10 +61,7 @@ describe("listInstances uses tRPC", () => {
     const result = await listInstances();
 
     expect(mockListInstances).toHaveBeenCalled();
-    expect(fetchSpy).not.toHaveBeenCalledWith(
-      expect.stringContaining("/fleet/"),
-      expect.anything(),
-    );
+    expect(fetchSpy).not.toHaveBeenCalledWith(expect.stringContaining("/fleet/"), expect.anything());
     expect(result).toHaveLength(1);
     expect(result[0].id).toBe("bot-1");
     expect(result[0].status).toBe("running");
@@ -187,10 +184,7 @@ describe("getInstanceLogs uses tRPC", () => {
 
   it("calls trpcVanilla.fleet.getInstanceLogs.query", async () => {
     mockGetInstanceLogs.mockResolvedValueOnce({
-      logs: [
-        "2026-02-20T10:00:00Z [INFO] Bot started",
-        "2026-02-20T10:00:01Z [ERROR] Connection failed",
-      ],
+      logs: ["2026-02-20T10:00:00Z [INFO] Bot started", "2026-02-20T10:00:01Z [ERROR] Connection failed"],
     });
 
     const { getInstanceLogs } = await import("@/lib/api");

package/src/__tests__/api-null-guards.test.ts

@@ -22,6 +22,7 @@ interface MockTrpcVanilla {
   billing: {
     creditsBalance: MockQuery;
     creditsHistory: MockQuery;
+    creditsDailySummary: MockQuery;
     creditOptions: MockQuery;
     affiliateStats: MockQuery;
     affiliateReferrals: MockQuery;
@@ -61,6 +62,7 @@ vi.mock("@/lib/trpc", () => ({
     billing: {
       creditsBalance: { query: vi.fn() },
       creditsHistory: { query: vi.fn() },
+      creditsDailySummary: { query: vi.fn() },
       creditOptions: { query: vi.fn() },
       affiliateStats: { query: vi.fn() },
       affiliateReferrals: { query: vi.fn() },
@@ -154,7 +156,7 @@ describe("API null guards", () => {
   it("getCreditHistory handles empty response", async () => {
     const { trpcVanilla } = await import("@/lib/trpc");
     const { billing } = trpcVanilla as unknown as MockTrpcVanilla;
-    billing.creditsHistory.query.mockResolvedValue({});
+    billing.creditsDailySummary.query.mockResolvedValue({});
 
     const { getCreditHistory } = await import("@/lib/api");
     const result = await getCreditHistory();

package/src/__tests__/audit-log-table-pagination.test.tsx

@@ -56,9 +56,7 @@ describe("AuditLogTable pagination", () => {
     render(<AuditLogTable />);
 
     await screen.findByText("Bot 0");
-    expect(mockFetchAuditLog).toHaveBeenCalledWith(
-      expect.objectContaining({ offset: 0, limit: 50 }),
-    );
+    expect(mockFetchAuditLog).toHaveBeenCalledWith(expect.objectContaining({ offset: 0, limit: 50 }));
   });
 
   it("clicking Next requests offset 50 (page 2)", async () => {
@@ -79,9 +77,7 @@ describe("AuditLogTable pagination", () => {
     await user.click(screen.getByRole("button", { name: "Next" }));
     await screen.findByText("Bot 50");
 
-    expect(mockFetchAuditLog).toHaveBeenLastCalledWith(
-      expect.objectContaining({ offset: 50, limit: 50 }),
-    );
+    expect(mockFetchAuditLog).toHaveBeenLastCalledWith(expect.objectContaining({ offset: 50, limit: 50 }));
   });
 
   it("disables Next when hasMore is false", async () => {

package/src/__tests__/auth-password-reset.test.tsx

@@ -26,12 +26,8 @@ vi.mock("better-auth/react", () => ({
 
 vi.mock("framer-motion", () => ({
   motion: {
-    div: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => (
-      <div {...props}>{children}</div>
-    ),
-    p: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => (
-      <p {...props}>{children}</p>
-    ),
+    div: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => <div {...props}>{children}</div>,
+    p: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => <p {...props}>{children}</p>,
   },
 }));
 
@@ -150,12 +146,8 @@ describe("Reset password page", () => {
   beforeEach(() => {
     mockFetch = vi.fn();
     mockPush.mockClear();
-    vi.mocked(useRouter).mockReturnValue({ push: mockPush } as unknown as ReturnType<
-      typeof useRouter
-    >);
-    vi.mocked(useSearchParams).mockReturnValue(
-      new URLSearchParams() as ReturnType<typeof useSearchParams>,
-    );
+    vi.mocked(useRouter).mockReturnValue({ push: mockPush } as unknown as ReturnType<typeof useRouter>);
+    vi.mocked(useSearchParams).mockReturnValue(new URLSearchParams() as ReturnType<typeof useSearchParams>);
   });
 
   it("shows access denied when no token is present", async () => {
@@ -346,9 +338,7 @@ describe("Reset password page", () => {
     await user.click(screen.getByRole("button", { name: "Reset password" }));
 
     await waitFor(() => {
-      expect(
-        screen.getByText("Password must contain at least one uppercase letter"),
-      ).toBeInTheDocument();
+      expect(screen.getByText("Password must contain at least one uppercase letter")).toBeInTheDocument();
     });
     expect(mockFetch).not.toHaveBeenCalled();
   });
@@ -366,9 +356,7 @@ describe("Reset password page", () => {
     await user.click(screen.getByRole("button", { name: "Reset password" }));
 
     await waitFor(() => {
-      expect(
-        screen.getByText("Password must contain at least one lowercase letter"),
-      ).toBeInTheDocument();
+      expect(screen.getByText("Password must contain at least one lowercase letter")).toBeInTheDocument();
     });
     expect(mockFetch).not.toHaveBeenCalled();
   });
@@ -404,9 +392,7 @@ describe("Reset password page", () => {
     await user.click(screen.getByRole("button", { name: "Reset password" }));
 
     await waitFor(() => {
-      expect(
-        screen.getByText("Password must contain at least one special character"),
-      ).toBeInTheDocument();
+      expect(screen.getByText("Password must contain at least one special character")).toBeInTheDocument();
     });
     expect(mockFetch).not.toHaveBeenCalled();
   });

package/src/__tests__/auth-redirect.test.tsx

@@ -4,6 +4,12 @@ import { beforeEach, describe, expect, it, vi } from "vitest";
 const mockReplace = vi.fn();
 vi.mock("next/navigation", () => ({
   useRouter: () => ({ replace: mockReplace }),
+  useSearchParams: () => new URLSearchParams(),
+}));
+
+vi.mock("@/lib/utils", () => ({
+  sanitizeRedirectUrl: (url: string) => url,
+  cn: (...args: unknown[]) => args.filter(Boolean).join(" "),
 }));
 
 const mockUseSession = vi.fn();
@@ -20,14 +26,14 @@ describe("AuthRedirect", () => {
     mockReplace.mockClear();
   });
 
-  it("redirects authenticated user to /marketplace", async () => {
+  it("redirects authenticated user to homePath", async () => {
     mockUseSession.mockReturnValue({
       data: { user: { id: "1", email: "test@test.com" } },
       isPending: false,
     });
     render(<AuthRedirect />);
     await waitFor(() => {
-      expect(mockReplace).toHaveBeenCalledWith("/marketplace");
+      expect(mockReplace).toHaveBeenCalledWith("/");
     });
   });
 

package/src/__tests__/auth.test.tsx

@@ -27,31 +27,37 @@ vi.mock("better-auth/react", () => ({
   }),
 }));
 
-// Mock tRPC so OAuthButtons can query enabled social providers
-vi.mock("@/lib/trpc", () => ({
-  trpc: {
-    authSocial: {
-      enabledSocialProviders: {
-        useQuery: () => ({ data: ["github", "discord", "google"], isLoading: false }),
-      },
-    },
-  },
-  TRPCProvider: ({ children }: { children: React.ReactNode }) => children,
+// Mock @/lib/api-config for OAuthButtons fetch URL
+vi.mock("@/lib/api-config", () => ({
+  API_BASE_URL: "https://api.test/api",
+  PLATFORM_BASE_URL: "https://api.test",
+  SITE_URL: "https://api.test",
 }));
 
 // Mock framer-motion to prevent animation issues in JSDOM
 vi.mock("framer-motion", () => ({
   motion: {
-    div: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => (
-      <div {...props}>{children}</div>
-    ),
-    p: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => (
-      <p {...props}>{children}</p>
-    ),
+    div: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => <div {...props}>{children}</div>,
+    p: ({ children, ...props }: React.PropsWithChildren<Record<string, unknown>>) => <p {...props}>{children}</p>,
   },
 }));
 
 describe("Login page", () => {
+  beforeEach(() => {
+    // OAuthButtons now fetches providers from API_BASE_URL/auth/providers via fetch
+    vi.stubGlobal(
+      "fetch",
+      vi.fn().mockResolvedValue({
+        ok: true,
+        json: () => Promise.resolve(["github", "discord", "google"]),
+      }),
+    );
+  });
+
+  afterEach(() => {
+    vi.unstubAllGlobals();
+  });
+
   it("renders email and password fields", async () => {
     const { default: LoginPage } = await import("../app/(auth)/login/page");
     render(<LoginPage />);
@@ -71,7 +77,7 @@ describe("Login page", () => {
     const { default: LoginPage } = await import("../app/(auth)/login/page");
     render(<LoginPage />);
 
-    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();
   });
@@ -152,13 +158,9 @@ describe("OAuth callback page", () => {
   beforeEach(() => {
     vi.useFakeTimers();
     mockPush.mockClear();
-    vi.mocked(useRouter).mockReturnValue({ push: mockPush } as unknown as ReturnType<
-      typeof useRouter
-    >);
+    vi.mocked(useRouter).mockReturnValue({ push: mockPush } as unknown as ReturnType<typeof useRouter>);
     vi.mocked(useParams).mockReturnValue({ provider: "github" });
-    vi.mocked(useSearchParams).mockReturnValue(
-      new URLSearchParams() as ReturnType<typeof useSearchParams>,
-    );
+    vi.mocked(useSearchParams).mockReturnValue(new URLSearchParams() as ReturnType<typeof useSearchParams>);
   });
 
   afterEach(() => {

package/src/__tests__/auto-topup-card.test.tsx

@@ -12,9 +12,7 @@ const { mockGetAutoTopupSettings, mockUpdateAutoTopupSettings } = vi.hoisted(()
 // Mock framer-motion to prevent animation issues in JSDOM
 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>,
   },
 }));
 
@@ -163,9 +161,7 @@ describe("AutoTopupCard", () => {
     const { AutoTopupCard } = await import("../components/billing/auto-topup-card");
     render(<AutoTopupCard />);
 
-    expect(
-      await screen.findByText("Add a payment method to enable auto-topup."),
-    ).toBeInTheDocument();
+    expect(await screen.findByText("Add a payment method to enable auto-topup.")).toBeInTheDocument();
   });
 
   it("shows error state with retry on fetch failure", async () => {
@@ -229,9 +225,7 @@ describe("AutoTopupCard", () => {
     render(<AutoTopupCard />);
 
     expect(
-      await screen.findByText(
-        "Tip: scheduled top-ups keep you in the dividend pool automatically.",
-      ),
+      await screen.findByText("Tip: scheduled top-ups keep you in the dividend pool automatically."),
     ).toBeInTheDocument();
   });
 
@@ -241,9 +235,7 @@ describe("AutoTopupCard", () => {
     render(<AutoTopupCard />);
 
     expect(
-      await screen.findByText(
-        "Tip: scheduled top-ups keep you in the dividend pool for 7 days each month.",
-      ),
+      await screen.findByText("Tip: scheduled top-ups keep you in the dividend pool for 7 days each month."),
     ).toBeInTheDocument();
   });
 

package/src/__tests__/backups-tab.test.tsx

@@ -1,6 +1,7 @@
 import { render, screen, waitFor } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
 import { beforeEach, describe, expect, it, vi } from "vitest";
+import { trpcVanillaProxy } from "./setup.js";
 
 vi.mock("@/lib/api", () => ({
   listSnapshots: vi.fn(),
@@ -20,7 +21,7 @@ vi.mock("@/lib/fetch-utils", () => ({
 }));
 
 vi.mock("@/lib/trpc", () => ({
-  trpcVanilla: {},
+  trpcVanilla: trpcVanillaProxy,
 }));
 
 vi.mock("sonner", () => ({
@@ -183,9 +184,7 @@ describe("BackupsTab", () => {
   });
 
   it("retries load when retry button is clicked", async () => {
-    mockListSnapshots
-      .mockRejectedValueOnce(new Error("Network error"))
-      .mockResolvedValueOnce(MOCK_SNAPSHOTS);
+    mockListSnapshots.mockRejectedValueOnce(new Error("Network error")).mockResolvedValueOnce(MOCK_SNAPSHOTS);
     const user = userEvent.setup();
     render(<BackupsTab botId="bot-1" />);
     await waitFor(() => {

package/src/__tests__/billing-layout-nav-hidden.test.tsx

@@ -1,47 +1,15 @@
 import { render, screen } from "@testing-library/react";
-import { expect, test, vi } from "vitest";
+import { expect, test } from "vitest";
 import BillingLayout from "@/app/(dashboard)/billing/layout";
 
-// Mock next/navigation
-vi.mock("next/navigation", () => ({
-  usePathname: () => "/billing/plans",
-}));
-
-// Mock next/link
-vi.mock("next/link", () => ({
-  default: ({
-    children,
-    href,
-    ...props
-  }: {
-    children: React.ReactNode;
-    href: string;
-    [key: string]: unknown;
-  }) => (
-    <a href={href} {...props}>
-      {children}
-    </a>
-  ),
-}));
-
-// Mock API — mode never resolves (stays null)
-vi.mock("@/lib/api", () => ({
-  apiFetch: vi.fn(),
-  getInferenceMode: () =>
-    new Promise((_resolve) => {
-      /* never resolves */
-    }),
-}));
-
-test("hostedOnly nav item is hidden (not just invisible) while mode is loading", () => {
+test("billing layout renders children and footer links", () => {
   render(
     <BillingLayout>
       <div>child</div>
     </BillingLayout>,
   );
 
-  const hostedLink = screen.getByText("Hosted Usage");
-  const li = hostedLink.closest("li");
-  expect(li?.className).toMatch(/\bhidden\b/);
-  expect(li?.className).not.toMatch(/\binvisible\b/);
+  expect(screen.getByText("child")).toBeInTheDocument();
+  expect(screen.getByText("Terms of Service")).toBeInTheDocument();
+  expect(screen.getByText("Privacy Policy")).toBeInTheDocument();
 });

package/src/__tests__/billing-payment-org-invoices.test.tsx

@@ -58,27 +58,11 @@ vi.mock("framer-motion", () => ({
   AnimatePresence: ({ children }: { children: React.ReactNode }) => children,
   motion: {
     div: ({ children, ...props }: Record<string, unknown>) => {
-      const {
-        initial: _i,
-        animate: _a,
-        exit: _e,
-        variants: _v,
-        custom: _c,
-        transition: _t,
-        ...rest
-      } = props;
+      const { initial: _i, animate: _a, exit: _e, variants: _v, custom: _c, transition: _t, ...rest } = props;
       return <div {...(rest as Record<string, unknown>)}>{children as React.ReactNode}</div>;
     },
     tr: ({ children, ...props }: Record<string, unknown>) => {
-      const {
-        initial: _i,
-        animate: _a,
-        exit: _e,
-        variants: _v,
-        custom: _c,
-        transition: _t,
-        ...rest
-      } = props;
+      const { initial: _i, animate: _a, exit: _e, variants: _v, custom: _c, transition: _t, ...rest } = props;
       return <tr {...(rest as Record<string, unknown>)}>{children as React.ReactNode}</tr>;
     },
   },

package/src/__tests__/billing.test.tsx

@@ -233,16 +233,6 @@ describe("Plans page", () => {
     expect(screen.getByText(/\/month/)).toBeInTheDocument();
   });
 
-  it("shows BYOK callout", async () => {
-    const { default: PlansPage } = await import("../app/(dashboard)/billing/plans/page");
-    render(<PlansPage />);
-
-    expect(await screen.findByText("Bring Your Own Keys")).toBeInTheDocument();
-    expect(
-      screen.getAllByText(/Platform never touches your inference/).length,
-    ).toBeGreaterThanOrEqual(1);
-  });
-
   it("links to full pricing page", async () => {
     const { default: PlansPage } = await import("../app/(dashboard)/billing/plans/page");
     render(<PlansPage />);
@@ -251,14 +241,13 @@ describe("Plans page", () => {
     expect(link).toHaveAttribute("href", "/pricing");
   });
 
-  it("shows included features", async () => {
+  it("renders features section (empty by default brand config)", async () => {
     const { default: PlansPage } = await import("../app/(dashboard)/billing/plans/page");
     render(<PlansPage />);
 
-    expect(screen.getByText(/signup credit included/)).toBeInTheDocument();
-    expect(screen.getByText("All channels")).toBeInTheDocument();
-    expect(screen.getByText("All plugins")).toBeInTheDocument();
-    expect(screen.getByText("All providers")).toBeInTheDocument();
+    // Default brand config has empty planFeatures[], so no feature items render.
+    // Just verify the page doesn't crash.
+    expect(screen.getByText("Your Plan")).toBeInTheDocument();
   });
 });
 
@@ -310,9 +299,7 @@ describe("Usage page", () => {
 
     expect(await screen.findByText("Anthropic")).toBeInTheDocument();
     expect(screen.getByText("OpenAI")).toBeInTheDocument();
-    expect(
-      screen.getByText((_, element) => element?.textContent === "~$23.40"),
-    ).toBeInTheDocument();
+    expect(screen.getByText((_, element) => element?.textContent === "~$23.40")).toBeInTheDocument();
     expect(screen.getByText((_, element) => element?.textContent === "~$8.12")).toBeInTheDocument();
     expect(screen.getByText(/Platform does not charge for inference/)).toBeInTheDocument();
   });
@@ -455,7 +442,7 @@ describe("Payment page", () => {
 });
 
 describe("Billing layout", () => {
-  it("renders billing navigation links", async () => {
+  it("renders child content and footer", async () => {
     const { default: BillingLayout } = await import("../app/(dashboard)/billing/layout");
     render(
       <BillingLayout>
@@ -463,27 +450,9 @@ describe("Billing layout", () => {
       </BillingLayout>,
     );
 
-    expect(screen.getByText("Your Plan")).toBeInTheDocument();
-    expect(screen.getByText("Usage")).toBeInTheDocument();
-    expect(screen.getByText("Payment")).toBeInTheDocument();
     expect(screen.getByText("child content")).toBeInTheDocument();
-  });
-
-  it("shows Hosted Usage nav for hosted users", async () => {
-    const api = await import("@/lib/api");
-    vi.mocked(api.getInferenceMode).mockResolvedValue("hosted");
-
-    const { default: BillingLayout } = await import("../app/(dashboard)/billing/layout");
-    render(
-      <BillingLayout>
-        <div>child content</div>
-      </BillingLayout>,
-    );
-
-    expect(await screen.findByText("Hosted Usage")).toBeInTheDocument();
-
-    // Reset for other tests
-    vi.mocked(api.getInferenceMode).mockResolvedValue("byok");
+    expect(screen.getByText("Terms of Service")).toBeInTheDocument();
+    expect(screen.getByText("Privacy Policy")).toBeInTheDocument();
   });
 });
 

package/src/__tests__/bot-settings/resources-tab.test.tsx

@@ -99,9 +99,7 @@ describe("ResourcesTab error state", () => {
 
   it("retries loading when Retry button is clicked", async () => {
     const { getResourceTier } = await import("@/lib/bot-settings-data");
-    vi.mocked(getResourceTier)
-      .mockRejectedValueOnce(new Error("fail"))
-      .mockResolvedValueOnce({ tier: "pro" });
+    vi.mocked(getResourceTier).mockRejectedValueOnce(new Error("fail")).mockResolvedValueOnce({ tier: "pro" });
 
     render(<ResourcesTab botId="bot-1" />);
     await waitFor(() => {

package/src/__tests__/bot-settings/storage-tab.test.tsx

@@ -72,9 +72,7 @@ describe("StorageTab", () => {
   it("shows billing explanation text", async () => {
     render(<StorageTab botId="bot-1" />);
     await waitFor(() => {
-      expect(
-        screen.getByText(/Storage costs are billed daily from your credit balance/),
-      ).toBeInTheDocument();
+      expect(screen.getByText(/Storage costs are billed daily from your credit balance/)).toBeInTheDocument();
     });
   });
 });

package/src/__tests__/bot-settings/vps-upgrade-card.test.tsx

@@ -23,9 +23,7 @@ describe("VpsUpgradeCard", () => {
 
   it("displays the description", () => {
     render(<VpsUpgradeCard botId="bot-1" />);
-    expect(
-      screen.getByText(/dedicated persistent container with fixed monthly pricing/),
-    ).toBeInTheDocument();
+    expect(screen.getByText(/dedicated persistent container with fixed monthly pricing/)).toBeInTheDocument();
   });
 
   it("lists all VPS features", () => {