eslint-config-complete 4.17.0 → 4.18.0

package/dist/src/base/base-eslint.js

@@ -264,7 +264,6 @@ const SUGGESTIONS = {
     "no-param-reassign": [
         "error",
         {
-            props: true,
             ignorePropertyModificationsFor: [
                 "acc", // for reduce accumulators
                 "accumulator", // for reduce accumulators
@@ -278,6 +277,7 @@ const SUGGESTIONS = {
                 "$scope", // for Angular 1 scopes
                 "staticContext", // for ReactRouter context
             ],
+            props: true,
         },
     ],
     /**
@@ -310,12 +310,12 @@ const SUGGESTIONS = {
     "no-restricted-globals": [
         "error",
         {
-            name: "isFinite",
             message: "Use Number.isFinite instead: https://github.com/airbnb/javascript#standard-library--isfinite",
+            name: "isFinite",
         },
         {
-            name: "isNaN",
             message: "Use Number.isNaN instead: https://github.com/airbnb/javascript#standard-library--isnan",
+            name: "isNaN",
         },
         ...confusingBrowserGlobals,
     ],
@@ -328,52 +328,52 @@ const SUGGESTIONS = {
     "no-restricted-properties": [
         "error",
         {
+            message: "arguments.callee is deprecated",
             object: "arguments",
             property: "callee",
-            message: "arguments.callee is deprecated",
         },
         {
+            message: "Please use Number.isFinite instead",
             object: "global",
             property: "isFinite",
-            message: "Please use Number.isFinite instead",
         },
         {
+            message: "Please use Number.isFinite instead",
             object: "self",
             property: "isFinite",
-            message: "Please use Number.isFinite instead",
         },
         {
+            message: "Please use Number.isFinite instead",
             object: "window",
             property: "isFinite",
-            message: "Please use Number.isFinite instead",
         },
         {
+            message: "Please use Number.isNaN instead",
             object: "global",
             property: "isNaN",
-            message: "Please use Number.isNaN instead",
         },
         {
+            message: "Please use Number.isNaN instead",
             object: "self",
             property: "isNaN",
-            message: "Please use Number.isNaN instead",
         },
         {
+            message: "Please use Number.isNaN instead",
             object: "window",
             property: "isNaN",
-            message: "Please use Number.isNaN instead",
         },
         {
-            property: "__defineGetter__",
             message: "Please use Object.defineProperty instead.",
+            property: "__defineGetter__",
         },
         {
-            property: "__defineSetter__",
             message: "Please use Object.defineProperty instead.",
+            property: "__defineSetter__",
         },
         {
+            message: "Use the exponentiation operator (**) instead.",
             object: "Math",
             property: "pow",
-            message: "Use the exponentiation operator (**) instead.",
         },
     ],
     /** Disabled because it is intended for disallowing specific language features per-project. */

package/dist/src/base/base-import-x.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"base-import-x.d.ts","sourceRoot":"","sources":["../../../src/base/base-import-x.js"],"names":[],"mappings":"AA4NA;;;;;;;;;GASG;AACH,2DAsFE"}
+{"version":3,"file":"base-import-x.d.ts","sourceRoot":"","sources":["../../../src/base/base-import-x.js"],"names":[],"mappings":"AA4NA;;;;;;;;;GASG;AACH,2DAqFE"}

package/dist/src/base/base-import-x.js

@@ -189,6 +189,12 @@ export const baseImportX = defineConfig({
     plugins: {
         "import-x": esLintPluginImportX,
     },
+    rules: {
+        ...HELPFUL_WARNINGS,
+        ...MODULE_SYSTEMS,
+        ...STATIC_ANALYSIS,
+        ...STYLE_GUIDE,
+    },
     // Beyond just specifying the plugin, additional configuration is necessary to make the plugin
     // work properly with TypeScript:
     // - First, the "eslint-import-resolver-typescript" package needs to be installed, or else an
@@ -211,12 +217,6 @@ export const baseImportX = defineConfig({
             typescript: true,
         },
     },
-    rules: {
-        ...HELPFUL_WARNINGS,
-        ...MODULE_SYSTEMS,
-        ...STATIC_ANALYSIS,
-        ...STYLE_GUIDE,
-    },
 }, 
 // Disable some TypeScript-specific rules in JavaScript files.
 {

package/dist/src/base/base-jsdoc.js

@@ -139,8 +139,8 @@ export const baseJSDoc = defineConfig({
             {
                 contexts: [
                     {
-                        context: "FunctionDeclaration",
                         comment: 'JsdocBlock:has(JsdocTag[tag="param"])',
+                        context: "FunctionDeclaration",
                     },
                 ],
             },

package/dist/src/base/base-stylistic.js

@@ -20,8 +20,8 @@ export const baseStylistic = defineConfig({
             "error",
             {
                 blankLine: "always",
-                prev: "function",
                 next: "function",
+                prev: "function",
             },
         ],
         /**
@@ -33,8 +33,8 @@ export const baseStylistic = defineConfig({
             "error",
             "double",
             {
-                avoidEscape: true,
                 allowTemplateLiterals: "never",
+                avoidEscape: true,
             },
         ],
         /**

package/dist/src/base/base-typescript-eslint.js

@@ -5,9 +5,10 @@ import tseslint from "typescript-eslint";
  * https://typescript-eslint.io/rules/
  */
 export const baseTypeScriptESLint = defineConfig({
-    plugins: {
-        "@typescript-eslint": tseslint.plugin,
-    },
+    // Rules that require type information will throw an error on ".json" files. (This is needed
+    // when using `eslint-plugin-package-json`. Even though this config does not currently use the
+    // plugin, we include it here defensively.)
+    ignores: ["**/*.json", "**/*.jsonc", "**/*.code-workspace"],
     // We need to provide some special configuration to ESLint in order for it to parse TypeScript
     // files. From:
     // https://typescript-eslint.io/packages/typescript-eslint/#advanced-usage
@@ -34,6 +35,9 @@ export const baseTypeScriptESLint = defineConfig({
             },
         },
     },
+    plugins: {
+        "@typescript-eslint": tseslint.plugin,
+    },
     rules: {
         "@typescript-eslint/adjacent-overload-signatures": "error",
         /**
@@ -99,23 +103,23 @@ export const baseTypeScriptESLint = defineConfig({
             // Allow camelCase variables (23.2), PascalCase variables (23.8), and UPPER_CASE variables
             // (23.10).
             {
-                selector: "variable",
                 format: ["camelCase", "PascalCase", "UPPER_CASE"],
                 leadingUnderscore: "allow",
+                selector: "variable",
             },
             // Allow camelCase functions (23.2), and PascalCase functions (23.8).
             {
-                selector: "function",
                 format: ["camelCase", "PascalCase"],
                 leadingUnderscore: "allow",
+                selector: "function",
             },
             // Airbnb recommends PascalCase for classes (23.3), and although Airbnb does not make
             // TypeScript recommendations, we are assuming this rule would similarly apply to anything
             // "type like", including interfaces, type aliases, and enums.
             {
-                selector: "typeLike",
                 format: ["PascalCase"],
                 leadingUnderscore: "allow",
+                selector: "typeLike",
             },
         ],
         "@typescript-eslint/no-array-constructor": "error",
@@ -299,11 +303,11 @@ export const baseTypeScriptESLint = defineConfig({
         "@typescript-eslint/prefer-destructuring": [
             "error",
             {
-                VariableDeclarator: {
+                AssignmentExpression: {
                     array: false,
                     object: true,
                 },
-                AssignmentExpression: {
+                VariableDeclarator: {
                     array: false,
                     object: true,
                 },
@@ -374,15 +378,15 @@ export const baseTypeScriptESLint = defineConfig({
         "@typescript-eslint/strict-boolean-expressions": [
             "error",
             {
-                allowString: false,
-                allowNumber: false,
-                allowNullableObject: true,
+                allowAny: false,
                 allowNullableBoolean: false,
-                allowNullableString: false,
-                allowNullableNumber: false,
                 allowNullableEnum: false,
-                allowAny: false,
+                allowNullableNumber: false,
+                allowNullableObject: true,
+                allowNullableString: false,
+                allowNumber: false,
                 allowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing: false,
+                allowString: false,
             },
         ],
         "@typescript-eslint/strict-void-return": "error",
@@ -396,8 +400,8 @@ export const baseTypeScriptESLint = defineConfig({
             "error",
             {
                 allowDefaultCaseForExhaustiveSwitch: false,
-                requireDefaultForNonUnion: true,
                 considerDefaultExhaustiveForUnions: true,
+                requireDefaultForNonUnion: true,
             },
         ],
         "@typescript-eslint/triple-slash-reference": "error",
@@ -411,10 +415,6 @@ export const baseTypeScriptESLint = defineConfig({
         "@typescript-eslint/unified-signatures": "error",
         "@typescript-eslint/use-unknown-in-catch-callback-variable": "error",
     },
-    // Rules that require type information will throw an error on ".json" files. (This is needed
-    // when using `eslint-plugin-package-json`. Even though this config does not currently use the
-    // plugin, we include it here defensively.)
-    ignores: ["**/*.json", "**/*.jsonc", "**/*.code-workspace"],
 }, 
 // Enable linting on TypeScript file extensions.
 { files: ["**/*.ts", "**/*.tsx", "**/*.mts", "**/*.cts"] }, 

package/dist/src/base/base-unicorn.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"base-unicorn.d.ts","sourceRoot":"","sources":["../../../src/base/base-unicorn.js"],"names":[],"mappings":"AAGA;;;GAGG;AACH,2DAgWG"}
+{"version":3,"file":"base-unicorn.d.ts","sourceRoot":"","sources":["../../../src/base/base-unicorn.js"],"names":[],"mappings":"AAGA;;;GAGG;AACH,2DAsYG"}

package/dist/src/base/base-unicorn.js

@@ -18,6 +18,7 @@ export const baseUnicorn = defineConfig({
         "unicorn/consistent-boolean-name": "off",
         "unicorn/consistent-class-member-order": "error",
         "unicorn/consistent-compound-words": "error",
+        "unicorn/consistent-conditional-object-spread": "error",
         "unicorn/consistent-date-clone": "error",
         "unicorn/consistent-destructuring": "error",
         "unicorn/consistent-empty-array-spread": "error",
@@ -29,6 +30,7 @@ export const baseUnicorn = defineConfig({
         "unicorn/consistent-optional-chaining": "error",
         "unicorn/consistent-template-literal-escape": "error",
         "unicorn/custom-error-definition": "error",
+        "unicorn/default-export-style": "error",
         "unicorn/dom-node-dataset": "error",
         "unicorn/empty-brace-spaces": "off", // eslint-config-prettier
         "unicorn/error-message": "error",
@@ -43,6 +45,8 @@ export const baseUnicorn = defineConfig({
         "unicorn/isolated-functions": "error",
         "unicorn/logical-assignment-operators": "error",
         "unicorn/max-nested-calls": "error",
+        /** Disabled since it is common to use the variable name of "i". */
+        "unicorn/name-replacements": "off",
         "unicorn/new-for-builtins": "error",
         /**
          * Disabled because if a line breaks three or more ESLint rules, then it is useful to use a
@@ -50,31 +54,42 @@ export const baseUnicorn = defineConfig({
          */
         "unicorn/no-abusive-eslint-disable": "off",
         "unicorn/no-accessor-recursion": "error",
+        "unicorn/no-accidental-bitwise-operator": "error",
         "unicorn/no-anonymous-default-export": "error",
         /** Disabled since it is not helpful when using TypeScript. */
         "unicorn/no-array-callback-reference": "off",
+        "unicorn/no-array-concat-in-loop": "error",
         "unicorn/no-array-fill-with-reference-type": "error",
         "unicorn/no-array-from-fill": "error",
+        "unicorn/no-array-front-mutation": "error",
         "unicorn/no-array-method-this-argument": "error",
         "unicorn/no-array-reduce": "error",
         "unicorn/no-array-reverse": "error",
         "unicorn/no-array-sort": "error",
+        "unicorn/no-array-sort-for-min-max": "error",
         "unicorn/no-array-splice": "error",
         /** Disabled because this goes against the standard JSDoc format. */
         "unicorn/no-asterisk-prefix-in-documentation-comments": "off",
         "unicorn/no-await-expression-member": "error",
         "unicorn/no-await-in-promise-methods": "error",
         "unicorn/no-blob-to-file": "error",
+        "unicorn/no-boolean-sort-comparator": "error",
         /** Disabled since simple double nested loops are common. */
         "unicorn/no-break-in-nested-loop": "off",
         "unicorn/no-canvas-to-image": "error",
+        "unicorn/no-chained-comparison": "error",
+        "unicorn/no-collection-bracket-access": "error",
         /** Disabled since it is unnecessary with TypeScript objects that are properly typed. */
         "unicorn/no-computed-property-existence-check": "off",
         "unicorn/no-confusing-array-splice": "error",
         "unicorn/no-confusing-array-with": "error",
         "unicorn/no-console-spaces": "error",
+        "unicorn/no-constant-zero-expression": "error",
         "unicorn/no-declarations-before-early-exit": "error",
         "unicorn/no-document-cookie": "error",
+        "unicorn/no-double-comparison": "error",
+        "unicorn/no-duplicate-if-branches": "error",
+        "unicorn/no-duplicate-logical-operands": "error",
         "unicorn/no-duplicate-loops": "error",
         "unicorn/no-duplicate-set-values": "error",
         "unicorn/no-empty-file": "error",
@@ -84,10 +99,12 @@ export const baseUnicorn = defineConfig({
         "unicorn/no-for-loop": "error",
         "unicorn/no-global-object-property-assignment": "error",
         "unicorn/no-immediate-mutation": "error",
+        "unicorn/no-impossible-length-comparison": "error",
         "unicorn/no-incorrect-query-selector": "error",
         "unicorn/no-incorrect-template-string-interpolation": "error",
         "unicorn/no-instanceof-builtins": "error",
         "unicorn/no-invalid-argument-count": "error",
+        "unicorn/no-invalid-character-comparison": "error",
         "unicorn/no-invalid-fetch-options": "error",
         "unicorn/no-invalid-file-input-accept": "error",
         "unicorn/no-invalid-remove-event-listener": "error",
@@ -95,10 +112,12 @@ export const baseUnicorn = defineConfig({
         "unicorn/no-keyword-prefix": "off",
         "unicorn/no-late-current-target-access": "error",
         "unicorn/no-lonely-if": "error",
+        "unicorn/no-loop-iterable-mutation": "error",
         "unicorn/no-magic-array-flat-depth": "error",
         /** Superseded by the `complete/format-line-comments` rule. */
         "unicorn/no-manually-wrapped-comments": "off",
         "unicorn/no-mismatched-map-key": "error",
+        "unicorn/no-misrefactored-assignment": "error",
         "unicorn/no-named-default": "error",
         "unicorn/no-negated-array-predicate": "error",
         "unicorn/no-negated-comparison": "error",
@@ -108,6 +127,7 @@ export const baseUnicorn = defineConfig({
         "unicorn/no-new-array": "error",
         "unicorn/no-new-buffer": "error",
         "unicorn/no-non-function-verb-prefix": "error",
+        "unicorn/no-nonstandard-builtin-properties": "error",
         "unicorn/no-null": "error",
         "unicorn/no-object-as-default-parameter": "error",
         "unicorn/no-object-methods-with-collections": "error",
@@ -119,6 +139,7 @@ export const baseUnicorn = defineConfig({
         "unicorn/no-process-exit": "off",
         "unicorn/no-redundant-comparison": "error",
         "unicorn/no-return-array-push": "error",
+        "unicorn/no-selector-as-dom-name": "error",
         "unicorn/no-single-promise-in-promise-methods": "error",
         "unicorn/no-static-only-class": "error",
         "unicorn/no-subtraction-comparison": "error",
@@ -134,6 +155,7 @@ export const baseUnicorn = defineConfig({
         "unicorn/no-unnecessary-array-flat-depth": "error",
         "unicorn/no-unnecessary-array-splice-count": "error",
         "unicorn/no-unnecessary-await": "error",
+        "unicorn/no-unnecessary-boolean-comparison": "error",
         "unicorn/no-unnecessary-global-this": "error",
         "unicorn/no-unnecessary-nested-ternary": "error",
         "unicorn/no-unnecessary-polyfills": "error",
@@ -154,13 +176,16 @@ export const baseUnicorn = defineConfig({
         "unicorn/no-useless-boolean-cast": "error",
         "unicorn/no-useless-coercion": "error",
         "unicorn/no-useless-collection-argument": "error",
+        "unicorn/no-useless-compound-assignment": "error",
         "unicorn/no-useless-concat": "error",
         "unicorn/no-useless-continue": "error",
+        "unicorn/no-useless-delete-check": "error",
         "unicorn/no-useless-else": "error",
         "unicorn/no-useless-error-capture-stack-trace": "error",
         "unicorn/no-useless-fallback-in-spread": "error",
         "unicorn/no-useless-iterator-to-array": "error",
         "unicorn/no-useless-length-check": "error",
+        "unicorn/no-useless-logical-operand": "error",
         "unicorn/no-useless-override": "error",
         "unicorn/no-useless-promise-resolve-reject": "error",
         "unicorn/no-useless-recursion": "error",
@@ -169,6 +194,7 @@ export const baseUnicorn = defineConfig({
         "unicorn/no-useless-template-literals": "error",
         /** Disabled since it does not work properly with TypeScript. */
         "unicorn/no-useless-undefined": "off",
+        "unicorn/no-xor-as-exponentiation": "error",
         "unicorn/no-zero-fractions": "error",
         "unicorn/number-literal-case": "off", // eslint-config-prettier
         "unicorn/numeric-separators-style": "error",
@@ -178,8 +204,10 @@ export const baseUnicorn = defineConfig({
         "unicorn/prefer-array-find": "error",
         "unicorn/prefer-array-flat": "error",
         "unicorn/prefer-array-flat-map": "error",
+        "unicorn/prefer-array-from-async": "error",
         "unicorn/prefer-array-from-map": "error",
         "unicorn/prefer-array-index-of": "error",
+        "unicorn/prefer-array-iterable-methods": "error",
         "unicorn/prefer-array-last-methods": "error",
         "unicorn/prefer-array-slice": "error",
         "unicorn/prefer-array-some": "error",
@@ -187,9 +215,11 @@ export const baseUnicorn = defineConfig({
         "unicorn/prefer-await": "error",
         "unicorn/prefer-bigint-literals": "error",
         "unicorn/prefer-blob-reading-methods": "error",
+        "unicorn/prefer-boolean-return": "error",
         "unicorn/prefer-class-fields": "error",
         "unicorn/prefer-classlist-toggle": "error",
         "unicorn/prefer-code-point": "error",
+        "unicorn/prefer-continue": "error",
         "unicorn/prefer-date-now": "error",
         "unicorn/prefer-default-parameters": "error",
         "unicorn/prefer-direct-iteration": "error",
@@ -202,10 +232,12 @@ export const baseUnicorn = defineConfig({
         "unicorn/prefer-else-if": "error",
         "unicorn/prefer-event-target": "error",
         "unicorn/prefer-export-from": "error",
+        "unicorn/prefer-flat-math-min-max": "error",
         "unicorn/prefer-get-or-insert-computed": "error",
         "unicorn/prefer-global-number-constants": "error",
         "unicorn/prefer-global-this": "error",
         "unicorn/prefer-has-check": "error",
+        "unicorn/prefer-hoisting-branch-code": "error",
         "unicorn/prefer-https": "error",
         "unicorn/prefer-identifier-import-export-specifiers": "error",
         "unicorn/prefer-import-meta-properties": "error",
@@ -225,6 +257,7 @@ export const baseUnicorn = defineConfig({
         "unicorn/prefer-logical-operator-over-ternary": "error",
         "unicorn/prefer-map-from-entries": "error",
         "unicorn/prefer-math-abs": "error",
+        "unicorn/prefer-math-constants": "error",
         "unicorn/prefer-math-min-max": "error",
         "unicorn/prefer-math-trunc": "error",
         "unicorn/prefer-minimal-ternary": "error",
@@ -244,10 +277,12 @@ export const baseUnicorn = defineConfig({
         "unicorn/prefer-optional-catch-binding": "error",
         "unicorn/prefer-path2d": "error",
         "unicorn/prefer-private-class-fields": "error",
+        "unicorn/prefer-promise-with-resolvers": "error",
         "unicorn/prefer-prototype-methods": "error",
         "unicorn/prefer-query-selector": "error",
         "unicorn/prefer-queue-microtask": "error",
         "unicorn/prefer-reflect-apply": "error",
+        "unicorn/prefer-regexp-escape": "error",
         "unicorn/prefer-regexp-test": "error",
         "unicorn/prefer-response-static-json": "error",
         "unicorn/prefer-scoped-selector": "error",
@@ -259,6 +294,7 @@ export const baseUnicorn = defineConfig({
         "unicorn/prefer-single-array-predicate": "error",
         "unicorn/prefer-single-call": "error",
         "unicorn/prefer-single-object-destructuring": "error",
+        "unicorn/prefer-single-replace": "error",
         "unicorn/prefer-smaller-scope": "error",
         "unicorn/prefer-split-limit": "error",
         "unicorn/prefer-spread": "error",
@@ -283,10 +319,11 @@ export const baseUnicorn = defineConfig({
         "unicorn/prefer-type-error": "error",
         "unicorn/prefer-type-literal-last": "error",
         "unicorn/prefer-uint8array-base64": "error",
+        "unicorn/prefer-unary-minus": "error",
         "unicorn/prefer-unicode-code-point-escapes": "error",
+        "unicorn/prefer-url-can-parse": "error",
         "unicorn/prefer-url-href": "error",
-        /** Disabled since it is common to use the variable name of "i". */
-        "unicorn/prevent-abbreviations": "off",
+        "unicorn/prefer-while-loop-condition": "error",
         "unicorn/relative-url-style": "error",
         "unicorn/require-array-join-separator": "error",
         /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-complete",
-  "version": "4.17.0",
+  "version": "4.18.0",
   "description": "A sharable ESLint config for TypeScript projects.",
   "keywords": [
     "eslint",
@@ -35,24 +35,24 @@
     "@stylistic/eslint-plugin": "5.10.0",
     "confusing-browser-globals": "1.0.11",
     "eslint-import-resolver-typescript": "4.4.5",
-    "eslint-plugin-complete": "1.9.0",
+    "eslint-plugin-complete": "1.10.0",
     "eslint-plugin-eslint-plugin": "7.4.0",
     "eslint-plugin-import-x": "4.16.2",
-    "eslint-plugin-jsdoc": "63.0.6",
+    "eslint-plugin-jsdoc": "63.0.7",
     "eslint-plugin-n": "18.1.0",
     "eslint-plugin-perfectionist": "5.9.1",
     "eslint-plugin-regexp": "3.1.0",
-    "eslint-plugin-unicorn": "67.0.0",
+    "eslint-plugin-unicorn": "68.0.0",
     "typescript-eslint": "8.61.1"
   },
   "devDependencies": {
     "@eslint/js": "10.0.1",
     "@types/confusing-browser-globals": "1.0.3",
-    "@types/node": "25.9.3",
+    "@types/node": "26.0.0",
     "@typescript-eslint/typescript-estree": "8.61.1",
     "@typescript-eslint/utils": "8.61.1",
     "complete-common": "2.29.0",
-    "complete-node": "17.0.2",
+    "complete-node": "18.0.0",
     "eslint-config-prettier": "10.1.8",
     "typescript": "6.0.3"
   },