Enhancement: [no-unused-constructor-member] Should report unused members in constructor

[muuvmuuv]

Before You File a Proposal Please Confirm You Have Done The Following...

• I have searched for related issues and found none that match my proposal.
• I have searched the current rule list and found no rules that match my proposal.
• I have read the FAQ and my problem is not listed.

My proposal is suitable for this project

• I believe my proposal would be useful to the broader TypeScript community (meaning it is not a niche proposal).

Link to the rule's documentation

There is none yet?

Description

Should report a warning when a private member in a constructor is unused.

Fail

class Test {
  constructor(private foo: Foo) {}
}

Pass

class Test {
  constructor(private foo: Foo) {}

  test() {
    return this.foo;
  }
}

Additional Info

Maybe as well for non-private, not declared as public.

[JoshuaKGoldberg]

Thanks for posting! This is a duplicate of #4571. Before filing an issue, please use our issue search to check for open and closed issues that already address what you're looking for.

[muuvmuuv]

Oh ok, I thought this issue is just for private class members, my proposal is for constructor members/definitions.

[JoshuaKGoldberg]

Ah I can see why you'd suggest a new issue. Constructor parameter properties with the private modifier are private class members though.

[muuvmuuv]

Ohh, sorry, didn't know, thought those are handled different. 👍🏻 hope both are addressed the same in that issue, really think this would lets us find a lot of unused props in our codebase.