@fullstacksjs/eslint-config 13.9.0 → 13.10.0

package/README.md

@@ -51,7 +51,7 @@ You can fine-tune module detection by overriding it, the `defineConfig` function
 ```typescript
 interface Options {
   react?: boolean | { additionalEffectHooks?: string }; // controls react, react-hooks, jsx/a11y plugins
-  typescript?: ParserOptions // https://typescript-eslint.io/packages/parser#configuration
+  typescript?: boolean | ParserOptions // https://typescript-eslint.io/packages/parser#configuration
   node?: boolean; // controls node plugin
   sort?: boolean; // controls perfectionist plugin
   strict?: boolean; // controls strict rules

package/cjs/index.d.ts

@@ -93,7 +93,7 @@ export interface Options extends Linter.Config {
    * Controls [TypeScript plugin](https://www.npmjs.com/package/typescript-eslint).
    * @default true - If you have `typescript` in your dependencies.
    */
-  typescript?: { projectService?: boolean | ProjectService; tsconfigRootDir?: string; cacheLifetime?: number };
+  typescript?: boolean | { projectService?: boolean | ProjectService; tsconfigRootDir?: string; cacheLifetime?: number };
   /**
    * Disables expensive rules.
    * @default false
@@ -111,7 +111,7 @@ export interface Options extends Linter.Config {
   gitignore?: string | false;
   /**
    * Controls regex plugin.
-   * @default true
+   * @default { allowedCharacterRanges: ['all'] }
    */
   regex?: boolean | { allowedCharacterRanges: string[] };
 }

package/cjs/modules/jest.js

@@ -5,6 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 
 var _eslintPluginJest = _interopRequireDefault(require("eslint-plugin-jest"));
+var _globals = _interopRequireDefault(require("globals"));
 var _globs = require("../utils/globs.js");
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 /**
@@ -16,6 +17,11 @@ function jest() {
     plugins: {
       jest: _eslintPluginJest.default
     },
+    languageOptions: {
+      globals: {
+        ..._globals.default.jest
+      }
+    },
     rules: {
       'jest/consistent-test-it': 'off',
       'jest/expect-expect': 'off',

package/cjs/modules/playwright.js

@@ -15,9 +15,6 @@ function playwright() {
     },
     files: _globs.globs.e2e,
     rules: {
-      'jest/no-standalone-expect': ['error', {
-        additionalTestBlockFunctions: ['test.jestPlaywrightDebug', 'it.jestPlaywrightDebug', 'test.jestPlaywrightSkip', 'it.jestPlaywrightSkip', 'test.jestPlaywrightConfig', 'it.jestPlaywrightConfig']
-      }],
       'playwright/expect-expect': 'error',
       'playwright/missing-playwright-await': 'error',
       'playwright/max-nested-describe': ['warn', {
@@ -39,6 +36,7 @@ function playwright() {
       'playwright/no-useless-await': 'error',
       'playwright/no-unsafe-references': 'warn',
       'playwright/no-useless-not': 'warn',
+      'playwright/no-unused-locators': 'warn',
       'playwright/no-wait-for-navigation': 'warn',
       'playwright/no-wait-for-selector': 'warn',
       'playwright/no-wait-for-timeout': 'warn',

package/cjs/modules/promise.js

@@ -5,9 +5,10 @@ Object.defineProperty(exports, "__esModule", {
 });
 
 var _eslintPluginPromise = _interopRequireDefault(require("eslint-plugin-promise"));
+var _conditions = require("../utils/conditions.js");
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 /** @return { import('eslint').Linter.Config } */
-function promise() {
+function promise(options = {}) {
   return {
     plugins: {
       promise: _eslintPluginPromise.default
@@ -26,8 +27,10 @@ function promise() {
       'promise/param-names': 'warn',
       'promise/prefer-await-to-callbacks': 'off',
       'promise/prefer-await-to-then': 'off',
+      'promise/prefer-catch': (0, _conditions.strict)(options, 'warn'),
       'promise/valid-params': 'warn',
-      'promise/no-multiple-resolved': 'warn'
+      'promise/no-multiple-resolved': 'warn',
+      'promise/spec-only': 'warn'
     }
   };
 }

package/cjs/modules/react.js

@@ -54,6 +54,7 @@ function react(options = {}) {
       '@eslint-react/dom/no-script-url': 'warn',
       '@eslint-react/dom/no-unsafe-iframe-sandbox': 'warn',
       '@eslint-react/dom/no-unsafe-target-blank': 'warn',
+      '@eslint-react/jsx-dollar': 'warn',
       '@eslint-react/jsx-key-before-spread': 'error',
       '@eslint-react/jsx-no-undef': 'off',
       '@eslint-react/jsx-no-iife': 'warn',

package/cjs/modules/tests.js

@@ -13,31 +13,25 @@ function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e
  * @param { import('../index.js').Options } options
  * @return { import('eslint').Linter.Config }
  */
+
 function tests(options = {}) {
   return {
+    files: [..._globs.globs.test, ..._globs.globs.e2e],
     plugins: {
       vitest: _eslintPlugin.default
     },
-    files: [..._globs.globs.test, ..._globs.globs.e2e],
     languageOptions: {
       globals: {
-        ..._globals.default['shared-node-browser'],
-        ..._globals.default.jest
+        ..._globals.default['shared-node-browser']
       }
     },
     rules: {
-      'vitest/padding-around-after-all-blocks': 'warn',
-      'vitest/padding-around-after-each-blocks': 'warn',
-      'vitest/padding-around-before-all-blocks': 'warn',
-      'vitest/padding-around-before-each-blocks': 'warn',
-      'vitest/padding-around-describe-blocks': 'warn',
-      'vitest/padding-around-expect-groups': 'warn',
-      'vitest/padding-around-test-blocks': 'warn',
       'max-lines-per-function': 'off',
       'no-sparse-arrays': 'off',
       'no-empty-function': 'off',
+      'vitest/padding-around-all': 'warn',
       ...(0, _conditions.predicate)(options.react, {
-        'react/jsx-no-constructed-context-values': 'off'
+        '@eslint-react/no-unstable-context-value': 'off'
       }),
       ...(0, _conditions.predicate)(options.typescript, {
         '@typescript-eslint/method-signature-style': 'off',

package/cjs/modules/vitest.js

@@ -95,11 +95,13 @@ function vitest() {
       'vitest/require-local-test-context-for-concurrent-snapshots': 'warn',
       'vitest/require-mock-type-parameters': 'off',
       'vitest/require-to-throw-message': 'warn',
-      'vitest/require-top-level-describe': 'warn',
+      'vitest/require-top-level-describe': 'off',
       'vitest/valid-describe-callback': 'warn',
       'vitest/valid-expect': 'warn',
       'vitest/valid-expect-in-promise': 'warn',
-      'vitest/valid-title': 'warn',
+      'vitest/valid-title': ['warn', {
+        allowArguments: true
+      }],
       'vitest/warn-todo': 'warn'
     }
   };

package/cjs/utils/globs.js

@@ -29,7 +29,7 @@ const globs = exports.globs = {
   html: '**/*.html',
   astro: '**/*.astro',
   graphql: '**/*.{gql,graphql}',
-  storybook: ['**/*.stories.{js,jsx,ts,tsx}'],
+  storybook: ['**/*.stories.{js,mjs,cjs,jsx,ts,tsx}'],
   test: [`**/__tests__/*.${srcExt}`, `**/*.spec.${srcExt}`, `**/*.test.${srcExt}`],
   e2e: [`**/e2e/*.${srcExt}`]
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fullstacksjs/eslint-config",
-  "version": "13.9.0",
+  "version": "13.10.0",
   "license": "MIT",
   "author": "fullstacks <fullstacksjs@gmail.com>",
   "description": "fullstacks eslint config",
@@ -35,11 +35,11 @@
     }
   },
   "dependencies": {
-    "@eslint-react/eslint-plugin": "2.2.3",
-    "@eslint/compat": "1.4.0",
-    "@next/eslint-plugin-next": "16.0.0",
+    "@eslint-react/eslint-plugin": "2.3.1",
+    "@eslint/compat": "1.4.1",
+    "@next/eslint-plugin-next": "16.0.1",
     "@stylistic/eslint-plugin": "5.5.0",
-    "@vitest/eslint-plugin": "1.3.24",
+    "@vitest/eslint-plugin": "1.4.1",
     "deepmerge": "4.3.1",
     "eslint-plugin-better-tailwindcss": "3.7.10",
     "eslint-plugin-cypress": "5.2.0",
@@ -48,31 +48,32 @@
     "eslint-plugin-jsx-a11y": "6.10.2",
     "eslint-plugin-n": "17.23.1",
     "eslint-plugin-perfectionist": "4.15.1",
-    "eslint-plugin-playwright": "2.2.2",
+    "eslint-plugin-playwright": "2.3.0",
     "eslint-plugin-prettier": "5.5.4",
     "eslint-plugin-promise": "7.2.1",
     "eslint-plugin-react-hooks": "7.0.1",
     "eslint-plugin-regexp": "2.10.0",
-    "eslint-plugin-storybook": "0.12.0",
-    "globals": "16.4.0",
+    "eslint-plugin-storybook": "10.0.6",
+    "globals": "16.5.0",
     "local-pkg": "1.1.2",
-    "typescript-eslint": "8.46.2"
+    "typescript-eslint": "8.46.3"
   },
   "devDependencies": {
     "@eslint/eslintrc": "3.3.1",
-    "@eslint/js": "9.38.0",
-    "@semantic-release/github": "12.0.0",
+    "@eslint/js": "9.39.1",
+    "@semantic-release/github": "12.0.1",
     "@semantic-release/npm": "13.1.1",
     "@semantic-release/release-notes-generator": "14.1.0",
     "@types/eslint": "9.6.1",
-    "cspell": "9.2.2",
-    "eslint": "9.38.0",
+    "cspell": "9.3.0",
+    "eslint": "9.39.1",
     "husky": "9.1.7",
     "jest": "30.2.0",
     "lint-staged": "16.2.6",
     "pinst": "3.0.0",
     "prettier": "3.6.2",
-    "semantic-release": "25.0.1",
+    "semantic-release": "25.0.2",
+    "storybook": "10.0.6",
     "typescript": "5.9.3",
     "unbuild": "3.6.1"
   },

package/src/index.d.ts

@@ -93,7 +93,7 @@ export interface Options extends Linter.Config {
    * Controls [TypeScript plugin](https://www.npmjs.com/package/typescript-eslint).
    * @default true - If you have `typescript` in your dependencies.
    */
