Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=true, idempotentHint=true, and destructiveHint=false, so the safety profile is clear. The description adds valuable behavioral context beyond annotations by describing conditional outputs: authenticated:true and empty pending[] when all providers are connected, and connect_url plus per-install URLs when credentials are missing. This gives the agent a model of what to expect, though partial-connection states are not explicitly described.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.