node-app-root-path icon indicating copy to clipboard operation
node-app-root-path copied to clipboard

Incompatible with Browserify

Open Bluebie opened this issue 5 years ago • 1 comments

Browserify 16.5.0 does not seem to provide a .main property on the require object, so requiring app-root-path is failing because browser cannot read property .filename from require.main (which is undefined)

https://github.com/inxilpro/node-app-root-path/blob/54d06fef2340a36070fbfc24973bb676079aa7a1/browser-shim.js#L3

#22 seems to be related to the same problem, but was closed without any more information about why.

You can reproduce breaking browserify with:

echo "console.log(require('app-root-path').toString())" > test.js
browserify test.js -o bundle.js

and then trying to load bundle.js with a script tag in a browser.

Bluebie avatar Feb 02 '20 08:02 Bluebie

Would you be willing to submit a PR for the browser shim?

inxilpro avatar Aug 17 '21 17:08 inxilpro