Skip to content

Python: Autodetect version issue (26.2) [STUD-78782]#511

Open
viogroza wants to merge 3 commits intodevelopfrom
fix/STUD-78782_autodetect
Open

Python: Autodetect version issue (26.2) [STUD-78782]#511
viogroza wants to merge 3 commits intodevelopfrom
fix/STUD-78782_autodetect

Conversation

@viogroza
Copy link
Collaborator

Search for python and python3 when detecting python
Search in the install folder and in ../bin folder
Remove no longer needed preprocessor directive

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Python runtime autodetection to be more flexible across installations and simplifies OS detection logic by removing now-unneeded preprocessor guards.

Changes:

  • Extend Python autodetection to probe multiple executable names (python/python3) and locations (<path> and <path>/bin).
  • Refactor autodetection to try multiple candidates and aggregate failures for easier troubleshooting.
  • Remove #if NETCOREAPP guards around RuntimeInformation.IsOSPlatform(...) checks.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
Activities/Shared/UiPath.Shared.Service/Client/Controller.cs Removes NETCOREAPP conditional around Windows detection.
Activities/Python/UiPath.Python/Service/PythonProxy.cs Removes NETCOREAPP conditional around Windows detection.
Activities/Python/UiPath.Python/Impl/Engine.cs Removes NETCOREAPP conditional around Windows detection.
Activities/Python/UiPath.Python/EngineProvider.cs Implements multi-candidate Python executable probing and aggregated failure handling.
Activities/Python/UiPath.Python.Host.Shared/PythonService.cs Removes NETCOREAPP conditional around Windows detection.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AlexMarinescuUiPath
Copy link
Contributor

branch name does not contain the package name
my suggestion: fix/python/python-scope-search-for-framework-in-path

@viogroza
Copy link
Collaborator Author

viogroza commented Feb 18, 2026

address part of the review comments in this file
code_review_fix_STUD-78782_autodetect.txt

  1. Resource leak
  2. Potential read of the process output before the process exit
  3. Typo

@viogroza
Copy link
Collaborator Author

viogroza commented Feb 18, 2026

branch name does not contain the package name my suggestion: fix/python/python-scope-search-for-framework-in-path

leave as it is for now (changing the name will require new PR)
@AlexMarinescuUiPath

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

viogroza added 3 commits March 3, 2026 09:58
Search for python and python3 when detecting python
Search in the install folder and in ../bin folder
Remove no longer needed preprocessor directive
@viogroza viogroza force-pushed the fix/STUD-78782_autodetect branch from 3da41d2 to cc7c8fa Compare March 3, 2026 08:10
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants