Skip to content

Conversation

@cristipufu
Copy link
Member

@cristipufu cristipufu commented Jul 4, 2025

Pull Request Overview

This PR enhances the session runtime to better handle invalid stdout messages, tracks the last message id for error reporting, and bumps the project version.

  • Adds handling and logging for exceptions returned from the stdout reader (server print statements are sometimes logged as exceptions)
  • Introduces self._last_message_id tracking and uses it when sending error responses
  • Updates project version from 0.0.94 to 0.0.95

@cristipufu cristipufu requested review from Copilot and edis-uipath July 4, 2025 14:33
@cristipufu cristipufu self-assigned this Jul 4, 2025
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 enhances the session runtime to better handle invalid stdout messages, tracks the last message ID for error reporting, and bumps the project version.

  • Adds handling and logging for exceptions returned from the stdout reader
  • Introduces self._last_message_id tracking and uses it when sending error responses
  • Updates project version from 0.0.94 to 0.0.95

Reviewed Changes

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

File Description
src/uipath_mcp/_cli/_runtime/_session.py Handle Exception from receive(), track last_message_id, and reformat error responses
pyproject.toml Incremented version to 0.0.95
Comments suppressed due to low confidence (1)

src/uipath_mcp/_cli/_runtime/_session.py:132

  • [nitpick] New branch for handling Exception from receive() should have accompanying tests to verify logging and continuation behavior.
                                if isinstance(session_message, Exception):

@cristipufu cristipufu force-pushed the fix/better_error_handling branch from 72686f6 to 0d19491 Compare July 4, 2025 14:36
@cristipufu cristipufu merged commit 4f3160f into main Jul 4, 2025
6 checks passed
@cristipufu cristipufu deleted the fix/better_error_handling branch July 8, 2025 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants