#8142 (comment) reminded me that we don't have any linting to prevent unused catch locals.

In our repo right now, this would present zero lint complaints from @typescript-eslint/no-unused-vars:

try {
  // ...
} catch (error) {
  // ...
}

Proposal: let's set https://eslint.org/docs/latest/rules/no-unused-vars#caughterrors to "all" in our lint config?