unicorn/prefer-optional-catch-binding

catch doesn't need a parameter. A name that's never read sends the reader looking for where the error is handled.

--fix removes the parameter and its parentheses.

Configure

Off by default. To turn it on:

[lint.rules]
"unicorn/prefer-optional-catch-binding" = "error" # or "warn"

Examples

Reported:

function parse(text) {
  try {
    return JSON.parse(text);
  } catch (error) {
    return undefined;
  }
}

try {
  await upload(file);
} catch ({ message }) {
  retry(file);
}

Not reported:

function parse(text) {
  try {
    return JSON.parse(text);
  } catch {
    return undefined;
  }
}

try {
  await upload(file);
} catch ({ message }) {
  console.error(message);
}