Calling any MCP tool with a non-existent instanceId override returns a generic 500 error instead of a meaningful response. This happens because the validation layer only checks the format of the ID, not whether it exists. When the lookup fails, the error is swallowed and converted into a generic 500 response
How to reproduce:
Call get_health({ instanceId: 'nonexistent' })
Expected behavior:
Return a clear 404 error indicating the instance does not exist
Actual behavior:
Returns 500 Failed to get health
Scope:
Affects all MCP tools that accept an optional instanceId override
Calling any MCP tool with a non-existent instanceId override returns a generic 500 error instead of a meaningful response. This happens because the validation layer only checks the format of the ID, not whether it exists. When the lookup fails, the error is swallowed and converted into a generic 500 response
How to reproduce:
Call get_health({ instanceId: 'nonexistent' })
Expected behavior:
Return a clear 404 error indicating the instance does not exist
Actual behavior:
Returns 500 Failed to get health
Scope:
Affects all MCP tools that accept an optional instanceId override