Fix: Add bundled uv to PATH for Linux/macOS package launches - #1721
Merged
Conversation
- Prepend the bundled uv directory (Assets/uv) to PATH in UvVenvRunner.RunDetached on non-Windows platforms - Lets package launch scripts that invoke `uv` (e.g. Forge Neo) find it without a system install - Declare the uv path once, above the platform branches, so both Windows and non-Windows paths reuse it
mohnjiles
approved these changes
Aug 23, 2026
ionite34
approved these changes
Aug 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The bundled UV currently does not get passed to package launches on Linux/MacOS installs, just on Windows.
This leads to WebUI packages not being able to utilize UV if required/attempted when the user does not have a system-wide UV installed in their OS, or otherwise added to PATH by the user.
This is currently causing Forge Neo installs to fail on Linux with 'Error: 2 uv not found'.
This fix applies the SM bundled UV to be passed to package launches regardless of OS env path. While keeping the PortableGit path specific to Windows installs.
Potentially fixes #1713 as Forge Neo is very uv-dependent and the package config for forge-neo in the project by default launches Neo with --uv extended from ForgeClassic.cs:
var launchArgs = new List<string> { "launch.py", "--uv", "--exit" }which runs Neo's launch process through its UV path inmodules_forge/uv_hook.pywhich requires UV to be on PATH. User may have had UV installed system-wide but still cascaded to failure.