Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=true and idempotentHint=true, so the safe read behavior is covered. The description adds value by explaining the semantics of the returned fields (url, repo, branch, root_directory) and their null cases, as well as the context-flooding concern behind the default limit. This goes beyond the annotations without contradicting them.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.