Add clippy lint: use_destructuring

[emilk]

New lint: [use_destructuring]

Adds a pedantic lint that suggests using destructuring when a function accesses all fields of a struct via the self parameter (e.g. self.x, self.y, self.z), and the struct has 3 or more fields. This helps keep code in sync with struct definitions — the compiler will error on the destructuring if a field is added.

Works for both named structs and tuple structs.

Configuration

Two options in clippy.toml:

Option	Type	Default	Description
use-destructuring-min-fields	u64	3	Minimum number of struct fields for the lint to trigger.
use-destructuring-scope	string	"self"	Which variables to lint. See below.

use-destructuring-scope values:

Value	Description
"self"	Only lint the self parameter (default, most conservative).
"Self"	Lint any variable whose type matches Self in the enclosing impl block.
"crate"	Lint any variable whose type is defined in the current crate.
"*"	Lint all struct types, including external ones.

Suppression

The lint does NOT fire when:

• The struct has fewer fields than the configured minimum (default 3).
• Not all fields are accessed.
• The variable is also used as a whole (passed to a function, returned, etc.).
• Any field is mutated (self.x = 5).
• The struct is #[non_exhaustive].
• The struct is a union or enum.
• The field access is from macro expansion.
• Destructuring would cause a name collision with an existing local variable.

Example

struct Point { x: f32, y: f32, z: f32 }

impl std::fmt::Display for Point {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        // Before:
        write!(f, "({}, {}, {})", self.x, self.y, self.z)

        // After:
        // let Self { x, y, z } = self;
        // write!(f, "({x}, {y}, {z})")
    }
}

---

• Followed lint naming conventions
• Added passing UI tests (including committed .stderr file)
• cargo test passes locally
• Executed cargo dev update_lints
• Added lint documentation
• Run cargo dev fmt

changelog: [use_destructuring]: new pedantic lint suggesting destructuring when all fields of a struct are accessed individually

---

Implemented with a lot of help from Claude Code

[github-actions]

Lintcheck changes for 12e7dab

Lint	Added	Removed	Changed
clippy::use_destructuring	522	0	0

This comment will be updated if you push new changes

[rustbot]

r? @dswij

rustbot has assigned @dswij.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

• Owners of files modified in this PR: 7 candidates
• 7 candidates expanded to 7 candidates
• Random selection from Jarcho, dswij, llogiq, samueltardieu

[dswij]

r? clippy

[rustbot]

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.