perf(watches): remove unnecessary listener

Author: igorlfs

lua/dap-view/listeners.lua

@@ -153,16 +153,6 @@ dap.listeners.after.stackTrace[SUBSCRIPTION_ID] = function(_, err, _, payload)
     end
 end
 
-dap.listeners.after.variables[SUBSCRIPTION_ID] = function()
-    -- When setting a variable for some adapters, the request may be slow.
-    -- And by the time we refresh the view "regularly", the data might not be up-to-date.
-    -- To avoid dealing with such edge cases, we force redrawing after the variables request.
-    -- This introduces an overrhead, but its robustness should be worth it
-    if state.current_section == "watches" then
-        require("dap-view.views").switch_to_view("watches")
-    end
-end
-
 local reeval = { "setExpression", "setVariable" }
 
 for _, listener in ipairs(reeval) do

lua/dap-view/watches/actions.lua

@@ -167,7 +167,9 @@ M.expand_or_collapse = function(line, co)
                 if variable_view then
                     variable_view.expanded = not variable_view.expanded
 
-                    variable_view.children, variable_view.err = eval.expand_variable(reference)
+                    variable_view.children, variable_view.err = eval.expand_variable(reference, nil, co)
+
+                    return true
                 end
             else
                 vim.notify("Nothing to expand")

lua/dap-view/watches/eval.lua

@@ -91,7 +91,8 @@ end
 
 ---@param variables_reference number
 ---@param previous_expansion_result dapview.VariableView[]?
-M.expand_variable = function(variables_reference, previous_expansion_result)
+---@param co thread?
+M.expand_variable = function(variables_reference, previous_expansion_result, co)
     local session = assert(require("dap").session(), "has active session")
 
     local frame_id = session.current_frame and session.current_frame.id
@@ -159,6 +160,10 @@ M.expand_variable = function(variables_reference, previous_expansion_result)
         varible_views[k] = new_variable_view
     end
 
+    if co then
+        coroutine.resume(co)
+    end
+
     return #varible_views > 0 and varible_views or nil, err
 end