[v5]: do not depend on use-sync-external-store

[dai-shi]

for #2138

Just forgot it in #2237 (comment)

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
zustand-demo	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 20, 2024 6:38am

[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 9823aa5:

Sandbox	Source
React	Configuration
React TypeScript	Configuration
React Browserify	Configuration
React Snowpack	Configuration
React Parcel	Configuration
Next.js	Configuration
@pavlobu/zustand demo	Configuration

[github-actions]

Size Change: +71 B (+1%)

Total Size: 9.04 kB

Filename	Size	Change
dist/index.js	600 B	+71 B (+13%)	⚠️

ℹ️ View Unchanged

Filename	Size
dist/middleware.js	4.08 kB
dist/middleware/immer.js	328 B
dist/react/shallow.js	1.07 kB
dist/shallow.js	1.08 kB
dist/traditional.js	464 B
dist/vanilla.js	436 B
dist/vanilla/shallow.js	969 B

_{compressed-size-action}

[charkour]

Thanks for catching this.

We need to keep use-sync-external-store in traditional.ts to support the selector API, correct?

[dai-shi]

Yes, not for selector but for equalityFn.

[dai-shi]

Warning: The result of getServerSnapshot should be cached to avoid an infinite loop

I see...

[dai-shi]