fix(getNodeProcess): check if process is available

[danilowoz]

Fix #787 (comment)

[vercel]

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

Name	Status	Preview	Comments	Updated
sandpack-docs	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Mar 3, 2023 at 9:41AM (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 e18f77c:

Sandbox	Source
Sandpack	Configuration
strange-bardeen-brj7y5	Configuration

[Jokcy]

@danilowoz This breaks the SANDPACK_BARE_COMPONENTS ability for disabling default css solution. We need to figure out a better way for this feature, it's not a good idea to depend on process.env for client packages, different bundler treat this in default way.

For example vite recommend we use import.mate.env which is supported out box, but if we want to use process.env we need additional configuration. So process.env is not a common standard for client packages.

I guess it's better we directly build two version dist files and make SANDPACK_BARE_COMPONENTS take effect at build time. The default build will include the default styles and a new build maybe called styleless/index.js exculde it.

In this way we can complete exclude stitches related code in the styleless build, in the current solution, although we can disable the default style but the code still being bundled into the application which is a waste. And because of the typeof process condition tree shaking will take effect to reduce the useless code.