Q? How to prevent Hydrogen from prepending $PATH with lots of node_modules\.bin paths?

[kiwi0fruit]

How to prevent Hydrogen from prepending $PATH with lots of node_modules\.bin paths?

[BenRussert]

I dont understand the question, could you elaborate?

[rgbkrk]

Where does this happen @kiwi0fruit? Can you show the behavior? This doesn't sound right.

[kylebarron]

If I open up a Python file in Atom and run:

import os
os.getenv('PATH')

I get:

'/Users/kyle/tmp/node_modules/.bin:/Users/kyle/node_modules/.bin:/Users/node_modules/.bin:/node_modules/.bin:/Applications/Atom.app/Contents/Frameworks/Atom Helper.app/Contents/MacOS:/Library/Java/JavaVirtualMachines/jdk1.8.0_181.jdk/Contents/Home/bin:/Users/kyle/local/bin:/Users/kyle/bin:/usr/local/bin:/Applications/Sublime Text.app/Contents/MacOS:/Users/kyle/anaconda/bin:/Applications/Julia-0.6.app/Contents/Resources/julia/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Library/TeX/texbin:/usr/local/go/bin:/usr/local/MacGPG2/bin:/opt/X11/bin:/Users/kyle/github/go/bin'

I find the same thing in R and Stata kernels as well, so I'm guessing that the PATH is changed in Atom somewhere, and the kernels inherit that.

[kylebarron]

echo $PATH in the directory I opened Atom from gives:

/Library/Java/JavaVirtualMachines/jdk1.8.0_181.jdk/Contents/Home/bin:/Users/kyle/local/bin:/Users/kyle/bin:/usr/local/bin:/Applications/Sublime Text.app/Contents/MacOS:/Users/kyle/anaconda/bin:/Applications/Julia-0.6.app/Contents/Resources/julia/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Library/TeX/texbin:/usr/local/go/bin:/usr/local/MacGPG2/bin:/opt/X11/bin:/Users/kyle/github/go/bin

so the added paths are:

/Users/kyle/tmp/node_modules/.bin:/Users/kyle/node_modules/.bin:/Users/node_modules/.bin:/node_modules/.bin:/Applications/Atom.app/Contents/Frameworks/Atom Helper.app/Contents/MacOS

[rgbkrk]

Does Hydrogen use shell-env underneath? I imagine we put some things in to inherit the user's shell so we can detect the right python etc. environment.

[kiwi0fruit]

nteract desktop app also prepends PATH this same way.

[BenRussert]

This is coming from spawnteract's execa dependency (maybe?)

xref sindresorhus/execa#153