Bash
# Review checklist for immediate assessment
1. Does it solve the stated problem?
2. Are there any obvious bugs or edge cases?
3. Is the code readable and well-structured?
4. Are tests adequate?
5. Any security concerns?

Progress:

• Read the PR description and linked issues
• Check the diff size (>400 lines = request split)
• Review architecture/design changes first
• Examine code quality and patterns
• Verify test coverage and quality
• Check for security vulnerabilities
• Validate documentation updates
• Test locally if complex changes

Review Priority:

1. Correctness (bugs, logic errors)
2. Security (injection, auth, data exposure)
3. Performance (algorithms, database queries)
4. Maintainability (readability, patterns)
5. Style (formatting, naming)

Example 1: Input: API endpoint that processes user input without validation Output: "❌ SECURITY: This endpoint is vulnerable to injection attacks. Add input validation and sanitization before processing user data."

Example 2: Input: Function with 8 parameters and 150 lines Output: "🔄 REFACTOR: This function violates SRP. Consider breaking into smaller functions or using a parameter object."

Example 3: Input: Database query in a loop Output: "⚡ PERFORMANCE: N+1 query detected. Use bulk operations or eager loading instead."