@dvukovic/style-guide 0.3.93 → 0.3.94

package/src/eslint/configs/playwright.js

@@ -1,4 +1,5 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    extends: ["../plugins/playwright.js"],
-}
+import playwrightPlugin from "../plugins/playwright.js"
+
+const playwrightConfig = [playwrightPlugin]
+
+export default playwrightConfig

package/src/eslint/configs/playwright.test.js

@@ -0,0 +1,17 @@
+import { ESLint } from "eslint"
+
+import config from "./playwright.js"
+
+const eslint = new ESLint({
+    overrideConfig: config,
+    overrideConfigFile: true,
+})
+
+describe("playwright", () => {
+    test("loads without errors", async () => {
+        const results = await eslint.lintText("const x = 1\n", { filePath: "test.spec.ts" })
+
+        expect(results).toBeDefined()
+        expect(results[0].fatalErrorCount).toBe(0)
+    })
+})

package/src/eslint/configs/react.js

@@ -1,4 +1,6 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    extends: ["../plugins/react.js", "../plugins/react-hooks.js"],
-}
+import reactPlugin from "../plugins/react.js"
+import reactHooksPlugin from "../plugins/react-hooks.js"
+
+const reactConfig = [reactPlugin, reactHooksPlugin]
+
+export default reactConfig

package/src/eslint/configs/react.test.js

@@ -0,0 +1,17 @@
+import { ESLint } from "eslint"
+
+import config from "./react.js"
+
+const eslint = new ESLint({
+    overrideConfig: config,
+    overrideConfigFile: true,
+})
+
+describe("react", () => {
+    test("loads without errors", async () => {
+        const results = await eslint.lintText("const x = 1\n", { filePath: "test.tsx" })
+
+        expect(results).toBeDefined()
+        expect(results[0].fatalErrorCount).toBe(0)
+    })
+})

package/src/eslint/configs/storybook.js

@@ -1,11 +1,16 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    extends: ["../plugins/storybook"],
-    rules: {
-        "no-console": "off",
-        "react-hooks/rules-of-hooks": "off",
-        "react/no-array-index-key": "off",
-        "security-node/detect-crlf": "off",
-        "unicorn/consistent-function-scoping": "off",
+import storybookPlugin from "../plugins/storybook.js"
+
+const storybookConfig = [
+    storybookPlugin,
+    {
+        rules: {
+            "no-console": "off",
+            "react-hooks/rules-of-hooks": "off",
+            "react/no-array-index-key": "off",
+            "security-node/detect-crlf": "off",
+            "unicorn/consistent-function-scoping": "off",
+        },
     },
-}
+]
+
+export default storybookConfig

package/src/eslint/configs/storybook.test.js

@@ -0,0 +1,17 @@
+import { ESLint } from "eslint"
+
+import config from "./storybook.js"
+
+const eslint = new ESLint({
+    overrideConfig: config,
+    overrideConfigFile: true,
+})
+
+describe("storybook", () => {
+    test("loads without errors", async () => {
+        const results = await eslint.lintText("const x = 1\n", { filePath: "test.stories.tsx" })
+
+        expect(results).toBeDefined()
+        expect(results[0].fatalErrorCount).toBe(0)
+    })
+})

package/src/eslint/configs/typescript-strict.js

@@ -1,25 +1,32 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["@typescript-eslint"],
-    rules: {
-        "@typescript-eslint/no-unsafe-argument": "error",
-        "@typescript-eslint/no-unsafe-assignment": "error",
-        "@typescript-eslint/no-unsafe-call": "error",
-        "@typescript-eslint/no-unsafe-declaration-merging": "error",
-        "@typescript-eslint/no-unsafe-enum-comparison": "error",
-        "@typescript-eslint/no-unsafe-member-access": "error",
-        "@typescript-eslint/no-unsafe-return": "error",
-        "@typescript-eslint/no-unsafe-unary-minus": "error",
-        "@typescript-eslint/restrict-template-expressions": [
-            "error",
-            {
-                allowAny: false,
-                allowBoolean: false,
-                allowNever: false,
-                allowNullish: false,
-                allowNumber: true,
-                allowRegExp: false,
-            },
-        ],
+import typescriptEslint from "typescript-eslint"
+
+const typescriptStrictConfig = [
+    {
+        plugins: {
+            "@typescript-eslint": typescriptEslint.plugin,
+        },
+        rules: {
+            "@typescript-eslint/no-unsafe-argument": "error",
+            "@typescript-eslint/no-unsafe-assignment": "error",
+            "@typescript-eslint/no-unsafe-call": "error",
+            "@typescript-eslint/no-unsafe-declaration-merging": "error",
+            "@typescript-eslint/no-unsafe-enum-comparison": "error",
+            "@typescript-eslint/no-unsafe-member-access": "error",
+            "@typescript-eslint/no-unsafe-return": "error",
+            "@typescript-eslint/no-unsafe-unary-minus": "error",
+            "@typescript-eslint/restrict-template-expressions": [
+                "error",
+                {
+                    allowAny: false,
+                    allowBoolean: false,
+                    allowNever: false,
+                    allowNullish: false,
+                    allowNumber: true,
+                    allowRegExp: false,
+                },
+            ],
+        },
     },
-}
+]
+
+export default typescriptStrictConfig

