prefer-regex-literals: should report regex literals passed to RegExp constructor

What rule do you want to change?

prefer-regex-literals

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

It will produce more warnings.

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

I think this makes sense as default behavior.

Please provide some example code that this change will affect:

const foo = new RegExp(/foo/);

What does the rule currently do for this code?

It is not recognized as a problem.

What will the rule do after it’s changed?

The rule should report this as problem and suggest the following autofix:

-const foo = new RegExp(/foo/);
+const foo = /foo/;

Are you willing to submit a pull request to implement this change?

Yes

[mdjermanovic]

I’ll champion this, only one more 👍 needed.

[lo1tuma]

I’ve updated the PR and implemented the change behind a new option disallowRedundantWrapping.