Can't properly serialize digit strings (or other strings that happen to be valid JSON)

[lynn]

When I write

  const [x, setX] = useLocalStorage<string>("test", "");
  useEffect(() => setX("123456789123456789123"));
  console.log("x =", x);

I see x = 123456789123456800000 after refreshing the page.

This is because the value is identified as being JSON when reading it back out of LocalStorage, and then JSON.parse turns it into a float, with loss of precision.

In fact, x is now indeed a number, even though the type signature of useLocalStorage promises that it is string. 😱

Always converting JSON has other weird consequences:

• You can type something like {"wow": "json injection"} into a useLocalStorage-backed text field and break stuff!
• When serializing the word "true", it comes back out as a boolean value true.
• …

It might be nice to have a useLocalStorageRaw that only works with strings and does not "helpfully" parse strings as JSON whenever it has the opportunity.

[jharrilim]

That is a really good point. It seems to come from this line:

local-storage/src/local-storage-events.ts

Line 67 in db301e6

storage.setItem(key, typeof value === 'object' ? JSON.stringify(value) : `${value}`);

I don't remember why this was done truthfully, but if we just did JSON.stringify on all values, it would avoid this problem

[iamsolankiamit]

• Start data from localStorage not working #76
  Seems to be connected to this. as we are not converting the data on first load but only on subsequent calls.