post-robot icon indicating copy to clipboard operation
post-robot copied to clipboard

Mismatch between post-robot and @types/post-robot

Open malsabbagh opened this issue 2 years ago • 1 comments

I noticed a couple of instances that would be nice to fix.

once definition doesn't return cancel function in its type. It should return

export function once(
    name: string,
    options?: ServerOptionsType | HandlerType,
    handler?: HandlerType,
): ZalgoPromise<{ source: any; origin: string; data: object }> & CancelableType;

https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/post-robot/index.d.ts#L39-L43

HandlerType doesn't allow non-promise functions it should be both

type HandlerType = (event: {
    source: CrossDomainWindowType,
    origin: string,
    data: any
}) => ZalgoPromise<any> | void;

https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/post-robot/index.d.ts#L15-L19

I didn't go over all other definitions, I just noticed these two, there could be more issues.

malsabbagh avatar Apr 28 '22 15:04 malsabbagh

I can take this up, have been busy since I wrote the definitions couple of years ago. @mstuart

Thank you @malsabbagh for pointing these out :)

Cijin avatar Sep 07 '23 02:09 Cijin