-  typescript?: { projectService?: boolean | ProjectService; tsconfigRootDir?: string; cacheLifetime?: number };
+  typescript?: boolean | { projectService?: boolean | ProjectService; tsconfigRootDir?: string; cacheLifetime?: number };
   /**
    * Disables expensive rules.
    * @default false
@@ -111,7 +111,7 @@ export interface Options extends Linter.Config {
   gitignore?: string | false;
   /**
    * Controls regex plugin.
-   * @default true
+   * @default { allowedCharacterRanges: ['all'] }
    */
   regex?: boolean | { allowedCharacterRanges: string[] };
 }

package/src/modules/jest.mjs

@@ -1,4 +1,5 @@
 import plugin from 'eslint-plugin-jest';
+import globals from 'globals';
 
 import { globs } from '../utils/globs.mjs';
 
@@ -9,6 +10,11 @@ function jest() {
   return {
     files: globs.test,
     plugins: { jest: plugin },
+    languageOptions: {
+      globals: {
+        ...globals.jest,
+      },
+    },
     rules: {
       'jest/consistent-test-it': 'off',
       'jest/expect-expect': 'off',

package/src/modules/playwright.mjs

@@ -8,19 +8,6 @@ function playwright() {
     plugins: { playwright: plugin },
     files: globs.e2e,
     rules: {
-      'jest/no-standalone-expect': [
-        'error',
-        {
-          additionalTestBlockFunctions: [
-            'test.jestPlaywrightDebug',
-            'it.jestPlaywrightDebug',
-            'test.jestPlaywrightSkip',
-            'it.jestPlaywrightSkip',
-            'test.jestPlaywrightConfig',
-            'it.jestPlaywrightConfig',
-          ],
-        },
-      ],
       'playwright/expect-expect': 'error',
       'playwright/missing-playwright-await': 'error',
       'playwright/max-nested-describe': ['warn', { max: 5 }],
@@ -40,6 +27,7 @@ function playwright() {
       'playwright/no-useless-await': 'error',
       'playwright/no-unsafe-references': 'warn',
       'playwright/no-useless-not': 'warn',
+      'playwright/no-unused-locators': 'warn',
       'playwright/no-wait-for-navigation': 'warn',
       'playwright/no-wait-for-selector': 'warn',
       'playwright/no-wait-for-timeout': 'warn',

package/src/modules/promise.mjs

@@ -1,7 +1,9 @@
 import plugin from 'eslint-plugin-promise';
 
+import { strict } from '../utils/conditions.mjs';
+
 /** @return { import('eslint').Linter.Config } */
-function promise() {
+function promise(options = {}) {
   return {
     plugins: { promise: plugin },
     rules: {
@@ -18,8 +20,10 @@ function promise() {
       'promise/param-names': 'warn',
       'promise/prefer-await-to-callbacks': 'off',
       'promise/prefer-await-to-then': 'off',
+      'promise/prefer-catch': strict(options, 'warn'),
       'promise/valid-params': 'warn',
       'promise/no-multiple-resolved': 'warn',
+      'promise/spec-only': 'warn',
     },
   };
 }

package/src/modules/react.mjs

@@ -49,6 +49,7 @@ function react(options = {}) {
       '@eslint-react/dom/no-unsafe-iframe-sandbox': 'warn',
       '@eslint-react/dom/no-unsafe-target-blank': 'warn',
 
+      '@eslint-react/jsx-dollar': 'warn',
       '@eslint-react/jsx-key-before-spread': 'error',
       '@eslint-react/jsx-no-undef': 'off',
       '@eslint-react/jsx-no-iife': 'warn',

package/src/modules/tests.mjs

@@ -8,28 +8,23 @@ import { globs } from '../utils/globs.mjs';
  * @param { import('..').Options } options
  * @return { import('eslint').Linter.Config }
  */
+
 function tests(options = {}) {
   return {
-    plugins: { vitest: plugin },
     files: [...globs.test, ...globs.e2e],
+    plugins: { vitest: plugin },
     languageOptions: {
-      globals: { ...globals['shared-node-browser'], ...globals.jest },
+      globals: { ...globals['shared-node-browser'] },
     },
     rules: {
-      'vitest/padding-around-after-all-blocks': 'warn',
-      'vitest/padding-around-after-each-blocks': 'warn',
-      'vitest/padding-around-before-all-blocks': 'warn',
-      'vitest/padding-around-before-each-blocks': 'warn',
-      'vitest/padding-around-describe-blocks': 'warn',
-      'vitest/padding-around-expect-groups': 'warn',
-      'vitest/padding-around-test-blocks': 'warn',
-
       'max-lines-per-function': 'off',
       'no-sparse-arrays': 'off',
       'no-empty-function': 'off',
 
+      'vitest/padding-around-all': 'warn',
+
       ...predicate(options.react, {
-        'react/jsx-no-constructed-context-values': 'off',
+        '@eslint-react/no-unstable-context-value': 'off',
       }),
 
       ...predicate(options.typescript, {

package/src/modules/vitest.mjs

@@ -58,6 +58,7 @@ function vitest() {
       'vitest/no-standalone-expect': 'warn',
       'vitest/no-test-prefixes': 'warn',
       'vitest/no-test-return-statement': 'warn',
+
       'vitest/prefer-called-once': 'warn',
       'vitest/prefer-called-times': 'off',
       'vitest/prefer-called-with': 'warn',
@@ -88,11 +89,11 @@ function vitest() {
       'vitest/require-local-test-context-for-concurrent-snapshots': 'warn',
       'vitest/require-mock-type-parameters': 'off',
       'vitest/require-to-throw-message': 'warn',
-      'vitest/require-top-level-describe': 'warn',
+      'vitest/require-top-level-describe': 'off',
       'vitest/valid-describe-callback': 'warn',
       'vitest/valid-expect': 'warn',
       'vitest/valid-expect-in-promise': 'warn',
-      'vitest/valid-title': 'warn',
+      'vitest/valid-title': ['warn', { allowArguments: true }],
       'vitest/warn-todo': 'warn',
     },
   };

package/src/utils/globs.mjs

@@ -28,7 +28,7 @@ export const globs = {
   astro: '**/*.astro',
   graphql: '**/*.{gql,graphql}',
 
-  storybook: ['**/*.stories.{js,jsx,ts,tsx}'],
+  storybook: ['**/*.stories.{js,mjs,cjs,jsx,ts,tsx}'],
 
   test: [`**/__tests__/*.${srcExt}`, `**/*.spec.${srcExt}`, `**/*.test.${srcExt}`],
   e2e: [`**/e2e/*.${srcExt}`],