refactor: move rules to /rules, merge ast.js and react.js

Author: NickvanDyke

src/index.js

@@ -1,15 +1,14 @@
-import noEmptyEffect from "./no-empty-effect.js";
-import noAdjustStateOnPropChange from "./no-adjust-state-on-prop-change.js";
-import noResetAllStateOnPropChange from "./no-reset-all-state-on-prop-change.js";
-import noEventHandler from "./no-event-handler.js";
-import noPassLiveStateToParent from "./no-pass-live-state-to-parent.js";
-import noInitializeState from "./no-initialize-state.js";
-import noChainStateUpdates from "./no-chain-state-updates.js";
-import noDerivedState from "./no-derived-state.js";
-import noPassDataToParent from "./no-pass-data-to-parent.js";
-import noManageParent from "./no-manage-parent.js";
+import noEmptyEffect from "./rules/no-empty-effect.js";
+import noAdjustStateOnPropChange from "./rules/no-adjust-state-on-prop-change.js";
+import noResetAllStateOnPropChange from "./rules/no-reset-all-state-on-prop-change.js";
+import noEventHandler from "./rules/no-event-handler.js";
+import noPassLiveStateToParent from "./rules/no-pass-live-state-to-parent.js";
+import noInitializeState from "./rules/no-initialize-state.js";
+import noChainStateUpdates from "./rules/no-chain-state-updates.js";
+import noDerivedState from "./rules/no-derived-state.js";
+import noPassDataToParent from "./rules/no-pass-data-to-parent.js";
+import noManageParent from "./rules/no-manage-parent.js";
 import globals from "globals";
-import "./util/javascript.js";
 
 /**
  * @type {import("eslint").ESLint.Plugin}
@@ -68,3 +67,8 @@ Object.assign(plugin.configs, {
 });
 
 export default plugin;
+
+// Wraps `Array.every()` to return false for empty arrays.
+Array.prototype.notEmptyEvery = function (predicate) {
+  return this.length > 0 && this.every(predicate);
+};

src/no-adjust-state-on-prop-change.js …/rules/no-adjust-state-on-prop-change.jssrc/no-adjust-state-on-prop-change.js renamed to src/rules/no-adjust-state-on-prop-change.js src/no-adjust-state-on-prop-change.js renamed to src/rules/no-adjust-state-on-prop-change.js

@@ -1,4 +1,4 @@
-import { getCallExpr } from "./util/ast.js";
+import { getCallExpr } from "../util/ast.js";
 import {
   getEffectDepsRefs,
   getEffectFnRefs,
@@ -7,7 +7,7 @@ import {
   isProp,
   isStateSetter,
   isUseEffect,
-} from "./util/react.js";
+} from "../util/ast.js";
 
 /**
  * @type {import("eslint").Rule.RuleModule}

src/no-chain-state-updates.js src/rules/no-chain-state-updates.jssrc/no-chain-state-updates.js renamed to src/rules/no-chain-state-updates.js

@@ -1,4 +1,4 @@
-import { getCallExpr } from "./util/ast.js";
+import { getCallExpr } from "../util/ast.js";
 import {
   getEffectDepsRefs,
   getEffectFnRefs,
@@ -8,7 +8,7 @@ import {
   isState,
   isStateSetter,
   isUseEffect,
-} from "./util/react.js";
+} from "../util/ast.js";
 
 /**
  * @type {import("eslint").Rule.RuleModule}

src/no-derived-state.js src/rules/no-derived-state.jssrc/no-derived-state.js renamed to src/rules/no-derived-state.js

@@ -9,8 +9,8 @@ import {
   hasCleanup,
   isUseEffect,
   getUpstreamRefs,
-} from "./util/react.js";
-import { getCallExpr, getDownstreamRefs } from "./util/ast.js";
+} from "../util/ast.js";
+import { getCallExpr, getDownstreamRefs } from "../util/ast.js";
 
 /**
  * @type {import('eslint').Rule.RuleModule}

src/no-empty-effect.js src/rules/no-empty-effect.jssrc/no-empty-effect.js renamed to src/rules/no-empty-effect.js

@@ -1,4 +1,4 @@
-import { isUseEffect, getEffectFnRefs } from "./util/react.js";
+import { isUseEffect, getEffectFnRefs } from "../util/ast.js";
 
 /**
  * @type {import("eslint").Rule.RuleModule}

src/no-event-handler.js src/rules/no-event-handler.jssrc/no-event-handler.js renamed to src/rules/no-event-handler.js

@@ -3,9 +3,9 @@ import {
   getEffectDepsRefs,
   hasCleanup,
   isUseEffect,
-} from "./util/react.js";
-import { findDownstreamNodes, getDownstreamRefs } from "./util/ast.js";
-import { isState } from "./util/react.js";
+} from "../util/ast.js";
+import { findDownstreamNodes, getDownstreamRefs } from "../util/ast.js";
+import { isState } from "../util/ast.js";
 
 /**
  * @type {import("eslint").Rule.RuleModule}

src/no-initialize-state.js src/rules/no-initialize-state.jssrc/no-initialize-state.js renamed to src/rules/no-initialize-state.js

@@ -1,12 +1,12 @@
-import { getCallExpr } from "./util/ast.js";
+import { getCallExpr } from "../util/ast.js";
 import {
   getEffectDepsRefs,
   getEffectFnRefs,
   getUseStateNode,
   isImmediateCall,
   isStateSetter,
   isUseEffect,
-} from "./util/react.js";
+} from "../util/ast.js";
 
 /**
  * @type {import("eslint").Rule.RuleModule}

src/no-manage-parent.js src/rules/no-manage-parent.jssrc/no-manage-parent.js renamed to src/rules/no-manage-parent.js

@@ -2,8 +2,8 @@ import {
   getEffectFnRefs,
   getEffectDepsRefs,
   isUseEffect,
-} from "./util/react.js";
-import { isProp } from "./util/react.js";
+} from "../util/ast.js";
+import { isProp } from "../util/ast.js";
 
 /**
  * @type {import("eslint").Rule.RuleModule}

src/no-pass-data-to-parent.js src/rules/no-pass-data-to-parent.jssrc/no-pass-data-to-parent.js renamed to src/rules/no-pass-data-to-parent.js

@@ -7,8 +7,8 @@ import {
   isProp,
   hasCleanup,
   isUseEffect,
-} from "./util/react.js";
-import { getCallExpr, getDownstreamRefs } from "./util/ast.js";
+} from "../util/ast.js";
+import { getCallExpr, getDownstreamRefs } from "../util/ast.js";
 
 /**
  * @type {import("eslint").Rule.RuleModule}

src/no-pass-live-state-to-parent.js src/rules/no-pass-live-state-to-parent.jssrc/no-pass-live-state-to-parent.js renamed to src/rules/no-pass-live-state-to-parent.js

@@ -5,8 +5,8 @@ import {
   isPropCallback,
   isState,
   isUseEffect,
-} from "./util/react.js";
-import { getCallExpr, getDownstreamRefs } from "./util/ast.js";
+} from "../util/ast.js";
+import { getCallExpr, getDownstreamRefs } from "../util/ast.js";
 
 /**
  * @type {import("eslint").Rule.RuleModule}