Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
The annotations already declare the safety profile (readOnlyHint=false, destructiveHint=false, idempotentHint=false), so the description does not need to restate them. It adds the useful explicit-request policy, but it says nothing about preconditions (what happens if the memory was never deleted), what is actually restored (versions, links, metadata), or the consequences of the non-idempotent flag.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.