johnpapa / vscode-angular-snippets

Angular Snippets for VS Code
MIT License
575 stars 163 forks source link

Express configuration support requires workspace access - breaks extensionKind 'ui' #111

Closed egamma closed 4 years ago

egamma commented 4 years ago

The extension is currently defined as a 'ui' extension. This is correct when it comes to snippets, they run fine in the UI extension host only.

However, the extension also provides a command that configures an express setup "Add Node.js express file to workspace". This command writes files to the workspace https://github.com/johnpapa/vscode-angular-snippets/blob/ecf7345e6bd782c8b72862b1e5d84f69cab73604/src/configureWorkspace/configure.ts#L80. This will fail when run in a remote setup, since the extension runs in the UI extension host where there is no access to the workspace.

There are several options:

In general we prefer that customizations like themes and snippets are have the following "extensionKind": ["ui", "workspace"].

johnpapa commented 4 years ago

thanks for the details and options.

given your recommendation on making this a ui and workspace, i'll merge that PR and close this issue. fixed in 9.1.2