eslint-plugin-nextfriday 1.22.0 → 1.23.1

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # eslint-plugin-nextfriday
 
+## 1.23.1
+
+### Patch Changes
+
+- [#96](https://github.com/next-friday/eslint-plugin-nextfriday/pull/96) [`1821f08`](https://github.com/next-friday/eslint-plugin-nextfriday/commit/1821f08a00cf60a0b1ab6d654e700b54b86f364b) Thanks [@joetakara](https://github.com/joetakara)! - Upgrade dependencies to latest compatible versions: typescript-eslint 8.58 → 8.59, eslint-plugin-unicorn 63 → 64, eslint-plugin-sonarjs 4.0.2 → 4.0.3, plus dev tooling (prettier, changesets, commitlint, swc). Pinned eslint, typescript, and @types/node to their latest 9.x/5.x/22.x lines for ecosystem and engine compatibility.
+
+## 1.23.0
+
+### Minor Changes
+
+- [#94](https://github.com/next-friday/eslint-plugin-nextfriday/pull/94) [`7eaf83e`](https://github.com/next-friday/eslint-plugin-nextfriday/commit/7eaf83e365bb0867679f9abe764ade784eeb4594) Thanks [@joetakara](https://github.com/joetakara)! - Add auto-fix support to jsx-sort-props rule. Running eslint --fix will automatically reorder JSX props by value type.
+
 ## 1.22.0
 
 ### Minor Changes

package/README.md

@@ -231,7 +231,7 @@ export default [
 | [jsx-no-variable-in-callback](docs/rules/JSX_NO_VARIABLE_IN_CALLBACK.md)                 | Disallow variable declarations inside callback functions in JSX       | ❌      |
 | [jsx-require-suspense](docs/rules/JSX_REQUIRE_SUSPENSE.md)                               | Require lazy-loaded components to be wrapped in Suspense              | ❌      |
 | [jsx-simple-props](docs/rules/JSX_SIMPLE_PROPS.md)                                       | Enforce simple prop values (strings, variables, callbacks, ReactNode) | ❌      |
-| [jsx-sort-props](docs/rules/JSX_SORT_PROPS.md)                                           | Enforce JSX props are sorted by value type                            | ❌      |
+| [jsx-sort-props](docs/rules/JSX_SORT_PROPS.md)                                           | Enforce JSX props are sorted by value type                            | ✅      |
 | [prefer-jsx-template-literals](docs/rules/PREFER_JSX_TEMPLATE_LITERALS.md)               | Enforce template literals instead of mixing text and JSX expressions  | ✅      |
 | [react-props-destructure](docs/rules/REACT_PROPS_DESTRUCTURE.md)                         | Enforce destructuring props inside React component body               | ❌      |
 | [enforce-props-suffix](docs/rules/ENFORCE_PROPS_SUFFIX.md)                               | Enforce 'Props' suffix for interfaces and types in \*.tsx files       | ❌      |

package/docs/rules/JSX_SORT_PROPS.md

@@ -2,6 +2,8 @@
 
 Enforce JSX props are sorted by value type.
 
+> This rule is auto-fixable using `--fix`.
+
 ## Rule Details
 
 This rule enforces a consistent ordering of JSX props based on the type of their value. Props must appear in the following order:

package/lib/index.cjs

@@ -42,7 +42,7 @@ var import_eslint_plugin_unicorn = __toESM(require("eslint-plugin-unicorn"), 1);
 // package.json
 var package_default = {
   name: "eslint-plugin-nextfriday",
-  version: "1.22.0",
+  version: "1.23.1",
   description: "A comprehensive ESLint plugin providing custom rules and configurations for Next Friday development workflows.",
   keywords: [
     "eslint",
@@ -106,29 +106,29 @@ var package_default = {
     typecheck: "tsc --noEmit"
   },
   dependencies: {
-    "@typescript-eslint/utils": "^8.57.2",
+    "@typescript-eslint/utils": "^8.59.1",
     "emoji-regex": "^10.6.0",
-    "eslint-plugin-sonarjs": "^4.0.2",
-    "eslint-plugin-unicorn": "^63.0.0",
+    "eslint-plugin-sonarjs": "^4.0.3",
+    "eslint-plugin-unicorn": "^64.0.0",
     tsup: "^8.5.1"
   },
   devDependencies: {
     "@changesets/changelog-github": "^0.6.0",
-    "@changesets/cli": "^2.30.0",
-    "@commitlint/cli": "^20.5.0",
+    "@changesets/cli": "^2.31.0",
+    "@commitlint/cli": "^20.5.2",
     "@commitlint/config-conventional": "^20.5.0",
-    "@eslint/js": "^9.35.0",
+    "@eslint/js": "^9.39.4",
     "@jest/globals": "^30.3.0",
     "@stylistic/eslint-plugin": "^5.10.0",
-    "@swc/core": "^1.15.21",
+    "@swc/core": "^1.15.32",
     "@swc/jest": "^0.2.39",
     "@types/eslint": "^9.6.1",
     "@types/jest": "^30.0.0",
-    "@types/node": "^22.18.1",
+    "@types/node": "^22.19.17",
     "@types/ramda": "^0.31.1",
-    "@typescript-eslint/parser": "^8.57.2",
-    "@typescript-eslint/rule-tester": "^8.57.2",
-    eslint: "^9.39.3",
+    "@typescript-eslint/parser": "^8.59.1",
+    "@typescript-eslint/rule-tester": "^8.59.1",
+    eslint: "^9.39.4",
     "eslint-config-airbnb-extended": "^3.0.1",
     "eslint-config-prettier": "^10.1.8",
     "eslint-import-resolver-typescript": "^4.4.4",
@@ -138,12 +138,12 @@ var package_default = {
     husky: "^9.1.7",
     jest: "^30.3.0",
     "lint-staged": "^16.4.0",
-    prettier: "^3.8.1",
+    prettier: "^3.8.3",
     "sort-package-json": "^3.6.1",
     "ts-jest": "^29.4.6",
     "ts-node": "^10.9.2",
-    typescript: "^5.6.2",
-    "typescript-eslint": "^8.57.2"
+    typescript: "^5.9.3",
+    "typescript-eslint": "^8.59.1"
   },
   peerDependencies: {
     eslint: "^9.0.0"
@@ -1814,6 +1814,29 @@ function hasUnsortedProps(attributes) {
     return false;
   });
 }
+function sortSegment(segment, sourceCode, fixer) {
+  const sorted = [...segment].sort((a, b) => (getTypeGroup(a) ?? 0) - (getTypeGroup(b) ?? 0));
+  const sortedTexts = sorted.map((attr) => sourceCode.getText(attr));
+  return segment.map((attr, i) => ({ attr, sortedText: sortedTexts[i] })).filter(({ attr, sortedText }) => sourceCode.getText(attr) !== sortedText).map(({ attr, sortedText }) => fixer.replaceText(attr, sortedText));
+}
+function getSegments(attributes) {
+  const result = [];
+  let current = [];
+  attributes.forEach((attr) => {
+    if (attr.type === import_utils24.AST_NODE_TYPES.JSXSpreadAttribute) {
+      if (current.length > 0) {
+        result.push(current);
+        current = [];
+      }
+    } else {
+      current.push(attr);
+    }
+  });
+  if (current.length > 0) {
+    result.push(current);
+  }
+  return result;
+}
 var jsxSortProps = createRule22({
   name: "jsx-sort-props",
   meta: {
@@ -1821,6 +1844,7 @@ var jsxSortProps = createRule22({
     docs: {
       description: "Enforce JSX props are sorted by value type: strings, hyphenated strings, numbers/booleans, expressions, objects/arrays, functions, JSX elements, then shorthand booleans"
     },
+    fixable: "code",
     messages: {
       unsortedProps: "JSX props should be sorted by value type: strings, hyphenated strings, numbers/booleans/null, expressions, objects/arrays, functions, JSX elements, then shorthand booleans."
     },
@@ -1828,14 +1852,19 @@ var jsxSortProps = createRule22({
   },
   defaultOptions: [],
   create(context) {
+    const { sourceCode } = context;
     return {
       JSXOpeningElement(node) {
-        if (hasUnsortedProps(node.attributes)) {
-          context.report({
-            node,
-            messageId: "unsortedProps"
-          });
+        if (!hasUnsortedProps(node.attributes)) {
+          return;
         }
+        context.report({
+          node,
+          messageId: "unsortedProps",
+          fix(fixer) {
+            return getSegments(node.attributes).flatMap((segment) => sortSegment(segment, sourceCode, fixer));
+          }
+        });
       }
     };
   }