Please review this file:
[File path or code]

Review the following PR:
Changes: [Description of changes]
Files: [List of modified files]

Focus areas:
- Security
- Performance
- Code readability

Review this code from a security perspective:
- SQL injection
- XSS vulnerabilities
- Authentication/authorization issues
- Sensitive information leaks

Refactoring suggestions for this legacy code:
- Conversion to modern syntax
- Performance improvements
- Testability enhancements

undefined

query = f"SELECT * FROM users WHERE username = '{username}'"
cursor.execute(query)

query = "SELECT * FROM users WHERE username = ?"
cursor.execute(query, (username,))

undefined

undefined

for (const post of posts) {
  post.author = await db.users.findById(post.authorId);
}

const authorIds = posts.map(p => p.authorId);
const authors = await db.users.findByIds(authorIds);
const authorMap = new Map(authors.map(a => [a.id, a]));
posts.forEach(post => post.author = authorMap.get(post.authorId));

undefined

undefined

// Current: One long function
public void processOrder(Order order) {
  // Validation logic (20 lines)
  // Inventory check (30 lines)
  // Payment processing (40 lines)
  // Notification sending (25 lines)
  // Logging (15 lines)
  // Database update (30 lines)
}

// Recommended: Split into functions
public void processOrder(Order order) {
  validateOrder(order);
  checkInventory(order);
  processPayment(order);
  sendNotifications(order);
  updateDatabase(order);
  logOrderProcessing(order);
}

undefined

undefined

undefined

Custom review rules:
- Implement rate limiting for all API endpoints
- Database migrations must be rollbackable
- All public functions require JSDoc comments
- Error responses must follow standard format

undefined

undefined

#!/bin/bash

undefined

Please review the following PR:

Title: Add user authentication feature
Modified Files:
- src/auth/login.ts (new)
- src/auth/jwt.ts (new)
- src/middleware/auth.ts (new)
- src/routes/api.ts (modified)

Changes:
- Implemented JWT-based authentication
- Added login endpoint
- Created authentication middleware

Review this bug fix:
Issue: User list loading is slow
Fix: Query optimization and caching addition

Review this refactoring:
Change: Split 500-line function into 10 small functions
Purpose: Improve maintainability and testability

Please review this PR:

File: src/payment/processor.py

Changes: Implementation of credit card payment processing

Key check items:
- Security (handling of card information)
- Error handling
- Transaction management
- PCI DSS compliance