Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=false, destructiveHint=false and openWorldHint=false, so safety is covered. The description adds the configuration prerequisite but says nothing about whether adding a duplicate product merges line items, how quantity behaves on repeat adds, or what errors result from missing option ids. With annotations doing the safety lift, this is adequate but thin.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.