Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
No annotations are provided, so the description carries the full burden. It describes the core behavior ('Say hello') and purpose ('test that the Salt MCP server is working'), which implies a safe, read-only operation. However, it doesn't disclose any behavioral traits like error conditions, response format, or whether it has side effects. For a simple tool with no annotations, this is adequate but minimal.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.