Global not defined in ESM bundles.

[dsm23]

• three version: 0.125.2
• react-three-fiber version: 5.3.18
• @react-three/drei version: 3.2.2
• node version: 15.5.1
• npm (or yarn) version: 2.4.0

Problem description:

It appears that importing from the @react-three/drei library in one of the 2 snowpack react templates throws the ReferenceError: global is not defined. Issues appears to be specific to this library

[joshuaellis]

Hi @dsm23,

Have you seen this issue? #273

it contains a work around, however if you want to provide a fix that we can permanently implement that would be great! We currently use global as a fix for SSR.

I’m not clued up enough on Snowpack to suggest a solution for the long term yet.

[joshuaellis]

This is the offending line:

drei/src/helpers/base64.ts

Line 3 in 9f30f82

global['Blob'] = Blob

. I’m wondering if we need it. Whether importing Blob would simply fix our old SSR issue and now new ESM issue...

I could remove this line and release a beta tomorrow to see...

[dsm23]

I had not seen the other issue, thank you for point it out to me. If you do need the offending statement would a globalThis not also work?

[joshuaellis]

Hi @dsm23,

I've released a patch on 3.2.3-beta.1 if you could try it out in snowpack that'd be great. I can test later in the day with SSR.

[github-actions]

🎉 This issue has been resolved in version 3.3.0-beta.2 🎉

The release is available on:

• npm package (@beta dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[github-actions]

🎉 This issue has been resolved in version 3.3.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[dsm23]

works well, thank you

[joshuaellis]

You're welcome @dsm23 👌🏼

[cedeber]

This is good news, thanks. I confirm that it works with snowpack without the Node polyfill. Now, I'll test it with Vite too :-D

[cedeber]

Also works with Vitejs 🥳