Renaming a struct field should rename variables that destructure the field

The title is not as clear as it could be - here's an example:

        let Item { attrs, name, source, visibility, def_id, inner, stability, deprecation } = item;

When I rename inner to kind, I expect this to be changed to

        let Item { attrs, name, source, visibility, def_id, kind, stability, deprecation } = item;

and all usages of inner in this scope updated. Instead, rust-analyzer changes it to

        let Item { attrs, name, source, visibility, def_id, kind: inner, stability, deprecation } = item;

which is not super helpful and messes up rustfmt.

Reproduce the rename using the Rust destructuring example in the issue and inspect rust-analyzer's rename behavior for the field and its local uses. The work is done when renaming inner to kind produces a shorthand kind binding, updates usages in scope, and no longer leaves the kind: inner form that upsets rustfmt.