fix(types): mapWritableState array

Fix #2014
vuejs · Feb 21, 2023 · a7ad90d · a7ad90d
1 parent e48bab4
commit a7ad90d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/packages/pinia/src/mapHelpers.ts b/packages/pinia/src/mapHelpers.ts
@@ -467,7 +467,12 @@ export function mapWritableState<
 >(
   useStore: StoreDefinition<Id, S, G, A>,
   keys: readonly Keys[]
-): { [K in Keys]: S[K] }
+): {
+  [K in Keys]: {
+    get: () => S[K]
+    set: (value: S[K]) => any
+  }
+}
 /**
  * Allows using state and getters from one store without using the composition
  * API (`setup()`) by generating an object to be spread in the `computed` field

diff --git a/packages/pinia/test-dts/mapHelpers.test-d.ts b/packages/pinia/test-dts/mapHelpers.test-d.ts
@@ -90,7 +90,12 @@ expectType<{
   newToggleA: () => void
 }>(mapActions(useStore, { newSetToggle: 'setToggle', newToggleA: 'toggleA' }))
 
-expectType<{ a: 'on' | 'off' }>(mapWritableState(useStore, ['a']))
+expectType<{
+  a: {
+    get: () => 'on' | 'off'
+    set: (v: 'on' | 'off') => any
+  }
+}>(mapWritableState(useStore, ['a']))
 // @ts-expect-error: only defined in array
 mapWritableState(useStore, ['a']).b