-
Notifications
You must be signed in to change notification settings - Fork 148
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set extensionKind in package.json to support Remote Development #95
Comments
Fixes wakatime#95. See that issue for details
Fixes #95. See that issue for details
Merged, thanks. |
No, it's not a UI extension as it needs access to files on the filesystem. |
@mjbvz is that correct, that since the extension needs to read the current file contents, and the contents of surrounding folders, that it must be |
@alanhamlett Just encountered this problem, WakaTime is not working with remote development now (and sadly no option to install on remote, possibly caused by the incorrect "extensionKind"). |
@vilic can you add this to your settings and see if it works:
|
That helps the installation on remote, but then I get this:
And |
It seems that file |
Relaunching VS Code causes the WakaTime extension to extract that zip file into the |
@alanhamlett Seems that I just keep getting 0 length |
Looks like that my issue was caused by VS Code applying local proxy settings (implicit imported from system) to the remote extensions. microsoft/vscode#60773 |
@alanhamlett So now basically it's working. There's some issue opening WakaTime dashboard from status bar though ( |
@vilic do you have Mac or Linux? if Mac, which OS version? |
@gandarez local Windows 10 with remote Ubuntu 18.04 |
you mean you're using a |
@gandarez Sorry for getting you confused. I was just saying the Live Share (installed remotely) button can open local browser successfully while WakaTime can't: I am ussing Remote-SSH to connect (there's nothing to do with Live Share). |
For me |
@alanhamlett It depends on which files you are accessing and how you are doing this. If you are reading files from the workspace (and not using the vscode apis to do this) then you must be a workspace extension I'll remove your extension from the whitelist so its entirely up to you to decide where it should be run |
Thanks! It needs to be a workspace extension so I'll add the necessary config changes. |
Fixed in v2.1.0. |
@mjbvz are there vscode apis that allow a UI extension to read the contents of an opened file on a remote workspace? |
The |
What is the current status of this issue? I have wakatime added to "always installed extensions" in my VSCode settings. Every time I rebuild the container (which is a lot), I have to re-enter the API key. One possible solution as (partly) suggested in #130, would be allow Wakatime to read the API key as an environment variable and somehow pass the wakatime API key to the container. For example (in {
// other stuff
"remoteEnv": {
"WAKATIME_API_KEY": "${localEnv:WAKATIME_API_KEY}"
}
} Then wakatime can first check if This has the additional benefit of not causing any problems for those people who do not use wakatime. For them the I think we should keep it up to the users to ensure that the @alanhamlett What do you think about this? I can try making a PR for this too. |
Hi, I'm on the VS Code team. We recently released support for Remote Development and I noticed that your may extension need a small change to ensure users have a good experience when using it remote workspaces.
What is the issue?
To make remote development as transparent as possible to users, VS Code distinguishes two classes of extensions:
UI Extensions: These extensions make contributions to the VS Code user interface and are always run on the user's local machine. UI Extensions cannot directly access files in the workspace, or run scripts/tools installed in that workspace or on the machine. Example UI Extensions include: themes, snippets, language grammars, and keymaps.
Workspace Extensions: These extensions are run on the same machine as where the workspace is located. When in a local workspace, Workspace Extensions are run on the local machine. When in a remote workspace, Workspace Extensions are run on the remote machine. Workspace Extensions can access files in the workspace to provide rich, multi-file language services, debugger support, or perform complex operations on multiple files in workspace (either themselves or by invoking scripts/tools).
You can find more details about this architecture here.
VS Code currently infers that your extension is a Workspace Extension. This means that users who have your extension installed must also install it to the remote in order to use it in remote workspaces. I believe that your extension should probably be a UI extension instead. That way your extension will always be enabled for users who install it, even if they open a remote workspace.
How do I fix this?
To tell VS Code that your extension is a UI extension, just add
"extensionKind": "ui"
to your extension's package.json.UI Extensions always run on the user's local machine, even when they open a remote workspace.
I'll submit a PR that does this. Please let me know if you have any questions or concerns about the issue. We've also put together a guide to help you test your extension in remote workspaces
PS: As a temporary workaround for a few popular extensions, we've automatically added your extension to an internal whitelist so that is always treated as a UI extension
The text was updated successfully, but these errors were encountered: