Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare the safety profile (readOnly=false, destructive=false), and the description adds beyond that: it explains why the tool is non-OAuth, that secrets are validated and never echoed back, and that existing config is replaced. One mild tension exists between 'store or replace' (upsert-flavored) and idempotentHint=false, but it is not a clear safety-profile contradiction.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.