@pumpn/eslint-config 15.6.1 → 16.1.1

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": "15.6.1",
+	"version": "16.1.1",
 	"description": "standard eslint config for pumpn code projects",
 	"main": "config.js",
 	"author": "Nano Miratus <nanomiratus@gmail.com>",
@@ -17,31 +17,30 @@
 		"test": "echo 'no tests yet'"
 	},
 	"devDependencies": {
-		"eslint": "^9.19.0"
+		"eslint": "^9.33.0"
 	},
 	"dependencies": {
-		"@eslint-community/eslint-plugin-eslint-comments": "^4.4.0",
-		"@eslint-react/eslint-plugin": "^1.15.0",
-		"@stylistic/eslint-plugin": "^3.0.1",
-		"@typescript-eslint/parser": "^8.11.0",
-		"eslint-plugin-depend": "^0.12.0",
-		"eslint-plugin-disable-autofix": "^5.0.1",
-		"eslint-plugin-func-params-args": "^4.0.0",
-		"eslint-plugin-import-x": "^4.3.1",
-		"eslint-plugin-jsdoc": "^50.4.3",
+		"@eslint-community/eslint-plugin-eslint-comments": "^4.5.0",
+		"@eslint-react/eslint-plugin": "^1.52.6",
+		"@stylistic/eslint-plugin": "^5.2.3",
+		"@typescript-eslint/parser": "^8.40.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.1.0",
 		"eslint-plugin-no-secrets": "^2.2.1",
-		"eslint-plugin-perfectionist": "^4.8.0",
-		"eslint-plugin-promise": "^7.1.0",
-		"eslint-plugin-react": "^7.35.0",
-		"eslint-plugin-redos": "^4.4.5",
-		"eslint-plugin-regexp": "^2.6.0",
+		"eslint-plugin-perfectionist": "^4.15.0",
+		"eslint-plugin-promise": "^7.2.1",
+		"eslint-plugin-redos": "^4.5.0",
+		"eslint-plugin-regexp": "^2.10.0",
 		"eslint-plugin-security": "^3.0.1",
-		"eslint-plugin-sonarjs": "^3.0.1",
-		"eslint-plugin-tailwindcss": "^3.18.0",
-		"eslint-plugin-unicorn": "^56.0.0",
-		"globals": "^15.14.0"
+		"eslint-plugin-sonarjs": "^3.0.4",
+		"eslint-plugin-tailwindcss": "^4.0.0-beta.0",
+		"eslint-plugin-unicorn": "^60.0.0",
+		"globals": "^16.3.0"
 	},
 	"peerDependencies": {
-		"eslint": "^9.19.0"
+		"eslint": "^9.33.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";
@@ -8,16 +8,15 @@
 import eslintComments from "@eslint-community/eslint-plugin-eslint-comments";
 import react from "@eslint-react/eslint-plugin";
 import stylistic from "@stylistic/eslint-plugin";
+import command from "eslint-plugin-command";
 import depend from "eslint-plugin-depend";
-import disableAutofix from "eslint-plugin-disable-autofix";
-import funcParamsArgs from "eslint-plugin-func-params-args";
 import importX from "eslint-plugin-import-x";
 import jsdoc from "eslint-plugin-jsdoc";
 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";
@@ -39,23 +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,
-	"disable-autofix": disableAutofix,
-	"func-params-args": funcParamsArgs,
+	// 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/_exports.js

@@ -1,6 +1,5 @@
 export { default as dependRules } from "./depend.js";
 export { default as eslintCommentsRules } from "./eslint-comments.js";
-export { default as funcParamsArgsRules } from "./func-params-args.js";
 // export { default as githubRules } from "./github.js";
 export { default as importXRules } from "./import-x.js";
 export { default as jsdocRules } from "./jsdoc.js";
@@ -14,6 +13,7 @@ export { default as perfectionistRules } from "./perfectionist.js";
 export { default as possibleProblemsRules } from "./possible-problems.js";
 export { default as reactRules } from "./react.js";
 // export { default as redosRules } from "./redos.js";
+export { default as commandRules } from "./command.js";
 export { default as regexpRules } from "./regexp.js";
 export { default as securityRules } from "./security.js";
 export { default as sonarjsRules } from "./sonarjs.js";

package/rules/command.js

@@ -0,0 +1,12 @@
+/**
+ * @import { Linter } from "eslint";
+ */
+
+/**
+ * @satisfies {Linter.RulesRecord}
+ */
+const commandRules = {
+	"command/command": "error"
+};
+
+export default commandRules;

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": [
@@ -31,12 +38,7 @@ const styleGuideRules = {
 	"import-x/no-duplicates": "error",
 	"import-x/no-named-default": "error",
 	"import-x/no-named-export": "off",
-	"import-x/no-namespace": [
-		"error",
-		{
-			ignore: ["{./**/,**/}_exports.?(*.)js", "@typescript-eslint/parser"]
-		}
-	],
+	"import-x/no-namespace": "off",
 	"import-x/no-unassigned-import": "error",
 	"import-x/order": [
 		"off",
@@ -80,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: true,
-				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": [
@@ -39,11 +40,14 @@ const perfectionistRules = {
 				["private-method", "private-function-property"],
 				"unknown"
 			],
-			partitionByComment: String.raw`\/\/`
+			partitionByComment: String.raw`\/\/`,
+			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",
 		{
@@ -79,28 +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"
-			]
+			groups: ["shorthand", "unknown", "multiline"]
 		}
 	],
-	"perfectionist/sort-maps": ["error"],
-	"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",
 		{
@@ -109,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].*)|$)"],
@@ -187,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.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.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/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";
  */
 
 /**