Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=true, so the safety profile is covered. The description adds useful return-shape context (lines, supplier, totals), but says nothing about error behavior for an invalid/unknown Guid, auth requirements, or rate limits, so it does not go far beyond the annotation baseline.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.