Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnly/idempotent/non-destructive, so the safety bar is met structurally. The description adds genuine context beyond that: the brand budget is conditional ('when it has one') and the intended downstream behavior is to quote cost to the user. It stops short of noting any caching/freshness behavior, hence 4.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.