Code Review

Comprehensive code review following project standards.

Review Checklist

Code Quality

• Clear naming conventions

• No code duplication

• Proper error handling with { success, error } pattern

• TypeScript types explicit (no any )

Project Patterns

• Uses hooks for Supabase queries (not direct queries in components)

• Services return { success, data?, error? }

• AI calls go through aiGatewayService.ts

• Data transforms follow 4-layer schema (Raw → Parsed → Database → Display)

Security

• No hardcoded secrets or API keys

• Input validation on external data

• SQL injection prevention

• XSS protection

Testing

• E2E tests use ?e2e=1 for auth bypass

• Schema transformers have validation tests

Output Format

For each finding:

• Severity: Critical / Warning / Suggestion

• File:Line: Location

• Issue: Description

• Fix: Recommended solution

Trigger Phrases

• "review my changes"

• "code review"

• "check this code"

• "/review"