release/1.0: all P0/P1 fixes + quick wins (#323-#331, #336, #339, #341, #342)

.env.example

@@ -25,5 +25,8 @@ ADMIN_BOOTSTRAP_TOKEN=
 # Generate: node -e "console.log(require('crypto').randomBytes(32).toString('hex'))"
 API_KEY_HMAC_SECRET=
 
+# Self-registration toggle — set to "false" to disable /signup (optional, default: true)
+# REGISTRATION_ENABLED=true
+
 # Tenant ID — defaults to "default" if unset (optional)
 # TENANT_ID=default

app/e2e/charts.spec.ts

@@ -1224,21 +1224,15 @@ test.describe("Column mapping overlay", () => {
       timeout: 10_000,
     });
 
-    // The column mapping overlay should be visible on the grid in edit mode
+    // Column mapping overlay should NOT appear on dashboard cards (#331)
     await expect(
       page.locator("[data-testid='column-mapping-overlay']").first(),
-    ).toBeVisible({ timeout: 15_000 });
-
-    // X and Y triggers should be present
-    await expect(
-      page.locator("[data-testid='column-mapping-x-trigger']").first(),
-    ).toBeVisible();
-    await expect(
-      page.locator("[data-testid='column-mapping-y-trigger']").first(),
-    ).toBeVisible();
+    ).not.toBeVisible({ timeout: 5_000 });
   });
 
