Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare destructiveHint=true, readOnlyHint=false, and non-idempotent behaviour, so the safety profile is covered elsewhere. The description adds internal routing context (the server selects the upscale path), but it never explains what the destructive hint actually means here, e.g., whether the original image is replaced or a new asset is created, nor the cost or latency.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.