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

DapAdapter Updates #40

Merged

Conversation

Anthony-Eid
Copy link
Collaborator

@Anthony-Eid Anthony-Eid commented Sep 22, 2024

Release Notes:

The aim of this PR is start making debug adapters more inline with how Zed implements LSPAdapters. This PR will start this off by setting up the automatic downloads of debug adapters (With users permission) and change our debugger implementation to use DAPBinary's.

TODO:

  • Share http_client with dap_store to enable granting http access to debug adapters that are downloaded from git
  • Change the DebugAdapter trait to be more inline with the LSPAdapter trait
  • Use DebugAdapterBinary to start debugger client
  • Create dap_adapters crate to organize language specific debug adapter trait implementations
  • Automatically download debug adapters (debugpy)

@Anthony-Eid Anthony-Eid changed the title Pass http client to dap store DapAdapter Updates Sep 27, 2024
Anthony-Eid and others added 9 commits September 27, 2024 09:46
Co-authored-by: Remco Smits <djsmits12@gmail.com>
This is the first phase of dap::client refactor to organize debug adapters with zed lsp adapters.
Eventually dap::client will have a TransportParams pass to it instead of an adapter, and adapters
will handle custom debug events.

Co-authored-by: Remco Smits <djsmits12@gmail.com>
This change was done to make dap::client more in line with zed's lsp::client, it might be reverted depending
on if this solution is cleaner or not.
@Anthony-Eid Anthony-Eid marked this pull request as ready for review October 7, 2024 20:32
@Anthony-Eid Anthony-Eid merged commit 187d909 into RemcoSmitsDev:debugger Oct 7, 2024
0 of 4 checks passed
@Anthony-Eid Anthony-Eid deleted the debug-adapter-downloads branch October 7, 2024 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant