Что плохого в ожидании цепочки обещаний?

Под капотом async / await - это просто обещания.

То есть, когда у вас есть код, который выглядит так:

const result = await myAsyncFunction();   
console.log(result): 

Это точно так же, как писать:

myAsyncFunction().then(data => {
   const result = data; 
   console.log(result); 
}); 

Причина в том, что вы не должны смешивать цепочки async / await и .then, в том, что это сбивает с толку.

Лучше просто выбрать один стиль и придерживаться его.

И пока вы выбираете один - вы также можете выбрать async / await - это более понятно.

Мне сказали, что ожидание цепочки обещаний сломает мой код.

Не обязательно, что два ваших фрагмента кода действительно работают одинаково (если someFunction() действительно возвращает обещание).

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

Это сложнее понять и поддерживать, смешивать разные стили сбивает с толку. Путаница приводит к ошибкам.

Учтите, что вам нужно будет добавить еще один вызов обещания в месте вызова console.log() или даже условный возврат из функции. Можете ли вы использовать await в обратном вызове, как в другом месте функции, нужно ли вам return результат обратного вызова then, возможно ли даже return из внешней функции? Все эти вопросы даже не возникают в первом фрагменте. И хотя на вашем примере с игрушкой на них можно легко ответить, это может быть не так просто в реальном коде с более сложным и вложенным потоком управления.

Так что лучше отдать предпочтение более лаконичному и чистому. Придерживайтесь await для единообразия, избегайте then в async functions ¹.

_{1: Конечно, всегда есть исключение из правил. Я бы сказал, что проще использовать цепочку обещаний для обработки ошибок в некоторых случаях, когда вы использовали бы catch или второй then обратный вызов .}

Если ваш then код вернул обещание вместо вызова console.log, ваш первый пример будет await, а ваш второй - нет.

Когда вы используете async/await, вы будете улавливать отказы в try/catch блоках. Ваш код будет менее вложенным и понятным.

Использование then часто приводит к большей вложенности и затрудняет чтение кода.

Вы можете await что угодно, независимо от того, возвращает ли он promise. Иногда это оправдывает себя вызовом метода, который однажды может стать асинхронным или просто вернуть обещание без объявления async.

Минусами являются сложность, производительность и совместимость, которые бледнеют по сравнению с достижениями.

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