Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=false, idempotentHint=true, destructiveHint=true, and openWorldHint=false, so the safety profile is covered. The description adds a real behavioral fact the annotations lack — the required project_comments:write scope — and the 'existing' qualifier implies the record must already exist, but it never explains what is destroyed, whether content is replaced or merged, or what happens to parent_id if omitted.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.