goose icon indicating copy to clipboard operation
goose copied to clipboard

Run the developer mcp in a login shell

Open jamadeo opened this issue 5 months ago • 1 comments

I'm not sure we want to do this, but if we do, this would probably be the way

jamadeo avatar Nov 06 '25 20:11 jamadeo

I think this may be worthwhile, previously, it would inherit it from the forced loading on macos, but now people may find it is missing things they expect are on their $PATH, so this would be closer to what you run if you run from cli, so principle of least surprise: I think yes?

What is downside to this?

michaelneale avatar Nov 07 '25 01:11 michaelneale

I just opened #5680 before I realized this draft PR was already open for this, FWIW I am definitely in favor of running a login shell bc I've found the developer extension in goose UI/desktop to be pretty limited now - it can no longer run any non system PATH command which includes rg, node/npm, pip/uv, jq, yq, etc. Not having rg in particular is tough since goose makes heavy use of it during local development and when exploring local codebases

This also breaks Presidio MCP for Block users in UI/desktop, because that MCP implementation uses subprocess.run() to launch sq presidio which similarly fails since sq is also installed via Homebrew

wpfleger96 avatar Nov 11 '25 17:11 wpfleger96

done in #5774

jamadeo avatar Nov 19 '25 19:11 jamadeo