Fix false positives for unknown method in regexp/no-unused-capturing-group (#82)

Author: ota-meshi

lib/rules/no-unused-capturing-group.ts

@@ -383,6 +383,7 @@ export default createRule("no-unused-capturing-group", {
                 return
             }
             if (!typeTracer.isString(node.callee.object)) {
+                capturingData.markAsCannotTrack()
                 return
             }
             if (capturingData.flags.includes("g")) {
@@ -401,6 +402,7 @@ export default createRule("no-unused-capturing-group", {
                 return
             }
             if (!typeTracer.isString(node.callee.object)) {
+                capturingData.markAsCannotTrack()
                 return
             }
             // String.prototype.search()
@@ -424,6 +426,7 @@ export default createRule("no-unused-capturing-group", {
                 return
             }
             if (!typeTracer.isString(node.callee.object)) {
+                capturingData.markAsCannotTrack()
                 return
             }
             const replacementNode = node.arguments[1]
@@ -535,6 +538,7 @@ export default createRule("no-unused-capturing-group", {
                 return
             }
             if (!typeTracer.isString(node.callee.object)) {
+                capturingData.markAsCannotTrack()
                 return
             }
             capturingData.markAsUsed()
@@ -629,6 +633,7 @@ export default createRule("no-unused-capturing-group", {
                 return
             }
             if (!typeTracer.isString(node.callee.object)) {
+                capturingData.markAsCannotTrack()
                 return
             }
             capturingData.markAsUsed()

tests/lib/rules/no-unused-capturing-group.ts

@@ -10,6 +10,19 @@ const tester = new RuleTester({
 
 tester.run("no-unused-capturing-group", rule as any, {
     valid: [
+        `
+        const computed = \`---\\n\${Object.keys(fileIntro)
+            .map((key) => \`\${key}: \${yamlValue(fileIntro[key])}\`)
+            .join("\\n")}\\n---\\n\`
+
+        const fileIntroPattern = /^---\\n(.*\\n)+---\\n*/gu
+
+        if (fileIntroPattern.test(this.content)) {
+            this.content = this.content.replace(fileIntroPattern, computed)
+        } else {
+            this.content = \`\${computed}\${this.content.trim()}\\n\`
+        }
+        `,
         String.raw`
         var replaced = '2000-12-31'.replace(/(\d{4})-(\d{2})-(\d{2})/, '$1/$2/$3') // "2000/12/31"
         var replaced = '2000-12-31'.replace(/(?<y>\d{4})-(?<m>\d{2})-(?<d>\d{2})/u, '$<y>/$<m>/$<d>') // "2000/12/31"