Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already indicate this is a non-readonly, idempotent mutation, so the bar is lower. The description adds the behavioral note that the thread must already exist and that the IDs must come from add_comment, and warns against commandId. However, it does not disclose what 'resolving' does to the thread state or that commandId is still required for idempotency, which is a notable gap.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.