Thanks for the proposal. Would this only apply to variables declared with const, or to all variables this rule reports (let, var, functions, parameters, classes, imports...)?

My idea would be to support all kinds of unused variables to propose the suggestion, for example :

1.

const { useState, useEffect } = require("react");
useEffect();

suggests :

const { useEffect } = require("react");
useEffect();

2.

function() {
  const { useState } = require("react");
  let a = 1 + 1;
  return a
}

suggests :

function() {
  let a = 1 + 1;
  return a
}

3.

function({ foo, bar}) {
    return bar;
}

suggests :

function({ bar}) {
    return bar;
}

4.

function({ foo, bar}) {
    return null;
}

suggests:

function() {
    return null;
}

5.

function() {
    let a = 1, b = 2;
    const x = 1;
   var c = 0;
   return 1;
}

suggests :

function() {
   return 1;
}

I support this. There will be a lot of details (for example, #17546 (comment)) that I think we can figure out while working on the PR, or if the team prefers here in advance.

Waiting for more opinions from other team members.

Oops! It looks like we lost track of this issue. What do we want to do here? This issue will auto-close in 7 days without an update.

This issue was auto-closed due to inactivity. While we wish we could keep responding to every issue, we unfortunately don't have the bandwidth and need to focus on high-value issues.

@eslint/eslint-team what do you think of this proposal?

I think adding suggestions is a good idea. Marking as accepted. @edi9999 would you be interested in submitting a PR?

Currently, I'm not going to be able to work on this in the foreseeable future.

I don't have that much of eslint internals knowledge yet and don't have much time to do it, I'm sorry.

I can try to implement this.
But i think it's good to work on this after #17662 PR.