Skip to content

fix(desktop): restore augmentProcessPath() lost in #1334 rebase#1361

Merged
esengine merged 1 commit into
mainfrom
fix/desktop-restore-augment-path
May 20, 2026
Merged

fix(desktop): restore augmentProcessPath() lost in #1334 rebase#1361
esengine merged 1 commit into
mainfrom
fix/desktop-restore-augment-path

Conversation

@esengine

Copy link
Copy Markdown
Owner

Summary

Test plan

  • tests/desktop-login-shell-path.test.ts still green
  • Full npm run verify green (235 files / 3256 tests via pre-push hook)

#1331 added augmentProcessPath() in desktopCommand so the Tauri-spawned
Node could see nvm/asdf/fnm/volta/mise PATH entries — the GUI process
never runs the user's shell init. #1334 was branched before #1331 landed
and the rebase dropped the import + call without it being part of the
QQ-runtime work.

Re-add the import and the call site, no behavior change beyond restoring
the previous fix.
@esengine esengine merged commit 95864a9 into main May 20, 2026
4 checks passed
@esengine esengine deleted the fix/desktop-restore-augment-path branch May 20, 2026 00:58
ChasLui pushed a commit to ChasLui/DeepSeek-Reasonix that referenced this pull request May 23, 2026
…se (esengine#1361)

esengine#1331 added augmentProcessPath() in desktopCommand so the Tauri-spawned
Node could see nvm/asdf/fnm/volta/mise PATH entries — the GUI process
never runs the user's shell init. esengine#1334 was branched before esengine#1331 landed
and the rebase dropped the import + call without it being part of the
QQ-runtime work.

Re-add the import and the call site, no behavior change beyond restoring
the previous fix.

Co-authored-by: reasonix <reasonix@deepseek.com>
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