Experience-Monks / Experience-Monks/devtool

Supporting NODE_PATH

Open
#31 0 comments 8 reactions 0 assignees View on GitHub
bug
Dominant language
JavaScript
Stars
3.7k
Forks
147
PR merge metrics
No merged PRs in 30d

Description

This is related to #6.

When Electron initializes the Module paths, it will grab everything from `NODE_PATH` and merge it with the user's home dirs.

However, at some point in execution, this `Module.globalPaths` is mutated and replaced with a new set of paths pointing to electron globals so that `require('electron')` will work. It is unclear where this mutation is happening, and at what point we need to re-patch the `Module.globalPaths` array to fix this.

To recap, these two features are broken right now:
- `NODE_PATH` is not respected
- `.node_modules` and `.node_libraries` in home dir is not respected

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.