pu sh возвращает обещание, которое разрешается после завершения записи в базу данных. Если вы не дождетесь его разрешения, вы не узнаете, не получилось ли это, поскольку ваша функция вернется успешно. Вместо этого вы получите необработанный отказ. Кроме того, все, что запускается после возврата функций, получит меньше ресурсов процессора и памяти в базе данных. Поэтому перепишите его так:
const createPost = async (newPost) => {
await app.database().ref('posts').push(newPost);
}
Что касается исходного вопроса, вам не следует объявлять переменную, если она используется только один раз, поэтому я бы выбрал первый вариант.