* Fix clippy

* Remove increment request sequence (wrong while merging)

* Send correct request to stop thread based on capabilities

* Fix used wrong request args in set breakpoints request

Some debug adapters depend on getting the exact data that you passed in `launch` or `attach` request.

* Send correct request for stopping debug adapter

I changed the name to be more in line with the request name. We now also send the correct request values on the `support_terminate_debuggee` and `support_terminate_debuggee` capabilities.

* Send disconnect request for terminate threads if it does not support it

* Only send configuration done request if its supported

* Add disconnect icon

* Only send step over request params when adapter supports it

* Only send resume(continue) request params if adapter supports it

* Step in only send request args if adapter supports it

* Step out only send request args if adapter supports it

* Step back only send request args if adapter supports it

* Log error using `detach_and_log_err` instead of manually

I also made a new DS specifically for breakpoints that aren't part of any open buffers.
It should be easier to serialize and deserialize breakpoints now.

…is released

…alization

Code action's that overlap with breakpoints will now show as red. Still need to
add a code action to toggle a breakpoint & handle overlaps with code runners

…s loading

I also handle overlaps with code test buttons as well

…ning instead of starting

We shouldn't consider a debugger active until it's running because it may fail during it's starting phase.
Also, this check is used determine if we should update a file's breakpoints when one is toggled.

…y warnings

Remco Smits <djsmits12@gmail.com>

…en file when booting zed up

* WIP Show variables in toggable list items

* Use more unique element id so we can group variables by name and reference

* Fix correct pass in listItem toggle value

* Fix we did not set the current stack frame id

* WIP start fetching variables recursively

* Remove duplicated code for current stack frame id

* Fetch all variables for each stack frame parallel

* Fetch all variable on the same level parallel

* Rename vars so its more clear

* Refactor how we store thread data

We now store the information in a better way so we could filter on it easier and faster.

* Remove unused code

* Use stack frame id as top level key instead of struct itself

* Add has_children to scope thread entry

* Fix allow switching current stack frame

Also fixed building list entries twice right after each other

* Show message when scope does not have variables

* Remove scope if it does not have variables

* Add allow collapsing scopes

* Add allow collapsing variables

* Add docs to determine collapsed variables

* Allow collapsing variables and always show first scope variables

* Correctly fix collapse variables

* Fix clippy

Oopss, after testing i did commit the wrong value.

* Only terminate thread if it did not already end

* Remove debug client when its terminated

Co-authored-by: Remco Smits <djsmits12@gmail.com>

Co-authored-by: Remco Smits <djsmits12@gmail.com>

* Lazy fetch

* Remove unused code

* Clean up fetch nested variable

* Include scope id for variable id to be more unique

* Clean up not needed get_mut function calls

Co-authored-by: Remco Smits <djsmits12@gmail.com>

Co-authored-by: Remco Smits <djsmits12@gmail.com>

Make breakpoints persistent across Zed sessions

* Add debug icon to toggle debug panel

* Add setting to wether to show the debug button

* Fix clippy

* Only clear highlights op open tabs

This much better so we don't have to open each path of each stack frame in each thread.

* Don't open the same file twice for clearing thread highlights

Refactored this because its being refactored inside: zed-industries#16744

* Create debugger console

* Get console to output console messages during output events

* Use match expression in handle_output_event

* Move debug console code to it's own file console

This commit also changes the default granularity to line, before this was statement but most editors have line as the default.

* Show warning when session did not stop but exited

* Fix code was not formatted any more

* Fix clippy

* Wip render set variable value editor

* Remove unused subscriptions

* Set current variable value & select all when setting value

* Send set variable request

* Rename tread entry to VariableListEntry

* WIP allow setting variables for nested structures

* Refactor & rename vars on thread state to be only the ids

* Fix we did not correct notify the right context when updating variable list

* Clean open entries when debugger stops

* Use SetExpression if adapter supports it when setting value

* Refetch scope variables after setting value

This commit also reworks how we store scopes & variables

* Remove debug code

* Make Clippy happy

* Rename variable

* Change order for variable id

* Allow cancelling set value using escape key

* Move breakpoint and client code to dap store

This also changes how we sync breakpoints. Now we just update the dap store model instead of having a RWlock.

The goal is to prepare for making inlay hints for debugging work.

* Remove debug code

* Remove unused method

* Fix don't grow the amount tasks for sending all the breakpoints

* Partially implement terminate clients when app quits

* Sync open breakpoints to closed breakpoints when buffer is closed

* Call terminate request also for not already started clients

* Fix missing import

* Remove not needed read_with call

* Start setting up new debug task format

* Set up blueprint for converting from debug task to regular task

* Create debug adapter trait

* Get debug.json schema to json lsp to show users hints

* Start debugger task refactor to enable easier debugger setup

* Get python adapter to work within task.json

co-authored-by: Piotr <piotr@zed.dev>

* Start work on getting Php Debug adapter working

* Make debug adapter trait work with async functions

Fix CICD spell check & clippy warnings

Co-authored-by: Remco Smits <djsmits12@gmail.com>

---------

Co-authored-by: Piotr <piotr@zed.dev>
Co-authored-by: Remco Smits <djsmits12@gmail.com>

This fixes a crash that happened when placing a breakpoint within a multi buffer
where it's excerpt_id != 1.

Co-authored-by: Remco Smits <djsmits12@gmail.com>

This is the first step of merging open/close breakpoints into
one data structure.

Co-authored-by: Remco Smits <djsmits12@gmail.com>

This is done by making breakpoint.position field optional and
adding a cached_position field too. I also added dap_store to
buffer_store and got buffer_store to initialize breakpoints
when a new buffer is opened. Fixing a bug where some breakpoints
wouldn't appear within multi buffers

Co-authored-by: Piotr <piotr@zed.dev>