`prefer-named-capture-group` should be disabled when using `.test()` or `.replace()`

What rule do you want to change?

prefer-named-capture-group suggested in https://github.com/eslint/eslint/issues/11381

Does this change cause the rule to produce more or fewer warnings?

Fewer warnings

How will the change be implemented? (New option, new default behavior, etc.)?

New default behavior

Please provide some example code that this change will affect:

const hasPlanet = /with (mars|pluto)/i.test(object);

return string.replace(/on (mars|venus)/gi, 'on Pluto');

What does the rule currently do for this code?

The rule suggests adding ?:

This has no advantages:

• the result is unchanged
• the group isn’t meant to be captured at all
• it lengthens the regex

.replace can capture the groups but it doesn’t support named groups.

What will the rule do after it’s changed?

Not report an issue when a regex is used inline with .test or .replace()

[mdjermanovic]

I think that String.prototype.replace supports named capture groups

https://github.com/tc39/proposal-regexp-named-groups#replacement-targets

[g-plane]

Thanks for this issue.

Personally, I think this is expected. You’re using regular expressions groups, but you aren’t going to capture anything, so this rule suggests you adding prefix ?: which means using groups without capturing.