Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=true, destructiveHint=false and openWorldHint=false, and the description's 'Read-only' simply restates that. The list of included sub-resources (prices, variants, stock, tax, file settings) adds some scope context, but nothing about permissions, error behavior, or data staleness is disclosed, which is acceptable given the lower bar set by annotations.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.