@sabzidev/eslint-rules 1.0.6 → 1.0.8

package/README.md

@@ -161,17 +161,17 @@ export default config;
 
 ## 📋 Available Rules
 
-| Rule                                     | Description                              |
-| :--------------------------------------- | :--------------------------------------- |
-| `merge-duplicate-id-and-classname-props` | Merges duplicate id and className props. |
-| `merge-exports`                          | Merges or organizes export declarations. |
-| `no-invalid-id-and-classname-value`      | Detects invalid id and className values. |
-| `no-useless-empty-type`                  | Detects unnecessary empty types.         |
-| `no-useless-template-literal`            | Detects unnecessary template literals.   |
-| `padding-before-jump-statement`          | Enforces padding before jump statements. |
-| `sort-comments`                          | Sorts comments.                          |
-| `sort-jsx-props`                         | Sorts JSX props.                         |
-| `sort-object-props`                      | Sorts object properties.                 |
+| Rule                                     | Description                                                                |
+| :--------------------------------------- | :------------------------------------------------------------------------- |
+| `merge-duplicate-id-and-classname-props` | Merges duplicate id and className props.                                   |
+| `merge-exports`                          | Merges or organizes export declarations.                                   |
+| `no-invalid-id-and-classname-value`      | Detects invalid id and className values.                                   |
+| `no-useless-empty-type`                  | Detects unnecessary empty types.                                           |
+| `no-useless-template-literal`            | Detects unnecessary template literals.                                     |
+| `padding-before-jump-statement`          | Enforces padding before jump statements.                                   |
+| `sort-comments`                          | Sorts comments.                                                            |
+| `sort-jsx-props`                         | Sorts JSX props.                                                           |
+| `sort-props`                             | Sorts object properties, function parameters, and TypeScript type members. |
 
 ---
 

package/dist/index.js

@@ -17,7 +17,7 @@ var e = {
 	},
 	license: "MIT",
 	author: "Abolfazl Sabzmohammadi <SabziDev@gmail.com> (https://Sabzi.Dev)",
-	version: "1.0.6",
+	version: "1.0.8",
 	type: "module",
 	main: "./dist/index.js",
 	exports: "./dist/index.js",
