@almadar/eslint-plugin 2.1.1 → 2.2.0

package/index.js

@@ -9,6 +9,7 @@ const templateSerializableProps = require("./rules/template-serializable-props")
 const noTypeProp = require("./rules/no-type-prop");
 const noAsAny = require("./rules/no-as-any");
 const noImportGenerated = require("./rules/no-import-generated");
+const noUseNavigate = require("./rules/no-use-navigate");
 const requireEventBus = require("./rules/require-event-bus");
 const requireDisplayName = require("./rules/require-display-name");
 const requireClosedCircuitProps = require("./rules/require-closed-circuit-props");
@@ -47,6 +48,7 @@ const plugin = {
     "no-type-prop": noTypeProp,
     "no-as-any": noAsAny,
     "no-import-generated": noImportGenerated,
+    "no-use-navigate": noUseNavigate,
     "require-event-bus": requireEventBus,
     "require-display-name": requireDisplayName,
     "require-closed-circuit-props": requireClosedCircuitProps,
@@ -89,6 +91,7 @@ plugin.configs.recommended = {
     "almadar/no-type-prop": "error",
     "almadar/no-as-any": "error",
     "almadar/no-import-generated": "error",
+    "almadar/no-use-navigate": "error",
 
     // Warn-level — conventions that should be followed
     "almadar/require-event-bus": "warn",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@almadar/eslint-plugin",
-  "version": "2.1.1",
+  "version": "2.2.0",
   "private": false,
   "description": "Custom ESLint rules for Almadar component architecture",
   "main": "index.js",

package/rules/no-use-navigate.js

@@ -0,0 +1,57 @@
+"use strict";
+
+module.exports = {
+  meta: {
+    type: "problem",
+    docs: {
+      description:
+        "Disallow useNavigate from react-router-dom. Use useEventBus and emit UI:NAVIGATE instead.",
+      category: "Almadar Architecture",
+    },
+    messages: {
+      noUseNavigate:
+        "Do not use useNavigate() from react-router-dom. Emit 'UI:NAVIGATE' via useEventBus instead for closed circuit compliance.",
+      noReactRouterNavigate:
+        "Do not import useNavigate from react-router-dom. Use useEventBus().emit('UI:NAVIGATE', { url }) instead.",
+    },
+    schema: [],
+  },
+
+  create(context) {
+    return {
+      // Catch: import { useNavigate } from 'react-router-dom'
+      ImportDeclaration(node) {
+        if (
+          node.source &&
+          node.source.value === "react-router-dom" &&
+          node.specifiers
+        ) {
+          for (const specifier of node.specifiers) {
+            if (
+              specifier.type === "ImportSpecifier" &&
+              specifier.imported &&
+              specifier.imported.name === "useNavigate"
+            ) {
+              context.report({
+                node: specifier,
+                messageId: "noReactRouterNavigate",
+              });
+            }
+          }
+        }
+      },
+      // Catch: useNavigate() call expressions
+      CallExpression(node) {
+        if (
+          node.callee.type === "Identifier" &&
+          node.callee.name === "useNavigate"
+        ) {
+          context.report({
+            node,
+            messageId: "noUseNavigate",
+          });
+        }
+      },
+    };
+  },
+};