@rpcbase/eslint-config 0.23.0 → 0.25.0

package/eslint.config.js

@@ -1,15 +1,19 @@
 import gitignore from "eslint-config-flat-gitignore"
 import globals from "globals"
-import pluginImport from "eslint-plugin-import"
+import tseslint from "typescript-eslint"
 import pluginJs from "@eslint/js"
+// plugins
+import pluginImport from "eslint-plugin-import"
 import pluginReact from "eslint-plugin-react"
-import tseslint from "typescript-eslint"
 import reactRefresh from "eslint-plugin-react-refresh"
+import pluginPlaywright from "eslint-plugin-playwright"
+import pluginVitest from "@vitest/eslint-plugin"
+import pluginReactHooks from "eslint-plugin-react-hooks"
 
 import noLoneTextNode from "./src/rules/no-lone-text-node"
 
 
-/** @type {import('eslint').Linter.Config[]} */
+/** @type {import("eslint").Linter.Config[]} */
 export const config = [
   { languageOptions: { globals: { ...globals.browser, ...globals.node } } },
   gitignore(),
@@ -30,7 +34,7 @@ export const config = [
     ...pluginReact.configs.flat.recommended,
     settings: {
       react: {
-        version: "detect",
+        version: "19",
       },
       "import/resolver": {
         typescript: {
@@ -108,4 +112,29 @@ export const config = [
       "rb/no-lone-text-node": "error",
     },
   },
-];
+  {
+    files: ["**/*.test.{js,mjs,cjs,ts,jsx,tsx}"],
+    plugins: {
+      vitest: pluginVitest,
+    },
+    rules: {
+      ...pluginVitest.configs.recommended.rules, // you can also use vitest.configs.all.rules to enable all rules
+      "vitest/no-commented-out-tests": "off",
+    },
+  },
+  {
+    ...pluginPlaywright.configs["flat/recommended"],
+    files: ["spec/**"],
+    rules: {
+      ...pluginPlaywright.configs["flat/recommended"].rules,
+    },
+  },
+  {
+    files: ["**/*.{js,ts,tsx}"],
+    plugins: { "react-hooks": pluginReactHooks },
+    rules: {
+      "react-hooks/rules-of-hooks": "error",
+      "react-hooks/exhaustive-deps": "warn",
+    }
+  },
+]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rpcbase/eslint-config",
-  "version": "0.23.0",
+  "version": "0.25.0",
   "description": "",
   "license": "MIT",
   "type": "module",
@@ -35,17 +35,19 @@
   "dependencies": {
     "@eslint/js": "9.26.0",
     "@typescript-eslint/experimental-utils": "5.62.0",
+    "@vitest/eslint-plugin": "1.3.3",
     "eslint-config-flat-gitignore": "2.1.0",
     "eslint-import-resolver-typescript": "4.3.4",
     "eslint-plugin-import": "2.31.0",
+    "eslint-plugin-playwright": "2.2.0",
     "eslint-plugin-react": "7.37.5",
+    "eslint-plugin-react-hooks": "5.2.0",
     "eslint-plugin-react-refresh": "0.4.20",
     "globals": "16.2.0",
     "typescript-eslint": "8.35.0"
   },
   "devDependencies": {
     "@types/eslint": "9.6.1",
-    "eslint": "^9.x",
     "typescript": "^5.x"
   }
 }

package/src/rules/no-lone-text-node.ts

@@ -1,5 +1,6 @@
 import { TSESTree, ESLintUtils } from "@typescript-eslint/experimental-utils"
 
+
 type MessageIds = "loneText"
 
 /**
@@ -20,7 +21,7 @@ const createRule = ESLintUtils.RuleCreator(
   name => `@rpcbase/eslint-config/rules/${name}`
 )
 
-const defaultIgnoredChars = ["©", "*", ":", ".", "·", "•", "—", "–", "…", "-", "+", "(", ")", "\"", "\'"]
+const defaultIgnoredChars = ["©", "*", ":", ".", "·", "•", "—", "–", "…", "-", "+", "(", ")", "\"", "'"]
 
 function isIgnoredText(text: string, ignored: string[]) {
   // ignore if every non-whitespace char is in the list