Skip to content

fix: subprocess encoding on windows#1386

Merged
cristipufu merged 1 commit intomainfrom
fix/subprocess_windows_encode
Feb 27, 2026
Merged

fix: subprocess encoding on windows#1386
cristipufu merged 1 commit intomainfrom
fix/subprocess_windows_encode

Conversation

@cristipufu
Copy link
Member

@cristipufu cristipufu commented Feb 27, 2026

Description

This PR fixes subprocess encoding issues on Windows by improving how stdout/stderr encoding is configured. The fix addresses a problem where parent processes couldn't properly decode output when the CLI was invoked as a subprocess with piped output.

@cristipufu cristipufu requested a review from Copilot February 27, 2026 05:59
@cristipufu cristipufu self-assigned this Feb 27, 2026
@github-actions github-actions bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository labels Feb 27, 2026
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

This PR fixes subprocess encoding issues on Windows by improving how stdout/stderr encoding is configured. The fix addresses a problem where parent processes couldn't properly decode output when the CLI was invoked as a subprocess with piped output.

Changes:

  • Version bumped from 2.10.0 to 2.10.1 (patch release)
  • Modified Windows encoding initialization to differentiate between TTY and piped output scenarios
  • TTY output uses UTF-8 encoding; piped output keeps system encoding with error replacement

Reviewed changes

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

File Description
uv.lock Version bump to 2.10.1
pyproject.toml Version bump to 2.10.1
src/uipath/_cli/init.py Conditional encoding reconfiguration based on TTY detection

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

@cristipufu cristipufu merged commit c522b2e into main Feb 27, 2026
57 checks passed
@cristipufu cristipufu deleted the fix/subprocess_windows_encode branch February 27, 2026 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants