Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already cover read-only, idempotent and open-world behavior, and the description adds real signal: the "not_required" sentinel value and the conditional authorizeUrl on "not_started"/"pending". It doesn't discuss latency, caching, or error modes for invalid yield IDs, so it's strong but not exhaustive.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.