Как при рефакторинге кода понимаешь что нужно остановиться

Рефакторинг нужно останавливать, когда:

1. Код стал чище, но функциональность не изменилась
2. Дальнейшие изменения не дают заметного улучшения
3. Начинаешь вносить изменения, выходящие за рамки текущей задачи
4. Тесты проходят, покрытие не ухудшилось

Пример плохого рефакторинга:

// Было (работает)
function sum(a, b) {
  return a + b;
}

// Стало (избыточно)
class Calculator {
  add(a, b) {
    return a + b;
  }
}

Лучше следовать правилу бойскаута: "Оставь код чище, чем нашёл", но не переусердствовать.