Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare destructiveHint=true, readOnlyHint=false, and idempotentHint=false, and 'restores ... to its previous state' is consistent with that mutation profile. The description adds a bit of context about the undo semantics but does not disclose the key behavioral traits an agent would want, such as whether the undoId is single-use/consumed, what exactly is overwritten, or whether the restore is reversible.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.