First parameter of getters should be the state about pinia

[posva]

I pushed a new test at 40d0996 but I remember it wasn't possible to type this and the first argument at the same time. On top of that, this can be typed with other getters while the argument cannot.

In my tests, state has type any, so not sure how you manage to get that one but I don't think there is a way to support both this and the argument types. If you want to give it a try or TS end up somehow supporting it, feel free to give it a try and open a pull request

[MartinMalinda]

I tried to do a quick hack of G extends Record<string, (this: { state: S } & A & G, state: S) => any> but it doesn't work well, maybe because G references itself there. It is indeed strange how this stops being inferred once an argument type is declared.

[posva]

I gave this a try to at least have the state as the first parameter but couldn't make all the existing type tests pass. It's on https://github.com/posva/pinia/tree/types/getters-state if anyone can help.

Ideally, one should be able to have an automatically inferred type for a getter like this:

double: state => state.count * 2