quotemark: Invalid regular expression: /\\)/: Unmatched ')'

[mjomble]

Bug Report

• TSLint version: 5.11.0
• TypeScript version: 3.1.6
• Running TSLint via: CLI

TypeScript code being linted

('a)

with tslint.json configuration:

{ "rules": { "quotemark": true } }

Actual behavior

The 'quotemark' rule threw an error in 'test.ts':
SyntaxError: Invalid regular expression: /\\)/: Unmatched ')'
    at new RegExp (<anonymous>)
    at cb (...\tslint\lib\rules\quotemarkRule.js:106:33)
    ...

Expected behavior

No error.

Looks like the same issue as #3073 and #4181 but those issues got closed as there was no reproducible case.

[JoshuaKGoldberg]

Ha, thanks for the repro @mjomble! Quotemark bugs have been popping up every now and then...

If the problem is that new RegExp(\\\\${actualQuoteMark}, "g") is taking in an invalid string "/\\)/", it seems like it would be a pretty quick fix to add a case to the actualQuoteMark === expectedQuoteMark if statement to return early if the actualQuoteMark is not a ', ", or ```.

Related: #3808.