Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare non-read-only, non-destructive, non-idempotent, closed-world, and the description reinforces this consistently by disclosing PATCH semantics ('Omitted fields stay unchanged'), the optimistic-concurrency requirement (expectedRevision), the frozen-theme lock, and the fact that published pages are untouched. What it omits is failure behavior on a stale revision and any auth/capability caveat, so it is strong but not exhaustive.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.