verify-nextjs

name	verify-nextjs
description	Next.js/React verification checklist — components, data fetching, types, performance, Tailwind v4.
disable-model-invocation	true

Skill: Verify Next.js

Purpose

Systematic verification checklist for Next.js/React code. Used by the reviewer agent after code execution.

Verification Process

1. Tests

• Tests exist for new/changed components and logic
• Tests pass (npm test / vitest)
• Tests cover user interactions, not implementation details
• API mocking is realistic (matches actual response shapes)

2. Components

• Server vs Client Components used appropriately
• No unnecessary "use client" directives
• Props are typed — no any types
• Components are reasonably sized (extract if >150 lines)
• Key props on list items

3. Data Fetching

Match the project's architecture mode (see CLAUDE.md / nextjs-patterns): SSR-centric projects use route-level conventions, frontend-centric projects manage state in client components.

• No waterfalls — parallel fetches where possible
• Loading states handled (SSR: loading.tsx or Suspense; frontend-centric: loading flags in state, reset in finally)
• Error states handled (SSR: error.tsx or error boundaries; frontend-centric: error state + user-visible message)
• Caching strategy is intentional (not accidental)
• API endpoints use trailing slashes (Django REST Framework convention)
• API calls live in a single lib/api.ts — a generic apiFetch<T> base plus namespaced exports (authApi, itemsApi), not scattered fetch calls

4. Types

• API response types match the integration summary
• No any escape hatches without comments explaining why
• Shared types are in types/ not duplicated across files
• Backend dates are typed as string (ISO 8601) and parsed with dayjs — not read as Date straight off the wire
• tsc --noEmit passes cleanly

5. Performance

• Images use next/image
• No unnecessary re-renders (check effect dependencies)
• Large lists are virtualized if >100 items
• Bundle impact is reasonable (no giant libraries for small tasks)

For a deeper performance pass, run the /react skill (react-best-practices) — its 57 rules across 8 categories are prioritized by impact.

6. Accessibility

• Interactive elements are keyboard accessible
• Form inputs have labels
• Images have alt text
• Semantic HTML used (not div soup)

7. Security

• No secrets or API keys in client code
• User input is sanitized before rendering
• API calls include appropriate auth headers
• Server actions validate input

8. Code Quality

• No commented-out code
• No console.logs left in
• ESLint passes
• Consistent naming conventions

9. Development Environment

• Dockerfile.dev exists and uses a supported LTS Node image (node:22-alpine or newer)
• docker-compose.yml exists with volume mounts and WATCHPACK_POLLING
• Makefile exists and wraps all commands via docker compose exec
• No npm run / npx commands used directly on host

10. Styling — Tailwind v4 (Tailwind stacks only: shadcn / plain Next.js — skip for MUI)

• No tailwind.config.js/ts — configuration lives in an @theme block in CSS, with @import "tailwindcss" (not @tailwind base/components/utilities)
• Colors use semantic tokens in OKLCH (bg-primary, not bg-blue-500 or hardcoded hex); new tokens go in @theme rather than arbitrary values
• Component variants use CVA (class-variance-authority), merged through cn()
• size-* shorthand (size-10) over h-10 w-10; no forwardRef (React 19 passes ref as a prop)
• Dark mode via @custom-variant dark + the .dark class, not a config darkMode flag

Output

Produce a verification-report.md (same format as verify-django).