fix(build): make build pass and fully esm bundle

[ecyrbe]

• export client and server correctly as node16 package
• fix mising react-dom dep
• pin and upgrade dependencies
• put shared deps (react, zod) as peer dependencies
• don't use use client on lib level, this is app responsibility

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
zact-example	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 4, 2023 8:29pm

[t3dotgg]

Decent chance this won't work without this

[ecyrbe]

I'm pretty confident about this.
Here you are in a library.
Library code is built and bundled without knowing anything about RSC boundaries.
This should be at component level in your next app :

'use client'
import { useZact } from 'zact/client';

cf https://github.com/reactjs/rfcs/blob/main/text/0227-server-module-conventions.md

[chungweileong94]

Well true, we should use this hook in client component anyway, user will have to specify the use client instead, so the directive is not really necessary in here.

However, it doesn't hurt to have the use client directive here tho, to tell React that this is one of the client entry, it's safer than nothing.

https://nextjs.org/docs/getting-started/react-essentials#third-party-packages

[ecyrbe]

The issue is, tsup used here to bundle does not honor 'use client' placement at the top of the file.
So that's why i propose to remove it, since it's not packaged correctly