Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=true and openWorldHint=true, so safety is covered. The description adds genuinely useful semantic context — the data is realized (not unrealized) P&L and is bounded by 'tracked history' — but says nothing about latency, rate limits, or how the two granularities are combined in the response.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.