Also, on the naming front, would it be better if runWithHub was called something like runWithAsyncContext to better describe what it's actually doing?

Can we make runWithHub just pass through it's arguments? We lose the type safety, but gain the flexibility.

function runWithHub<T, A extends unknown[]>(callback: (hub: Hub, ...args: A) => T,  ...args: A): T;

runWithAsyncContext

I like this - it's more backwards compat as well.

@lforst will be a fan :)

runWithAsyncContext

I like this - it's more backwards compat as well.

@lforst will be a fan :)

indeed <3

Can we make runWithHub just pass through it's arguments?

Yeah possibly, and then just if (arg instanceof EventEmitter) before passing the args to add as a safety check.

This is safer since we know all the args will be of the same type:

function runWithAsyncContext<T, A>(callback: (hub: Hub, ...args: A[]) => T, ...args: A[]): T;

This is safer since we know all the args will be of the same type:

function runWithAsyncContext<T, A>(callback: (hub: Hub, ...args: A[]) => T, ...args: A[]): T;
Maybe we'll need a default type of unknown for A 🤔

How about the following? I think this should work (at least as the user-facing type - internally we may need some casts):

function runWithAsyncContext<C extends (hub: Hub, ...args: any[]) => any>(callback: C): ReturnType<C>;

How about the following? I think this should work (at least as the user-facing type - internally we may need some casts):

function runWithAsyncContext<C extends (hub: Hub, ...args: any[]) => any>(callback: C): ReturnType<C>;

We need to pass the callback args as parameters to the top level function so we can use them in our AsyncContextStrategy implementation like this.

I tried this PR in its current state and replaced some run and bind usages with runWithAsyncContext and the types look good. For the single usage where these parameters are used (ie. the node request handler) it needs to be called with types (runWithAsyncContext<void, EventEmitter>() since request and response are different types that both inherit EventEmitter. Everywhere else where there aren't args passed through the types are not needed.

In SDKs where these additional arguments are not required or a bit smelly, we could re-export runWithAsyncContext with these additional parameters removed. Since they're optional parameters, it wont matter if users import from core or the SDK.