fix: handle debug signalR bridge race condition (#1590) [2.10.37.post1] - #1887
Merged
andreiancuta-uipath merged 5 commits intoSep 8, 2026
Merged
Conversation
github-actions
Bot
added
test:uipath-langchain
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The new SignalR start-race behavior is not test-covered and the added inline comment should be corrected for clarity.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR is a cherry-pick hotfix onto the uipath 2.10.37 release line to address a remote-debugging (SignalR) race condition, bump the patch version to 2.10.37.post1, and ensure CI/CD workflows run appropriately on release/** branches.
Changes:
- Emit
OnExecutionStartedwhen handling the SignalRStartcommand to reduce the chance of a remote-debug session waiting indefinitely due to connection ordering. - Bump
packages/uipathversion from2.10.37to2.10.37.post1(including lockfile). - Expand CI/CD workflow branch triggers to
release/**and guard CD package detection so it doesn’t run on branch creation pushes.
File summaries
| File | Description |
|---|---|
packages/uipath/src/uipath/_cli/_debug/_bridge.py |
Adds emit_execution_started() call in _handle_start to mitigate a SignalR start/receiver race. |
packages/uipath/pyproject.toml |
Updates the uipath package version to 2.10.37.post1. |
packages/uipath/uv.lock |
Aligns locked editable uipath version with 2.10.37.post1. |
.github/workflows/ci.yml |
Enables CI to run for PRs targeting release/** branches. |
.github/workflows/cd.yml |
Enables CD on release/** pushes and avoids branch-creation publish detection via output fallbacks + github.event.created guard. |
Review details
- Files reviewed: 4/5 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+744
to
+745
| # handle race conditions, runtime connected to debug bridge before the receiver | ||
| await self.emit_execution_started() |
Comment on lines
+744
to
+745
| # handle race conditions, runtime connected to debug bridge before the receiver | ||
| await self.emit_execution_started() |
andreitava-uip
approved these changes
Sep 8, 2026
andreiancuta-uipath
force-pushed
the
fix/debug-bridge-race-2.10.37
branch
from
September 8, 2026 14:16
38cc295 to
01067fb
Compare
|
andreiancuta-uipath
merged commit Sep 8, 2026
b3ef728
into
release/v2.10.37.post1
43 of 55 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cherry-pick of #1590 onto uipath 2.10.37, released as
2.10.37.post1._bridge.py: emitexecution_startedfrom_handle_startso a runtime that connects before the receiver is not left waiting (three lines from 64399bf).2.10.37->2.10.37.post1in pyproject.toml and uv.lock.release/**to the CD push trigger and CI pull_request trigger, with the skip-on-branch-creation guard (ci: run checks and publish on release branches #1879, ci: do not run cd when a branch is created #1881).sonar-project.properties(from ci: add coverage report on PRs #1618, without coverage upload) because therelease/*ruleset requires the "SonarCloud Code Analysis" check, which did not exist in CI at this commit.The
test-uipath-langchain/test-uipath-llamaindexcross-tests are label-triggered and expected to fail here: they resolve current uipath-langchain main (uipath>=2.14.6) against the 2.10.37 wheels. They are not required checks.Pairs with uipath-runtime 0.10.0.post1 (cherry-pick of uipath-runtime-python#161), already on PyPI.
🤖 Generated with Claude Code