Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=true, idempotentHint=true, destructiveHint=false and openWorldHint=false, so the description's 'Read-only' line adds nothing structured data doesn't cover. It does contribute useful behavioral context about the payload — that each item carries a deal rationale, price and stock status — which goes beyond the annotations.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.