Closed edemaine closed 1 day ago
Our current AutoPromise wrapper is broken in many cases. TS seems to be splitting up AutoPromise<A | B> into AutoPromise<A> | AutoPromise<B> which fails the check for a single Promise<...> wrapper.
AutoPromise
AutoPromise<A | B>
AutoPromise<A> | AutoPromise<B>
Promise<...>
But the arguably simpler Promise<Awaited<T>> wrapper seems to work great!
Promise<Awaited<T>>
This shouldn't affect help hovers; the user will still see AutoPromise<their type> just as before (that's why we gave it a name in the first place).
AutoPromise<their type>
Our current
AutoPromisewrapper is broken in many cases. TS seems to be splitting upAutoPromise<A | B>intoAutoPromise<A> | AutoPromise<B>which fails the check for a singlePromise<...>wrapper.But the arguably simpler
Promise<Awaited<T>>wrapper seems to work great!This shouldn't affect help hovers; the user will still see
AutoPromise<their type>just as before (that's why we gave it a name in the first place).