fix(mcp): only wait for next pause in browser_resume when step or location is set#41293
fix(mcp): only wait for next pause in browser_resume when step or location is set#41293SebTardif wants to merge 1 commit into
Conversation
…ation is set browser_resume unconditionally awaited a pausedPromise that only resolved when the debugger paused again. For a plain resume (no step or location), if execution completed without hitting another breakpoint, the promise never resolved and the tool hung indefinitely. Only create and await the pausedPromise when params.step or params.location is set, since those are the only cases where waiting for re-pause is expected. Introduced in microsoft#39717 (2026-03-17).
|
@SebTardif Could you please file an issue first? I wonder what's the usecase for this? Currently, calling |
|
Filed #41304. The use case is an MCP agent that pauses a script (via The fix makes the wait conditional: when |
Summary
browser_resumeunconditionally awaited a promise that only resolved when the debugger paused again. For a plain resume (no step or location), if execution completed without another breakpoint, the tool hung indefinitely.steporlocationis set.Introduced in #39717