Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=false and destructiveHint=true, so the mutation profile is covered. The description usefully adds that the *current selection is replaced* (disclosing what gets destroyed), which is real value beyond the annotations. However, the presence of a 'revision' parameter hints at concurrency/versioning behavior that goes entirely unexplained.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.