README.md missleading: Primitive Store can only be used with strings

[RaphaelBossek]

The Primitive Store example and the explanation from README.md suggest that persistentAtom also allows types other than string, which is not the case for the type definition

[ai]

Are you talking about this example:

import { persistentAtom } from '@nanostores/persistent'

export const shoppingCart = persistentAtom<Product[]>('cart', [], {
  encode: JSON.stringify,
  decode: JSON.parse,
})

It has encode/decode options to convert object to string and back.

What exact problem do you have?

[ivan-suhorukov]

I issued the same problem then i tried to use persistendAtom without third argument. Maybe there will be a clener code if you make it JSON encode/decode by default without need to use a third argument?

[ai]

@ivan-suhorukov JSON.parse is not the best option for serialisation since we may also need validation, etc.

Do you use TypeScript? It should warn you.

[RaphaelBossek]

I've created a small project at https://codesandbox.io/p/github/RaphaelBossek/nanostores-persistent-example/main?file=%2Fsrc%2Fcounter.ts%3A16%2C1 for the TypeScript issue with the example:

[ai]

TS error looks good, it highlights need of encoding.

How do you suggest changing the doc?