Maintenance: Update eslintrc and some fix (#173)

Author: ota-meshi

.eslintrc.js

@@ -1,6 +1,13 @@
 "use strict"
 
-// const version = require("./package.json").version
+const { rules } = require("eslint-plugin-regexp")
+
+const enableAllRules = Object.keys(rules)
+    .map((name) => `regexp/${name}`)
+    .reduce((p, c) => {
+        p[c] = "error"
+        return p
+    }, {})
 
 module.exports = {
     parserOptions: {
@@ -18,6 +25,8 @@ module.exports = {
         "plugin:@ota-meshi/+prettier",
     ],
     rules: {
+        // eslint-disable-next-line node/no-unsupported-features/es-syntax -- Lint only
+        ...enableAllRules,
         "require-jsdoc": "error",
         "no-warning-comments": "warn",
         "no-lonely-if": "off",
@@ -42,29 +51,13 @@ module.exports = {
             },
         ],
 
-        // https://github.com/ota-meshi/eslint-plugin-regexp/pull/49
         "no-empty-character-class": "error",
-        "regexp/negation": "error",
-        "regexp/no-dupe-disjunctions": "error",
-        "regexp/no-useless-character-class": "error",
-        "regexp/no-useless-escape": "error",
-        "regexp/no-useless-non-capturing-group": "error",
-        "regexp/no-useless-non-greedy": "error",
-        "regexp/no-useless-range": "error",
-        "regexp/prefer-character-class": "error",
-        "regexp/prefer-range": "error",
-        "regexp/prefer-unicode-codepoint-escapes": "error",
 
+        "regexp/no-dupe-disjunctions": ["error", { disallowNeverMatch: true }],
         "regexp/letter-case": [
             "error",
             { hexadecimalEscape: "lowercase", controlEscape: "uppercase" },
         ],
-
-        // others
-        "regexp/order-in-character-class": "error",
-        "regexp/prefer-quantifier": "error",
-        "regexp/prefer-regexp-exec": "error",
-        "regexp/prefer-regexp-test": "error",
     },
     overrides: [
         {

lib/rules/no-useless-character-class.ts

@@ -115,7 +115,7 @@ export default createRule("no-useless-character-class", {
                                 element.type === "CharacterClassRange"
                             ) {
                                 if (
-                                    /^[$(-+./?[{|]$/u.test(text) ||
+                                    /^[$()*+./?[{|]$/u.test(text) ||
                                     (flags.unicode && text === "}")
                                 ) {
                                     text = `\\${text}`

lib/utils/type-tracker/jsdoc/index.ts

@@ -90,7 +90,7 @@ export class JSDoc {
         }
         const params = (this.params = new JSDocParams())
         for (const param of this.genTags("param")) {
-            const paths: string[] = (param.name || "").split(/\./gu)
+            const paths: string[] = (param.name || "").split(/\./u)
             params.add(paths, param)
         }
         return params

tools/update-docs.ts

@@ -25,7 +25,7 @@ const ROOT = path.resolve(__dirname, "../docs/rules")
 
 //eslint-disable-next-line require-jsdoc -- tools
 function pickSince(content: string): string | null {
-    const fileIntro = /^---\n(.*\n)+---\n*/g.exec(content)
+    const fileIntro = /^---\n(.*\n)+---\n*/.exec(content)
     if (fileIntro) {
         const since = /since: "?(v\d+\.\d+\.\d+)"?/.exec(fileIntro[0])
         if (since) {
@@ -173,11 +173,11 @@ This rule was introduced in eslint-plugin-regexp ${this.since}
     public adjustCodeBlocks() {
         // Adjust the necessary blank lines before and after the code block so that GitHub can recognize `.md`.
         this.content = this.content.replace(
-            /(<eslint-code-block[\s\S]*?>)\n+```/gmu,
+            /(<eslint-code-block[\s\S]*?>)\n+```/gu,
             "$1\n\n```",
         )
         this.content = this.content.replace(
-            /```\n+<\/eslint-code-block>/gmu,
+            /```\n+<\/eslint-code-block>/gu,
             "```\n\n</eslint-code-block>",
         )
         return this