fix(vanilla): should recompute state after onMount

[tmkx]

Related Issues or Discussions

Fixes #2098

Summary

Check List

• yarn run prettier for formatting code and docs

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
jotai	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 31, 2023 2:36am

[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 4aa774d:

Sandbox	Source
React	Configuration
React Typescript	Configuration
React Browserify	Configuration
React Snowpack	Configuration
Next.js	Configuration
Next.js with custom Babel config	Configuration
React with custom Babel config	Configuration
optimistic-bush-9rccdd	Issue #2098

[dai-shi]

Thanks for working on it!

Can you add a test with store api, without atomWithStorage, and without React?

[dai-shi]

The tricky thing is CI tests.

[dai-shi]

It looks good. Nice work.

Let me try and see if I can refactor.

[dai-shi]

I guess the fundamental issue is that dependent graph isn't ready when onMount is invoked. So, the queue is introduced. Not really sure if it's the ideal solution, but it works so far.

[fturmel]

Thank you both!