-
-
Notifications
You must be signed in to change notification settings - Fork 2k
Open
Description
I am using metamcp´s mcp inspector to try and figure out, how this mcp server works. The mcp inspector marks required parameters with a small star. When i try to run the list_workflows tool with the inspector, i see these parameters:
My problem here is, that from my testing, the "limit", "cursor" and the "projectId" parameters are all three required. I get these issues:
For empty limit:
{
"limit": 0,
"cursor": "",
"active": false,
"tags": [],
"projectId": "",
"excludePinnedData": false
}
{
"content": [
{
"type": "text",
"text": "{\n \"success\": false,\n \"error\": \"Invalid input\",\n \"details\": {\n \"errors\": [\n {\n \"code\": \"too_small\",\n \"minimum\": 1,\n \"type\": \"number\",\n \"inclusive\": true,\n \"exact\": false,\n \"message\": \"Number must be greater than or equal to 1\",\n \"path\": [\n \"limit\"\n ]\n }\n ]\n }\n}"
}
]
}
For empty cursor:
{
"limit": 1,
"cursor": "",
"active": false,
"tags": [],
"projectId": "",
"excludePinnedData": false
}
{
"content": [
{
"type": "text",
"text": "{\n \"success\": false,\n \"error\": \"Invalid request: Empty value found for query parameter 'cursor'\",\n \"code\": \"VALIDATION_ERROR\"\n}"
}
]
}
For empty projectId:
{
"limit": 1,
"cursor": "1",
"active": false,
"tags": [],
"projectId": "",
"excludePinnedData": false
}
{
"content": [
{
"type": "text",
"text": "{\n \"success\": false,\n \"error\": \"Invalid request: Empty value found for query parameter 'projectId'\",\n \"code\": \"VALIDATION_ERROR\"\n}"
}
]
}
In my opinion, the limit and cursor parameters might make sense, but projectId is useless here. I want to list all workflows, not filter for any. I am not 100% sure if this is a problem with the mcp inspector, but so far it worked with every mcp server i threw at it.
Metadata
Metadata
Assignees
Labels
No labels