Skip to content

fix: use execFile to bypass shell command resolution-W-23662030 - #897

Open
soridalac wants to merge 2 commits into
mainfrom
sl/W-23662030
Open

fix: use execFile to bypass shell command resolution-W-23662030#897
soridalac wants to merge 2 commits into
mainfrom
sl/W-23662030

Conversation

@soridalac

Copy link
Copy Markdown
Contributor

Summary

  • Replace child_process.exec() with child_process.execFile() when invoking npm
  • execFile does not spawn a shell, preventing unintended command resolution from the working directory on Windows

Test plan

  • Existing unit tests pass
  • Verified sf version --verbose works correctly on Windows and macOS
  • Verified npm registry lookup still returns version info for user-installed plugins

@W-23662030@

@git2gus

git2gus Bot commented Aug 3, 2026

Copy link
Copy Markdown

Git2Gus App is installed but the .git2gus/config.json doesn't have right values. You should add the required configuration.

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