Enable search of local Typescript Language Service Plugins (fix #45856) #45858
Conversation
|
|
| @@ -74,6 +76,7 @@ export class TypeScriptServiceConfiguration { | |||
| && this.localTsdk === other.localTsdk | |||
| && this.npmLocation === other.npmLocation | |||
| && this.tsServerLogLevel === other.tsServerLogLevel | |||
| && this.tsServerPluginPaths === other.tsServerPluginPaths | |||
There was a problem hiding this comment.
Should be array contents comparison
| @@ -106,6 +106,16 @@ | |||
| "description": "%typescript.tsserver.log%", | |||
| "scope": "window" | |||
| }, | |||
| "typescript.tsserver.pluginPaths": { | |||
There was a problem hiding this comment.
Add "isExecutable": true here. This prevents this from ever being used as a workspace setting
There was a problem hiding this comment.
Why do you think it should be a user-only setting?
Specifying pluginPaths on workspace level can be quite useful, especially in conjunction with relative paths. In this case plugins configuration can be committed to a source control and shared between team members.
If this is this because of the security concerns, I don't believe this is a more dangerous option than ability to specify typescript.tsdk on a workspace-level.
There was a problem hiding this comment.
Yes it is for security reasons. Tsdk asks for user confirmation before executing any code in the workspace but that flow is complicated so I don't want to duplicate it here. Instead we should just limit these this to be a user only setting
|
@mjbvz Changes have been made according to your comments. |
killerDJO
force-pushed
the
master
branch
3 times, most recently
from
f5c8e12 to
9c43780
Compare
…discovery locations
…, setting made executable
|
Thanks! On track for the March vscode release |
Proposed change for #45856. Added option to discover plugins in project-local node modules