## Code Review Excellence

### Review Priority Order

1. **Correctness**: Does code do what it claims? Edge cases?
2. **Security**: Input validation, auth/authz, keychain usage
3. **Performance**: Re-renders, memory leaks, lazy loading
4. **Maintainability**: Naming, function size, SRP
5. **Architecture**: Module boundaries, patterns, coupling

### Finding Severity

- Blocking: bugs, security holes, data loss, architecture violations
- Important: quality impact, performance issues
- Suggestion: style improvements, minor refactoring

### Feedback Formula: [What] + [Why] + [How]

### iOS Security Checklist

- [ ] No hardcoded secrets or API keys
- [ ] Sensitive data in Keychain, not UserDefaults
- [ ] All inputs validated and sanitized
- [ ] HTTPS only (ATS compliant)
- [ ] No logging of sensitive data
- [ ] Auth tokens not in plain text
- [ ] Permission strings accurate