Господа помогите понять вот такой код -который я пытаюсь сделать

typescript-овым с типами

module.exports = {
getSubscribersList: async function (contract:Contract, startBlock, endBlock, showLog):Promise<void | SubscribersResult> {
return new Promise<SubscribersResult>(async (resolve, reject) => {
const subscribeEvents:ChannelData = await this.getSubscribeEvents(contract, startBlock, endBlock, showLog)
const unsubscribeEvents:ChannelData = await this.getUnsubscribeEvents(
contract,
startBlock,
endBlock,
showLog
)
resolve({ subscribeEvents, unsubscribeEvents })
}).catch((err) => {
reject(err)
})
}
}

эти типы я сам добавил

export type SubscribersResult = { subscribeEvents:ChannelData, unsubscribeEvents:ChannelData };

export type ChannelData = {
[channelName: string]: { users: string[]}; // 'channel:signature:blockNumber'[]
};

что я не понимаю
1 почему верхний метод должен возвращать
Promise<void | SubscribersResult>
а не
Promise<SubscribersResult>
(иначе ide пишет тип не тот!)

2 что делает конструкция
return await new Promise()
и чем она отличается от
return new Promise().catch()

семантически разница же только в том ловить эксепшен прямо тут в месте await или нет

у нас много кода c return await new Promise()

3 почему даже когда я покрываю этот метод типами
webstorm не видит эти типы из соседнего файла

это из за module.exports?

3.1*
вот даже в этом методе
this.getSubscribeEvents
this.getUnsubscribeEvents
типизированы
но WebStorm этого не видит прямо в этом методе - приходится писать тип руками у переменной

код менять сильно не могу (опасно - не потестить нормально) - такое вот наследие

как написать все с await я знаю - регулярно перелопачиваю - но натыкаюсь на крупные файлы которые не хочется трогать

Скрин кода отправь

не понял - ? - кода до добавления типов ? - ну убери <> и будет код

Вот эта часть не делает ничего + бажная, т.к. нет reject в этом контексте. Она же ничего не возвращает (нет return), отсюда и void .catch((err) => { reject(err) }) 2. return await — это практически антипаттерн. Только если он не внутри блока try. Ибо await можно только внутри async функций использовать, а они и так промис возвращают, а тут мы как бы ждём его выполнения и возвращаем как бы уже не промис :-/ new Promise().catch() или даже await new Promise().catch() вполне допустимо использовать, но это не то же самое, что await new Promise(). catch возвращает новый промис, в котором, например, можно погасить ошибку от изначального промиса. 3. лучше не смешивать типы модулей: либо export const/function/class либо module.exports. Вообще если это весь файл, то я тоже не вижу типы) Точно ли всё хорошо с импортами? Если в IDE на методы с ctrl курсор навести, то типы показывает? Что в данном контексте есть this?

2. return await в v8 работает на 1 тик быстрее, чем return promise https://github.com/tc39/proposal-faster-promise-adoption

Интересно. Но, как я понял, это ускорение работает из-за оптимизации для fulfilled промисов

https://github.com/graphql/graphql-js/pull/3754/files#diff-1d705c6a4c73cd3ce46190029e75abd3015b49de3ec250357dcf9b44a35cb7d0R935-R938 люди именно так оптимизируют кстати

это же вроде оптимизация для фуллфиллед промисов

Действительно, даже из eslint:recommended убрали no-return-await. Спасибо! Обновлю линтер)

крутую штуку раскопали но думаю у нас не такие писатели - скорее всего просто что попало пишут)