test(serverReady): assert breakpoint action runs

Fix serverReady breakpoint handling during startup hops.

Author: dkattan

src/session.ts

@@ -2899,6 +2899,45 @@ export async function startDebuggingAndWaitForStop(params: StartDebuggingAndWait
           )
           && hops < maxHops
         ) {
+          // Special-case: serverReady breakpoint stop (after entry).
+          // If we don't process it here, we'll keep re-stopping on the same breakpoint.
+          if (
+            serverReady
+            && serverReadySource
+            && typeof nextStop.line === "number"
+            && nextStop.line === serverReady.trigger?.line
+          ) {
+            logger.info(
+              `Hit serverReady breakpoint during startup hops at line ${nextStop.line}; executing action then continuing to user breakpoint.`,
+            );
+            if (!serverReadyTriggerSummary) {
+              serverReadyTriggerSummary = serverReady.trigger?.path
+                ? `Breakpoint ${serverReady.trigger.path}:${serverReady.trigger.line}`
+                : "serverReady breakpoint hit";
+            }
+            await executeServerReadyAction("late");
+            // Remove serverReady breakpoint to avoid re-trigger.
+            vscode.debug.removeBreakpoints([serverReadySource]);
+            serverReadySource = undefined;
+            try {
+              await nextStop.session.customRequest("continue", {
+                threadId: nextStop.threadId,
+              });
+            }
+            catch (e) {
+              const msg = e instanceof Error ? e.message : String(e);
+              throw new Error(
+                `Failed to continue after serverReady breakpoint (DAP 'continue'): ${msg}`,
+              );
+            }
+            const elapsed = Date.now() - loopStart;
+            const remaining = Math.max(0, remainingMs - elapsed);
+            nextStop = await waitForDebuggerStopBySessionId({
+              sessionId,
+              timeout: remaining,
+            });
+            continue;
+          }
           hops++;
           const elapsed = Date.now() - loopStart;
           const remaining = Math.max(0, remainingMs - elapsed);

src/test/serverReady.test.ts

@@ -75,6 +75,29 @@ describe("serverReady breakpoint", function () {
       },
     });
 
+    assert.strictEqual(
+      context.serverReadyInfo.configured,
+      true,
+      "serverReady should be reported as configured",
+    );
+    assert.strictEqual(
+      context.serverReadyInfo.triggerMode,
+      "breakpoint",
+      "serverReady trigger mode should be breakpoint",
+    );
+    assert.ok(
+      context.serverReadyInfo.phases.some(
+        p => p.phase === "entry" || p.phase === "late",
+      ),
+      `Expected serverReady action to execute for breakpoint trigger, but phases were: ${JSON.stringify(
+        context.serverReadyInfo.phases,
+      )}`,
+    );
+    assert.ok(
+      context.serverReadyInfo.triggerSummary?.includes("Breakpoint"),
+      `Expected serverReady trigger summary to mention Breakpoint, but was: ${context.serverReadyInfo.triggerSummary}`,
+    );
+
     // Assert we stopped at the user breakpoint (not serverReady line)
     assert.strictEqual(
       context.frame.line,
@@ -145,6 +168,29 @@ describe("serverReady breakpoint", function () {
         action: { type: "httpRequest", url: "http://localhost:31337/health" },
       },
     });
+
+    assert.strictEqual(
+      context.serverReadyInfo.configured,
+      true,
+      "serverReady should be reported as configured (httpRequest)",
+    );
+    assert.strictEqual(
+      context.serverReadyInfo.triggerMode,
+      "breakpoint",
+      "serverReady trigger mode should be breakpoint (httpRequest)",
+    );
+    assert.ok(
+      context.serverReadyInfo.phases.some(
+        p => p.phase === "entry" || p.phase === "late",
+      ),
+      `Expected serverReady action to execute for breakpoint trigger (httpRequest), but phases were: ${JSON.stringify(
+        context.serverReadyInfo.phases,
+      )}`,
+    );
+    assert.ok(
+      context.serverReadyInfo.triggerSummary?.includes("Breakpoint"),
+      `Expected serverReady trigger summary to mention Breakpoint (httpRequest), but was: ${context.serverReadyInfo.triggerSummary}`,
+    );
     assert.strictEqual(
       context.frame.line,
       userBreakpointLine,