Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint, idempotentHint, non-destructive and closed-world, so the safety profile is covered and this is a pure computation with no side effects to disclose. The description adds the return summary ('final price and effective single-discount equivalent'), which is useful context even though an output schema exists. One caveat it does not disclose is that stacking is capped at two discounts, which matters for a tool pitched as handling 'multiple promotions.'
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.