-
-
Notifications
You must be signed in to change notification settings - Fork 720
Description
I see #13542 was merged an hour ago. This only covers the case with the relative path, correct? I could not share
oxc.path.serveras a project wide setting in my.vscode/settings.json, if I have a mix of Windows and non-Windows developers as I sometimes would have to use.exeand sometimes not. (Referring to this comment.)
Originally posted by @donaldpipowitch in #12849
Created an own issue for it, because it is more work than expected.
There is no .exe, only .CMD and .ps1 (in NPM/PNPM).
Bun has .exe file in node_modules/.bin.
There could be 2 solutions:
Enable shell option, so VSCode can use node_modules/.bin/language_server too.
This will enable a security hole (#13732 (comment)), which should be covered by #13740 and #13734.
I would like to force somehow more checks before enabling this.
Search for .exe inside node_modules.
This could be a bit difficult because the package manager could but the file somewhere else than expected.
The other editors could have solved this already. But we should check if this works with different package managers as expected.