Returning `never` from `redirect` function makes incorrect behavior about solid-router HOT 5 OPEN

The important part is this can't impact the return type of the function that returns it. It is intended that redirect is always thrown or returned as it doesn't do the redirect but returns the redirect. So in a sense the code that is incorrect here would never work anyway. The fact is never is odd but it is a TypeScript trick. We could I suppose make it always throw as well but we did this to give more control. If void works without impacting types would consider it.

from solid-router.

Yes it is more like, createRedirect 🌝 but yeah I see the intention now. Redirecting the actual type of the redirect function, could pollute the return type and caller scope may behave incorrect and bring type-level complications.
I just tried void, it gets unionized to the return type but that's alright I guess.

Although, I'm just a starter and don't know all the cases yet to use the redirect function, maybe action and cache or any other wrapper function could strip out redirect type from the return type by using infer

from solid-router.