package/src/eslint/configs/typescript-strict.test.js

@@ -0,0 +1,17 @@
+import { ESLint } from "eslint"
+
+import config from "./typescript-strict.js"
+
+const eslint = new ESLint({
+    overrideConfig: config,
+    overrideConfigFile: true,
+})
+
+describe("typescript-strict", () => {
+    test("loads without errors", async () => {
+        const results = await eslint.lintText("const x: number = 1\n", { filePath: "test.ts" })
+
+        expect(results).toBeDefined()
+        expect(results[0].fatalErrorCount).toBe(0)
+    })
+})

package/src/eslint/configs/typescript.js

@@ -1,4 +1,6 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    extends: ["../plugins/typescript-eslint.js", "../plugins/typescript-sort-keys.js"],
-}
+import typescriptEslintPlugin from "../plugins/typescript-eslint.js"
+import typescriptSortKeysPlugin from "../plugins/typescript-sort-keys.js"
+
+const typescriptConfig = [typescriptEslintPlugin, typescriptSortKeysPlugin]
+
+export default typescriptConfig

package/src/eslint/configs/typescript.test.js

@@ -0,0 +1,17 @@
+import { ESLint } from "eslint"
+
+import config from "./typescript.js"
+
+const eslint = new ESLint({
+    overrideConfig: config,
+    overrideConfigFile: true,
+})
+
+describe("typescript", () => {
+    test("loads without errors", async () => {
+        const results = await eslint.lintText("const x: number = 1\n", { filePath: "test.ts" })
+
+        expect(results).toBeDefined()
+        expect(results[0].fatalErrorCount).toBe(0)
+    })
+})

package/src/eslint/configs/vitest.js

@@ -1,4 +1,5 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    extends: ["../plugins/vitest.js"],
-}
+import vitestPlugin from "../plugins/vitest.js"
+
+const vitestConfig = [vitestPlugin]
+
+export default vitestConfig

package/src/eslint/configs/vitest.test.js

@@ -0,0 +1,17 @@
+import { ESLint } from "eslint"
+
+import config from "./vitest.js"
+
+const eslint = new ESLint({
+    overrideConfig: config,
+    overrideConfigFile: true,
+})
+
+describe("vitest", () => {
+    test("loads without errors", async () => {
+        const results = await eslint.lintText("const x = 1\n", { filePath: "test.test.ts" })
+
+        expect(results).toBeDefined()
+        expect(results[0].fatalErrorCount).toBe(0)
+    })
+})

package/src/eslint/plugins/es-x.js

