@ornikar/eslint-config 23.0.0 → 24.0.0

package/rules/security.js

@@ -1,22 +1,24 @@
 'use strict';
 
-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',
+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',
+    },
   },
-};
+];

package/rules/sort-imports-exports.js

@@ -1,14 +1,18 @@
 'use strict';
 
-module.exports = {
-  plugins: ['simple-import-sort'],
-  rules: {
-    'sort-imports': [
-      'error',
-      {
-        ignoreDeclarationSort: true,
-      },
-    ],
-    'simple-import-sort/exports': 'error',
+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',
+    },
   },
-};
+];

package/rules/style.js

@@ -1,39 +1,41 @@
 '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',
+        },
+      ],
+    },
   },
-};
+];