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 usefully adds what is retrieved (rates, stock settings, custom fields), but it discloses nothing further about auth needs, error behavior when the id is absent, or how the optional include affects the result.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.