@bigbinary/eslint-plugin-neeto 1.1.20 → 1.1.21

package/README.md

@@ -113,6 +113,7 @@ module.exports = {
 - [use-functions-and-hooks-from-neeto-commons-frontend.md](./docs/src/rules/use-functions-and-hooks-from-neeto-commons-frontend.md)
 - [zustand-use-with-immutable-actions-wrapper-inside-create](./docs/src/rules/zustand-use-with-immutable-actions-wrapper-inside-create.md)
 - [add-required-peer-dependencies](./docs/src/rules/add-required-peer-dependencies.md)
+- [move-spread-attributes-to-top-position](./docs/src/rules/move-spread-attributes-to-top-position.md)
 
 ## Other references
 

package/dist/index.js

@@ -151,7 +151,8 @@
     "@bigbinary/neeto/merge-spread-attributes": "error",
     "@bigbinary/neeto/props-inside-spread-attributes-should-be-sorted": "error",
     "@bigbinary/neeto/props-with-different-key-value-should-be-outside-spread-attribute": "error",
-    "@bigbinary/neeto/use-common-constants": "warn"
+    "@bigbinary/neeto/use-common-constants": "warn",
+    "@bigbinary/neeto/move-spread-attributes-to-top-position": "error"
   };
 
   var recommended = {
@@ -547,6 +548,7 @@
   var USE_WITH_T_HOC_IN_DUMB_COMPONENTS = 'Use "withT" HOC from "neeto-commons-frontend" instead of useTranslation hook in Dumb components';
   var NO_WITH_T_IN_SMART_COMPONENTS = '"withT" HOC should only be used in dumb components. "useTranslation" hook should be used here.';
   var USE_STANDARD_DATE_TIME_FORMATS = 'Use "dateFormat/timeFormat" from "@bigbinary/neeto-commons-frontend/utils" instead. Refer: https://github.com/bigbinary/neeto-commons-frontend/blob/main/docs/utils/README.md#dateformattimeformat';
+  var MOVE_SPREAD_ATTRIBUTES_TO_TOP = "Move spread attribute to top";
 
   var PACKAGES_TO_CHECK = ["@bigbinary/neeto-molecules", "@bigbinary/neetoui", "@bigbinary/neeto-commons-frontend"];
 
@@ -6879,6 +6881,13 @@
       array.splice(index, 1);
     }
   };
+  var findIndexToInsert = function findIndexToInsert(endIndex, attributes) {
+    var index = endIndex - 1;
+    for (; index >= 0; --index) {
+      if (attributes[index].type === TYPES.JSX_SPREAD_ATTRIBUTE) return index + 1;
+    }
+    return index + 1;
+  };
 
   var mergeSpreadAttributes = {
     meta: {
@@ -7004,6 +7013,43 @@
     }
   };
 
+  var moveSpreadAttributesToTopPosition = {
+    meta: {
+      type: "problem",
+      docs: {
+        url: "https://github.com/bigbinary/eslint-plugin-neeto/blob/main/docs/src/rules/move-spread-attributes-to-top-position.md",
+        description: "Enforce moving spread attributes to top most position in oprning elements"
+      },
+      fixable: "code"
+    },
+    create: function create(context) {
+      return {
+        JSXSpreadAttribute: function JSXSpreadAttribute(node) {
+          var _node$argument$proper;
+          if (!pure.matches(SPREAD_ATTRIBUTE_WITH_OBJECT_EXPRESSION, node) || (_node$argument$proper = node.argument.properties) !== null && _node$argument$proper !== void 0 && _node$argument$proper.find(pure.matches(OTHER_PROPS_INSIDE_SPREAD_ATTRIBUTE))) {
+            return;
+          }
+          var attributes = node.parent.attributes;
+          var nodeIndex = pure.findIndexBy({
+            range: node.range
+          }, attributes);
+          var indexToInsert = findIndexToInsert(nodeIndex, attributes);
+          if (indexToInsert === nodeIndex) return;
+          var indentLevel = context.getSourceCode().getFirstToken(attributes[0]).loc.start.column;
+          context.report({
+            node: node,
+            message: MOVE_SPREAD_ATTRIBUTES_TO_TOP,
+            fix: function fix(fixer) {
+              var removeNode = fixer.remove(node);
+              var moveNode = fixer.insertTextBefore(attributes[indexToInsert], "".concat(context.getSourceCode().getText(node), "\n").concat(" ".repeat(indentLevel)));
+              return [moveNode, removeNode];
+            }
+          });
+        }
+      };
+    }
+  };
+
   var noRepetitionInPropKeyAndValueName = {
     meta: {
       type: "problem",
@@ -26901,7 +26947,8 @@
       "no-unnecessary-if-condition": noUnnecessaryIfCondition,
       "zustand-use-with-immutable-actions-wrapper-inside-create": useWithImmutableActionsWrapperInsideCreate,
       "add-required-peer-dependencies": addRequiredPeerDependencies,
-      "use-common-constants": useCommonConstants
+      "use-common-constants": useCommonConstants,
+      "move-spread-attributes-to-top-position": moveSpreadAttributesToTopPosition
     },
     configs: {
       recommended: recommended,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bigbinary/eslint-plugin-neeto",
-  "version": "1.1.20",
+  "version": "1.1.21",
   "main": "dist/index.js",
   "license": "UNLICENSED",
   "scripts": {