@@ -421,8 +421,10 @@ var e = {
 				wrongOrder: "Object properties should be ordered!",
 				wrongDestructure: "Destructured properties should be ordered!",
 				wrongParams: "Function parameters should be ordered!",
+				wrongCallArguments: "Function call arguments should be ordered!",
 				wrongTypeProperties: "Type properties should be ordered!"
-			}
+			},
+			schema: []
 		},
 		create(e) {
 			let { sourceCode: t } = e, n = new Set(l), r = (e) => /^on[A-Z]/.test(e), i = (e) => {
@@ -470,42 +472,60 @@ var e = {
 				if (!r || r.length <= 1) return;
 				let i = d(r);
 				g(e, r, i, n, (e) => e.map((e) => t.getText(e)).join(", "));
-			}, v = (n) => {
-				let { params: a } = n;
-				if (!a || a.length === 0) return;
-				for (let e of a) e.type === "ObjectPattern" && _(e, "wrongParams");
-				if (a.length <= 1 || a.some((e) => e.type !== "Identifier")) return;
-				let o = (e) => {
-					let t = l.indexOf(e);
-					return t === -1 ? r(e) ? {
-						category: 2,
-						priority: i(e)
-					} : e === "className" ? {
-						category: 3,
-						priority: 0
-					} : {
-						category: e === "style" ? 4 : 1,
-						priority: 0
-					} : {
-						category: 0,
-						priority: t
-					};
-				}, s = a.map((e, t) => ({
-					param: e,
-					index: t,
-					...o(e.name)
-				})).sort((e, t) => e.category === t.category ? e.priority === t.priority ? e.index - t.index : e.priority - t.priority : e.category - t.category).map(({ param: e }) => e);
-				if (!p(a, s)) return;
-				let c = a[0], u = a.at(-1);
+			}, v = (e) => {
+				let t = l.indexOf(e);
+				return t === -1 ? r(e) ? {
+					category: 2,
+					priority: i(e)
+				} : e === "className" ? {
+					category: 3,
+					priority: 0
+				} : {
+					category: e === "style" ? 4 : 1,
+					priority: 0
+				} : {
+					category: 0,
+					priority: t
+				};
+			}, y = (e) => e.map((e, t) => ({
+				item: e,
+				index: t,
+				...v(e.name)
+			})).sort((e, t) => e.category === t.category ? e.priority === t.priority ? e.index - t.index : e.priority - t.priority : e.category - t.category).map(({ item: e }) => e), b = (n) => {
+				let { params: r } = n;
+				if (!r || r.length === 0) return;
+				for (let e of r) e.type === "ObjectPattern" && _(e, "wrongParams");
+				if (r.length <= 1 || r.some((e) => e.type !== "Identifier")) return;
+				let i = y(r);
+				if (!p(r, i)) return;
+				let a = r[0], o = r.at(-1);
 				e.report({
 					node: n,
 					messageId: "wrongParams",
 					fix(e) {
-						let n = s.map((e) => t.getText(e)).join(", ");
-						return e.replaceTextRange([c.range[0], u.range[1]], n);
+						let n = i.map((e) => t.getText(e)).join(", ");
+						return e.replaceTextRange([a.range[0], o.range[1]], n);
+					}
+				});
+			}, x = (e) => e.type === "Identifier" ? e.name : e.type === "Literal" || e.type === "StringLiteral" || e.type === "NumericLiteral" ? String(e.value) : null, S = (n) => {
+				let { arguments: r } = n;
+				if (r.length <= 1 || r.some((e) => e.type === "SpreadElement" || x(e) === null)) return;
+				let i = r.map((e, t) => ({
+					argument: e,
+					index: t,
+					...v(x(e))
+				})).sort((e, t) => e.category === t.category ? e.priority === t.priority ? e.index - t.index : e.priority - t.priority : e.category - t.category).map(({ argument: e }) => e);
+				if (!p(r, i)) return;
+				let a = r[0], o = r.at(-1);
+				e.report({
+					node: n,
+					messageId: "wrongCallArguments",
+					fix(e) {
+						let n = i.map((e) => t.getText(e)).join(", ");
+						return e.replaceTextRange([a.range[0], o.range[1]], n);
 					}
 				});
-			}, y = (e) => {
+			}, C = (e) => {
 				let t = e.body ?? e.members;
 				if (!t || t.length <= 1) return;
 				let n = f(t);
@@ -527,16 +547,17 @@ var e = {
 				ObjectPattern(e) {
 					_(e, "wrongDestructure");
 				},
-				TSTypeLiteral: y,
-				TSInterfaceBody: y,
-				FunctionDeclaration: v,
-				FunctionExpression: v,
-				ArrowFunctionExpression: v,
-				TSDeclareFunction: v,
-				TSFunctionType: v,
-				TSCallSignatureDeclaration: v,
-				TSConstructSignatureDeclaration: v,
-				TSMethodSignature: v
+				TSTypeLiteral: C,
+				TSInterfaceBody: C,
+				FunctionDeclaration: b,
+				FunctionExpression: b,
+				ArrowFunctionExpression: b,
+				TSDeclareFunction: b,
+				TSFunctionType: b,
+				TSCallSignatureDeclaration: b,
+				TSConstructSignatureDeclaration: b,
+				TSMethodSignature: b,
+				CallExpression: S
 			};
 		}
 	}

package/package.json

@@ -17,7 +17,7 @@
   },
   "license": "MIT",
   "author": "Abolfazl Sabzmohammadi <SabziDev@gmail.com> (https://Sabzi.Dev)",
-  "version": "1.0.6",
+  "version": "1.0.8",
   "type": "module",
   "main": "./dist/index.js",
   "exports": "./dist/index.js",