Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint, idempotentHint and destructiveHint=false, so the safety profile is covered. The description adds value by naming the margin concept and some output fields (mid_market_amount, amount_after_margin, fees_lost), but it repeats return-value content that a dedicated output schema already carries and says nothing about defaults, rounding, or rate-source expectations.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.