Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?
Annotations already declare the safety profile (readOnlyHint=false, destructiveHint=false, openWorldHint=false), so the bar is lower, and the description clears it by disclosing what actually gets written (connection env vars) and the operational consequence that a redeploy is required. It stops short of saying whether existing env vars are overwritten or whether detach reverses it, so it is not fully complete.
Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.