Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnly, idempotent, non-destructive and closed-world, so safety is covered. The description adds real behavioral value by listing what the converter handles (attributes, nested elements, arrays, CDATA, namespaces) and the default-on value parsing, which materially changes output shape. It does not say what happens with malformed XML, keeping it at 4.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.