Test if process is defined in browser

[pelikhan]

Hello all!

First thanks for this very well written library. It's a breeze to work with it.

I am hitting a snag when compiling graphql into a ES6 library. The node.js process object is queried in a mjs file which crashes in the browser. It would be great to add a check to make sure process is defined.

graphql-js/src/jsutils/instanceOf.js

Line 14 in 2736f45

export default process.env.NODE_ENV === 'production'

Thanks!

[IvanGoncharov]

@pelikhan Thanks for taking the time to report it 👍
I have plan to remove process completely in the new major version and switch to having development and production build inside the NPM package similar to how it's done by React.

[schrotie]

Temporary workaround until this fix is in the release:

Import an ES module with this content before importing GraphQL:
window.process = {env: {}};

[yaacovCR]

Closed by #3501