feat(mcp): add Secure Parameters support for mcp v2026-06#3394
feat(mcp): add Secure Parameters support for mcp v2026-06#3394duwenxin99 wants to merge 6 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces support for secure parameters in the Model Context Protocol (MCP) server. It updates tool manifest generation to separate standard and secure parameters, filters out tools with secure parameters if the client does not support them, and validates and merges standard and secure arguments during tool execution. The review feedback highlights an issue where authentication parameters for secure parameters are discarded instead of being merged, suggests defensive nil checks to prevent potential nil pointer dereferences when accessing parameters, and recommends returning INVALID_PARAMS instead of METHOD_NOT_FOUND when a client lacks the secure-params capability to comply with the JSON-RPC specification.
No description provided.