nextjs-hackathon-stack 0.1.23 → 0.1.24

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nextjs-hackathon-stack",
-  "version": "0.1.23",
+  "version": "0.1.24",
   "description": "Scaffold a full-stack Next.js hackathon starter",
   "type": "module",
   "bin": {

package/template/eslint.config.ts

@@ -61,6 +61,10 @@ export default tseslint.config(
     },
   },
   {
-    ignores: [".next/**", "node_modules/**", "dist/**", "eslint.config.ts", "next-env.d.ts", "next.config.ts", "playwright.config.ts", "drizzle.config.ts", "vitest.config.ts"],
+    files: ["**/error.tsx"],
+    rules: { "no-console": "off" },
+  },
+  {
+    ignores: [".next/**", "node_modules/**", "dist/**", "eslint.config.ts", "next-env.d.ts", "next.config.ts", "playwright.config.ts", "drizzle.config.ts", "vitest.config.ts", "postcss.config.mjs"],
   }
 );

package/template/middleware.ts

@@ -1,4 +1,5 @@
-import { type NextRequest } from "next/server";
+import type { NextRequest } from "next/server";
+
 import { updateSession } from "@/shared/lib/supabase/middleware";
 
 export async function middleware(request: NextRequest) {

package/template/src/app/(protected)/page.tsx

@@ -1,17 +1,20 @@
-import { TodoList } from "@/features/todos/components/todo-list";
-import { AddTodoForm } from "@/features/todos/components/add-todo-form";
+import { eq } from "drizzle-orm";
+import { redirect } from "next/navigation";
+
 import { logoutAction } from "@/features/auth/actions/logout.action";
-import { createClient } from "@/shared/lib/supabase/server";
+import { AddTodoForm } from "@/features/todos/components/add-todo-form";
+import { TodoList } from "@/features/todos/components/todo-list";
 import { db } from "@/shared/db";
 import { todos } from "@/shared/db/schema";
-import { eq } from "drizzle-orm";
+import { createClient } from "@/shared/lib/supabase/server";
 
 export default async function HomePage() {
   const supabase = await createClient();
   const {
     data: { user },
   } = await supabase.auth.getUser();
-  const items = await db.select().from(todos).where(eq(todos.userId, user!.id));
+  if (!user) redirect("/login");
+  const items = await db.select().from(todos).where(eq(todos.userId, user.id));
 
   return (
     <main className="container mx-auto max-w-2xl p-8">

package/template/src/app/__tests__/protected-layout.test.tsx

@@ -12,7 +12,7 @@ vi.mock("@/shared/lib/supabase/server", () => ({
 }));
 
 vi.mock("next/navigation", () => ({
-  redirect: (url: string) => mockRedirect(url),
+  redirect: (url: string): unknown => mockRedirect(url),
 }));
 
 describe("ProtectedLayout", () => {

package/template/src/app/__tests__/protected-page.test.tsx

@@ -1,7 +1,8 @@
-import { describe, it, expect, vi } from "vitest";
+import { describe, it, expect, vi, beforeEach } from "vitest";
 
 const mockGetUser = vi.fn();
 const mockSelect = vi.fn();
+const mockRedirect = vi.fn();
 
 vi.mock("@/shared/lib/supabase/server", () => ({
   createClient: vi.fn(() =>
@@ -29,7 +30,26 @@ vi.mock("@/features/todos/components/add-todo-form", () => ({
   AddTodoForm: () => <div data-testid="add-todo-form" />,
 }));
 
+vi.mock("next/navigation", () => ({
+  redirect: (url: string): unknown => mockRedirect(url),
+}));
+
 describe("HomePage (protected)", () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    mockRedirect.mockImplementation(() => { throw new Error("NEXT_REDIRECT"); });
+  });
+
+  it("redirects to /login when user is not authenticated", async () => {
+    // Arrange
+    mockGetUser.mockResolvedValue({ data: { user: null } });
+    const { default: HomePage } = await import("../(protected)/page");
+
+    // Act + Assert
+    await expect(HomePage()).rejects.toThrow("NEXT_REDIRECT");
+    expect(mockRedirect).toHaveBeenCalledWith("/login");
+  });
+
   it("renderiza la página con las tareas del usuario", async () => {
     // Arrange
     mockGetUser.mockResolvedValue({ data: { user: { id: "u1", email: "user@example.com" } } });

package/template/src/features/auth/__tests__/login-form.test.tsx

@@ -7,7 +7,7 @@ import { LoginForm } from "../components/login-form";
 const mockLoginAction = vi.fn().mockResolvedValue({ status: "idle" });
 
 vi.mock("../actions/login.action", () => ({
-  loginAction: (...args: unknown[]) => mockLoginAction(...args),
+  loginAction: (...args: unknown[]): unknown => mockLoginAction(...args),
 }));
 
 describe("LoginForm", () => {

package/template/src/features/auth/__tests__/login.action.test.ts

@@ -12,7 +12,7 @@ vi.mock("@/shared/lib/supabase/server", () => ({
 }));
 
 vi.mock("next/navigation", () => ({
-  redirect: (url: string) => mockRedirect(url),
+  redirect: (url: string): unknown => mockRedirect(url),
 }));
 
 describe("loginAction", () => {

package/template/src/features/auth/__tests__/logout.action.test.ts

@@ -12,7 +12,7 @@ vi.mock("@/shared/lib/supabase/server", () => ({
 }));
 
 vi.mock("next/navigation", () => ({
-  redirect: (url: string) => mockRedirect(url),
+  redirect: (url: string): unknown => mockRedirect(url),
 }));
 
 describe("logoutAction", () => {

package/template/src/features/auth/actions/login.action.ts

@@ -25,7 +25,7 @@ export async function loginAction(
   });
 
   if (!parsed.success) {
-    return { status: "error", message: parsed.error.issues[0]!.message };
+    return { status: "error", message: (parsed.error.issues[0] as { message: string }).message };
   }
 
   const supabase = await createClient();

package/template/src/features/auth/components/login-form.tsx

@@ -35,7 +35,7 @@ export function LoginForm() {
     const fd = new FormData();
     fd.set("email", data.email);
     fd.set("password", data.password);
-    startTransition(() => formAction(fd));
+    startTransition(() => { formAction(fd); });
   });
 
   return (

package/template/src/features/todos/__tests__/add-todo-form.test.tsx

@@ -7,7 +7,7 @@ import { AddTodoForm } from "../components/add-todo-form";
 const mockAdd = vi.fn().mockResolvedValue(undefined);
 
 vi.mock("../actions/todos.action", () => ({
-  addTodoAction: (...args: unknown[]) => mockAdd(...args),
+  addTodoAction: (...args: unknown[]): unknown => mockAdd(...args),
 }));
 
 describe("AddTodoForm", () => {

package/template/src/features/todos/__tests__/todo-list.test.tsx

@@ -8,8 +8,8 @@ const mockToggle = vi.fn().mockResolvedValue(undefined);
 const mockDelete = vi.fn().mockResolvedValue(undefined);
 
 vi.mock("../actions/todos.action", () => ({
-  toggleTodoAction: (...args: unknown[]) => mockToggle(...args),
-  deleteTodoAction: (...args: unknown[]) => mockDelete(...args),
+  toggleTodoAction: (...args: unknown[]): unknown => mockToggle(...args),
+  deleteTodoAction: (...args: unknown[]): unknown => mockDelete(...args),
 }));
 
 const baseTodo = {

package/template/src/features/todos/__tests__/todos.action.test.ts

@@ -8,7 +8,7 @@ const mockRevalidatePath = vi.fn();
 const mockGetUser = vi.fn();
 
 vi.mock("next/cache", () => ({
-  revalidatePath: (path: string) => mockRevalidatePath(path),
+  revalidatePath: (path: string): unknown => mockRevalidatePath(path),
 }));
 
 vi.mock("@/shared/lib/supabase/server", () => ({

package/template/src/features/todos/actions/todos.action.ts

@@ -1,7 +1,8 @@
 "use server";
 
-import { revalidatePath } from "next/cache";
 import { eq } from "drizzle-orm";
+import { revalidatePath } from "next/cache";
+
 import { db } from "@/shared/db";
 import { todos } from "@/shared/db/schema";
 import { createClient } from "@/shared/lib/supabase/server";

package/template/src/features/todos/components/add-todo-form.tsx

@@ -1,14 +1,16 @@
 "use client";
 
 import { useRef, useTransition } from "react";
+
 import { addTodoAction } from "../actions/todos.action";
+
 import { Button } from "@/shared/components/ui/button";
 
 export function AddTodoForm() {
   const [isPending, startTransition] = useTransition();
   const inputRef = useRef<HTMLInputElement>(null);
 
-  function handleSubmit(e: React.FormEvent<HTMLFormElement>) {
+  function handleSubmit(e: React.SyntheticEvent<HTMLFormElement>) {
     e.preventDefault();
     const formData = new FormData(e.currentTarget);
     startTransition(async () => {

package/template/src/features/todos/components/todo-list.tsx

@@ -1,9 +1,11 @@
 "use client";
 
 import { useTransition } from "react";
-import { toggleTodoAction, deleteTodoAction } from "../actions/todos.action";
-import type { SelectTodo } from "@/shared/db/schema";
+
+import { deleteTodoAction, toggleTodoAction } from "../actions/todos.action";
+
 import { Spinner } from "@/shared/components/ui/spinner";
+import type { SelectTodo } from "@/shared/db/schema";
 
 interface TodoListProps {
   items: SelectTodo[];
@@ -33,7 +35,7 @@ export function TodoList({ items }: TodoListProps) {
               checked={todo.completed}
               disabled={isPending}
               onChange={() => {
-                startTransition(() => toggleTodoAction(todo.id));
+                startTransition(() => { void toggleTodoAction(todo.id); });
               }}
               aria-label={`Marcar "${todo.title}" como completada`}
             />
@@ -46,7 +48,7 @@ export function TodoList({ items }: TodoListProps) {
             type="button"
             disabled={isPending}
             onClick={() => {
-              startTransition(() => deleteTodoAction(todo.id));
+              startTransition(() => { void deleteTodoAction(todo.id); });
             }}
             className="rounded px-2 py-1 text-sm text-red-500 hover:bg-red-50"
             aria-label={`Eliminar "${todo.title}"`}

package/template/src/shared/__tests__/middleware.test.ts

@@ -1,10 +1,10 @@
-import { describe, it, expect, vi } from "vitest";
 import { NextRequest } from "next/server";
+import { describe, it, expect, vi } from "vitest";
 
 const mockUpdateSession = vi.fn();
 
 vi.mock("@/shared/lib/supabase/middleware", () => ({
-  updateSession: (...args: unknown[]) => mockUpdateSession(...args),
+  updateSession: (...args: unknown[]): unknown => mockUpdateSession(...args),
 }));
 
 describe("middleware", () => {
@@ -12,11 +12,11 @@ describe("middleware", () => {
     // Arrange
     const mockResponse = new Response(null, { status: 200 });
     mockUpdateSession.mockResolvedValue(mockResponse);
-    const { middleware } = await import("../../middleware");
+    const mod = await import("../../../middleware") as { middleware: (req: NextRequest) => Promise<Response> };
     const req = new NextRequest("http://localhost/dashboard");
 
     // Act
-    const response = await middleware(req);
+    const response = await mod.middleware(req);
 
     // Assert
     expect(mockUpdateSession).toHaveBeenCalledWith(req);
@@ -25,10 +25,10 @@ describe("middleware", () => {
 
   it("exports a matcher config", async () => {
     // Arrange + Act
-    const { config } = await import("../../middleware");
+    const mod = await import("../../../middleware") as { config: { matcher: string[] } };
 
     // Assert
-    expect(config.matcher).toBeDefined();
-    expect(Array.isArray(config.matcher)).toBe(true);
+    expect(mod.config.matcher).toBeDefined();
+    expect(Array.isArray(mod.config.matcher)).toBe(true);
   });
 });

package/template/src/shared/__tests__/supabase-middleware.test.ts

@@ -1,31 +1,34 @@
+import type * as NextServerTypes from "next/server";
 import { describe, it, expect, vi, beforeEach } from "vitest";
 
 const mockGetUser = vi.fn();
 const mockCreateServerClient = vi.fn();
 const mockNextResponseNext = vi.fn();
 const mockNextResponseRedirect = vi.fn();
+const mockRequestGetAll = vi.fn(() => []);
+const mockRequestSet = vi.fn();
 
 vi.mock("@supabase/ssr", () => ({
-  createServerClient: (...args: unknown[]) => mockCreateServerClient(...args),
+  createServerClient: (...args: unknown[]): unknown => mockCreateServerClient(...args),
 }));
 
 vi.mock("next/server", async () => {
-  const actual = await vi.importActual<typeof import("next/server")>("next/server");
+  const actual = await vi.importActual<typeof NextServerTypes>("next/server");
   return {
     ...actual,
     NextResponse: {
-      next: (...args: unknown[]) => mockNextResponseNext(...args),
-      redirect: (...args: unknown[]) => mockNextResponseRedirect(...args),
+      next: (...args: unknown[]): unknown => mockNextResponseNext(...args),
+      redirect: (...args: unknown[]): unknown => mockNextResponseRedirect(...args),
     },
   };
 });
 
-type CookieOptions = { path?: string; maxAge?: number };
-type CookieItem = { name: string; value: string; options: CookieOptions };
-type CookieCallbacks = {
+interface CookieOptions { path?: string; maxAge?: number }
+interface CookieItem { name: string; value: string; options: CookieOptions }
+interface CookieCallbacks {
   getAll: () => CookieItem[];
   setAll: (cookies: CookieItem[]) => void;
-};
+}
 
 function makeRequest(pathname: string) {
   return {
@@ -33,10 +36,10 @@ function makeRequest(pathname: string) {
       pathname,
       clone: () => ({ pathname, toString: () => `http://localhost${pathname}` }),
     },
-    cookies: { getAll: vi.fn(() => []), set: vi.fn() },
+    cookies: { getAll: mockRequestGetAll, set: mockRequestSet },
     headers: new Headers(),
     url: `http://localhost${pathname}`,
-  } as unknown as import("next/server").NextRequest;
+  } as unknown as NextServerTypes.NextRequest;
 }
 
 describe("supabase middleware (updateSession)", () => {
@@ -85,10 +88,11 @@ describe("supabase middleware (updateSession)", () => {
     await updateSession(req);
 
     // Act
-    capturedCookieCallbacks!.getAll();
+    if (!capturedCookieCallbacks) throw new Error("capturedCookieCallbacks not set");
+    capturedCookieCallbacks.getAll();
 
     // Assert
-    expect(req.cookies.getAll).toHaveBeenCalled();
+    expect(mockRequestGetAll).toHaveBeenCalled();
   });
 
   it("invokes setAll cookie callback and updates request + response cookies", async () => {
@@ -99,12 +103,13 @@ describe("supabase middleware (updateSession)", () => {
     await updateSession(req);
 
     // Act
-    capturedCookieCallbacks!.setAll([
+    if (!capturedCookieCallbacks) throw new Error("capturedCookieCallbacks not set");
+    capturedCookieCallbacks.setAll([
       { name: "sb-token", value: "abc", options: { path: "/" } },
     ]);
 
     // Assert
-    expect(req.cookies.set).toHaveBeenCalledWith("sb-token", "abc");
+    expect(mockRequestSet).toHaveBeenCalledWith("sb-token", "abc");
     expect(fakeResponse.cookies.set).toHaveBeenCalledWith("sb-token", "abc", { path: "/" });
   });
 

package/template/src/shared/__tests__/supabase-server.test.ts

@@ -5,7 +5,7 @@ const mockGetAll = vi.fn().mockReturnValue([]);
 const mockCreateServerClient = vi.fn();
 
 vi.mock("@supabase/ssr", () => ({
-  createServerClient: (...args: unknown[]) => mockCreateServerClient(...args),
+  createServerClient: (...args: unknown[]): unknown => mockCreateServerClient(...args),
 }));
 
 vi.mock("next/headers", () => ({
@@ -43,7 +43,8 @@ describe("supabase server", () => {
 
     // Act
     await createClient();
-    capturedGetAll!();
+    if (!capturedGetAll) throw new Error("capturedGetAll not set");
+    capturedGetAll();
 
     // Assert
     expect(mockGetAll).toHaveBeenCalled();
@@ -64,7 +65,8 @@ describe("supabase server", () => {
     await createClient();
 
     // Act
-    capturedSetAll!([{ name: "token", value: "abc", options: { path: "/" } }]);
+    if (!capturedSetAll) throw new Error("capturedSetAll not set");
+    capturedSetAll([{ name: "token", value: "abc", options: { path: "/" } }]);
 
     // Assert
     expect(mockSet).toHaveBeenCalledWith("token", "abc", { path: "/" });
@@ -88,8 +90,10 @@ describe("supabase server", () => {
     await createClient();
 
     // Act + Assert
-    expect(() =>
-      capturedSetAll!([{ name: "token", value: "abc", options: {} }])
-    ).not.toThrow();
+    if (!capturedSetAll) throw new Error("capturedSetAll not set");
+    const setAll = capturedSetAll;
+    expect(() => {
+      setAll([{ name: "token", value: "abc", options: {} }]);
+    }).not.toThrow();
   });
 });

package/template/src/shared/__tests__/utils.test.ts

@@ -9,8 +9,11 @@ describe("cn", () => {
   });
 
   it("omits falsy conditional classes", () => {
-    // Arrange + Act + Assert
-    expect(cn("foo", false && "bar", "baz")).toBe("foo baz");
+    // Arrange
+    const falsy = false as boolean;
+
+    // Act + Assert
+    expect(cn("foo", falsy && "bar", "baz")).toBe("foo baz");
   });
 
   it("handles undefined values", () => {

package/template/src/shared/components/ui/button.tsx

@@ -2,9 +2,10 @@ import { Slot } from "@radix-ui/react-slot"
 import { cva, type VariantProps } from "class-variance-authority"
 import * as React from "react"
 
-import { cn } from "@/shared/lib/utils"
 import { Spinner } from "./spinner"
 
+import { cn } from "@/shared/lib/utils"
+
 const buttonVariants = cva(
   "inline-flex items-center justify-center gap-2 whitespace-nowrap rounded-md text-sm font-medium transition-all disabled:pointer-events-none disabled:opacity-50 [&_svg]:pointer-events-none [&_svg:not([class*='size-'])]:size-4 shrink-0 [&_svg]:shrink-0 outline-none focus-visible:border-ring focus-visible:ring-ring/50 focus-visible:ring-[3px] aria-invalid:ring-destructive/20 dark:aria-invalid:ring-destructive/40 aria-invalid:border-destructive",
   {

package/template/src/shared/components/ui/spinner.tsx

@@ -1,4 +1,5 @@
 import { LoaderCircle } from "lucide-react";
+
 import { cn } from "@/shared/lib/utils";
 
 export function Spinner({ className }: { className?: string }) {