Uncaught ReferenceError: process is not defined

[ericsoco]

Subject of the issue

The VFile constructor references process directly. In a vanilla browser environment, this throws an error. However, many build systems polyfill process so this may have gone unnoticed in most applications -- process is mocked by default by Webpack, and Browserify does something similar.

Your environment

• OS: Mac OSX 10.14.5
• Packages: unified@6.2.0, vfile@2.3.0 (via react-markdown@4.2.2)
• Env:
  • node@10.16.0
  • npm@6.9.0
  • yarn@1.16.0
  • Chrome v76.0.3809.132

Steps to reproduce

In an environment without a global process object (e.g. a vanilla browser, a webpack build with the process mock disabled):

• Call vfile(), either as a standalone function (which redirects to call as a constructor) or as a constructor (new VFile()).
• or call ☝️from e.g. processor.parseFile()

Expected behaviour

The VFile() constructor instantiates an object.

Actual behaviour

The VFile() constructor throws.

[wooorm]

Correct, vfile depends on process, so the mocking of process in other environments is indeed expected. I see it as expected behaviour, not as an issue 🤷‍♂️

[ericsoco]

vfile claims to work in the browser. From the README:

Plus, they work in the browser.

It does work in the browser, but only if process (specifically, process.cwd()) is polyfilled. This is a non-obvious gotcha.

How is VFile.cwd used by vfile consumers? Browsing the source (via GH search, which can yield spotty results), I don't see it referenced in either vfile or unified...

[wooorm]

Btw, we talked about this with rexxars from react-markdown already: #16 (comment)

It does work in the browser, but only if process (specifically, process.cwd()) is polyfilled. This is a non-obvious gotcha.

This is indeed how it is intended: it depends on certain aspects, which are available in Node, and typically available in bundlers as well. Unless explicitly turned off.

How is VFile.cwd used by vfile consumers? Browsing the source (via GH search, which can yield spotty results), I don't see it referenced in either vfile or unified...

file.cwd is used because file.path can be relative, in which case, it should be read relative to file.cwd. Indeed, GH search isn’t great, but searching across the collective yields more results.

---

There are several downsides for touching/changing/removing cwd, as outlined in the issue mentioned above, but also: It’s breaking. vfile itself was downloaded 100m times this year. In total, the projects in the collective are downloaded more than 2B times a year. These numbers are multiplying manyfold per year. Breaking stuff that’s used so much has consequences.

[ericsoco]

Understood. Thanks for the context. For our project, I've restored the Webpack process polyfill that our framework removes. Probably safe to close this as a wontfix given the risk involved in a change, given the change will likely benefit a much smaller percentage of users than those negatively impacted by an error in a change.

[wooorm]

Sweet, thanks for raising this and for understanding!

Best

[ericsoco]

Heads up that Webpack is considering removing default polyfilling, so this may become more of an issue moving forward.

You might consider leaving them feedback if you have thoughts (they're asking for feedback on the decision).