Это неизбежное зло :D

Замыкания же, замыкания!

function do_something() {
  console.log("step1");
  open_modal_dialog().onconfirm = function() {
    console.log("step2");
  };
}

Но, конечно, генератором лучше, если кроссбраузерность не нужна.

В функции надо выносить повторяющийся код.

на функции/методы надо делить весь код, даже если он не повторяется. Чем меньше методы - тем лучше.

Потенциально повторяющийся или неочевидный.

Необязательно. Или тебе приятно читать портянки бесконечных функций из сотен строк?

Код может быть разделен не только по функциям.
Я, в общем, к тому, что во всем же надо знать меру, не стоит так категорично заявлять, что мало функций - быдлокод.