feat: allow using getters in other getters

Author: posva

__tests__/getters.spec.ts

@@ -11,6 +11,8 @@ describe('Store', () => {
       }),
       getters: {
         upperCaseName: ({ name }) => name.toUpperCase(),
+        composed: (state, { upperCaseName }) =>
+          (upperCaseName.value as string) + ': ok',
       },
     })()
   }
@@ -58,4 +60,9 @@ describe('Store', () => {
     aStore.state.a = 'b'
     expect(aStore.fromB.value).toBe('b b')
   })
+
+  it('can use other getters', () => {
+    const store = useStore()
+    expect(store.composed.value).toBe('EDUARDO: ok')
+  })
 })

src/store.ts

@@ -121,7 +121,7 @@ export function buildStore<
     computedGetters[getterName] = computed(() => {
       setActiveReq(_r)
       // eslint-disable-next-line @typescript-eslint/no-use-before-define
-      return getters[getterName](state.value)
+      return getters[getterName](state.value, computedGetters)
     }) as StoreWithGetters<S, G>[typeof getterName]
   }
 

src/types.ts

@@ -17,7 +17,8 @@ export function isPlainObject(
 export type NonNullObject = Record<any, any>
 
 export interface StoreGetter<S extends StateTree, T = any> {
-  (state: S): T
+  // TODO: would be nice to be able to define the getters here
+  (state: S, getters: Record<string, Ref<any>>): T
 }
 
 type TODO = any
@@ -30,13 +31,6 @@ export type SubscriptionCallback<S> = (
   state: S
 ) => void
 
-export type StoreReactiveGetters<
-  S extends StateTree,
-  G extends Record<string, (state: S, getters: any) => any>
-> = {
-  [k in keyof G]: G[k] extends (state: S, getters: any) => infer V ? V : never
-}
-
 export type StoreWithGetters<
   S extends StateTree,
   G extends Record<string, StoreGetter<S>>