Rule proposal: no-missing-g-flag

[fisker]

Motivation

When calling String#replaceAll() and String#matchAll() with non-global RegExp, it will be a runtime error. I hope to catch this pattern during linting.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/replaceAll#exceptions
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/matchAll#exceptions

Description

Forbid calling String#replaceAll() or String#matchAll() with non-global pattern.

Examples

/* ✓ GOOD */
foo.replaceAll(" ", '')
foo.replaceAll(/\s/g, '')
foo.matchAll(/\s/g)

/* ✗ BAD */
foo.replaceAll(/\s/, '')
foo.matchAll(/\s/)

[ota-meshi]

Thank you for the rule suggestions! That rule sounds good to me!