Review Changes

2. Deliverability Impact

• Suppression list checked before sends
• Bounce/complaint handling not broken
• DKIM/SPF/DMARC not affected
• Warm-up limits respected for new domains
• Email headers include required compliance fields

3. Queue Safety

• Jobs are idempotent (safe to retry)
• Dead letter queue handling for failures
• No race conditions in concurrent processing
• Worker graceful shutdown preserved
• Queue rate limiting not bypassed

4. API Contract

• Response shapes unchanged (or version bumped)
• Error codes consistent
• Pagination format consistent
• Rate limit headers present
• No new required fields on existing endpoints (breaking change)

5. Test Coverage

• New code paths have tests
• Edge cases covered (empty list, invalid email, rate limit hit)
• Integration tests for new flows (API → queue → SES → webhook)
• No tests removed or weakened

6. Performance

• No N+1 queries (check Prisma includes/joins)
• No unbounded queries (pagination enforced)
• No blocking operations in async handlers
• Redis operations are O(1) or O(log n)

Output Format

Review: N issues (X critical, Y warning, Z info)

[CRITICAL|WARNING|INFO] path:line — summary
  Fix: recommended action

Merge recommendation: APPROVE | APPROVE_WITH_FIXES | REQUEST_CHANGES | BLOCK