Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=false, idempotentHint=true and destructiveHint=false, so "Not read-only" and "is idempotent" restate structured data rather than add to it. The one genuine addition is "re-granting the same level is safe," which spells out what idempotency means operationally, plus the note that sharing permissions change — useful, but thin given the annotations already carry the safety profile.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.