Skip to content
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

VS Code Extension: Installation / update fails on Windows #2302

Closed
kryops opened this issue Apr 12, 2022 · 5 comments
Closed

VS Code Extension: Installation / update fails on Windows #2302

kryops opened this issue Apr 12, 2022 · 5 comments
Labels
bug lsp-server graphql-language-service-server

Comments

@kryops
Copy link

kryops commented Apr 12, 2022

Hi there,

the VS Code extension does not want to be installed or updated on Windows any longer. My system just tried and failed to update from 0.3.53 to 0.4.1. It's not just on my machine, a colleague of mine could reproduce as well.

It seems to affect automatic and manual updates as well as fresh installations. The only thing that seems to work is installing via the command line.

Sadly, I could not find any actual errors in the logs, the only thing that pops up is the "No ptyHost heartbeat after 12 seconds" error.

Here is the "Log (Shared)" output:

[2022-04-12 12:09:37.762] [sharedprocess] [info] Getting Manifest... graphql.vscode-graphql
[2022-04-12 12:09:38.372] [sharedprocess] [info] Installing extension: graphql.vscode-graphql
[2022-04-12 12:09:38.434] [sharedprocess] [info] Downloaded extension: graphql.vscode-graphql c:\Users\xxx\AppData\Roaming\Code\CachedExtensionVSIXs\graphql.vscode-graphql-0.4.1
[2022-04-12 12:09:54.395] [sharedprocess] [warning] No ptyHost heartbeat after 6 seconds
[2022-04-12 12:09:59.409] [sharedprocess] [error] No ptyHost heartbeat after 12 seconds

Environment:

  • VS Code 1.66.1
  • Windows 10 21H2

Seems to work fine on macOS.

Thanks for looking into it!

@acao
Copy link
Member

acao commented Apr 17, 2022

This looks quite bad!

it will be some time until I can reproduce on windows to find out more

@acao acao added bug lsp-server graphql-language-service-server labels Apr 17, 2022
@kryops
Copy link
Author

kryops commented Apr 25, 2022

I just had time to fiddle around with this a bit myself. I could not find the original cause of the problem, but it seems to be gone when building the current main branch locally.

@acao #2288 looks like you intended to publish a 0.4.2 for the VS Code extension? 0.4.1 is still the current version on the marketplace, so it probably didn't work.

When building locally, I had to change https://github.com/graphql/graphiql/blob/main/packages/vscode-graphql/esbuild.js#L4 to console to make it work, maybe that's the issue there as well.

@acao
Copy link
Member

acao commented Apr 25, 2022

The issue is with our publishing pipeline which I haven’t had time to fix. Atlassian changesets makes it quite difficult to define alternative publishing logic, compared to say, semantic release.

The version you are using isn’t bundled vs the version in the repo is bundled. Thus path issues go away because the extension and dependencies resolve from a single file.

Once I have stable internet and furniture I will get back to this, in a couple weeks

@acao
Copy link
Member

acao commented Apr 25, 2022

just published that version to vscode marketplace finally! I will fix the github actions later. let me know if your issue is resolved in the latest release!

@kryops
Copy link
Author

kryops commented Apr 26, 2022

0.4.3 installs perfectly again, thanks a lot! ☺️

@kryops kryops closed this as completed Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug lsp-server graphql-language-service-server
Projects
None yet
Development

No branches or pull requests

2 participants