Skip to content

Conversation

@cristipufu
Copy link
Member

@cristipufu cristipufu commented Jul 3, 2025

Summary

This PR adds the tool input schema to the registration payload and refines HTTP error handling during registration.

  • Include InputSchema in the tool_info payload
  • Update exception handling to log details for HTTP errors

@cristipufu cristipufu requested review from Copilot and edis-uipath July 3, 2025 11:00
@cristipufu cristipufu self-assigned this Jul 3, 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 adds the tool input schema to the registration payload and refines HTTP error handling during registration.

  • Include InputSchema in the tool_info payload
  • Remove debug logging of client_info
  • Update exception handling to log details for HTTP errors
Comments suppressed due to low confidence (2)

src/uipath_mcp/_cli/_runtime/_runtime.py:399

  • Double-encoding the schema as a JSON string may complicate downstream parsing. Consider passing the raw object (InputSchema: tool.inputSchema or {}) so it serializes as JSON only once.
                    else "{}",

src/uipath_mcp/_cli/_runtime/_runtime.py:399

  • Add unit tests to verify the registration payload includes InputSchema when defined and defaults to an empty object when missing.
                    else "{}",

@cristipufu cristipufu force-pushed the feat/save_tools_input_schema branch from f45b390 to 53b8f7f Compare July 3, 2025 11:36
@cristipufu cristipufu force-pushed the feat/save_tools_input_schema branch from dd59d66 to f5ccc3e Compare July 4, 2025 13:30
@cristipufu cristipufu merged commit c6c859d into main Jul 4, 2025
6 checks passed
@cristipufu cristipufu deleted the feat/save_tools_input_schema branch July 4, 2025 13:33
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