@ornikar/eslint-config 22.7.4 → 23.0.0

package/rules/security.js

@@ -1,24 +1,22 @@
 'use strict';
 
-const securityPlugin = require('eslint-plugin-security');
-
-module.exports = [
-  securityPlugin.configs.recommended,
-  {
-    rules: {
-      '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': 'off', // we use it when necessary and don't call it so not a security issue for us
-      'security/detect-possible-timing-attacks': 'error',
-      'security/detect-pseudoRandomBytes': 'error',
-      'security/detect-unsafe-regex': 'error',
-    },
+module.exports = {
+  // https://github.com/nodesecurity/eslint-plugin-security
+  plugins: ['security'],
+  extends: ['plugin:security/recommended-legacy'],
+  rules: {
+    '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': 'off', // we use it when necessary and don't call it so not a security issue for us
+    'security/detect-possible-timing-attacks': 'error',
+    'security/detect-pseudoRandomBytes': 'error',
+    'security/detect-unsafe-regex': 'error',
   },
-];
+};

package/rules/sort-imports-exports.js

@@ -1,18 +1,14 @@
 'use strict';
 
-const simpleImportSort = require('eslint-plugin-simple-import-sort');
-
-module.exports = [
-  {
-    plugins: { 'simple-import-sort': simpleImportSort },
-    rules: {
-      'sort-imports': [
-        'error',
-        {
-          ignoreDeclarationSort: true,
-        },
-      ],
-      'simple-import-sort/exports': 'error',
-    },
+module.exports = {
+  plugins: ['simple-import-sort'],
+  rules: {
+    'sort-imports': [
+      'error',
+      {
+        ignoreDeclarationSort: true,
+      },
+    ],
+    'simple-import-sort/exports': 'error',
   },
-];
+};

package/rules/style.js

@@ -1,41 +1,39 @@
 'use strict';
 
-module.exports = [
-  {
-    rules: {
-      /* changed rules */
+module.exports = {
+  rules: {
+    /* changed rules */
 
-      // http://eslint.org/docs/rules/no-plusplus
-      'no-plusplus': 'off',
+    // http://eslint.org/docs/rules/no-plusplus
+    'no-plusplus': 'off',
 
-      // when a function is exported, it makes more sense to use curly braces, but with callbacks use expressions.
-      // We should use eslint-plugin-proper-arrows or similar
-      // http://eslint.org/docs/rules/arrow-body-style
-      'arrow-body-style': 'off',
+    // when a function is exported, it makes more sense to use curly braces, but with callbacks use expressions.
+    // We should use eslint-plugin-proper-arrows or similar
+    // http://eslint.org/docs/rules/arrow-body-style
+    'arrow-body-style': 'off',
 
-      // http://eslint.org/docs/rules/no-unused-vars
-      'no-unused-vars': [
-        'error',
-        {
-          vars: 'all',
-          args: 'none', // changed after-used to none
-          // ESLint 9 changed the default from 'none' to 'all'; restore the v8 behaviour.
-          caughtErrors: 'none',
-          ignoreRestSiblings: true,
-        },
-      ],
+    // http://eslint.org/docs/rules/no-unused-vars
+    'no-unused-vars': [
+      'error',
+      {
+        vars: 'all',
+        args: 'none', // changed after-used to none
+        // ESLint 9 changed the default from 'none' to 'all'; restore the v8 behaviour.
+        caughtErrors: 'none',
+        ignoreRestSiblings: true,
+      },
+    ],
 
-      /* stricter rules */
+    /* stricter rules */
 
-      // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/order.md
-      'import/order': [
-        'error',
-        {
-          groups: ['builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
-          alphabetize: { order: 'asc', caseInsensitive: false },
-          'newlines-between': 'never',
-        },
-      ],
-    },
+    // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/order.md
+    'import/order': [
+      'error',
+      {
+        groups: ['builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
+        alphabetize: { order: 'asc', caseInsensitive: false },
+        'newlines-between': 'never',
+      },
+    ],
   },
-];
+};