@sarj/eslint-plugin 15.26.2 → 15.26.4

package/dist/index.cjs

@@ -652,6 +652,29 @@ var no_broad_return_type_default = createRule({
 // src/rules/no-reduce-accumulator-copy.ts
 var import_utils5 = require("@typescript-eslint/utils");
 var import_typescript3 = __toESM(require("typescript"), 1);
+
+// src/rules/_for-each-ast-child.ts
+function forEachAstChild(node, visitorKeys, visit) {
+  for (const key of visitorKeys[node.type] ?? []) {
+    if (forEachAstChildKey(node, key, visit)) return true;
+  }
+  return false;
+}
+function forEachAstChildKey(node, key, visit) {
+  const value = node[key];
+  if (Array.isArray(value)) {
+    const children = value;
+    for (const child of children) if (isNode(child) && visit(child) === true) return true;
+  } else if (isNode(value)) {
+    if (visit(value) === true) return true;
+  }
+  return false;
+}
+function isNode(value) {
+  return typeof value === "object" && value !== null && "type" in value && typeof value.type === "string";
+}
+
+// src/rules/no-reduce-accumulator-copy.ts
 var NO_REDUCE_ACCUMULATOR_COPY_DOCUMENTATION = {
   summary: "Review copies of the accumulated collection inside a built-in reduce callback.",
   rationale: "Copying a growing accumulator at every step can make collection quadratic instead of linear in its output size.",
@@ -722,12 +745,7 @@ var no_reduce_accumulator_copy_default = createRule({
             context.report({ node: current, messageId: "copy" });
           }
           if (current.type === import_utils5.AST_NODE_TYPES.CallExpression) inspectCall(current);
-          for (const key of context.sourceCode.visitorKeys[current.type] ?? []) {
-            const child = current[key];
-            if (Array.isArray(child)) {
-              for (const item of child) if (item !== null && typeof item === "object" && "type" in item) inspect(item);
-            } else if (child !== null && typeof child === "object" && "type" in child) inspect(child);
-          }
+          forEachAstChild(current, context.sourceCode.visitorKeys, inspect);
         };
         inspect(callback.body);
       }
@@ -3546,17 +3564,17 @@ function subtreeContainsStarLiteral(node) {
     const value = node[key];
     if (Array.isArray(value)) {
       for (const child of value) {
-        if (isNode(child) && subtreeContainsStarLiteral(child)) {
+        if (isNode2(child) && subtreeContainsStarLiteral(child)) {
           return true;
         }
       }
-    } else if (isNode(value) && subtreeContainsStarLiteral(value)) {
+    } else if (isNode2(value) && subtreeContainsStarLiteral(value)) {
       return true;
     }
   }
   return false;
 }
-function isNode(value) {
+function isNode2(value) {
   return typeof value === "object" && value !== null && typeof value.type === "string";
 }
 function calleeName(node) {
@@ -3973,6 +3991,17 @@ var import_utils27 = require("@typescript-eslint/utils");
 
 // src/rules/_sql.ts
 var import_utils26 = require("@typescript-eslint/utils");
+
+// src/rules/_for-each-own-ast-child.ts
+function forEachOwnAstChild(node, visit, includeKey = () => true) {
+  for (const key of Object.keys(node)) {
+    if (key === "parent" || !includeKey(key)) continue;
+    if (forEachAstChildKey(node, key, visit)) return true;
+  }
+  return false;
+}
+
+// src/rules/_sql.ts
 function stripSqlNoise(text) {
   return scanSqlNoise(text);
 }
@@ -4104,17 +4133,7 @@ function isJoinedFragmentArray(node) {
 }
 function markConsumed(node, consumed) {
   consumed.add(node);
-  for (const key of Object.keys(node)) {
-    if (key === "parent") {
-      continue;
-    }
-    const value = node[key];
-    for (const child of Array.isArray(value) ? value : [value]) {
-      if (child !== null && typeof child === "object" && "type" in child) {
-        markConsumed(child, consumed);
-      }
-    }
-  }
+  forEachOwnAstChild(node, (child) => markConsumed(child, consumed));
 }
 function createSqlListener(handler) {
   const consumed = /* @__PURE__ */ new WeakSet();
@@ -4570,15 +4589,7 @@ function functionComplexity(fn, visitorKeys) {
     visitChildren(node, nesting);
   }
   function visitChildren(node, nesting) {
-    for (const key of visitorKeys[node.type] ?? []) {
-      const value = node[key];
-      const candidates = Array.isArray(value) ? value : [value];
-      for (const candidate2 of candidates) {
-        if (typeof candidate2 === "object" && candidate2 !== null && "type" in candidate2) {
-          visit(candidate2, nesting);
-        }
-      }
-    }
+    forEachAstChild(node, visitorKeys, (child) => visit(child, nesting));
   }
   visit(fn.body, 0);
   return points;
@@ -4747,9 +4758,6 @@ var PURE_CONSTRUCTORS = /* @__PURE__ */ new Set([
   "Response",
   "AbortController"
 ]);
-function isNode2(value) {
-  return typeof value === "object" && value !== null && typeof value.type === "string";
-}
 function isPureCall(node) {
   const callee = node.callee;
   if (callee.type !== import_utils30.AST_NODE_TYPES.MemberExpression) {
@@ -4787,27 +4795,7 @@ function subtreeMatches(stmt, predicate, descendIntoFunctions = false) {
       found = true;
       return;
     }
-    for (const key of Object.keys(current)) {
-      if (key === "parent") {
-        continue;
-      }
-      if (!descendIntoFunctions && NESTED_FUNCTION_TYPES.has(current.type) && key === "body") {
-        continue;
-      }
-      const value = current[key];
-      if (Array.isArray(value)) {
-        for (const child of value) {
-          if (isNode2(child)) {
-            visit(child);
-          }
-        }
-      } else if (isNode2(value)) {
-        visit(value);
-      }
-      if (found) {
-        return;
-      }
-    }
+    forEachOwnAstChild(current, visit, (key) => descendIntoFunctions || !NESTED_FUNCTION_TYPES.has(current.type) || key !== "body");
   };
   visit(stmt);
   return found;
@@ -7163,17 +7151,7 @@ function nearestFunction(node) {
 }
 function walkOwnScope(node, predicate) {
   if (predicate(node)) return true;
-  for (const key of Object.keys(node)) {
-    if (key === "parent") continue;
-    const value = node[key];
-    for (const child of Array.isArray(value) ? value : [value]) {
-      if (typeof child === "object" && child !== null && typeof child.type === "string") {
-        const childNode = child;
-        if (!FUNCTION_TYPES3.has(childNode.type) && walkOwnScope(childNode, predicate)) return true;
-      }
-    }
-  }
-  return false;
+  return forEachOwnAstChild(node, (child) => !FUNCTION_TYPES3.has(child.type) && walkOwnScope(child, predicate));
 }
 function isAssertion(node, context) {
   if (node.type !== import_utils44.AST_NODE_TYPES.CallExpression) return false;
@@ -9976,21 +9954,10 @@ function walkWithinScope(node, visit) {
     if (isFunctionNode(current)) {
       return;
     }
-    for (const key of Object.keys(current)) {
-      if (key === "parent") {
-        continue;
-      }
-      const value = current[key];
-      if (Array.isArray(value)) {
-        for (const child of value) {
-          if (isNode3(child)) {
-            recurse(child);
-          }
-        }
-      } else if (isNode3(value)) {
-        recurse(value);
-      }
-    }
+    forEachOwnAstChild(current, (child) => {
+      recurse(child);
+      return found;
+    });
   };
   recurse(node);
   return found;
@@ -10037,22 +10004,10 @@ function subtreeReadsName(node, name) {
     if (shadowsName(current)) {
       return;
     }
-    for (const key of Object.keys(current)) {
-      if (key === "parent") {
-        continue;
-      }
-      if (isNonReadingProperty(current, key)) continue;
-      const value = current[key];
-      if (Array.isArray(value)) {
-        for (const child of value) {
-          if (isNode3(child)) {
-            recurse(child);
-          }
-        }
-      } else if (isNode3(value)) {
-        recurse(value);
-      }
-    }
+    forEachOwnAstChild(current, (child) => {
+      recurse(child);
+      return found;
+    }, (key) => !isNonReadingProperty(current, key));
   };
   recurse(node);
   return found;
@@ -10171,14 +10126,10 @@ function tryReturnsSafeParse(catchNode) {
     visitChildren(current);
   };
   function visitChildren(current) {
-    for (const key of Object.keys(current)) {
-      if (key === "parent") continue;
-      const value = current[key];
-      const children = Array.isArray(value) ? value : [value];
-      for (const child of children) {
-        if (isNode3(child)) recurse(child);
-      }
-    }
+    forEachOwnAstChild(current, (child) => {
+      recurse(child);
+      return sawUnsafeOperation;
+    });
   }
   recurse(tryBlock);
   return sawSafeParse && !sawUnsafeOperation;
@@ -16315,13 +16266,7 @@ function isEmptyGuard(node, access) {
 }
 function contains(node, visitorKeys, predicate) {
   if (predicate(node)) return true;
-  for (const key of visitorKeys[node.type] ?? []) {
-    const child = node[key];
-    for (const value of Array.isArray(child) ? child : [child]) {
-      if (value !== null && typeof value === "object" && "type" in value && contains(value, visitorKeys, predicate)) return true;
-    }
-  }
-  return false;
+  return forEachAstChild(node, visitorKeys, (child) => contains(child, visitorKeys, predicate));
 }
 function belongsToFunction(node, fn) {
   let current = node;
@@ -22085,14 +22030,7 @@ function referencedPropertyName(node) {
 function walk(node, visitorKeys, visit, nestedFunction = false) {
   visit(node, nestedFunction);
   const nested = nestedFunction || isFunction(node);
-  for (const key of visitorKeys[node.type] ?? []) {
-    const child = node[key];
-    if (Array.isArray(child)) {
-      for (const item of child) if (typeof item === "object" && item !== null && "type" in item) walk(item, visitorKeys, visit, nested);
-    } else if (typeof child === "object" && child !== null && "type" in child) {
-      walk(child, visitorKeys, visit, nested);
-    }
-  }
+  forEachAstChild(node, visitorKeys, (child) => walk(child, visitorKeys, visit, nested));
 }
 function classScope(context, node, computedReferenceNames) {
   const methods = node.body.body.filter(
@@ -23285,7 +23223,7 @@ var RULES = {
 };
 var meta = {
   name: "@sarj/eslint-plugin",
-  version: "15.26.2"
+  version: "15.26.4"
 };
 var APPLICATION_ONLY_RULES = [];
 var LIBRARY_IMPORT_POLICY = ["error", {

package/dist/index.d.cts

@@ -996,7 +996,7 @@ type FlatPreset = {
 declare const PLUGIN: {
     readonly meta: {
         readonly name: "@sarj/eslint-plugin";
-        readonly version: "15.26.2";
+        readonly version: "15.26.4";
     };
     readonly rules: {
         readonly "no-conditional-empty-object-spread": DocumentedRule<[], "avoid">;

package/dist/index.d.ts

@@ -996,7 +996,7 @@ type FlatPreset = {
 declare const PLUGIN: {
     readonly meta: {
         readonly name: "@sarj/eslint-plugin";
-        readonly version: "15.26.2";
+        readonly version: "15.26.4";
     };
     readonly rules: {
         readonly "no-conditional-empty-object-spread": DocumentedRule<[], "avoid">;

package/dist/index.js

@@ -608,6 +608,29 @@ var no_broad_return_type_default = createRule({
 // src/rules/no-reduce-accumulator-copy.ts
 import { AST_NODE_TYPES as AST_NODE_TYPES4, ASTUtils as ASTUtils2, ESLintUtils as ESLintUtils4 } from "@typescript-eslint/utils";
 import ts3 from "typescript";
+
+// src/rules/_for-each-ast-child.ts
+function forEachAstChild(node, visitorKeys, visit) {
+  for (const key of visitorKeys[node.type] ?? []) {
+    if (forEachAstChildKey(node, key, visit)) return true;
+  }
+  return false;
+}
+function forEachAstChildKey(node, key, visit) {
+  const value = node[key];
+  if (Array.isArray(value)) {
+    const children = value;
+    for (const child of children) if (isNode(child) && visit(child) === true) return true;
+  } else if (isNode(value)) {
+    if (visit(value) === true) return true;
+  }
+  return false;
+}
+function isNode(value) {
+  return typeof value === "object" && value !== null && "type" in value && typeof value.type === "string";
+}
+
+// src/rules/no-reduce-accumulator-copy.ts
 var NO_REDUCE_ACCUMULATOR_COPY_DOCUMENTATION = {
   summary: "Review copies of the accumulated collection inside a built-in reduce callback.",
   rationale: "Copying a growing accumulator at every step can make collection quadratic instead of linear in its output size.",
@@ -678,12 +701,7 @@ var no_reduce_accumulator_copy_default = createRule({
             context.report({ node: current, messageId: "copy" });
           }
           if (current.type === AST_NODE_TYPES4.CallExpression) inspectCall(current);
-          for (const key of context.sourceCode.visitorKeys[current.type] ?? []) {
-            const child = current[key];
-            if (Array.isArray(child)) {
-              for (const item of child) if (item !== null && typeof item === "object" && "type" in item) inspect(item);
-            } else if (child !== null && typeof child === "object" && "type" in child) inspect(child);
-          }
+          forEachAstChild(current, context.sourceCode.visitorKeys, inspect);
         };
         inspect(callback.body);
       }
@@ -3510,17 +3528,17 @@ function subtreeContainsStarLiteral(node) {
     const value = node[key];
     if (Array.isArray(value)) {
       for (const child of value) {
-        if (isNode(child) && subtreeContainsStarLiteral(child)) {
+        if (isNode2(child) && subtreeContainsStarLiteral(child)) {
           return true;
         }
       }
-    } else if (isNode(value) && subtreeContainsStarLiteral(value)) {
+    } else if (isNode2(value) && subtreeContainsStarLiteral(value)) {
       return true;
     }
   }
   return false;
 }
-function isNode(value) {
+function isNode2(value) {
   return typeof value === "object" && value !== null && typeof value.type === "string";
 }
 function calleeName(node) {
@@ -3937,6 +3955,17 @@ import { AST_NODE_TYPES as AST_NODE_TYPES25 } from "@typescript-eslint/utils";
 
 // src/rules/_sql.ts
 import { AST_NODE_TYPES as AST_NODE_TYPES24 } from "@typescript-eslint/utils";
+
+// src/rules/_for-each-own-ast-child.ts
+function forEachOwnAstChild(node, visit, includeKey = () => true) {
+  for (const key of Object.keys(node)) {
+    if (key === "parent" || !includeKey(key)) continue;
+    if (forEachAstChildKey(node, key, visit)) return true;
+  }
+  return false;
+}
+
+// src/rules/_sql.ts
 function stripSqlNoise(text) {
   return scanSqlNoise(text);
 }
@@ -4068,17 +4097,7 @@ function isJoinedFragmentArray(node) {
 }
 function markConsumed(node, consumed) {
   consumed.add(node);
-  for (const key of Object.keys(node)) {
-    if (key === "parent") {
-      continue;
-    }
-    const value = node[key];
-    for (const child of Array.isArray(value) ? value : [value]) {
-      if (child !== null && typeof child === "object" && "type" in child) {
-        markConsumed(child, consumed);
-      }
-    }
-  }
+  forEachOwnAstChild(node, (child) => markConsumed(child, consumed));
 }
 function createSqlListener(handler) {
   const consumed = /* @__PURE__ */ new WeakSet();
@@ -4534,15 +4553,7 @@ function functionComplexity(fn, visitorKeys) {
     visitChildren(node, nesting);
   }
   function visitChildren(node, nesting) {
-    for (const key of visitorKeys[node.type] ?? []) {
-      const value = node[key];
-      const candidates = Array.isArray(value) ? value : [value];
-      for (const candidate2 of candidates) {
-        if (typeof candidate2 === "object" && candidate2 !== null && "type" in candidate2) {
-          visit(candidate2, nesting);
-        }
-      }
-    }
+    forEachAstChild(node, visitorKeys, (child) => visit(child, nesting));
   }
   visit(fn.body, 0);
   return points;
@@ -4711,9 +4722,6 @@ var PURE_CONSTRUCTORS = /* @__PURE__ */ new Set([
   "Response",
   "AbortController"
 ]);
-function isNode2(value) {
-  return typeof value === "object" && value !== null && typeof value.type === "string";
-}
 function isPureCall(node) {
   const callee = node.callee;
   if (callee.type !== AST_NODE_TYPES27.MemberExpression) {
@@ -4751,27 +4759,7 @@ function subtreeMatches(stmt, predicate, descendIntoFunctions = false) {
       found = true;
       return;
     }
-    for (const key of Object.keys(current)) {
-      if (key === "parent") {
-        continue;
-      }
-      if (!descendIntoFunctions && NESTED_FUNCTION_TYPES.has(current.type) && key === "body") {
-        continue;
-      }
-      const value = current[key];
-      if (Array.isArray(value)) {
-        for (const child of value) {
-          if (isNode2(child)) {
-            visit(child);
-          }
-        }
-      } else if (isNode2(value)) {
-        visit(value);
-      }
-      if (found) {
-        return;
-      }
-    }
+    forEachOwnAstChild(current, visit, (key) => descendIntoFunctions || !NESTED_FUNCTION_TYPES.has(current.type) || key !== "body");
   };
   visit(stmt);
   return found;
@@ -7145,17 +7133,7 @@ function nearestFunction(node) {
 }
 function walkOwnScope(node, predicate) {
   if (predicate(node)) return true;
-  for (const key of Object.keys(node)) {
-    if (key === "parent") continue;
-    const value = node[key];
-    for (const child of Array.isArray(value) ? value : [value]) {
-      if (typeof child === "object" && child !== null && typeof child.type === "string") {
-        const childNode = child;
-        if (!FUNCTION_TYPES3.has(childNode.type) && walkOwnScope(childNode, predicate)) return true;
-      }
-    }
-  }
-  return false;
+  return forEachOwnAstChild(node, (child) => !FUNCTION_TYPES3.has(child.type) && walkOwnScope(child, predicate));
 }
 function isAssertion(node, context) {
   if (node.type !== AST_NODE_TYPES37.CallExpression) return false;
@@ -9958,21 +9936,10 @@ function walkWithinScope(node, visit) {
     if (isFunctionNode(current)) {
       return;
     }
-    for (const key of Object.keys(current)) {
-      if (key === "parent") {
-        continue;
-      }
-      const value = current[key];
-      if (Array.isArray(value)) {
-        for (const child of value) {
-          if (isNode3(child)) {
-            recurse(child);
-          }
-        }
-      } else if (isNode3(value)) {
-        recurse(value);
-      }
-    }
+    forEachOwnAstChild(current, (child) => {
+      recurse(child);
+      return found;
+    });
   };
   recurse(node);
   return found;
@@ -10019,22 +9986,10 @@ function subtreeReadsName(node, name) {
     if (shadowsName(current)) {
       return;
     }
-    for (const key of Object.keys(current)) {
-      if (key === "parent") {
-        continue;
-      }
-      if (isNonReadingProperty(current, key)) continue;
-      const value = current[key];
-      if (Array.isArray(value)) {
-        for (const child of value) {
-          if (isNode3(child)) {
-            recurse(child);
-          }
-        }
-      } else if (isNode3(value)) {
-        recurse(value);
-      }
-    }
+    forEachOwnAstChild(current, (child) => {
+      recurse(child);
+      return found;
+    }, (key) => !isNonReadingProperty(current, key));
   };
   recurse(node);
   return found;
@@ -10153,14 +10108,10 @@ function tryReturnsSafeParse(catchNode) {
     visitChildren(current);
   };
   function visitChildren(current) {
-    for (const key of Object.keys(current)) {
-      if (key === "parent") continue;
-      const value = current[key];
-      const children = Array.isArray(value) ? value : [value];
-      for (const child of children) {
-        if (isNode3(child)) recurse(child);
-      }
-    }
+    forEachOwnAstChild(current, (child) => {
+      recurse(child);
+      return sawUnsafeOperation;
+    });
   }
   recurse(tryBlock);
   return sawSafeParse && !sawUnsafeOperation;
@@ -16316,13 +16267,7 @@ function isEmptyGuard(node, access) {
 }
 function contains(node, visitorKeys, predicate) {
   if (predicate(node)) return true;
-  for (const key of visitorKeys[node.type] ?? []) {
-    const child = node[key];
-    for (const value of Array.isArray(child) ? child : [child]) {
-      if (value !== null && typeof value === "object" && "type" in value && contains(value, visitorKeys, predicate)) return true;
-    }
-  }
-  return false;
+  return forEachAstChild(node, visitorKeys, (child) => contains(child, visitorKeys, predicate));
 }
 function belongsToFunction(node, fn) {
   let current = node;
@@ -22103,14 +22048,7 @@ function referencedPropertyName(node) {
 function walk(node, visitorKeys, visit, nestedFunction = false) {
   visit(node, nestedFunction);
   const nested = nestedFunction || isFunction(node);
-  for (const key of visitorKeys[node.type] ?? []) {
-    const child = node[key];
-    if (Array.isArray(child)) {
-      for (const item of child) if (typeof item === "object" && item !== null && "type" in item) walk(item, visitorKeys, visit, nested);
-    } else if (typeof child === "object" && child !== null && "type" in child) {
-      walk(child, visitorKeys, visit, nested);
-    }
-  }
+  forEachAstChild(node, visitorKeys, (child) => walk(child, visitorKeys, visit, nested));
 }
 function classScope(context, node, computedReferenceNames) {
   const methods = node.body.body.filter(
@@ -23306,7 +23244,7 @@ var RULES = {
 };
 var meta = {
   name: "@sarj/eslint-plugin",
-  version: "15.26.2"
+  version: "15.26.4"
 };
 var APPLICATION_ONLY_RULES = [];
 var LIBRARY_IMPORT_POLICY = ["error", {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sarj/eslint-plugin",
-  "version": "15.26.2",
+  "version": "15.26.4",
   "packageManager": "npm@12.0.2",
   "description": "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   "type": "module",