Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=true and idempotentHint=false, so the safety profile is covered. The description adds useful domain context (trade units = total units and depth of a pool) but says nothing about return shape, size, or why a read operation is flagged non-idempotent. Modest added value against a lower bar.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.