@perfective/eslint-config 0.25.1 → 0.26.1

package/LICENSE

@@ -1,6 +1,6 @@
 The MIT License
 
-Copyright (c) 2020-2022 Andrey Mikheychik (https://github.com/amikheychik)
+Copyright (c) 2020-2024 Andrey Mikheychik (https://github.com/amikheychik)
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/README.adoc

@@ -20,10 +20,11 @@ and https://eslint.style[ESLint Stylistic] plugin rules,
 * `link:https://github.com/dangreenisrael/eslint-plugin-jest-formatting[eslint-plugin-jest-formatting]`
 _(optional, required if `eslint-plugin-jest` is installed)_;
 * `link:https://github.com/gajus/eslint-plugin-jsdoc[eslint-plugin-jsdoc]`;
-* `link:https://github.com/mysticatea/eslint-plugin-node[eslint-plugin-node]`;
+* `link:https://github.com/eslint-community/eslint-plugin-n[eslint-plugin-n]`;
 * `link:https://github.com/TristonJ/eslint-plugin-prefer-arrow[eslint-plugin-prefer-arrow]`;
 * `link:https://github.com/eslint-community/eslint-plugin-promise[eslint-plugin-promise]`;
 * `link:https://github.com/cartant/eslint-plugin-rxjs[eslint-plugin-rxjs]` _(optional)_;
+* `link:https://github.com/eslint-community/eslint-plugin-security[eslint-plugin-security]`;
 * `link:https://github.com/lydell/eslint-plugin-simple-import-sort[eslint-plugin-simple-import-sort]`;
 * `link:https://github.com/SonarSource/eslint-plugin-sonarjs[eslint-plugin-sonarjs]`;
 * `link:https://github.com/testing-library/eslint-plugin-testing-library[eslint-plugin-testing-library]` _(optional)_;
@@ -55,9 +56,10 @@ npm install --save-dev \
     eslint-plugin-eslint-comments \
     eslint-plugin-import \
     eslint-plugin-jsdoc \
-    eslint-plugin-node \
+    eslint-plugin-n \
     eslint-plugin-prefer-arrow \
     eslint-plugin-promise \
+    eslint-plugin-security \
     eslint-plugin-simple-import-sort \
     eslint-plugin-sonarjs \
     eslint-plugin-unicorn \

package/README.md

@@ -20,10 +20,11 @@ and [ESlint Stylistic](https://eslint.style) plugin rules,
 -   [`eslint-plugin-jest-formatting`](https://github.com/dangreenisrael/eslint-plugin-jest-formatting)
     _(optional, required if `eslint-plugin-jest` is installed)_;
 -   [`eslint-plugin-jsdoc`](https://github.com/gajus/eslint-plugin-jsdoc);
--   [`eslint-plugin-node`](https://github.com/mysticatea/eslint-plugin-node);
+-   [`eslint-plugin-n`](https://github.com/eslint-community/eslint-plugin-n);
 -   [`eslint-plugin-prefer-arrow`](https://github.com/TristonJ/eslint-plugin-prefer-arrow);
 -   [`eslint-plugin-promise`](https://github.com/eslint-community/eslint-plugin-promise);
 -   [`eslint-plugin-rxjs`](https://github.com/cartant/eslint-plugin-rxjs) _(optional)_;
+-   [`eslint-plugin-security`](https://github.com/eslint-community/eslint-plugin-security);
 -   [`eslint-plugin-simple-import-sort`](https://github.com/lydell/eslint-plugin-simple-import-sort);
 -   [`eslint-plugin-sonarjs`](https://github.com/SonarSource/eslint-plugin-sonarjs);
 -   [`eslint-plugin-testing-library`](https://github.com/testing-library/eslint-plugin-testing-library) _(optional)_;
@@ -53,9 +54,10 @@ from issues that will be fixed automatically.
         eslint-plugin-eslint-comments \
         eslint-plugin-import \
         eslint-plugin-jsdoc \
-        eslint-plugin-node \
+        eslint-plugin-n \
         eslint-plugin-prefer-arrow \
         eslint-plugin-promise \
+        eslint-plugin-security \
         eslint-plugin-simple-import-sort \
         eslint-plugin-sonarjs \
         eslint-plugin-unicorn \

package/config/node.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.hasNodeModule = void 0;
+exports.hasNodeModule = hasNodeModule;
 function hasNodeModule(module) {
     try {
         return Boolean(require.resolve(module));
@@ -9,4 +9,3 @@ function hasNodeModule(module) {
         return false;
     }
 }
-exports.hasNodeModule = hasNodeModule;

package/config/plugin.js

@@ -1,6 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.hasEslintPlugin = exports.optionalRule = exports.optionalRules = exports.optionalOverrides = exports.optionalConfig = void 0;
+exports.optionalConfig = optionalConfig;
+exports.optionalOverrides = optionalOverrides;
+exports.optionalRules = optionalRules;
+exports.optionalRule = optionalRule;
+exports.hasEslintPlugin = hasEslintPlugin;
 const node_1 = require("./node");
 function optionalConfig(plugin, rules) {
     if (hasEslintPlugin(plugin)) {
@@ -11,21 +15,18 @@ function optionalConfig(plugin, rules) {
     }
     return {};
 }
-exports.optionalConfig = optionalConfig;
 function optionalOverrides(plugin, overrides) {
     if (hasEslintPlugin(plugin)) {
         return overrides;
     }
     return null;
 }
-exports.optionalOverrides = optionalOverrides;
 function optionalRules(plugin) {
     if (hasEslintPlugin(plugin)) {
         return `./rules/${plugin.replace(/^@/u, '')}`;
     }
     return null;
 }
-exports.optionalRules = optionalRules;
 function optionalRule(rule, config) {
     const plugin = pluginOfRule(rule);
     if (plugin === null) {
@@ -38,14 +39,12 @@ function optionalRule(rule, config) {
     }
     return {};
 }
-exports.optionalRule = optionalRule;
 function hasEslintPlugin(plugin) {
     if (plugin.startsWith('@')) {
         return (0, node_1.hasNodeModule)(`${plugin}/eslint-plugin`);
     }
     return (0, node_1.hasNodeModule)(`eslint-plugin-${plugin}`);
 }
-exports.hasEslintPlugin = hasEslintPlugin;
 function pluginOfRule(rule) {
     const parts = rule.split('/');
     if (parts.length > 1) {

package/index.js

@@ -16,9 +16,9 @@ module.exports = {
                 './rules/import',
                 './rules/jsdoc',
                 './rules/n',
-                './rules/node',
                 './rules/prefer-arrow',
                 './rules/promise',
+                './rules/security',
                 './rules/simple-import-sort',
                 './rules/sonarjs',
                 './rules/stylistic/js',

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@perfective/eslint-config",
-    "version": "0.25.1",
+    "version": "0.26.1",
     "description": "ESLint shareable rules configuration",
     "keywords": ["code quality", "code standard", "code style", "eslint", "eslint config", "lint", "perfective", "tslint", "tslint config", "typescript"],
     "author": "Andrey Mikheychik <a.mikheychik@gmail.com>",
@@ -12,10 +12,10 @@
     "license": "MIT",
     "peerDependencies": {
         "@babel/eslint-parser": "^7.24.7",
-        "@stylistic/eslint-plugin": "^2.1.0",
-        "@typescript-eslint/eslint-plugin": "^7.12.0",
+        "@stylistic/eslint-plugin": "^2.3.0",
+        "@typescript-eslint/eslint-plugin": "^7.14.1",
         "@typescript-eslint/eslint-plugin-tslint": "^7.0.2",
-        "@typescript-eslint/parser": "^7.12.0",
+        "@typescript-eslint/parser": "^7.14.1",
         "eslint": "^8.57.0",
         "eslint-import-resolver-typescript": "^3.6.1",
         "eslint-plugin-array-func": "^4.0.0",
@@ -26,16 +26,16 @@
         "eslint-plugin-jest": "^28.6.0",
         "eslint-plugin-jest-dom": "^5.4.0",
         "eslint-plugin-jest-formatting": "^3.1.0",
-        "eslint-plugin-jsdoc": "^48.2.9",
-        "eslint-plugin-n": "^17.8.1",
-        "eslint-plugin-node": "^11.1.0",
+        "eslint-plugin-jsdoc": "^48.5.0",
+        "eslint-plugin-n": "^17.9.0",
         "eslint-plugin-prefer-arrow": "^1.2.3",
         "eslint-plugin-promise": "^6.2.0",
         "eslint-plugin-rxjs": "^5.0.3",
+        "eslint-plugin-security": "^3.0.1",
         "eslint-plugin-simple-import-sort": "^12.1.0",
         "eslint-plugin-sonarjs": "^1.0.3",
         "eslint-plugin-testing-library": "^6.2.2",
-        "eslint-plugin-unicorn": "^53.0.0",
+        "eslint-plugin-unicorn": "^54.0.0",
         "tslint": "^6.1.3"
     },
     "peerDependenciesMeta": {

package/rules/import/rules/no-extraneous-dependencies.js

@@ -1,25 +1,24 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.cypressImportNoExtraneousDependencies = exports.jestImportNoExtraneousDependencies = exports.jsImportNoExtraneousDependencies = exports.importNoExtraneousDependencies = void 0;
+exports.importNoExtraneousDependencies = importNoExtraneousDependencies;
+exports.jsImportNoExtraneousDependencies = jsImportNoExtraneousDependencies;
+exports.jestImportNoExtraneousDependencies = jestImportNoExtraneousDependencies;
+exports.cypressImportNoExtraneousDependencies = cypressImportNoExtraneousDependencies;
 function importNoExtraneousDependencies(overrides = {}) {
     return Object.assign({ bundledDependencies: true, devDependencies: false, optionalDependencies: false, peerDependencies: false }, overrides);
 }
-exports.importNoExtraneousDependencies = importNoExtraneousDependencies;
 function jsImportNoExtraneousDependencies(devDependencies = ['**/.*.js', '**/*.config.js', '**/gulpfile.js']) {
     return importNoExtraneousDependencies({
         devDependencies,
     });
 }
-exports.jsImportNoExtraneousDependencies = jsImportNoExtraneousDependencies;
 function jestImportNoExtraneousDependencies(devDependencies = ['**/*.@(spec|test).[jt]s?(x)']) {
     return importNoExtraneousDependencies({
         devDependencies,
     });
 }
-exports.jestImportNoExtraneousDependencies = jestImportNoExtraneousDependencies;
 function cypressImportNoExtraneousDependencies(devDependencies = ['cypress/**/*.[jt]s']) {
     return importNoExtraneousDependencies({
         devDependencies,
     });
 }
-exports.cypressImportNoExtraneousDependencies = cypressImportNoExtraneousDependencies;

package/rules/n/index.js

@@ -29,9 +29,12 @@ module.exports = {
                 allowAtRootLevel: false,
             }],
         'n/no-unpublished-bin': 'error',
-        'n/no-unpublished-import': 'error',
+        'n/no-unpublished-import': ['error', {
+                ignorePrivate: false,
+            }],
         'n/no-unpublished-require': ['error', {
                 allowModules: [],
+                ignorePrivate: false,
                 tryExtensions: ['.js', '.json', '.node'],
             }],
         'n/no-unsupported-features/es-builtins': ['error', {

package/rules/security/index.js

@@ -0,0 +1,22 @@
+"use strict";
+module.exports = {
+    plugins: [
+        'security',
+    ],
+    rules: {
+        'security/detect-bidi-characters': 'error',
+        'security/detect-buffer-noassert': 'error',
+        'security/detect-child-process': 'error',
+        'security/detect-disable-mustache-escape': 'error',
+        'security/detect-eval-with-expression': 'error',
+        'security/detect-new-buffer': 'error',
+        'security/detect-no-csrf-before-method-override': 'error',
+        'security/detect-non-literal-fs-filename': 'error',
+        'security/detect-non-literal-regexp': 'error',
+        'security/detect-non-literal-require': 'error',
+        'security/detect-object-injection': 'error',
+        'security/detect-possible-timing-attacks': 'error',
+        'security/detect-pseudoRandomBytes': 'error',
+        'security/detect-unsafe-regex': 'error',
+    },
+};

package/rules/simple-import-sort/rules/imports.js

@@ -1,6 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.internalPackages = exports.scopedPackages = exports.simpleImportSortImports = void 0;
+exports.simpleImportSortImports = simpleImportSortImports;
+exports.scopedPackages = scopedPackages;
+exports.internalPackages = internalPackages;
 function simpleImportSortImports(internal = []) {
     return {
         groups: [
@@ -15,7 +17,6 @@ function simpleImportSortImports(internal = []) {
         ].filter(group => group.length > 0),
     };
 }
-exports.simpleImportSortImports = simpleImportSortImports;
 function sideEffects() {
     return ['^\\u0000'];
 }
@@ -31,11 +32,9 @@ function scopedPackages(exclude) {
     }
     return [`^(?!${exclude.join('|')})@`];
 }
-exports.scopedPackages = scopedPackages;
 function internalPackages(internal) {
     return internal.map(scope => [`^${scope}`]);
 }
-exports.internalPackages = internalPackages;
 function parentImports() {
     return ['^\\.\\.(?!/?$)', '^\\.\\./?$'];
 }

package/rules/stylistic/js/index.js

@@ -156,6 +156,7 @@ module.exports = {
         '@stylistic/js/quotes': ['warn', 'single', {
                 avoidEscape: true,
                 allowTemplateLiterals: true,
+                ignoreStringLiterals: false,
             }],
         '@stylistic/js/rest-spread-spacing': ['warn', 'never'],
         '@stylistic/js/semi': ['warn', 'always'],

package/rules/stylistic/jsx/index.js

@@ -28,10 +28,7 @@ module.exports = {
         '@stylistic/jsx/jsx-equals-spacing': ['warn', 'never'],
         '@stylistic/jsx/jsx-first-prop-new-line': ['warn', 'multiline-multiprop'],
         '@stylistic/jsx/jsx-function-call-newline': ['warn', 'multiline'],
-        '@stylistic/jsx/jsx-indent': ['warn', 4, {
-                checkAttributes: true,
-                indentLogicalExpressions: true,
-            }],
+        '@stylistic/jsx/jsx-indent': 'off',
         '@stylistic/jsx/jsx-indent-props': ['warn', {
                 indentMode: 1,
                 ignoreTernaryOperator: false,

package/rules/stylistic/ts/index.js

@@ -91,11 +91,20 @@ module.exports = {
             }],
         '@stylistic/js/no-extra-semi': 'off',
         '@stylistic/ts/no-extra-semi': 'warn',
+        '@stylistic/js/object-curly-newline': 'off',
+        '@stylistic/ts/object-curly-newline': ['warn', {
+                ImportDeclaration: { multiline: true },
+                ExportDeclaration: { multiline: true },
+            }],
         '@stylistic/js/object-curly-spacing': 'off',
         '@stylistic/ts/object-curly-spacing': ['warn', 'always', {
                 arraysInObjects: true,
                 objectsInObjects: true,
             }],
+        '@stylistic/js/object-property-newline': 'off',
+        '@stylistic/ts/object-property-newline': ['warn', {
+                allowAllPropertiesOnSameLine: true,
+            }],
         '@stylistic/js/padding-line-between-statements': 'off',
         '@stylistic/ts/padding-line-between-statements': 'off',
         '@stylistic/js/quote-props': 'off',

package/rules/typescript-eslint/rules/typescript-eslint-naming-convention.js

@@ -1,11 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.typescriptEslintNamingConvention = void 0;
+exports.typescriptEslintNamingConvention = typescriptEslintNamingConvention;
 function typescriptEslintNamingConvention(extensions = []) {
     return typescriptEslintNamingConventionDefaults()
         .concat(extensions);
 }
-exports.typescriptEslintNamingConvention = typescriptEslintNamingConvention;
 function typescriptEslintNamingConventionDefaults() {
     return [
         {

package/rules/typescript-eslint/rules/typescript-eslint-tslint-config.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.typescriptEslintTslintConfig = void 0;
+exports.typescriptEslintTslintConfig = typescriptEslintTslintConfig;
 const format = {
     'import-spacing': true,
 };
@@ -32,4 +32,3 @@ function typescriptEslintTslintConfig(rules = {}, directories = []) {
         rulesDirectory: directories,
     };
 }
-exports.typescriptEslintTslintConfig = typescriptEslintTslintConfig;

package/rules/unicorn/index.js

@@ -51,6 +51,7 @@ module.exports = {
         'unicorn/no-lonely-if': 'warn',
         'unicorn/no-magic-array-flat-depth': 'error',
         'unicorn/no-negated-condition': 'warn',
+        'unicorn/no-negation-in-equality-check': 'error',
         'unicorn/no-nested-ternary': 'off',
         'unicorn/no-new-array': 'warn',
         'unicorn/no-new-buffer': 'warn',
@@ -97,7 +98,9 @@ module.exports = {
                 },
             }],
         'unicorn/prefer-add-event-listener': 'warn',
-        'unicorn/prefer-array-find': 'warn',
+        'unicorn/prefer-array-find': ['warn', {
+                checkFromLast: true,
+            }],
         'unicorn/prefer-array-flat': 'off',
         'unicorn/prefer-array-flat-map': 'off',
         'unicorn/prefer-array-index-of': 'warn',

package/rules/unicorn/rules/prevent-abbreviations.js

@@ -1,9 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.unicornPreventAbbreviations = void 0;
+exports.unicornPreventAbbreviations = unicornPreventAbbreviations;
 function unicornPreventAbbreviations(replacements = {}, options = {}) {
     return Object.assign(Object.assign({ replacements: Object.assign({ arg: false, args: false, db: false, doc: false, docs: false, dev: false, env: false, i: false, j: false, lib: false, libs: false, param: false, params: false }, replacements), extendDefaultReplacements: true, allowList: {}, extendDefaultAllowList: true, checkDefaultAndNamespaceImports: 'internal', checkShorthandImports: 'internal', checkShorthandProperties: false, checkProperties: false, checkVariables: true, checkFilenames: true }, options), { ignore: [
             /e2e/u,
         ] });
 }
-exports.unicornPreventAbbreviations = unicornPreventAbbreviations;

package/rules/node/best-practices.js

@@ -1,6 +0,0 @@
-"use strict";
-module.exports = {
-    rules: {
-        'node/no-deprecated-api': 'off',
-    },
-};

package/rules/node/index.js

@@ -1,11 +0,0 @@
-"use strict";
-module.exports = {
-    plugins: [
-        'node',
-    ],
-    extends: [
-        './possible-errors',
-        './best-practices',
-        './stylistic-issues',
-    ],
-};

package/rules/node/possible-errors.js

@@ -1,23 +0,0 @@
-"use strict";
-module.exports = {
-    rules: {
-        'node/handle-callback-err': 'off',
-        'node/no-callback-literal': 'off',
-        'node/no-exports-assign': 'off',
-        'node/no-extraneous-import': 'off',
-        'node/no-extraneous-require': 'off',
-        'node/no-missing-import': 'off',
-        'node/no-missing-require': 'off',
-        'node/no-new-require': 'off',
-        'node/no-path-concat': 'off',
-        'node/no-process-exit': 'off',
-        'node/no-unpublished-bin': 'off',
-        'node/no-unpublished-import': 'off',
-        'node/no-unpublished-require': 'off',
-        'node/no-unsupported-features/es-builtins': 'off',
-        'node/no-unsupported-features/es-syntax': 'off',
-        'node/no-unsupported-features/node-builtins': 'off',
-        'node/process-exit-as-throw': 'off',
-        'node/shebang': 'off',
-    },
-};

package/rules/node/stylistic-issues.js

@@ -1,23 +0,0 @@
-"use strict";
-module.exports = {
-    rules: {
-        'node/callback-return': 'off',
-        'node/exports-style': ['off', 'module.exports', { allowBatchAssign: false }],
-        'node/file-extension-in-import': 'off',
-        'node/global-require': 'off',
-        'node/no-mixed-requires': 'off',
-        'node/no-process-env': 'off',
-        'node/no-restricted-import': 'off',
-        'node/no-restricted-require': 'off',
-        'node/no-sync': 'off',
-        'node/prefer-global/buffer': ['off', 'always'],
-        'node/prefer-global/console': ['off', 'always'],
-        'node/prefer-global/process': ['off', 'always'],
-        'node/prefer-global/text-decoder': ['off', 'always'],
-        'node/prefer-global/text-encoder': ['off', 'always'],
-        'node/prefer-global/url-search-params': ['off', 'always'],
-        'node/prefer-global/url': ['off', 'always'],
-        'node/prefer-promises/dns': 'off',
-        'node/prefer-promises/fs': 'off',
-    },
-};