Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=true, so the read-only nature is covered. The description adds context about using 'fresh or retained' snapshots, which is a behavioral trait beyond the annotation. However, it does not disclose how criteria in the selector are combined (AND/OR), what happens when no snapshot_id is provided, or what the return value looks like. Thus, some value is added but significant gaps remain.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.