Combiner function arguments have unknown type

[yndx-madfriend]

Combiner function arguments have unknown type

[markerikson]

You must provide correctly typed arguments for all of the input functions. As of Reselect 4.1, Reselect infers everything from those arguments.

Define (_state: RootState) => 5 (and similarly for any other arguments), and it will all work correctly!

[yndx-madfriend]

You must provide correctly typed arguments for all of the input functions. As of Reselect 4.1, Reselect infers everything from those arguments.

Define (_state: RootState) => 5 (and similarly for any other arguments), and it will all work correctly!

I do not understand why, with the new typings of createSelector, it makes such a difference (whether to have a function inlined or defined separately). Also, as I said, it all used to work in v3, and this new requirement of having separate definitions is not explicitly stated neither in changelog nor in readme.

[markerikson]

It's not about the "inline" or "defined separately".

It's literally that you must tell Reselect what the types of the arguments are.

// ❌ FAILS - no type for `state`
createSelector(
  (state) => state.x,
  output
)

// ✅ WORKS - TS knows the type of `state`
createSelector(
  (state: RootState) => state.x,
  output
)

Reselect's types now infer everything from those arguments, and tbh I'm not sure how this ever would have worked even with the old types if you don't provide any argument types at all.

[yndx-madfriend]

Thank you for clarifications @markerikson , it seems to work correctly now with declared types of arguments. 🎉