@@ -1,17 +1,10 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["es-x"],
-    rules: {
-        "es-x/no-iterator-prototype-drop": "error",
-        "es-x/no-iterator-prototype-every": "error",
-        "es-x/no-iterator-prototype-filter": "error",
-        "es-x/no-iterator-prototype-find": "error",
-        "es-x/no-iterator-prototype-flatmap": "error",
-        "es-x/no-iterator-prototype-foreach": "error",
-        "es-x/no-iterator-prototype-map": "error",
-        "es-x/no-iterator-prototype-reduce": "error",
-        "es-x/no-iterator-prototype-some": "error",
-        "es-x/no-iterator-prototype-take": "error",
-        "es-x/no-iterator-prototype-toarray": "error",
+import esX from "eslint-plugin-es-x"
+
+const plugin = {
+    plugins: {
+        "es-x": esX,
     },
+    rules: {},
 }
+
+export default plugin

package/src/eslint/plugins/eslint-comments.js

@@ -1,16 +1,18 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["@eslint-community/eslint-comments"],
+import eslintComments from "@eslint-community/eslint-plugin-eslint-comments"
+
+const plugin = {
+    plugins: {
+        "@eslint-community/eslint-comments": eslintComments,
+    },
     rules: {
-        "@eslint-community/eslint-comments/disable-enable-pair": [
-            "error",
-            { allowWholeFile: true },
-        ],
+        "@eslint-community/eslint-comments/disable-enable-pair": "error",
         "@eslint-community/eslint-comments/no-aggregating-enable": "error",
         "@eslint-community/eslint-comments/no-duplicate-disable": "error",
         "@eslint-community/eslint-comments/no-unlimited-disable": "error",
         "@eslint-community/eslint-comments/no-unused-disable": "error",
         "@eslint-community/eslint-comments/no-unused-enable": "error",
-        "@eslint-community/eslint-comments/require-description": ["error", { ignore: [] }],
+        "@eslint-community/eslint-comments/require-description": "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/eslint.js

@@ -1,8 +1,13 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    env: {
-        es6: true,
-        node: true,
+import globals from "globals"
+
+const plugin = {
+    languageOptions: {
+        ecmaVersion: 2024,
+        globals: {
+            ...globals.es2024,
+            ...globals.node,
+        },
+        sourceType: "module",
     },
     rules: {
         "array-callback-return": [
@@ -80,7 +85,6 @@ module.exports = {
         "no-lone-blocks": "error",
         "no-lonely-if": "error",
         "no-loop-func": "error",
-        "no-loss-of-precision": "error",
         "no-misleading-character-class": "error",
         "no-multi-assign": "error",
         "no-multi-str": "error",
@@ -115,10 +119,9 @@ module.exports = {
         "no-unsafe-optional-chaining": "error",
         "no-unused-expressions": "error",
         "no-unused-labels": "error",
-        "no-unused-private-class-members": "error",
         "no-unused-vars": "error",
         "no-use-before-define": "error",
-        // "no-useless-assignment": "error", NOTE: available in eslint 9
+        "no-useless-assignment": "error",
         "no-useless-backreference": "error",
         "no-useless-call": "error",
         "no-useless-catch": "error",
@@ -163,3 +166,5 @@ module.exports = {
         yoda: "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/etc.js

@@ -1,13 +1,3 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["etc"],
-    rules: {
-        "etc/no-assign-mutated-array": "error",
-        "etc/no-commented-out-code": "error",
-        "etc/no-deprecated": "error",
-        "etc/no-enum": "error",
-        "etc/no-internal": "error",
-        "etc/no-t": "error",
-        "etc/throw-error": "error",
-    },
-}
+const plugin = {}
+
+export default plugin

package/src/eslint/plugins/import-x.js

@@ -1,6 +1,9 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["import-x"],
+import importXPlugin from "eslint-plugin-import-x"
+
+const plugin = {
+    plugins: {
+        "import-x": importXPlugin,
+    },
     rules: {
         "import-x/default": "error",
         "import-x/export": "error",
@@ -10,7 +13,6 @@ module.exports = {
         "import-x/namespace": "error",
         "import-x/newline-after-import": "error",
         "import-x/no-absolute-path": "error",
-        "import-x/no-anonymous-default-export": "error",
         "import-x/no-cycle": "error",
         "import-x/no-deprecated": "error",
         "import-x/no-duplicates": "error",
@@ -26,3 +28,5 @@ module.exports = {
         "import-x/no-useless-path-segments": "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/jest.js

@@ -1,6 +1,9 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["jest"],
+import jest from "eslint-plugin-jest"
+
+const plugin = {
+    plugins: {
+        jest,
+    },
     rules: {
         "jest/consistent-test-it": ["error", { fn: "test", withinDescribe: "test" }],
         "jest/expect-expect": "error",
@@ -53,6 +56,9 @@ module.exports = {
         "jest/valid-describe-callback": "error",
         "jest/valid-expect": "error",
         "jest/valid-expect-in-promise": "error",
+        "jest/valid-mock-module-path": "error",
         "jest/valid-title": "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/mobx.js

@@ -1,9 +1,14 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["mobx"],
+import mobx from "eslint-plugin-mobx"
+
+const plugin = {
+    plugins: {
+        mobx,
+    },
     rules: {
         "mobx/exhaustive-make-observable": "error",
         "mobx/missing-make-observable": "error",
         "mobx/unconditional-make-observable": "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/n.js

@@ -1,6 +1,9 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["n"],
+import pluginNode from "eslint-plugin-n"
+
+const plugin = {
+    plugins: {
+        n: pluginNode,
+    },
     rules: {
         "n/callback-return": "error",
         "n/exports-style": "error",
@@ -35,3 +38,5 @@ module.exports = {
         "n/prefer-promises/fs": "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/next.js

@@ -1,6 +1,9 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["@next/eslint-plugin-next"],
+import next from "@next/eslint-plugin-next"
+
+const plugin = {
+    plugins: {
+        "@next/next": next,
+    },
     rules: {
         "@next/next/google-font-display": "error",
         "@next/next/google-font-preconnect": "error",
@@ -25,3 +28,5 @@ module.exports = {
         "@next/next/no-unwanted-polyfillio": "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/playwright.js

@@ -1,6 +1,9 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["playwright"],
+import playwright from "eslint-plugin-playwright"
+
+const plugin = {
+    plugins: {
+        playwright,
+    },
     rules: {
         "playwright/expect-expect": "error",
         "playwright/max-expects": ["error", { max: 15 }],
@@ -21,6 +24,7 @@ module.exports = {
         "playwright/no-slowed-test": "error",
         "playwright/no-standalone-expect": "error",
         "playwright/no-unsafe-references": "error",
+        "playwright/no-unused-locators": "error",
         "playwright/no-useless-await": "error",
         "playwright/no-useless-not": "error",
         "playwright/no-wait-for-selector": "error",
@@ -52,3 +56,5 @@ module.exports = {
         "playwright/valid-title": "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/prettier.js

@@ -0,0 +1,12 @@
+import eslintPluginPrettier from "eslint-plugin-prettier"
+
+const plugin = {
+    plugins: {
+        prettier: eslintPluginPrettier,
+    },
+    rules: {
+        "prettier/prettier": "error",
+    },
+}
+
+export default plugin

package/src/eslint/plugins/promise.js

@@ -1,26 +1,22 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["promise"],
+import promise from "eslint-plugin-promise"
+
+const plugin = {
+    plugins: {
+        promise,
+    },
     rules: {
-        "promise/always-return": [
-            "error",
-            {
-                ignoreLastCallback: true,
-            },
-        ],
-        "promise/catch-or-return": [
-            "error",
-            {
-                allowFinally: true,
-            },
-        ],
+        "promise/always-return": "error",
+        "promise/catch-or-return": "error",
         "promise/no-callback-in-promise": "error",
         "promise/no-multiple-resolved": "error",
         "promise/no-nesting": "error",
         "promise/no-new-statics": "error",
+        "promise/no-promise-in-callback": "error",
         "promise/no-return-in-finally": "error",
         "promise/no-return-wrap": "error",
         "promise/param-names": "error",
         "promise/valid-params": "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/react-hooks.js

@@ -1,7 +1,12 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["react-hooks"],
+import reactHooks from "eslint-plugin-react-hooks"
+
+const plugin = {
+    plugins: {
+        "react-hooks": reactHooks,
+    },
     rules: {
         "react-hooks/rules-of-hooks": "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/react.js

@@ -1,6 +1,9 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["react"],
+import react from "eslint-plugin-react"
+
+const plugin = {
+    plugins: {
+        react,
+    },
     rules: {
         "react/boolean-prop-naming": "error",
         "react/button-has-type": "error",
@@ -28,7 +31,6 @@ module.exports = {
                 props: "never",
             },
         ],
-        "react/jsx-filename-extension": "error",
         "react/jsx-filename-extension": [
             "error",
             { allow: "as-needed", extensions: [".tsx", ".jsx"] },
@@ -98,4 +100,11 @@ module.exports = {
         "react/style-prop-object": "error",
         "react/void-dom-elements-no-children": "error",
     },
+    settings: {
+        react: {
+            version: "detect",
+        },
+    },
 }
+
+export default plugin

package/src/eslint/plugins/rimac.js

@@ -1,8 +1,13 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["@rimac-technology"],
+import rimac from "@rimac-technology/eslint-plugin"
+
+const plugin = {
+    plugins: {
+        "@rimac-technology": rimac,
+    },
     rules: {
         "@rimac-technology/class-element-sorting": "error",
         "@rimac-technology/document-todos": ["error", { url: "http" }],
     },
 }
+
+export default plugin

package/src/eslint/plugins/security-node.js

@@ -1,6 +1,9 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["security-node"],
+import securityNode from "eslint-plugin-security-node"
+
+const plugin = {
+    plugins: {
+        "security-node": securityNode,
+    },
     rules: {
         "security-node/detect-absence-of-name-option-in-exrpress-session": "error",
         "security-node/detect-buffer-unsafe-allocation": "error",
@@ -26,3 +29,5 @@ module.exports = {
         "security-node/non-literal-reg-expr": "error",
     },
 }
+
+export default plugin

package/src/eslint/plugins/simple-import-sort.js

@@ -1,8 +1,13 @@
-/** @type {import("eslint").ESLint.ConfigData} */
-module.exports = {
-    plugins: ["simple-import-sort"],
+import simpleImportSort from "eslint-plugin-simple-import-sort"
+
+const plugin = {
+    plugins: {
+        "simple-import-sort": simpleImportSort,
+    },
     rules: {
         "simple-import-sort/exports": "error",
         "simple-import-sort/imports": "error",
     },
 }
+
+export default plugin