eslint-plugin-perfectionist 3.4.0 → 3.5.0

package/dist/index.d.ts

@@ -383,6 +383,7 @@ declare const _default: {
           groups: (string | string[])[]
           environment: 'node' | 'bun'
           internalPattern: string[]
+          sortSideEffects: boolean
           maxLineLength?: number
           order: 'desc' | 'asc'
           ignoreCase: boolean

package/dist/rules/sort-classes.js

@@ -292,6 +292,9 @@ const sortClasses = createEslintRule.createEslintRule({
             if ('properties' in nodeValue) {
               nodeValue.properties.forEach(traverseNode)
             }
+            if ('expressions' in nodeValue) {
+              nodeValue.expressions.forEach(traverseNode)
+            }
           }
           let traverseNode = nodeValue => {
             if (Array.isArray(nodeValue)) {

package/dist/rules/sort-classes.mjs

@@ -300,6 +300,9 @@ const sortClasses = createEslintRule({
             if ('properties' in nodeValue) {
               nodeValue.properties.forEach(traverseNode)
             }
+            if ('expressions' in nodeValue) {
+              nodeValue.expressions.forEach(traverseNode)
+            }
           }
           let traverseNode = nodeValue => {
             if (Array.isArray(nodeValue)) {

package/dist/rules/sort-imports.js

@@ -52,6 +52,11 @@ const sortImports = createEslintRule.createEslintRule({
             },
             type: 'array',
           },
+          sortSideEffects: {
+            description:
+              'Controls whether side-effect imports should be sorted.',
+            type: 'boolean',
+          },
           newlinesBetween: {
             description:
               'Specifies how new lines should be handled between import groups.',
@@ -148,6 +153,7 @@ const sortImports = createEslintRule.createEslintRule({
       order: 'asc',
       ignoreCase: true,
       internalPattern: ['~/**'],
+      sortSideEffects: false,
       newlinesBetween: 'always',
       maxLineLength: void 0,
       groups: [
@@ -180,6 +186,7 @@ const sortImports = createEslintRule.createEslintRule({
       customGroups: { type: {}, value: {} },
       internalPattern: ['~/**'],
       newlinesBetween: 'always',
+      sortSideEffects: false,
       type: 'alphabetical',
       environment: 'node',
       ignoreCase: true,
@@ -418,10 +425,14 @@ const sortImports = createEslintRule.createEslintRule({
             if (!(groupNum in grouped)) {
               grouped[groupNum] = [node]
             } else {
-              grouped[groupNum] = sortNodes.sortNodes(
-                [...grouped[groupNum], node],
-                options,
-              )
+              if (!options.sortSideEffects && isSideEffectImport(node.node)) {
+                grouped[groupNum] = [...grouped[groupNum], node]
+              } else {
+                grouped[groupNum] = sortNodes.sortNodes(
+                  [...grouped[groupNum], node],
+                  options,
+                )
+              }
             }
           }
           let formatted = Object.keys(grouped)
@@ -529,7 +540,9 @@ const sortImports = createEslintRule.createEslintRule({
             )
             if (
               !(
-                isSideEffectImport(left.node) && isSideEffectImport(right.node)
+                !options.sortSideEffects &&
+                isSideEffectImport(left.node) &&
+                isSideEffectImport(right.node)
               ) &&
               !hasContentBetweenNodes(left, right) &&
               (leftNum > rightNum ||

package/dist/rules/sort-imports.mjs

@@ -51,6 +51,11 @@ const sortImports = createEslintRule({
             },
             type: 'array',
           },
+          sortSideEffects: {
+            description:
+              'Controls whether side-effect imports should be sorted.',
+            type: 'boolean',
+          },
           newlinesBetween: {
             description:
               'Specifies how new lines should be handled between import groups.',
@@ -147,6 +152,7 @@ const sortImports = createEslintRule({
       order: 'asc',
       ignoreCase: true,
       internalPattern: ['~/**'],
+      sortSideEffects: false,
       newlinesBetween: 'always',
       maxLineLength: void 0,
       groups: [
@@ -179,6 +185,7 @@ const sortImports = createEslintRule({
       customGroups: { type: {}, value: {} },
       internalPattern: ['~/**'],
       newlinesBetween: 'always',
+      sortSideEffects: false,
       type: 'alphabetical',
       environment: 'node',
       ignoreCase: true,
@@ -412,10 +419,14 @@ const sortImports = createEslintRule({
             if (!(groupNum in grouped)) {
               grouped[groupNum] = [node]
             } else {
-              grouped[groupNum] = sortNodes(
-                [...grouped[groupNum], node],
-                options,
-              )
+              if (!options.sortSideEffects && isSideEffectImport(node.node)) {
+                grouped[groupNum] = [...grouped[groupNum], node]
+              } else {
+                grouped[groupNum] = sortNodes(
+                  [...grouped[groupNum], node],
+                  options,
+                )
+              }
             }
           }
           let formatted = Object.keys(grouped)
@@ -514,7 +525,9 @@ const sortImports = createEslintRule({
             )
             if (
               !(
-                isSideEffectImport(left.node) && isSideEffectImport(right.node)
+                !options.sortSideEffects &&
+                isSideEffectImport(left.node) &&
+                isSideEffectImport(right.node)
               ) &&
               !hasContentBetweenNodes(left, right) &&
               (leftNum > rightNum ||

package/dist/rules/sort-variable-declarations.js

@@ -69,15 +69,35 @@ const sortVariableDeclarations = createEslintRule.createEslintRule({
           }
           let dependencies = []
           let checkNode = nodeValue => {
+            if (
+              nodeValue.type === 'ArrowFunctionExpression' ||
+              nodeValue.type === 'FunctionExpression'
+            ) {
+              return
+            }
             if (nodeValue.type === 'Identifier') {
               dependencies.push(nodeValue.name)
             }
+            if (nodeValue.type === 'Property') {
+              traverseNode(nodeValue.key)
+              traverseNode(nodeValue.value)
+            }
+            if (nodeValue.type === 'ConditionalExpression') {
+              traverseNode(nodeValue.test)
+              traverseNode(nodeValue.consequent)
+              traverseNode(nodeValue.alternate)
+            }
             if (
-              'body' in nodeValue &&
-              nodeValue.body &&
-              !Array.isArray(nodeValue.body)
+              'expression' in nodeValue &&
+              typeof nodeValue.expression !== 'boolean'
             ) {
-              traverseNode(nodeValue.body)
+              traverseNode(nodeValue.expression)
+            }
+            if ('object' in nodeValue) {
+              traverseNode(nodeValue.object)
+            }
+            if ('callee' in nodeValue) {
+              traverseNode(nodeValue.callee)
             }
             if ('left' in nodeValue) {
               traverseNode(nodeValue.left)
@@ -89,9 +109,19 @@ const sortVariableDeclarations = createEslintRule.createEslintRule({
               nodeValue.elements
                 .filter(currentNode => currentNode !== null)
                 .forEach(traverseNode)
-            } else if ('arguments' in nodeValue) {
+            }
+            if ('argument' in nodeValue && nodeValue.argument) {
+              traverseNode(nodeValue.argument)
+            }
+            if ('arguments' in nodeValue) {
               nodeValue.arguments.forEach(traverseNode)
             }
+            if ('properties' in nodeValue) {
+              nodeValue.properties.forEach(traverseNode)
+            }
+            if ('expressions' in nodeValue) {
+              nodeValue.expressions.forEach(traverseNode)
+            }
           }
           let traverseNode = nodeValue => {
             checkNode(nodeValue)

package/dist/rules/sort-variable-declarations.mjs

@@ -68,15 +68,35 @@ const sortVariableDeclarations = createEslintRule({
           }
           let dependencies = []
           let checkNode = nodeValue => {
+            if (
+              nodeValue.type === 'ArrowFunctionExpression' ||
+              nodeValue.type === 'FunctionExpression'
+            ) {
+              return
+            }
             if (nodeValue.type === 'Identifier') {
               dependencies.push(nodeValue.name)
             }
+            if (nodeValue.type === 'Property') {
+              traverseNode(nodeValue.key)
+              traverseNode(nodeValue.value)
+            }
+            if (nodeValue.type === 'ConditionalExpression') {
+              traverseNode(nodeValue.test)
+              traverseNode(nodeValue.consequent)
+              traverseNode(nodeValue.alternate)
+            }
             if (
-              'body' in nodeValue &&
-              nodeValue.body &&
-              !Array.isArray(nodeValue.body)
+              'expression' in nodeValue &&
+              typeof nodeValue.expression !== 'boolean'
             ) {
-              traverseNode(nodeValue.body)
+              traverseNode(nodeValue.expression)
+            }
+            if ('object' in nodeValue) {
+              traverseNode(nodeValue.object)
+            }
+            if ('callee' in nodeValue) {
+              traverseNode(nodeValue.callee)
             }
             if ('left' in nodeValue) {
               traverseNode(nodeValue.left)
@@ -88,9 +108,19 @@ const sortVariableDeclarations = createEslintRule({
               nodeValue.elements
                 .filter(currentNode => currentNode !== null)
                 .forEach(traverseNode)
-            } else if ('arguments' in nodeValue) {
+            }
+            if ('argument' in nodeValue && nodeValue.argument) {
+              traverseNode(nodeValue.argument)
+            }
+            if ('arguments' in nodeValue) {
               nodeValue.arguments.forEach(traverseNode)
             }
+            if ('properties' in nodeValue) {
+              nodeValue.properties.forEach(traverseNode)
+            }
+            if ('expressions' in nodeValue) {
+              nodeValue.expressions.forEach(traverseNode)
+            }
           }
           let traverseNode = nodeValue => {
             checkNode(nodeValue)

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "eslint-plugin-perfectionist",
   "description": "ESLint plugin for sorting various data such as objects, imports, types, enums, JSX props, etc.",
-  "version": "3.4.0",
+  "version": "3.5.0",
   "homepage": "https://perfectionist.dev",
   "repository": "azat-io/eslint-plugin-perfectionist",
   "author": "Azat S. <to@azat.io>",

package/readme.md

@@ -188,6 +188,7 @@ module.exports = {
 | [sort-named-imports](https://perfectionist.dev/rules/sort-named-imports)                 | Enforce sorted named imports                | 🔧  |
 | [sort-object-types](https://perfectionist.dev/rules/sort-object-types)                   | Enforce sorted object types                 | 🔧  |
 | [sort-objects](https://perfectionist.dev/rules/sort-objects)                             | Enforce sorted objects                      | 🔧  |
+| [sort-sets](https://perfectionist.dev/rules/sort-sets)                                   | Enforce sorted Set elements                 | 🔧  |
 | [sort-svelte-attributes](https://perfectionist.dev/rules/sort-svelte-attributes)         | Enforce sorted Svelte attributes            | 🔧  |
 | [sort-switch-case](https://perfectionist.dev/rules/sort-switch-case)                     | Enforce sorted switch case statements       | 🔧  |
 | [sort-union-types](https://perfectionist.dev/rules/sort-union-types)                     | Enforce sorted union types                  | 🔧  |