The Developer's Guide to AI PR Reviews

// ❌ Vulnerable code
const query = `SELECT * FROM users WHERE id = ${userId}`;

// ✅ AI-suggested fix
const query = 'SELECT * FROM users WHERE id = ?';
db.query(query, [userId]);

// ❌ Inefficient code
for (let i = 0; i < users.length; i++) {
  await processUser(users[i]);
}

// ✅ AI-suggested improvement
await Promise.all(users.map(user => processUser(user)));

// ❌ Inconsistent style
function getUserData(id) {
  if (id == null) return null;
  const user = database.findUser(id);
  return user;
}

// ✅ AI-suggested improvements
async function getUserData(id: string): Promise<User | null> {
  if (!id) return null;
  return await database.findUser(id);
}