Rule Change: There should be a way to specify ignores for no-param-reassign

What rule do you want to change?

no-param-reassign

What change to do you want to make?

Generate fewer warnings

How do you think the change should be implemented?

A new option

Example code

function myWidget(spec) {
	spec ||= {};
	...
}

What does the rule currently do for this code?

Report that spec is reassigned.

What will the rule do after it’s changed?

With {ignore: ["spec"]}, the rule should stop reporting reassignments to spec.

Participation

• I am willing to submit a pull request to implement this change.

Additional comments

No response

[mdjermanovic]

There are tools to wrap and extend existing rules, like eslint-rule-composer and eslint-rule-extender, but note that rule definitions are not public API and therefore can change and possibly break extension rules in non-major versions. Copy & paste & edit is actually the recommended way.

[mdjermanovic]

Sorry for the delay.

I’m not in favor of adding this option to the core rule because I think there should be no reason to allow exceptions by param names in a new codebase. In particular, it would be confusing why the option exists and could lead to logic errors in the code, as in https://github.com/eslint/eslint/issues/15244. For specific use cases, such as an existing codebase that follows a certain pattern, it’s always possible to create a custom rule based on the core rule.