Как проводишь кросс-ревью

При кросс-ревью я придерживаюсь структурированного подхода:

1. Подготовка:

• Проверяю, что код соответствует ТЗ и макетам
• Убеждаюсь, что есть тесты и они проходят
• Проверяю соблюдение code style проекта

2. Анализ кода:

• Ищу потенциальные баги и edge cases
• Проверяю производительность решений
• Оцениваю читаемость и поддерживаемость
• Смотрю на соответствие принципам SOLID/DRY/KISS

3. Обратная связь:

• Комментарии делаю конструктивными, с предложениями по улучшению
• Отмечаю хорошие решения
• Для сложных моментов предлагаю альтернативы с примерами:

// Вместо:
function processData(data) {
  return data.map(item => ({...item, processed: true}));
}

// Лучше:
function markItemsAsProcessed(items) {
  return items.map(item => ({
    ...item,
    processed: true
  }));
}

4. Коммуникация:

• Обсуждаю спорные моменты вживую
• Всегда благодарю за проделанную работу