Skip to content

extensions: Yet another PR for debugger touchups #32822

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

Merged
merged 6 commits into from
Jun 17, 2025
Merged

Conversation

osiewicz
Copy link
Contributor

We'll now clean up DAP locators for unloaded extensions and load schemas proper

I can now load a custom Ruby extensions with all bells and whistles and use it as my debugger.

Release Notes:

  • N/A

We'll now clean up DAP locators for unloaded extensions and load schemas proper
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jun 16, 2025
@osiewicz osiewicz force-pushed the dap-extensions-v1000000 branch from 4b9d707 to a2784e7 Compare June 16, 2025 22:51
@osiewicz osiewicz enabled auto-merge (squash) June 17, 2025 07:33
@osiewicz osiewicz merged commit 0e794fa into main Jun 17, 2025
20 checks passed
@osiewicz osiewicz deleted the dap-extensions-v1000000 branch June 17, 2025 07:34
JosephTLyons pushed a commit that referenced this pull request Jun 17, 2025
We'll now clean up DAP locators for unloaded extensions and load schemas
proper

I can now load a custom Ruby extensions with all bells and whistles and
use it as my debugger.

Release Notes:

- N/A
osiewicz pushed a commit that referenced this pull request Jun 19, 2025
Closes #33039

This PR fixes a bug which causes the newest versions of the Biome and
Tombi extensions to not work with older Zed versions.

The bug occurs because in #32822, the type of the debug adapter and
debug locators was changed from a Vec to a BTreeMap. However, these
fields were already introduced much earlier in Zed, which now causes the
de-serialization of the `extension.toml` to fail for older Zed versions.
Any extension compiled with the newest extension CLI bumped in
zed-industries/extensions#2866 will not work
with older Zed versions prior to v0.191.

By adding this change and bumping the extension CLI again, this could be
prevented. On de-serialization, we would just fallback to either a Vec
for versions prior to v0.190 or a BTreeMap after. Feel free to let me
know what you think here.

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant