Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
readOnlyHint=true already tells the agent this is a safe read. The description adds the useful detail that the result is a series of 'ticks' containing equity, totalPnl and netTransfers, which is genuine context beyond the annotation. It is silent on pagination, ordering, and default range behavior, so it stops at adequate.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.