-  test("changing axis mapping updates chart", async ({ page }) => {
+  // Column mapping overlay removed from dashboard cards (#331) — axis mapping
+  // is now only available inside the widget editor modal.
+  test.skip("changing axis mapping updates chart", async ({ page }) => {
     test.setTimeout(60_000);
     await page.getByRole("button", { name: "Add Widget" }).first().click();
     const dialog = page.getByRole("dialog", { name: "Add Widget" });

app/src/app/(auth)/login/__tests__/page.test.tsx

@@ -0,0 +1,228 @@
+import { describe, it, expect, vi, beforeEach } from "vitest";
+import { render, screen, waitFor } from "@testing-library/react";
+import userEvent from "@testing-library/user-event";
+import React from "react";
+
+/* ---------- mocks ---------- */
+
+const mockPush = vi.fn();
+const mockSignIn = vi.fn();
+
+vi.mock("next-auth/react", () => ({
+  signIn: (...args: unknown[]) => mockSignIn(...args),
+}));
+
+vi.mock("next/navigation", () => ({
+  useRouter: () => ({ push: mockPush }),
+  useSearchParams: () => new URLSearchParams(),
+}));
+
+vi.mock("next/link", () => ({
+  __esModule: true,
+  default: ({
+    href,
+    children,
+    ...rest
+  }: {
+    href: string;
+    children: React.ReactNode;
+  }) => (
+    <a href={href} {...rest}>
+      {children}
+    </a>
+  ),
+}));
+
+vi.mock("@neoboard/components", () => ({
+  Card: ({
+    children,
+    className,
+  }: {
+    children: React.ReactNode;
+    className?: string;
+  }) => <div className={className}>{children}</div>,
+  CardContent: ({ children }: { children: React.ReactNode }) => (
+    <div>{children}</div>
+  ),
+  CardDescription: ({ children }: { children: React.ReactNode }) => (
+    <p>{children}</p>
+  ),
+  CardFooter: ({
+    children,
+    className,
+  }: {
+    children: React.ReactNode;
+    className?: string;
+  }) => <div className={className}>{children}</div>,
+  CardHeader: ({
+    children,
+    className,
+  }: {
+    children: React.ReactNode;
+    className?: string;
+  }) => <div className={className}>{children}</div>,
+  CardTitle: ({
+    children,
+    className,
+  }: {
+    children: React.ReactNode;
+    className?: string;
+  }) => <h2 className={className}>{children}</h2>,
+  Input: (props: React.InputHTMLAttributes<HTMLInputElement>) => (
+    <input {...props} />
+  ),
+  Label: ({
+    children,
+    htmlFor,
+  }: {
+    children: React.ReactNode;
+    htmlFor?: string;
+  }) => <label htmlFor={htmlFor}>{children}</label>,
+  Alert: ({ children }: { children: React.ReactNode; variant?: string }) => (
+    <div role="alert">{children}</div>
+  ),
+  AlertDescription: ({ children }: { children: React.ReactNode }) => (
+    <span>{children}</span>
+  ),
+  LoadingButton: ({
+    children,
+    loading,
+    loadingText,
+    ...rest
+  }: React.ButtonHTMLAttributes<HTMLButtonElement> & {
+    loading?: boolean;
+    loadingText?: string;
+  }) => (
+    <button {...rest} disabled={loading}>
+      {loading ? loadingText : children}
+    </button>
+  ),
+  PasswordInput: (props: React.InputHTMLAttributes<HTMLInputElement>) => (
+    <input type="password" {...props} />
+  ),
+}));
+
+/* ---------- import under test ---------- */
+import LoginPage from "../page";
+
+/* ---------- helpers ---------- */
+
+function mockFetchBootstrapStatus(registrationEnabled: boolean) {
+  global.fetch = vi.fn().mockResolvedValue({
+    json: () =>
+      Promise.resolve({
+        data: { bootstrapRequired: false, registrationEnabled },
+      }),
+  });
+}
+
+/* ---------- tests ---------- */
+
+describe("LoginPage", () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it("shows the signup link when registration is enabled", async () => {
+    mockFetchBootstrapStatus(true);
+
+    render(<LoginPage />);
+
+    await waitFor(() => {
+      expect(screen.getByText("Sign up")).toBeDefined();
+    });
+
+    const signupLink = screen.getByText("Sign up");
+    expect(signupLink.closest("a")).toHaveAttribute("href", "/signup");
+  });
+
+  it("hides the signup link when registration is disabled", async () => {
+    mockFetchBootstrapStatus(false);
+
+    render(<LoginPage />);
+
+    await waitFor(() => {
+      expect(screen.queryByText("Sign up")).toBeNull();
+    });
+  });
+
+  it("shows the signup link by default before fetch completes", () => {
+    // Fetch never resolves — default state should show the link
+    global.fetch = vi.fn().mockReturnValue(new Promise(() => {}));
+
+    render(<LoginPage />);
+
+    expect(screen.getByText("Sign up")).toBeDefined();
+  });
+
+  it("keeps the signup link when fetch fails", async () => {
+    global.fetch = vi.fn().mockRejectedValue(new Error("Network error"));
+
+    render(<LoginPage />);
+
+    // Default state is registrationEnabled=true, fetch error doesn't change it
+    await waitFor(() => {
+      expect(global.fetch).toHaveBeenCalled();
+    });
+
+    expect(screen.getByText("Sign up")).toBeDefined();
+  });
+
+  it("renders the login form with email and password fields", () => {
+    mockFetchBootstrapStatus(true);
+
+    render(<LoginPage />);
+
+    expect(screen.getByLabelText("Email")).toBeDefined();
+    expect(screen.getByLabelText("Password")).toBeDefined();
+    expect(screen.getByText("Sign in")).toBeDefined();
+  });
+
+  it("renders the NeoBoard title", () => {
+    mockFetchBootstrapStatus(true);
+
+    render(<LoginPage />);
+
+    expect(screen.getByText("NeoBoard")).toBeDefined();
+  });
+
+  it("shows error message when login fails", async () => {
+    mockFetchBootstrapStatus(true);
+    mockSignIn.mockResolvedValue({ error: "CredentialsSignin" });
+
+    const user = userEvent.setup();
+    render(<LoginPage />);
+
+    const emailInput = screen.getByLabelText("Email");
+    const passwordInput = screen.getByLabelText("Password");
+    const submitButton = screen.getByText("Sign in");
+
+    await user.type(emailInput, "test@example.com");
+    await user.type(passwordInput, "wrongpassword");
+    await user.click(submitButton);
+
+    await waitFor(() => {
+      expect(screen.getByText("Invalid email or password")).toBeDefined();
+    });
+  });
+
+  it("redirects to callbackUrl on successful login", async () => {
+    mockFetchBootstrapStatus(true);
+    mockSignIn.mockResolvedValue({ error: null });
+
+    const user = userEvent.setup();
+    render(<LoginPage />);
+
+    const emailInput = screen.getByLabelText("Email");
+    const passwordInput = screen.getByLabelText("Password");
+    const submitButton = screen.getByText("Sign in");
+
+    await user.type(emailInput, "test@example.com");
+    await user.type(passwordInput, "correctpassword");
+    await user.click(submitButton);
+
+    await waitFor(() => {
+      expect(mockPush).toHaveBeenCalledWith("/");
+    });
+  });
+});

app/src/app/(auth)/login/page.tsx

@@ -1,6 +1,6 @@
 "use client";
 
-import { Suspense, useState } from "react";
+import { Suspense, useState, useEffect } from "react";
 import { signIn } from "next-auth/react";
 import { useRouter, useSearchParams } from "next/navigation";
 import Link from "next/link";
@@ -16,10 +16,7 @@
   Alert,
   AlertDescription,
 } from "@neoboard/components";
-import {
-  LoadingButton,
-  PasswordInput,
-} from "@neoboard/components";
+import { LoadingButton, PasswordInput } from "@neoboard/components";
 
 function LoginForm() {
   const router = useRouter();
@@ -69,12 +66,7 @@
 
       <div className="space-y-2">
         <Label htmlFor="password">Password</Label>
-        <PasswordInput
-          id="password"
-          name="password"
-          required
-          minLength={6}
-        />
+        <PasswordInput id="password" name="password" required minLength={6} />
       </div>
 
       <LoadingButton
@@ -90,26 +82,43 @@
 }
 
 export default function LoginPage() {
+  const [registrationEnabled, setRegistrationEnabled] = useState(true);
+
+  useEffect(() => {
+    fetch("/api/auth/bootstrap-status")
+      .then((r) => r.json())
+      .then((body) => {
+        const payload = body?.data ?? body;
+        setRegistrationEnabled(payload?.registrationEnabled !== false);
+      })
+      .catch(() => {});
+  }, []);
+
   return (
     <div className="flex min-h-screen items-center justify-center">
       <Card className="w-full max-w-sm">
         <CardHeader className="text-center">
           <CardTitle className="text-2xl">NeoBoard</CardTitle>
+          <p className="text-sm text-muted-foreground">
+            Visual dashboards for Neo4j &amp; PostgreSQL
+          </p>
           <CardDescription>Sign in to your account</CardDescription>
         </CardHeader>
         <CardContent>
           <Suspense>
             <LoginForm />
           </Suspense>
         </CardContent>
-        <CardFooter className="justify-center">
-          <p className="text-sm text-muted-foreground">
-            Don&apos;t have an account?{" "}
-            <Link href="/signup" className="text-primary underline">
-              Sign up
-            </Link>
-          </p>
-        </CardFooter>
+        {registrationEnabled && (
+          <CardFooter className="justify-center">
+            <p className="text-sm text-muted-foreground">
+              Don&apos;t have an account?{" "}
+              <Link href="/signup" className="text-primary underline">
+                Sign up
+              </Link>
+            </p>
+          </CardFooter>
+        )}
       </Card>
     </div>
   );