Skip to content

feat: add MCP input gate for UiPath Sites plugin#561

Draft
ninja-shreyash wants to merge 2 commits into
mainfrom
codex/uipath-sites-mcp-ui
Draft

feat: add MCP input gate for UiPath Sites plugin#561
ninja-shreyash wants to merge 2 commits into
mainfrom
codex/uipath-sites-mcp-ui

Conversation

@ninja-shreyash

Copy link
Copy Markdown
Contributor

No description provided.

@ninja-shreyash ninja-shreyash requested a review from a team June 30, 2026 07:45
@ninja-shreyash ninja-shreyash marked this pull request as draft June 30, 2026 07:45
);

result(id, {
protocolVersion: params && params.protocolVersion ? params.protocolVersion : "2025-06-18",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The server echoes back whatever protocolVersion the client sent rather than advertising its own supported version. Per the MCP spec:

If the server does not support the client's protocol version, it SHOULD respond with its own highest supported version.

If a client sends a newer version (e.g. "2025-11-05"), this response will falsely claim the server supports it, which can mislead clients into using features the server does not implement.

Suggested change
protocolVersion: params && params.protocolVersion ? params.protocolVersion : "2025-06-18",
protocolVersion: "2025-06-18",

@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant