feat: support vpn mcp server - #206
Conversation
|
|
b35262a to
96d95d9
Compare
cuericlee
left a comment
cuericlee
left a comment
There was a problem hiding this comment.
Ask one more contributor or reviewer from VPN team to have a peer review.
|
|
||
| --- | ||
|
|
||
| ### Tool 1: describe_vpn_gateway_attributes |
There was a problem hiding this comment.
Consider to add create_xxx method to allow agent to create vpn gateway.
There was a problem hiding this comment.
current vpn mcp exposes only Describe* endpoints for read‑only queries.
| readme = "README.md" | ||
| requires-python = ">=3.10" | ||
| dependencies = [ | ||
| "mcp>=1.9.0", |
| parser.add_argument( | ||
| "--transport", | ||
| "-t", | ||
| choices=["stdio", "sse"], |
There was a problem hiding this comment.
add choice "streamable-http", using streamable-http instead of sse in the future.
| return vpn_client.describe_vpn_gateway_attributes(req) | ||
|
|
||
|
|
||
| @mcp.tool(description="调用DescribeVpnGatewayRouteAttributes查询指定VPN网关路由条目的信息") |
There was a problem hiding this comment.
avoid to use Chinese in the code, and describe more description for each arguments and output.
There was a problem hiding this comment.
solved, comments change to en, modelcontextprotocol/python‑sdk examples show description field is a concise summary of what the tool does, no need to include arguments and output.
f2bdbf4 to
593b351
Compare
|
/lgtm |
|
add uv.lock to ensure reproducible, consistent Python environments . |
5 participants
No description provided.