fix(utils): avoid catching storage error in atomWithStorage

[dai-shi]

close #1347

I ended up with introducing a symbol and removing big try-catch completely.
(while I originally thought about throwing a symbol or something, returning a symbol should be cleaner.)

I had to fix a test for #949.
Also, did a tweak for .subscribe #1004, to work with non defaultStorage. (There was an issue/discussion which I don't remember.)

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
jotai	✅ Ready (Inspect)	Visit Preview	Aug 26, 2022 at 0:01AM (UTC)

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 9c850c2:

Sandbox	Source
React	Configuration
React Typescript	Configuration
React Browserify	Configuration
React Snowpack	Configuration
Next.js with custom Babel config	Configuration
React with custom Babel config	Configuration
jotai-useAtomCallback-async-error (forked)	Issue #1347

[github-actions]

Size Change: +691 B (0%)

Total Size: 147 kB

Filename	Size	Change
dist/esm/utils.js	5.16 kB	+125 B (+2%)
dist/system/utils.development.js	5.52 kB	+129 B (+2%)
dist/system/utils.production.js	3.47 kB	+85 B (+3%)
dist/umd/utils.development.js	9.9 kB	+141 B (+1%)
dist/umd/utils.production.js	6.37 kB	+67 B (+1%)
dist/utils.js	9.69 kB	+144 B (+2%)

ℹ️ View Unchanged

Filename	Size
dist/babel/plugin-debug-label.js	945 B
dist/babel/plugin-react-refresh.js	1.16 kB
dist/babel/preset.js	1.41 kB
dist/devtools.js	4 kB
dist/esm/babel/plugin-debug-label.js	801 B
dist/esm/babel/plugin-react-refresh.js	1 kB
dist/esm/babel/preset.js	1.25 kB
dist/esm/devtools.js	3.12 kB
dist/esm/immer.js	643 B
dist/esm/index.js	5.21 kB
dist/esm/optics.js	668 B
dist/esm/query.js	1.17 kB
dist/esm/redux.js	254 B
dist/esm/urql.js	1.33 kB
dist/esm/valtio.js	540 B
dist/esm/xstate.js	872 B
dist/esm/zustand.js	289 B
dist/immer.js	726 B
dist/index.js	6.23 kB
dist/optics.js	938 B
dist/query.js	1.27 kB
dist/redux.js	314 B
dist/system/babel/plugin-debug-label.development.js	911 B
dist/system/babel/plugin-debug-label.production.js	677 B
dist/system/babel/plugin-react-refresh.development.js	1.1 kB
dist/system/babel/plugin-react-refresh.production.js	863 B
dist/system/babel/preset.development.js	1.36 kB
dist/system/babel/preset.production.js	1.05 kB
dist/system/devtools.development.js	3.29 kB
dist/system/devtools.production.js	2.44 kB
dist/system/immer.development.js	772 B
dist/system/immer.production.js	469 B
dist/system/index.development.js	5.44 kB
dist/system/index.production.js	3.01 kB
dist/system/optics.development.js	770 B
dist/system/optics.production.js	455 B
dist/system/query.development.js	1.32 kB
dist/system/query.production.js	1.08 kB
dist/system/redux.development.js	344 B
dist/system/redux.production.js	217 B
dist/system/urql.development.js	1.49 kB
dist/system/urql.production.js	1.01 kB
dist/system/valtio.development.js	652 B
dist/system/valtio.production.js	400 B
dist/system/xstate.development.js	981 B
dist/system/xstate.production.js	626 B
dist/system/zustand.development.js	377 B
dist/system/zustand.production.js	235 B
dist/umd/babel/plugin-debug-label.development.js	1.11 kB
dist/umd/babel/plugin-debug-label.production.js	839 B
dist/umd/babel/plugin-react-refresh.development.js	1.31 kB
dist/umd/babel/plugin-react-refresh.production.js	994 B
dist/umd/babel/preset.development.js	1.56 kB
dist/umd/babel/preset.production.js	1.19 kB
dist/umd/devtools.development.js	4.12 kB
dist/umd/devtools.production.js	2.85 kB
dist/umd/immer.development.js	871 B
dist/umd/immer.production.js	570 B
dist/umd/index.development.js	6.32 kB
dist/umd/index.production.js	3.46 kB
dist/umd/optics.development.js	1.08 kB
dist/umd/optics.production.js	671 B
dist/umd/query.development.js	1.42 kB
dist/umd/query.production.js	1.1 kB
dist/umd/redux.development.js	454 B
dist/umd/redux.production.js	322 B
dist/umd/urql.development.js	1.77 kB
dist/umd/urql.production.js	1.22 kB
dist/umd/valtio.development.js	722 B
dist/umd/valtio.production.js	500 B
dist/umd/xstate.development.js	1.45 kB
dist/umd/xstate.production.js	912 B
dist/umd/zustand.development.js	479 B
dist/umd/zustand.production.js	334 B
dist/urql.js	1.62 kB
dist/valtio.js	586 B
dist/xstate.js	1.31 kB
dist/zustand.js	344 B

_{compressed-size-action}

[joshmkennedy]

I'm unfamiliar with symbols. What is the benefit of Symbols as opposed to strings?

[dai-shi]

A string would work too. A symbol is unique, so no conflicting risks.
I'm actually fine with a string in this case, because exporting symbol isn't very nice. But, if some other libs happen to use the same string, it would catch it unexpectedly.

(edit) a string would work if we throw it, but it now returns, so a symbol would be required.