React Code Review Guidelines

Review Categories

Component Architecture

Server vs Client

Red Flags:

• 'use client' on component that doesn't use hooks or browser APIs
• Server Component trying to use hooks
• Client Component doing data fetching that should be on server

Good Practices:

• Server Components for data fetching and static content
• 'use client' only when necessary (hooks, events, browser APIs)
• Client Components receive data as props from Server Components

Component Structure