@pumpn/eslint-config 16.0.0 → 16.1.2

package/eslint.config.js

@@ -2,6 +2,18 @@ import config from "./config.js";
 
 const eslintConfig = [
 	...config,
+	{
+		rules: {
+			"tailwindcss/classnames-order": "off",
+			"tailwindcss/enforces-negative-arbitrary-values": "off",
+			"tailwindcss/enforces-shorthand": "off",
+			"tailwindcss/migration-from-tailwind-2": "off",
+			"tailwindcss/no-arbitrary-value": "off",
+			"tailwindcss/no-contradicting-classname": "off",
+			"tailwindcss/no-custom-classname": "off",
+			"tailwindcss/no-unnecessary-arbitrary-value": "off"
+		}
+	},
 	{
 		files: ["rules/unicorn.js"],
 		rules: {
@@ -9,11 +21,7 @@ const eslintConfig = [
 		}
 	},
 	{
-		files: [
-			"plugins.js",
-			"rules/_exports.js",
-			"rules/func-params-args.js"
-		],
+		files: ["plugins.js", "rules/_exports.js", "rules/func-params-args.js"],
 		rules: {
 			"unicorn/prevent-abbreviations": [
 				"error",

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@pumpn/eslint-config",
-	"version": "16.0.0",
+	"version": "16.1.2",
 	"description": "standard eslint config for pumpn code projects",
 	"main": "config.js",
 	"author": "Nano Miratus <nanomiratus@gmail.com>",
@@ -17,30 +17,30 @@
 		"test": "echo 'no tests yet'"
 	},
 	"devDependencies": {
-		"eslint": "^9.21.0"
+		"eslint": "^9.34.0"
 	},
 	"dependencies": {
-		"@eslint-community/eslint-plugin-eslint-comments": "^4.4.1",
-		"@eslint-react/eslint-plugin": "^1.28.0",
-		"@stylistic/eslint-plugin": "^4.1.0",
-		"@typescript-eslint/parser": "^8.25.0",
-		"eslint-plugin-command": "^3.1.0",
-		"eslint-plugin-depend": "^0.12.0",
-		"eslint-plugin-import-x": "^4.6.1",
-		"eslint-plugin-jsdoc": "^50.6.3",
+		"@eslint-community/eslint-plugin-eslint-comments": "^4.5.0",
+		"@eslint-react/eslint-plugin": "^1.53.0",
+		"@stylistic/eslint-plugin": "^5.3.1",
+		"@typescript-eslint/parser": "^8.42.0",
+		"eslint-import-resolver-typescript": "^4.4.4",
+		"eslint-plugin-command": "^3.3.1",
+		"eslint-plugin-depend": "^1.2.0",
+		"eslint-plugin-import-x": "^4.16.1",
+		"eslint-plugin-jsdoc": "^54.3.1",
 		"eslint-plugin-no-secrets": "^2.2.1",
-		"eslint-plugin-perfectionist": "^4.9.0",
+		"eslint-plugin-perfectionist": "^4.15.0",
 		"eslint-plugin-promise": "^7.2.1",
-		"eslint-plugin-react": "^7.37.4",
-		"eslint-plugin-redos": "^4.4.5",
-		"eslint-plugin-regexp": "^2.7.0",
+		"eslint-plugin-redos": "^4.5.0",
+		"eslint-plugin-regexp": "^2.10.0",
 		"eslint-plugin-security": "^3.0.1",
-		"eslint-plugin-sonarjs": "^3.0.2",
-		"eslint-plugin-tailwindcss": "^3.18.0",
-		"eslint-plugin-unicorn": "^57.0.0",
-		"globals": "^16.0.0"
+		"eslint-plugin-sonarjs": "^3.0.5",
+		"eslint-plugin-tailwindcss": "^4.0.0-beta.0",
+		"eslint-plugin-unicorn": "^60.0.0",
+		"globals": "^16.3.0"
 	},
 	"peerDependencies": {
-		"eslint": "^9.21.0"
+		"eslint": "^9.34.0"
 	}
-}
+}

package/plugins.js

@@ -1,5 +1,5 @@
-/* eslint-disable object-shorthand -- const types */
-/* eslint-disable import-x/no-rename-default -- bad export names */
+// TODO [2025-10-01]: Remove this directive once https://github.com/eslint-stylistic/eslint-stylistic/issues/573 is resolved
+/* eslint-disable object-shorthand -- see above */
 /* eslint-disable import-x/max-dependencies -- collection file */
 
 // import github from "eslint-plugin-github";
@@ -16,7 +16,7 @@ import noSecrets from "eslint-plugin-no-secrets";
 import perfectionist from "eslint-plugin-perfectionist";
 import promise from "eslint-plugin-promise";
 import redos from "eslint-plugin-redos";
-import regexp from "eslint-plugin-regexp";
+import * as regexp from "eslint-plugin-regexp";
 import security from "eslint-plugin-security";
 import sonarjs from "eslint-plugin-sonarjs";
 import tailwindcss from "eslint-plugin-tailwindcss";
@@ -38,22 +38,26 @@ const plugins = /** @type {const} */ ({
 	// "jsx-a11y": jsxA11y,
 	// "no-unsanitized": noUnsanitized,
 	"@eslint-community/eslint-comments": eslintComments,
-	"@stylistic": /** @type {ESLint.Plugin} */ (stylistic),
+	"@stylistic": stylistic,
 	command,
 	depend,
+	// TODO [2025-10-01]: Remove assertion once https://github.com/un-ts/eslint-plugin-import-x/issues/421 is resolved
 	"import-x": /** @type {ESLint.Plugin} */ (
-		/** @type {unknown} */ (importX)
+		/** @type {unknown} */ (
+			importX
+		)
 	),
 	jsdoc,
 	"no-secrets": noSecrets,
-	perfectionist: /** @type {ESLint.Plugin} */ (
-		/** @type {unknown} */ (perfectionist)
-	),
+	perfectionist,
 	promise,
 	redos,
 	regexp,
 	security,
-	sonarjs,
+	// TODO [2025-10-01]: Remove assertion once an issue is opened at https://community.sonarsource.com/ and resolved (:/)
+	sonarjs: /** @type {ESLint.Plugin} */ (
+		sonarjs
+	),
 	tailwindcss,
 	unicorn,
 	...reactPlugins

package/rules/command.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/depend.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/eslint-comments.js

@@ -1,7 +1,7 @@
 import * as eslintCommentsRuleSets from "./eslint-comments/_exports.js";
 
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/github.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/import-x/helpful-warnings.js

@@ -4,9 +4,9 @@ const helpfulWarningsRules = {
 	"import-x/no-empty-named-blocks": "error",
 	"import-x/no-extraneous-dependencies": "off",
 	"import-x/no-mutable-exports": "error",
-	"import-x/no-named-as-default": "error",
+	"import-x/no-named-as-default": "off",
 	"import-x/no-named-as-default-member": "error",
-	"import-x/no-rename-default": "error",
+	"import-x/no-rename-default": "off",
 	"import-x/no-unused-modules": [
 		"error",
 		{

package/rules/import-x/static-analysis.js

@@ -57,7 +57,7 @@ const staticAnalysisRules = {
 		}
 	],
 	"import-x/no-self-import": "error",
-	// TODO [2025-03-01]: Change to "error" when there is a resolver for deno (maybe https://github.com/redabacha/eslint-import-resolver-deno)
+	// TODO [2025-10-01]: Change to "error" when there is a resolver for deno (maybe https://github.com/redabacha/eslint-import-resolver-deno)
 	"import-x/no-unresolved": "off",
 	"import-x/no-useless-path-segments": [
 		"error",

package/rules/import-x/style-guide.js

@@ -4,7 +4,14 @@ const styleGuideRules = {
 	"import-x/consistent-type-specifier-style": "error",
 	"import-x/dynamic-import-chunkname": "off",
 	"import-x/exports-last": "error",
-	"import-x/extensions": ["error", "ignorePackages"],
+	"import-x/extensions": [
+		"error",
+		"ignorePackages",
+		{
+			checkTypeImports: true,
+			fix: true
+		}
+	],
 	"import-x/first": "error",
 	"import-x/group-exports": "error",
 	"import-x/max-dependencies": [
@@ -75,7 +82,8 @@ const styleGuideRules = {
 			warnOnUnassignedImports: false
 		}
 	],
-	"import-x/prefer-default-export": ["error", { target: "single" }]
+	"import-x/prefer-default-export": ["error", { target: "single" }],
+	"import-x/prefer-namespace-import": ["error", { patterns: ["react"] }]
 };
 
 export default styleGuideRules;

package/rules/import-x.js

@@ -1,7 +1,7 @@
 import * as importRuleSets from "./import-x/_exports.js";
 
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/jsdoc.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**
@@ -144,7 +144,7 @@ const jsdocRules = {
 		}
 	],
 	"jsdoc/require-description-complete-sentence": "off",
-	"jsdoc/require-example": "warn",
+	"jsdoc/require-example": "off",
 	"jsdoc/require-file-overview": [
 		"off",
 		{
@@ -163,31 +163,7 @@ const jsdocRules = {
 		}
 	],
 	"jsdoc/require-hyphen-before-param-description": ["warn", "always"],
-	"jsdoc/require-jsdoc": [
-		"warn",
-		{
-			contexts: [
-				`
-					:not(
-						:matches(
-							ObjectExpression > Property,
-							ArrayExpression:has(Literal ~ ArrowFunctionExpression),
-							CallExpression,
-							Property > AssignmentPattern
-						)
-					) > ArrowFunctionExpression
-				`.replaceAll(/\s/gv, "")
-			],
-			require: {
-				ArrowFunctionExpression: false,
-				ClassDeclaration: true,
-				ClassExpression: true,
-				FunctionDeclaration: true,
-				FunctionExpression: false,
-				MethodDefinition: true
-			}
-		}
-	],
+	"jsdoc/require-jsdoc": "off",
 	"jsdoc/require-param": ["warn", { unnamedRootBase: ["options"] }],
 	"jsdoc/require-param-description": [
 		"warn",

package/rules/jsx-a11y.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/layout-and-formatting.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/no-secrets.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 const tolerance = 5;

package/rules/no-unsanitized.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/perfectionist.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**
@@ -9,7 +9,8 @@ const perfectionistRules = {
 	"perfectionist/sort-array-includes": [
 		"error",
 		{
-			groupKind: "literals-first"
+			groupKind: "mixed",
+			groups: ["literal", "spread"]
 		}
 	],
 	"perfectionist/sort-classes": [
@@ -43,8 +44,10 @@ const perfectionistRules = {
 			partitionByNewLine: false
 		}
 	],
-	"perfectionist/sort-enums": ["error"],
-	"perfectionist/sort-exports": ["error"],
+	"perfectionist/sort-decorators": "error",
+	"perfectionist/sort-enums": "error",
+	"perfectionist/sort-exports": "error",
+	"perfectionist/sort-heritage-clauses": "error",
 	"perfectionist/sort-imports": [
 		"error",
 		{
@@ -80,36 +83,25 @@ const perfectionistRules = {
 				"index",
 				"unknown"
 			],
-			internalPattern: ["^@/.*"],
+			internalPattern: ["^@/.+"],
 			newlinesBetween: "always",
 			partitionByNewLine: false,
 			sortSideEffects: true
 		}
 	],
-	"perfectionist/sort-interfaces": ["error"],
-	"perfectionist/sort-intersection-types": ["error"],
+	"perfectionist/sort-interfaces": "error",
+	"perfectionist/sort-intersection-types": "error",
 	"perfectionist/sort-jsx-props": [
 		"error",
 		{
-			groups: [
-				"shorthand",
-				"unknown",
-				"multiline"
-			]
-		}
-	],
-	"perfectionist/sort-maps": [
-		"error",
-		{
-			ignoreCase: false,
-			order: "asc",
-			partitionByNewLine: true,
-			type: "natural"
+			groups: ["shorthand", "unknown", "multiline"]
 		}
 	],
-	"perfectionist/sort-named-exports": ["error"],
+	"perfectionist/sort-maps": "error",
+	"perfectionist/sort-modules": "off",
+	"perfectionist/sort-named-exports": "error",
 	"perfectionist/sort-named-imports": ["error", { partitionByComment: false }],
-	"perfectionist/sort-object-types": ["error"],
+	"perfectionist/sort-object-types": "error",
 	"perfectionist/sort-objects": [
 		"error",
 		{
@@ -118,11 +110,7 @@ const perfectionistRules = {
 				groupBeforeAfter1: ["^before.*"],
 				groupBeforeAfter2: ["^after.*"],
 				groupColor1: ["^r$", "^l$"],
-				groupColor2: [
-					"^g$",
-					"^c$",
-					"^a$"
-				],
+				groupColor2: ["^g$", "^c$", "^a$"],
 				groupColor3: ["^b$", "^h$"],
 				groupMinimumMaximum1: ["^minimum(([A-Z].*)|$)"],
 				groupMinimumMaximum2: ["^maximum(([A-Z].*)|$)"],
@@ -196,9 +184,9 @@ const perfectionistRules = {
 			]
 		}
 	],
-	"perfectionist/sort-sets": ["error"],
-	"perfectionist/sort-switch-case": ["error"],
-	"perfectionist/sort-union-types": ["error"],
+	"perfectionist/sort-sets": "error",
+	"perfectionist/sort-switch-case": "error",
+	"perfectionist/sort-union-types": "error",
 	"perfectionist/sort-variable-declarations": ["error", { partitionByComment: false }]
 };
 

package/rules/possible-problems.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/promise.js

@@ -1,20 +1,34 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**
  * @satisfies {Linter.RulesRecord}
  */
 const promiseRules = {
-	"promise/always-return": "error",
+	"promise/always-return": [
+		"error",
+		{
+			ignoreAssignmentVariable: ["globalThis"],
+			ignoreLastCallback: false
+		}
+	],
 	"promise/avoid-new": "off",
-	"promise/catch-or-return": "error",
+	"promise/catch-or-return": [
+		"error",
+		{
+			allowFinally: false,
+			allowThen: false,
+			allowThenStrict: false,
+			terminationMethod: ["catch"]
+		}
+	],
 	"promise/no-callback-in-promise": "error",
 	"promise/no-multiple-resolved": "error",
 	"promise/no-native": "off",
 	"promise/no-nesting": "error",
 	"promise/no-new-statics": "error",
-	"promise/no-promise-in-callback": "error",
+	"promise/no-promise-in-callback": ["error", { timeoutsErr: false }],
 	"promise/no-return-in-finally": "error",
 	"promise/no-return-wrap": "error",
 	"promise/param-names": [
@@ -26,8 +40,9 @@ const promiseRules = {
 	],
 	"promise/prefer-await-to-callbacks": "error",
 	"promise/prefer-await-to-then": ["error", { strict: true }],
+	"promise/prefer-catch": "error",
 	"promise/spec-only": "error",
-	"promise/valid-params": "error"
+	"promise/valid-params": ["error", { exclude: [] }]
 };
 
 export default promiseRules;

package/rules/react/naming-convention.js

@@ -3,8 +3,6 @@ const namingConventionRules = {
 		"error",
 		{
 			allowAllCaps: false,
-			allowLeadingUnderscore: false,
-			allowNamespace: false,
 			excepts: [],
 			rule: "PascalCase"
 		}

package/rules/react.js

@@ -1,7 +1,7 @@
 import * as reactRuleSets from "./react/_exports.js";
 
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/redos.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 const timeout = 10_000;

package/rules/regexp.js

@@ -1,7 +1,7 @@
 import * as regexpRuleSets from "./regexp/_exports.js";
 
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/security.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/sonarjs/bug-detection.js

@@ -6,7 +6,6 @@ const bugDetectionRules = {
 	"sonarjs/no-identical-conditions": "error",
 	"sonarjs/no-identical-expressions": "error",
 	"sonarjs/no-ignored-return": "error",
-	"sonarjs/no-one-iteration-loop": "error",
 	"sonarjs/no-use-of-empty-return-value": "error",
 	"sonarjs/non-existent-operator": "error"
 };

package/rules/sonarjs.js

@@ -1,7 +1,7 @@
 import * as sonarjsRuleSets from "./sonarjs/_exports.js";
 
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/stylistic/_common/min-items-for-array-newline.js

@@ -1,3 +1,3 @@
-const minItemsForArrayNewline = 3;
+const minItemsForArrayNewline = 4;
 
 export default minItemsForArrayNewline;

package/rules/stylistic/indent.js

@@ -8,7 +8,6 @@ const indentRules = {
 		}
 	],
 	"@stylistic/indent-binary-ops": ["error", "tab"],
-	"@stylistic/jsx-indent": ["error", "tab"],
 	"@stylistic/jsx-indent-props": ["error", "tab"]
 };
 

package/rules/stylistic/jsx.js

@@ -11,7 +11,6 @@ const jsxRules = {
 	],
 	"@stylistic/jsx-one-expression-per-line": ["error", { allow: "single-line" }],
 	"@stylistic/jsx-pascal-case": ["error", { allowNamespace: true }],
-	"@stylistic/jsx-props-no-multi-spaces": "error",
 	"@stylistic/jsx-self-closing-comp": "error",
 	"@stylistic/jsx-sort-props": [
 		"off",

package/rules/stylistic/linebreaks.js

@@ -38,25 +38,13 @@ const lineBreaksRules = {
 			blankLine: "always"
 		},
 		{
-			prev: [
-				"const",
-				"let",
-				"var"
-			],
+			prev: ["const", "let", "var"],
 			next: "*",
 			blankLine: "always"
 		},
 		{
-			prev: [
-				"const",
-				"let",
-				"var"
-			],
-			next: [
-				"const",
-				"let",
-				"var"
-			],
+			prev: ["const", "let", "var"],
+			next: ["const", "let", "var"],
 			blankLine: "any"
 		},
 		{

package/rules/stylistic.js

@@ -1,7 +1,7 @@
 import * as stylisticRuleSets from "./stylistic/_exports.js";
 
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**

package/rules/suggestions.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 const minIdentifierLength = 2;
@@ -53,11 +53,7 @@ const suggestionsRules = {
 	"dot-notation": "error",
 	eqeqeq: "error",
 	"func-name-matching": "error",
-	"func-names": [
-		"error",
-		"as-needed",
-		{ generators: "as-needed" }
-	],
+	"func-names": ["error", "as-needed", { generators: "as-needed" }],
 	"func-style": "error",
 	"grouped-accessor-pairs": "error",
 	"guard-for-in": "off",
@@ -67,11 +63,7 @@ const suggestionsRules = {
 		{
 			min: minIdentifierLength,
 			max: maxIdentifierLength,
-			exceptions: [
-				"x",
-				"y",
-				"z"
-			],
+			exceptions: ["x", "y", "z"],
 			properties: "never"
 		}
 	],
@@ -88,11 +80,7 @@ const suggestionsRules = {
 	"no-console": [
 		"error",
 		{
-			allow: [
-				"warn",
-				"error",
-				"info"
-			]
+			allow: ["warn", "error", "info"]
 		}
 	],
 	"no-continue": "off",

package/rules/tailwindcss.js

@@ -1,5 +1,5 @@
 /**
- * @import { Linter } from "eslint"
+ * @import { Linter } from "eslint";
  */
 
 /**