> const std::vector<std::string> editors = {"emacs", "vim", "nano"};

Зачем оно runtime? Тут должно быть constexpr/array.

> std::uniform_int_distribution<size_t> distribution(0, editors.size() - 1);

У тебя тут implicit conversion у аргументов (int, size_t). Если бы ты написал 0ul или явно привел 0 к size_t, то тип можно было бы не указывать у шаблонной функции.
Тут потенциальная обосрака если вектор у тебя пустой. Получишь распределение от 0 до std::numeric_limits<size_t>::max() ->  и в цикле выйдешь за пределы вектора.

И вообще, у автора оригинала логика была другая.