Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations (readOnlyHint=false, idempotentHint=false) already signal a mutating, non-idepotent operation, but the description adds no behavioral context beyond that. It does not disclose whether re-adding an already-favored collection fails or silently succeeds, what happens if the collection does not exist, or whether the favorite is per-user. No contradiction with annotations, but no added value either.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.