Fix auth prompt and buffer early stopped events

Two UX issues:

1. Auth: switch from newSession() (which always forces a new login
   prompt) to getSession() which silently reuses the existing session.

2. Variables not populated on connect: the runner sends a 'stopped'
   event immediately on websocket connect, before VS Code has finished
   the DAP initialization handshake (configurationDone). VS Code drops
   stopped events that arrive before configurationDone. The adapter now
   buffers any stopped events received before the client sends
   configurationDone and replays them once the handshake completes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: rentziass

src/debugger/debugger.ts

@@ -1,5 +1,5 @@
 import * as vscode from "vscode";
-import {newSession} from "../auth/auth";
+import {getSession} from "../auth/auth";
 import {log, logError} from "../log";
 import {validateTunnelUrl} from "./tunnelUrl";
 import {WebSocketDapAdapter} from "./webSocketDapAdapter";
@@ -56,11 +56,13 @@ async function connectToDebugger(): Promise<void> {
 
   // 2. Acquire a GitHub auth session. The token is used as a Bearer token
   //    against the Dev Tunnel relay, which accepts VS Code's GitHub app tokens.
-  let session: vscode.AuthenticationSession;
-  try {
-    session = await newSession("Sign in to GitHub to connect to the Actions job debugger.");
-  } catch (e) {
-    void vscode.window.showErrorMessage(`GitHub authentication required: ${(e as Error).message}`);
+  //    getSession() silently reuses the existing session; it only prompts if
+  //    the user has never signed in.
+  const session = await getSession();
+  if (!session) {
+    void vscode.window.showErrorMessage(
+      "GitHub authentication is required to connect to the Actions job debugger. Please sign in and try again."
+    );
     return;
   }
 

src/debugger/webSocketDapAdapter.ts

@@ -24,6 +24,16 @@ export class WebSocketDapAdapter implements vscode.DebugAdapter {
   private _pingTimer: ReturnType<typeof setInterval> | undefined;
   private _disposed = false;
 
+  /**
+   * Whether VS Code has completed the DAP initialization handshake. The
+   * runner sends a `stopped` event immediately on connect (before the client
+   * sends `configurationDone`), and VS Code ignores `stopped` events that
+   * arrive before configuration is done. We buffer early `stopped` events
+   * and replay them once the handshake completes.
+   */
+  private _configurationDone = false;
+  private _pendingStoppedEvents: vscode.DebugProtocolMessage[] = [];
+
   constructor(
     private readonly _tunnelUrl: string,
     private readonly _token: string
@@ -92,6 +102,20 @@ export class WebSocketDapAdapter implements vscode.DebugAdapter {
     const json = JSON.stringify(message);
     logDebug(`→ DAP: ${describeDapMessage(message)}`);
     this._ws.send(json);
+
+    // Detect configurationDone and replay any stopped events the runner
+    // sent before VS Code was ready to receive them.
+    const m = message as Record<string, unknown>;
+    if (m.command === "configurationDone" && m.type === "request") {
+      this._configurationDone = true;
+      if (this._pendingStoppedEvents.length > 0) {
+        logDebug(`Replaying ${this._pendingStoppedEvents.length} buffered stopped event(s)`);
+        for (const evt of this._pendingStoppedEvents) {
+          this._onDidSendMessage.fire(evt);
+        }
+        this._pendingStoppedEvents = [];
+      }
+    }
   }
 
   dispose(): void {
@@ -131,6 +155,17 @@ export class WebSocketDapAdapter implements vscode.DebugAdapter {
       }
 
       logDebug(`← DAP: ${describeDapMessage(message)}`);
+
+      // Buffer stopped events that arrive before configurationDone — the
+      // runner re-sends the stopped event on connect (before the client
+      // finishes the DAP handshake) and VS Code silently drops them.
+      const m = message as Record<string, unknown>;
+      if (m.type === "event" && m.event === "stopped" && !this._configurationDone) {
+        logDebug("Buffering stopped event (configurationDone not yet sent)");
+        this._pendingStoppedEvents.push(message);
+        return;
+      }
+
       this._onDidSendMessage.fire(message);
     });