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

ms-vscode.js-debug no longer included #123

Closed
Tracked by #13192
tsmaeder opened this issue Aug 14, 2023 · 5 comments · Fixed by #124
Closed
Tracked by #13192

ms-vscode.js-debug no longer included #123

tsmaeder opened this issue Aug 14, 2023 · 5 comments · Fixed by #124
Labels
bug Something isn't working

Comments

@tsmaeder
Copy link
Contributor

The ms-vscode.js-debug extension that provides typescript/javascript debugging in VS Code is no longer included in the theia built-in extension pack in the latest version. It used to be included in the built-in extension pack.

@tsmaeder tsmaeder added the bug Something isn't working label Aug 14, 2023
@tsmaeder
Copy link
Contributor Author

This extension is central to dog-fooding and included in VS Code out of the box.

@vince-fugnitto
Copy link
Member

According to the logs when we published it seems that it was skipped.
@marcdumais-work any ideas?

@marcdumais-work
Copy link
Contributor

According to the logs when we published it seems that it was skipped. @marcdumais-work any ideas?

Hummm, It seems this extension and its companion were skipped, when generating the built-ins pack, because they were not found on openvsx. I do not know if it's a bug in our script or if there was a glitch with the server.

Skipping extension, as it's not found in the registry : ms-vscode.js-debug-companion
Skipping extension, as it's not found in the registry : ms-vscode.js-debug

marcdumais-work added a commit to marcdumais-work/vscode-builtin-extensions that referenced this issue Aug 14, 2023
We use node-fetch to call open-vsx API end-points. For external
built-in extensions, we want to make sure they exist on open-vsx
before adding the mto the pack, to avoid issues installing the pack.
For this case, we were generating a URL that had a trailing slash,
which did not work:

Original URL, returns 404:
https://open-vsx.org/api/ms-vscode/js-debug-companion/

Corrected URL (works):
https://open-vsx.org/api/ms-vscode/js-debug-companion

Fixes eclipse-theia#123

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
@marcdumais-work
Copy link
Contributor

I think I found a bug that explains the above. We can fix it now, so that these extensions will be part of the pack next time.

marcdumais-work added a commit to marcdumais-work/vscode-builtin-extensions that referenced this issue Aug 14, 2023
We use node-fetch to call open-vsx API end-points. For external
built-in extensions, we want to make sure they exist on open-vsx
before adding them to the pack, to avoid issues when resolving the
pack(we do not check for a specifc version, because they may not
follow vscode's versioning).

It turns-out that For this case, we were generating a URL that had
a trailing slash, which did not work:

Original URL, returns 404:
https://open-vsx.org/api/ms-vscode/js-debug-companion/

Corrected URL (works):
https://open-vsx.org/api/ms-vscode/js-debug-companion

P.S. also aligned version of `node-fetch` and its typings

Fixes eclipse-theia#123

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
marcdumais-work added a commit to marcdumais-work/vscode-builtin-extensions that referenced this issue Aug 14, 2023
We use node-fetch to call open-vsx API end-points. For external
built-in extensions, we want to make sure they exist on open-vsx
before adding them to the pack, to avoid issues when resolving the
pack(we do not check for a specific version, because they may not
follow vscode's versioning).

It turns-out that For this case, we were generating a URL that had
a trailing slash, which did not work:

Original URL, returns 404:
https://open-vsx.org/api/ms-vscode/js-debug-companion/

Corrected URL (works):
https://open-vsx.org/api/ms-vscode/js-debug-companion

P.S. also aligned version of `node-fetch` and its typings

Fixes eclipse-theia#123

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
marcdumais-work added a commit to marcdumais-work/vscode-builtin-extensions that referenced this issue Aug 14, 2023
We use node-fetch to call open-vsx API end-points. For external
built-in extensions, we want to make sure they exist on open-vsx
before adding them to the pack, to avoid issues when resolving the
pack(we do not check for a specific version, because they may not
follow vscode's versioning).

It turns-out that For this case, we were generating a URL that had
a trailing slash, which did not work:

Original URL, returns 404:
https://open-vsx.org/api/ms-vscode/js-debug-companion/

Corrected URL (works):
https://open-vsx.org/api/ms-vscode/js-debug-companion

P.S. also aligned version of `node-fetch` and its typings

Fixes eclipse-theia#123

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
marcdumais-work added a commit that referenced this issue Aug 15, 2023
We use node-fetch to call open-vsx API end-points. For external
built-in extensions, we want to make sure they exist on open-vsx
before adding them to the pack, to avoid issues when resolving the
pack(we do not check for a specific version, because they may not
follow vscode's versioning).

It turns-out that For this case, we were generating a URL that had
a trailing slash, which did not work:

Original URL, returns 404:
https://open-vsx.org/api/ms-vscode/js-debug-companion/

Corrected URL (works):
https://open-vsx.org/api/ms-vscode/js-debug-companion

P.S. also aligned version of `node-fetch` and its typings

Fixes #123

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
@marcdumais-work
Copy link
Contributor

marcdumais-work commented Aug 15, 2023

@tsmaeder we found and fixed the bug, that mistakenly excluded ms-vscode.js-debug and its companion extension, from the vscode built-ins extension pack v1.79.0. When we next publish a new set of built-ins and a new pack along with it, the extensions should be present again.

The good news is that Blueprint is unaffected (probably like most Theia apps out- there) - It's not yet using that version of the built-ins pack, currently using slightly older v1.77.0. The extensions, listed in a pack, are not versioned, and always resolve to the latest compatible version, according to stated extensions API version supported (the version of the pack does not play a role). This means that, using a relatively recent version of the pack, should be virtually just as good. The only downside I see is that any extension added or legitimately removed in 1.79.0 would not be if we used the previous version. I do not think that's a problem ATM, but if it is for someone, they can work-around the issue by strategically explicitly adding and/or excluding extensions, to fit their needs.

Plan going forward: For Blueprint, wait until a new set of vscode built-ins is released before updating the built-ins pack it uses (currently v1.77.0)

WDYT?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants