@agilebot/eslint-plugin 0.5.3 → 0.5.5

package/dist/index.js

@@ -1,5 +1,5 @@
 /**
- * @license @agilebot/eslint-plugin v0.5.3
+ * @license @agilebot/eslint-plugin v0.5.5
  *
  * Copyright (c) Agilebot, Inc. and its affiliates.
  *
@@ -34,8 +34,8 @@ function _interopNamespaceDefault(e) {
 var fs__namespace = /*#__PURE__*/_interopNamespaceDefault(fs);
 var path__namespace = /*#__PURE__*/_interopNamespaceDefault(path);
 
-const rule$1 = require('eslint-plugin-deprecation/dist/rules/deprecation')["default"];
-const compatRule$1 = compat.fixupRule(rule$1);
+const rule = require('eslint-plugin-deprecation/dist/rules/deprecation')["default"];
+const compatRule = compat.fixupRule(rule);
 
 const docBaseUrl = 'https://github.com/sh-agilebot/frontend-toolkit/blob/master/packages/eslint-plugin/src/rules/';
 const hasDocs = typeof ["enforce-mui-icon-alias","intl-id-missing","intl-id-prefix","intl-no-default","react-better-exhaustive-deps"] !== 'undefined' ?
@@ -859,6 +859,7 @@ var reactBetterExhaustiveDeps = {
     const stateVariables = new WeakSet();
     const stableKnownValueCache = new WeakMap();
     const functionWithoutCapturedValueCache = new WeakMap();
+    const useEffectEventVariables = new WeakSet();
     function memoizeWithWeakMap(fn, map) {
       return function (arg) {
         if (map.has(arg)) {
@@ -882,7 +883,7 @@ var reactBetterExhaustiveDeps = {
         if (!isCustomHook) {
           reportProblem({
             node: node,
-            message: "Effect callbacks are synchronous to prevent race conditions. " + "Put the async function inside:\n\n" + 'useEffect(() => {\n' + '  async function fetchData() {\n' + '    // You can await here\n' + '    const response = await MyAPI.getData(someId);\n' + '    // ...\n' + '  }\n' + '  fetchData();\n' + "}, [someId]); // Or [] if effect doesn't need props or state\n\n" + 'Learn more about data fetching with Hooks: https://reactjs.org/link/hooks-data-fetching'
+            message: "Effect callbacks are synchronous to prevent race conditions. " + "Put the async function inside:\n\n" + 'useEffect(() => {\n' + '  async function fetchData() {\n' + '    // You can await here\n' + '    const response = await MyAPI.getData(someId);\n' + '    // ...\n' + '  }\n' + '  fetchData();\n' + "}, [someId]); // Or [] if effect doesn't need props or state\n\n" + 'Learn more about data fetching with Hooks: https://react.dev/link/hooks-data-fetching'
           });
         }
       }
@@ -921,7 +922,7 @@ var reactBetterExhaustiveDeps = {
         if (init == null) {
           return false;
         }
-        while (init.type === 'TSAsExpression') {
+        while (init.type === 'TSAsExpression' || init.type === 'AsExpression') {
           init = init.expression;
         }
         let declaration = def.node.parent;
@@ -951,7 +952,14 @@ var reactBetterExhaustiveDeps = {
         } = callee;
         if (name === 'useRef' && id.type === 'Identifier') {
           return true;
-        } else if (name === 'useState' || name === 'useReducer') {
+        } else if (isUseEffectEventIdentifier() && id.type === 'Identifier') {
+          for (const ref of resolved.references) {
+            if (ref !== id) {
+              useEffectEventVariables.add(ref.identifier);
+            }
+          }
+          return true;
+        } else if (name === 'useState' || name === 'useReducer' || name === 'useActionState') {
           if (id.type === 'ArrayPattern' && id.elements.length === 2 && isArray(resolved.identifiers)) {
             if (id.elements[1] === resolved.identifiers[0]) {
               if (name === 'useState') {
@@ -1238,13 +1246,16 @@ var reactBetterExhaustiveDeps = {
       }
       const declaredDependencies = [];
       const externalDependencies = new Set();
-      if (declaredDependenciesNode.type !== 'ArrayExpression') {
+      const isArrayExpression = declaredDependenciesNode.type === 'ArrayExpression';
+      const isTSAsArrayExpression = declaredDependenciesNode.type === 'TSAsExpression' && declaredDependenciesNode.expression.type === 'ArrayExpression';
+      if (!isArrayExpression && !isTSAsArrayExpression) {
         reportProblem({
           node: declaredDependenciesNode,
           message: "React Hook ".concat(getSource(reactiveHook), " was passed a ") + 'dependency list that is not an array literal. This means we ' + "can't statically verify whether you've passed the correct " + 'dependencies.'
         });
       } else {
-        declaredDependenciesNode.elements.forEach(declaredDependencyNode => {
+        const arrayExpression = isTSAsArrayExpression ? declaredDependenciesNode.expression : declaredDependenciesNode;
+        arrayExpression.elements.forEach(declaredDependencyNode => {
           if (declaredDependencyNode == null) {
             return;
           }
@@ -1255,6 +1266,18 @@ var reactBetterExhaustiveDeps = {
             });
             return;
           }
+          if (useEffectEventVariables.has(declaredDependencyNode)) {
+            reportProblem({
+              node: declaredDependencyNode,
+              message: 'Functions returned from `useEffectEvent` must not be included in the dependency array. ' + "Remove `".concat(getSource(declaredDependencyNode), "` from the list."),
+              suggest: [{
+                desc: "Remove the dependency `".concat(getSource(declaredDependencyNode), "`"),
+                fix(fixer) {
+                  return fixer.removeRange(declaredDependencyNode.range);
+                }
+              }]
+            });
+          }
           let declaredDependency;
           try {
             declaredDependency = analyzePropertyChain(declaredDependencyNode, null);
@@ -1532,7 +1555,8 @@ var reactBetterExhaustiveDeps = {
               extraWarning = " If '".concat(setStateRecommendation.setter, "' needs the ") + "current value of '".concat(setStateRecommendation.missingDep, "', ") + "you can also switch to useReducer instead of useState and " + "read '".concat(setStateRecommendation.missingDep, "' in the reducer.");
               break;
             case 'updater':
-              extraWarning = " You can also do a functional update '".concat(setStateRecommendation.setter, "(").concat(setStateRecommendation.missingDep.slice(0, 1), " => ...)' if you only need '").concat(setStateRecommendation.missingDep, "' in the '").concat(setStateRecommendation.setter, "' call.");
+              extraWarning = " You can also do a functional update '".concat(setStateRecommendation.setter, "(").concat(setStateRecommendation.missingDep.slice(0, 1), " => ...)' if you only need '").concat(setStateRecommendation.missingDep
+              , "'") + " in the '".concat(setStateRecommendation.setter, "' call.");
               break;
             default:
               throw new Error('Unknown case.');
@@ -1559,7 +1583,8 @@ var reactBetterExhaustiveDeps = {
       const callback = node.arguments[callbackIndex];
       const reactiveHook = node.callee;
       const reactiveHookName = getNodeWithoutReactNamespace(reactiveHook).name;
-      const declaredDependenciesNode = node.arguments[callbackIndex + 1];
+      const maybeNode = node.arguments[callbackIndex + 1];
+      const declaredDependenciesNode = maybeNode && !(maybeNode.type === 'Identifier' && maybeNode.name === 'undefined') ? maybeNode : undefined;
       const isEffect = /Effect($|[^a-z])/g.test(reactiveHookName);
       if (!callback) {
         reportProblem({
@@ -1582,6 +1607,9 @@ var reactBetterExhaustiveDeps = {
         case 'ArrowFunctionExpression':
           visitFunctionWithDependencies(callback, declaredDependenciesNode, reactiveHook, reactiveHookName, isEffect);
           return;
+        case 'TSAsExpression':
+          visitFunctionWithDependencies(callback.expression, declaredDependenciesNode, reactiveHook, reactiveHookName, isEffect);
+          return;
         case 'Identifier':
           if (!declaredDependenciesNode) {
             return;
@@ -1786,7 +1814,7 @@ function getConstructionExpressionType(node) {
       }
       return null;
     case 'TypeCastExpression':
-      return getConstructionExpressionType(node.expression);
+    case 'AsExpression':
     case 'TSAsExpression':
       return getConstructionExpressionType(node.expression);
   }
@@ -2008,6 +2036,9 @@ function isSameIdentifier(a, b) {
 function isAncestorNodeOf(a, b) {
   return a.range[0] <= b.range[0] && a.range[1] >= b.range[1];
 }
+function isUseEffectEventIdentifier(node) {
+  return false;
+}
 
 const Components = require('eslint-plugin-react/lib/util/Components');
 const RULE_NAME$7 = 'react-hook-use-ref';
@@ -2203,11 +2234,6 @@ var reactPreferSxProp = createRule({
   }
 });
 
-const pluginReactHooks = require('eslint-plugin-react-hooks');
-const rule = pluginReactHooks.rules['rules-of-hooks'];
-const compatRule = compat.fixupRule(rule);
-compatRule.meta.docs.recommended = false;
-
 function getBasicIdentifier(node) {
   if (node.type === 'Identifier') {
     return node.name;
@@ -2903,7 +2929,7 @@ var varNaming = createRule({
 
 var ruleFiles = /*#__PURE__*/Object.freeze({
   __proto__: null,
-  rules_deprecation: compatRule$1,
+  rules_deprecation: compatRule,
   rules_enforce_mui_icon_alias: enforceMuiIconAlias,
   rules_import_monorepo: importMonorepo,
   rules_intl_id_missing: intlIdMissing,
@@ -2919,7 +2945,6 @@ var ruleFiles = /*#__PURE__*/Object.freeze({
   rules_react_hook_use_ref: reactHookUseRef,
   rules_react_prefer_named_property_access: reactPreferNamedPropertyAccess,
   rules_react_prefer_sx_prop: reactPreferSxProp,
-  rules_react_rules_of_hooks: compatRule,
   rules_tss_class_naming: tssClassNaming,
   rules_tss_no_color_name: tssNoColorName,
   rules_tss_no_color_value: tssNoColorValue,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agilebot/eslint-plugin",
-  "version": "0.5.3",
+  "version": "0.5.5",
   "description": "Agilebot's ESLint plugin",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -18,12 +18,11 @@
     "node": "^18.18.0 || >=20.0.0"
   },
   "dependencies": {
-    "@eslint/compat": "^1.1.1",
+    "@eslint/compat": "^1.2.1",
     "@typescript-eslint/utils": "~8.7.0",
     "eslint-plugin-deprecation": "^3.0.0",
-    "eslint-plugin-react": "^7.35.0",
-    "eslint-plugin-react-hooks": "^4.6.2",
-    "@agilebot/eslint-utils": "0.5.3"
+    "eslint-plugin-react": "^7.37.2",
+    "@agilebot/eslint-utils": "0.5.5"
   },
   "peerDependencies": {
     "eslint": "^7.0.0 || ^8.0.0 || ^9.0.0"