Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=false, destructiveHint=true, and idempotentHint=true, and the description is consistent with all three. It adds real context beyond them: upsert semantics (one secret per key, setting an existing key overwrites its value) and a security trait not encoded anywhere else — values are write-only and never returned by any tool. Auth/permission requirements are the main omission.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.