Send stopDebugger notification when appropriate

Author: andyleejordan

src/PowerShellEditorServices/Services/PowerShellContext/PowerShellContextService.cs

@@ -162,8 +162,18 @@ public RunspaceDetails CurrentRunspace
         /// </summary>
         public string InitialWorkingDirectory { get; private set; }
 
+        /// <summary>
+        /// Tracks the state of the LSP debug server (not the PowerShell debugger).
+        /// </summary>
         internal bool IsDebugServerActive { get; set; }
 
+        /// <summary>
+        /// Tracks if the PowerShell session started the debug server itself (true), or if it was
+        /// started by an LSP notification (false). Essentially, this marks if we're responsible for
+        /// stopping the debug server (and thus need to send a notification to do so).
+        /// </summary>
+        internal bool OwnsDebugServerState { get; set; }
+
         internal DebuggerStopEventArgs CurrentDebuggerStopEventArgs { get; private set; }
 
         #endregion
@@ -777,6 +787,16 @@ public async Task<IEnumerable<TResult>> ExecuteCommandAsync<TResult>(
                         await this.sessionStateLock.ReleaseForExecuteCommand().ConfigureAwait(false);
                     }
 
+                    // This is the edge case where the debug server is running because it was
+                    // started by PowerShell (and not by an LSP event), and we're no longer in the
+                    // debugger within PowerShell, so since we own the state we need to stop the
+                    // debug server too.
+                    if (IsDebugServerActive && OwnsDebugServerState && !shell.IsNested)
+                    {
+                        logger.LogDebug("Stopping LSP debugger because PowerShell debugger stopped!");
+                        _languageServer?.SendNotification("powerShell/stopDebugger");
+                    }
+
                     if (shell.HadErrors)
                     {
                         var strBld = new StringBuilder(1024);
@@ -2422,9 +2442,14 @@ private void OnDebuggerStop(object sender, DebuggerStopEventArgs e)
             // when the DebugServer is fully started.
             CurrentDebuggerStopEventArgs = e;
 
+            // If this event has fired but the LSP debug server is not active, it means that the
+            // PowerShell debugger has started some other way (most likely an existing PSBreakPoint
+            // was executed). So not only do we have to start the server, but later we will be
+            // responsible for stopping it too.
             if (!IsDebugServerActive)
             {
                 _languageServer?.SendNotification("powerShell/startDebugger");
+                OwnsDebugServerState = true;
             }
 
             // We've hit a breakpoint so go to a new line so that the prompt can be rendered.