Skip to content

fix(desktop): enrich Electron runtime PATH#1652

Merged
src-opn merged 1 commit into
devfrom
path-mcp-issues
May 5, 2026
Merged

fix(desktop): enrich Electron runtime PATH#1652
src-opn merged 1 commit into
devfrom
path-mcp-issues

Conversation

@src-opn
Copy link
Copy Markdown
Collaborator

@src-opn src-opn commented May 5, 2026

Summary

  • Enrich Electron child-process PATH with sidecars plus common Homebrew, nvm, fnm, volta, pnpm, bun, cargo, pyenv, and local-bin locations.
  • Use macOS path_helper output as best-effort PATH input for Finder/Dock launches.
  • Apply the enriched PATH to Electron runtime children and binary resolution so local MCPs can find npm/node/npx in production.

Verification

  • node --check apps/desktop/electron/runtime.mjs
  • pnpm --filter @openwork/desktop check:electron

Notes

  • pnpm --filter @openwork/desktop typecheck:electron could not run in this worktree because node_modules/tsc are not installed.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-app Building Building Preview, Comment May 5, 2026 4:34pm
openwork-den Building Building Preview, Comment May 5, 2026 4:34pm
openwork-den-worker-proxy Building Building Preview, Comment May 5, 2026 4:34pm
openwork-landing Building Building Preview, Comment, Open in v0 May 5, 2026 4:34pm
openwork-share Ready Ready Preview, Comment May 5, 2026 4:34pm

@src-opn src-opn merged commit d1b2e38 into dev May 5, 2026
7 of 11 checks passed
benjaminshafii pushed a commit that referenced this pull request May 15, 2026
Co-authored-by: src-opn <src-opn@users.noreply.github.com>
benjaminshafii pushed a commit that referenced this pull request May 15, 2026
Co-authored-by: src-opn <src-opn@users.noreply.github.com>
@benjaminshafii benjaminshafii deleted the path-mcp-issues branch May 15, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant