Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare readOnlyHint=false, destructiveHint=false, idempotentHint=false, so the safety profile is covered. The description adds genuinely non-obvious behavior: expectedSpecVersion gates against stale writes, i.e. optimistic concurrency control, which is not derivable from the schema. It does not explain what happens when only one of light/dark is supplied or whether omission clears a basemap.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.