Skip to content

fix: continue execution when initial resume wait times out - #161

Open
andreitava-uip wants to merge 3 commits into
mainfrom
fix/debug-resume-timeout-continue
Open

fix: continue execution when initial resume wait times out#161
andreitava-uip wants to merge 3 commits into
mainfrom
fix/debug-resume-timeout-continue

Conversation

@andreitava-uip

@andreitava-uip andreitava-uip commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Problem

Intermittent job failures on resume. A debug run starts paused, waiting up to 60s for a resume command from the debug bridge. When the SignalR client disconnects and never sends one, UiPathDebugRuntime._stream_and_debug yielded a FAULTED result and returned, failing the job:

[Info ] Waiting for resume command...
[Warn ] Initial resume wait timed out after 60s, assuming debug bridge disconnected
[Info ] Execution completed: UiPathRuntimeStatus.FAULTED

Fix

On the initial resume-wait timeout, log the warning and fall through into the normal streaming loop instead of faulting — the run proceeds unattended with the options it was launched with. The explicit-quit path (UiPathDebugQuitError) is unchanged.

Testing

Added test_debug_runtime_continues_when_initial_resume_wait_times_out; the timeout is simulated by having wait_for_resume raise asyncio.TimeoutError, which asyncio.wait_for propagates identically. Full suite passes.

Copilot AI lite review requested due to automatic review settings August 19, 2026 11:52
@andreitava-uip
andreitava-uip requested review from a team and cristipufu as code owners August 19, 2026 11:52

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR changes debug runtime behavior so that if the initial “wait for resume” times out, execution continues (instead of faulting), and adds a regression test for that scenario.

Changes:

  • Update UiPathDebugRuntime._stream_and_debug to log a warning and proceed when the initial resume wait times out.
  • Add an async test verifying a timeout on the initial resume wait still results in successful execution and emits completion.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
tests/test_debugger.py Adds coverage for continuing execution when the initial resume wait times out.
src/uipath/runtime/debug/runtime.py Changes timeout handling to proceed rather than yield a faulted result and exit.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/uipath/runtime/debug/runtime.py
Comment thread src/uipath/runtime/debug/runtime.py Outdated
Comment thread src/uipath/runtime/debug/runtime.py Outdated
@andreitava-uip andreitava-uip changed the title fix(debug): continue execution when initial resume wait times out fix: continue execution when initial resume wait times out Aug 19, 2026
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants