Skip to content

Sync the application path with the shell - #358

Merged
RBilly merged 2 commits into
emdgroup:mainfrom
cthrall:fix/sync-path
Sep 11, 2026
Merged

RBilly merged 2 commits into
emdgroup:mainfrom
cthrall:fix/sync-path

Conversation

@cthrall

@cthrall cthrall commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Summary

When the application is launched from the Finder in Mac OS, the path that is passed to the application is the launchd path (/usr/bin:/bin:/usr/sbin:/sbin). It's possible that command-line tools like Git are installed elsewhere. This change uses the crate fix-path-env-rs to sync the application path with the user's shell path.

Test Plan

  • Launch Maestro from the Finder in Mac OS on a computer where Git is installed somewhere that is not in the launchd path and start an agent session that creates a new worktree.

Release Notes

Craig Thrall added 2 commits September 10, 2026 23:07
When the application is launched from the Finder in Mac OS, the path that is
passed to the application is the launchd path (/usr/bin:/bin:/usr/sbin:/sbin).
It's possible that command-line tools like Git are installed elsewhere.
This change uses the crate fix-path-env-rs to sync the application path with
the user's shell path.
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Craig Thrall seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@RBilly
RBilly merged commit a85f7e3 into emdgroup:main Sep 11, 2026
2 of 3 checks passed
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.

3 participants