eslint-plugin-nextfriday 1.22.0 → 1.23.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # eslint-plugin-nextfriday
 
+## 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.0",
   description: "A comprehensive ESLint plugin providing custom rules and configurations for Next Friday development workflows.",
   keywords: [
     "eslint",
@@ -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));
+          }
+        });
       }
     };
   }