@sarj/eslint-plugin 11.1.0 → 11.2.0

package/dist/index.cjs

@@ -869,6 +869,7 @@ var BANNER_RUN_RE = /={4,}|-{4,}|#{4,}|\*{4,}|~{4,}|[\u2500-\u257f]{4,}/;
 var REGION_MARKER_RE = /^#?(?:end)?region\b(.*)$/i;
 var REGION_TITLE_RE = /^[\s:\-\u2013\u2014]*\w[\w \-/&+]*$/;
 var REGION_TITLE_MAX_WORDS = 5;
+var REGION_PROSE_VERB_RE = /^(?:is|are|was|were|comes?|defaults?|derives?|inherits?|depends?|uses?|maps?|resolves?)\b/i;
 function stripCommentMarker(line) {
   return line.replace(/^\s*\/{1,2}/, "").replace(/^\s*\*+/, "").trim();
 }
@@ -930,7 +931,9 @@ var CALL_RE = /^[A-Za-z_$][\w.$]*\([^)]*\)\s*;?\s*$/;
 var ASSERTION_CODE_RE = /^(?:await\s+)?(?:expect(?:TypeOf)?|assert(?:\.\w+)?)\s*\(/;
 var HTTP_CONTRACT_RE = /\b(?:GET|HEAD|OPTIONS|PATCH|POST|PUT|DELETE)\s+(?:https?:\/\/|\/|\{[A-Za-z_$])/;
 var PROSE_ASSIGNMENT_RE = /^[A-Za-z_$][\w.$[\]]*\s*=\s*[A-Za-z][A-Za-z '-]{2,}\s+\((?:how|what|when|where|which|who|why)\b[^)]*\)[.!]?$/i;
-var PSEUDOCODE_RE = /%\w+%|\[opt\]|(?:^|\s)<[A-Za-z]\w*>|…|\.\.\./;
+var PSEUDOCODE_RE = /%\w+%|\[opt\]|(?:^|\s)<[A-Za-z]\w*>|\b[A-Za-z_$][\w$]*\s+x\d+\b|…|\.\.\./;
+var CALL_LABEL_RE = /^([A-Za-z_$][\w$]*(?:\.[A-Za-z_$][\w$]*)*)\([^;{}]*\)$/;
+var MULTIPLIER_PSEUDOCODE_RE = /\b[A-Za-z_$][\w$]*\s+x\d+\b/;
 function isBanner(text) {
   const t = text.trim();
   if (!t) return false;
@@ -942,6 +945,7 @@ function isRegionMarker(text) {
   if (match === null) return false;
   const title = (match[1] ?? "").trim();
   if (title.length === 0) return true;
+  if (REGION_PROSE_VERB_RE.test(title)) return false;
   if (!REGION_TITLE_RE.test(title)) return false;
   return title.split(/\s+/).length <= REGION_TITLE_MAX_WORDS;
 }
@@ -957,6 +961,15 @@ function looksLikeCode(text, allowCall = true) {
 function hasPseudocode(text) {
   return PSEUDOCODE_RE.test(text);
 }
+function testCallMatrixStems(comments) {
+  const stems = /* @__PURE__ */ new Set();
+  for (const comment of comments) {
+    const body2 = stripCommentMarker(comment.value);
+    const stem3 = CALL_LABEL_RE.exec(body2)?.[1];
+    if (stem3 !== void 0 && MULTIPLIER_PSEUDOCODE_RE.test(body2)) stems.add(stem3);
+  }
+  return stems;
+}
 function isEnumeratedProseItem(text) {
   const t = text.trim();
   return ENUMERATED_ITEM_RE.test(t) && t.split(/\s+/).length >= ENUMERATED_ITEM_MIN_WORDS;
@@ -1245,6 +1258,7 @@ var no_comment_cruft_default = createRule({
     return {
       Program() {
         const comments = sourceCode.getAllComments();
+        const callMatrixStems = isTestFile(context.filename) ? testCallMatrixStems(comments) : /* @__PURE__ */ new Set();
         const walls = findCommentWalls(comments);
         const wallByLeader = new Map(walls.map((wall) => [wall.leader, wall]));
         const wallMembers = new Set(walls.flatMap((wall) => [...wall.members]));
@@ -1282,6 +1296,8 @@ var no_comment_cruft_default = createRule({
             continue;
           }
           const firstText = texts[0];
+          const firstTextStem = firstText === void 0 ? void 0 : CALL_LABEL_RE.exec(firstText)?.[1];
+          if (firstTextStem !== void 0 && texts.length === 1 && callMatrixStems.has(firstTextStem)) continue;
           if (firstText !== void 0 && /^(?:todo|fixme)\b/i.test(firstText)) {
             if (!runCitesAReference(comments, i)) {
               context.report({
@@ -2211,6 +2227,20 @@ var PURE_METHODS = /* @__PURE__ */ new Set([
   "startsWith",
   "endsWith"
 ]);
+var SYNC_CALLBACK_METHODS = /* @__PURE__ */ new Set([
+  "every",
+  "filter",
+  "findIndex",
+  "findLast",
+  "findLastIndex",
+  "flatMap",
+  "forEach",
+  "map",
+  "reduce",
+  "reduceRight",
+  "some",
+  "sort"
+]);
 var PURE_NAMESPACES = /* @__PURE__ */ new Set([
   "Object",
   "Array",
@@ -2260,7 +2290,17 @@ function isPureCall(node) {
   if (callee.object.type === import_utils13.AST_NODE_TYPES.Identifier && PURE_NAMESPACES.has(callee.object.name)) {
     return !IMPURE_NAMESPACE_METHODS.has(`${callee.object.name}.${property.name}`);
   }
-  return PURE_METHODS.has(property.name);
+  return PURE_METHODS.has(property.name) && (!SYNC_CALLBACK_METHODS.has(property.name) || !synchronousCallbackCanThrow(node));
+}
+function synchronousCallbackCanThrow(node) {
+  return node.arguments.some((argument) => {
+    if (argument.type !== import_utils13.AST_NODE_TYPES.ArrowFunctionExpression && argument.type !== import_utils13.AST_NODE_TYPES.FunctionExpression) return false;
+    if (argument.async) return false;
+    return subtreeMatches2(
+      argument.body,
+      (current) => current.type === import_utils13.AST_NODE_TYPES.ThrowStatement || current.type === import_utils13.AST_NODE_TYPES.CallExpression && current.callee.type === import_utils13.AST_NODE_TYPES.MemberExpression && !current.callee.computed && current.callee.object.type === import_utils13.AST_NODE_TYPES.Identifier && current.callee.object.name === "JSON" && current.callee.property.type === import_utils13.AST_NODE_TYPES.Identifier && current.callee.property.name === "parse"
+    );
+  });
 }
 function isPureNew(node) {
   return node.callee.type === import_utils13.AST_NODE_TYPES.Identifier && PURE_CONSTRUCTORS.has(node.callee.name);
@@ -3522,6 +3562,8 @@ function typedFunction(node) {
 
 // src/rules/no-long-comment.ts
 var EXCESSIVE_SENTENCE_COUNT = 8;
+var EXCESSIVE_WORD_COUNT = 120;
+var PROSE_WORD_RE = /[\p{L}\p{N}][\p{L}\p{N}'’-]*/gu;
 var VERSIONED_DEPENDENCY_TREE_RE = /(?:^|\/)lib\/[^/]*-?v?\d+\.\d+(?:\.\d+)?[^/]*\//iu;
 function documentsTypeOrMember(sourceCode, comment) {
   const token = sourceCode.getTokenAfter(comment, { includeComments: false });
@@ -3533,6 +3575,9 @@ function documentsTypeOrMember(sourceCode, comment) {
   }
   return false;
 }
+function wordUnits(text) {
+  return text.match(PROSE_WORD_RE)?.length ?? 0;
+}
 var no_long_comment_default = createRule({
   name: "no-long-comment",
   meta: {
@@ -3550,7 +3595,7 @@ var no_long_comment_default = createRule({
     return {
       Program() {
         for (const group of proseGroups(context.filename, context.sourceCode)) {
-          if (group.comment.type !== "Block" || !group.comment.value.startsWith("*") || group.hasTypedTags || hasDocumentationStructure(group.text) || hasTechnicalAnchor(group.text) || documentsTypedFunction(context.sourceCode, group.comment) || documentsTypeOrMember(context.sourceCode, group.comment) || sentenceUnits(group.text) < EXCESSIVE_SENTENCE_COUNT) continue;
+          if (group.comment.type !== "Block" || !group.comment.value.startsWith("*") || group.hasTypedTags || hasDocumentationStructure(group.text) || hasTechnicalAnchor(group.text) || documentsTypedFunction(context.sourceCode, group.comment) || documentsTypeOrMember(context.sourceCode, group.comment) || sentenceUnits(group.text) < EXCESSIVE_SENTENCE_COUNT && wordUnits(group.text) < EXCESSIVE_WORD_COUNT) continue;
           context.report({ node: group.comment, messageId: "tooLong" });
         }
       }
@@ -8554,6 +8599,11 @@ var prefer_module_level_schema_default = createRule({
             type: "boolean",
             description: "Skip test files, where a fixture schema belongs next to its assertion."
           },
+          memoCallees: {
+            type: "array",
+            items: { type: "string" },
+            description: "Wrappers that construct their callback at most once. Defaults to lazy, memo, once, and useMemo."
+          },
           minProperties: {
             type: "number",
             minimum: 0,
@@ -8571,6 +8621,7 @@ var prefer_module_level_schema_default = createRule({
   create(context, [options]) {
     const factories = new Set(options?.factories ?? DEFAULT_FACTORIES);
     const ignoreTestFiles = options?.ignoreTestFiles ?? true;
+    const memoCallees = new Set(options?.memoCallees ?? MEMO_CALLEES);
     const minProperties = options?.minProperties ?? DEFAULT_MIN_PROPERTIES;
     const sourceCode = context.sourceCode;
     const filename = context.filename;
@@ -8590,7 +8641,7 @@ var prefer_module_level_schema_default = createRule({
         if (current !== node && isZodCall(current) && current.callee.type === import_utils56.AST_NODE_TYPES.MemberExpression && current.callee.property.type === import_utils56.AST_NODE_TYPES.Identifier && factories.has(current.callee.property.name)) {
           return true;
         }
-        if (current.type === import_utils56.AST_NODE_TYPES.CallExpression && (current.callee.type === import_utils56.AST_NODE_TYPES.Identifier && MEMO_CALLEES.has(current.callee.name) || current.callee.type === import_utils56.AST_NODE_TYPES.MemberExpression && !current.callee.computed && current.callee.property.type === import_utils56.AST_NODE_TYPES.Identifier && MEMO_CALLEES.has(current.callee.property.name))) {
+        if (current.type === import_utils56.AST_NODE_TYPES.CallExpression && (current.callee.type === import_utils56.AST_NODE_TYPES.Identifier && memoCallees.has(current.callee.name) || current.callee.type === import_utils56.AST_NODE_TYPES.MemberExpression && !current.callee.computed && current.callee.property.type === import_utils56.AST_NODE_TYPES.Identifier && memoCallees.has(current.callee.property.name))) {
           return true;
         }
         current = current.parent ?? void 0;
@@ -8640,6 +8691,9 @@ var prefer_module_level_schema_default = createRule({
           if (definition.type === "ImportBinding") {
             continue;
           }
+          if (definition.node.type === import_utils56.AST_NODE_TYPES.VariableDeclarator && definition.node.parent.type === import_utils56.AST_NODE_TYPES.VariableDeclaration && definition.node.parent.kind !== "const") {
+            return false;
+          }
           const [defStart, defEnd] = definition.node.range;
           if (defStart >= schemaStart && defEnd <= schemaEnd) {
             continue;
@@ -8874,7 +8928,7 @@ function sameAccess(node, access) {
   return node.type === import_utils58.AST_NODE_TYPES.MemberExpression && !node.computed && node.object.type === import_utils58.AST_NODE_TYPES.Identifier && node.object.name === access.object && node.property.type === import_utils58.AST_NODE_TYPES.Identifier && node.property.name === access.property;
 }
 function isNullGuard(node, access) {
-  if (node.type === import_utils58.AST_NODE_TYPES.UnaryExpression && node.operator === "!" && sameAccess(node.argument, access)) return true;
+  if (node.type === import_utils58.AST_NODE_TYPES.UnaryExpression && node.operator === "!" && (sameAccess(node.argument, access) || optionalMemberLengthOf(node.argument, access))) return true;
   if (node.type !== import_utils58.AST_NODE_TYPES.BinaryExpression || !["==", "==="].includes(node.operator)) {
     return false;
   }
@@ -8890,7 +8944,11 @@ function isEmptyGuard(node, access) {
   return memberLengthOf(node.left, access) && zero(node.right) || memberLengthOf(node.right, access) && zero(node.left);
 }
 function memberLengthOf(node, access) {
-  return node.type === import_utils58.AST_NODE_TYPES.MemberExpression && !node.computed && node.property.type === import_utils58.AST_NODE_TYPES.Identifier && node.property.name === "length" && sameAccess(node.object, access);
+  const target = node.type === import_utils58.AST_NODE_TYPES.ChainExpression ? node.expression : node;
+  return target.type === import_utils58.AST_NODE_TYPES.MemberExpression && !target.computed && target.property.type === import_utils58.AST_NODE_TYPES.Identifier && target.property.name === "length" && sameAccess(target.object, access);
+}
+function optionalMemberLengthOf(node, access) {
+  return node.type === import_utils58.AST_NODE_TYPES.ChainExpression && node.expression.type === import_utils58.AST_NODE_TYPES.MemberExpression && node.expression.optional && memberLengthOf(node, access);
 }
 function hasEquivalentLeadingGuard(fn, access, visitorKeys) {
   if (fn.body.type !== import_utils58.AST_NODE_TYPES.BlockStatement) return false;
@@ -8898,6 +8956,7 @@ function hasEquivalentLeadingGuard(fn, access, visitorKeys) {
   if (first?.type !== import_utils58.AST_NODE_TYPES.IfStatement) return false;
   const terminating = first.consequent.type === import_utils58.AST_NODE_TYPES.ReturnStatement || first.consequent.type === import_utils58.AST_NODE_TYPES.ThrowStatement || first.consequent.type === import_utils58.AST_NODE_TYPES.BlockStatement && first.consequent.body.length === 1 && (first.consequent.body[0]?.type === import_utils58.AST_NODE_TYPES.ReturnStatement || first.consequent.body[0]?.type === import_utils58.AST_NODE_TYPES.ThrowStatement);
   if (!terminating) return false;
+  if (contains(first.consequent, visitorKeys, (node) => sameAccess(node, access))) return false;
   return contains(first.test, visitorKeys, (node) => isNullGuard(node, access)) && contains(first.test, visitorKeys, (node) => isEmptyGuard(node, access));
 }
 function contains(node, visitorKeys, predicate) {
@@ -12641,7 +12700,7 @@ var rules = {
 };
 var meta = {
   name: "@sarj/eslint-plugin",
-  version: "11.1.0"
+  version: "11.2.0"
 };
 var applicationOnlyRules = [
   "no-restricted-library-load",

package/dist/index.d.cts

@@ -246,6 +246,7 @@ declare const rules: {
     readonly "prefer-module-level-schema": _typescript_eslint_utils_ts_eslint.RuleModule<"hoistSchema", readonly [{
         factories?: readonly string[];
         ignoreTestFiles?: boolean;
+        memoCallees?: readonly string[];
         minProperties?: number;
     }?], unknown, _typescript_eslint_utils_ts_eslint.RuleListener> & {
         name: string;
@@ -453,7 +454,7 @@ type FlatPreset = {
 declare const plugin: {
     readonly meta: {
         readonly name: "@sarj/eslint-plugin";
-        readonly version: "11.1.0";
+        readonly version: "11.2.0";
     };
     readonly rules: {
         readonly "duplicate-test-body": _typescript_eslint_utils_ts_eslint.RuleModule<"duplicateTestBody", readonly [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener> & {
@@ -614,6 +615,7 @@ declare const plugin: {
         readonly "prefer-module-level-schema": _typescript_eslint_utils_ts_eslint.RuleModule<"hoistSchema", readonly [{
             factories?: readonly string[];
             ignoreTestFiles?: boolean;
+            memoCallees?: readonly string[];
             minProperties?: number;
         }?], unknown, _typescript_eslint_utils_ts_eslint.RuleListener> & {
             name: string;

package/dist/index.d.ts

@@ -246,6 +246,7 @@ declare const rules: {
     readonly "prefer-module-level-schema": _typescript_eslint_utils_ts_eslint.RuleModule<"hoistSchema", readonly [{
         factories?: readonly string[];
         ignoreTestFiles?: boolean;
+        memoCallees?: readonly string[];
         minProperties?: number;
     }?], unknown, _typescript_eslint_utils_ts_eslint.RuleListener> & {
         name: string;
@@ -453,7 +454,7 @@ type FlatPreset = {
 declare const plugin: {
     readonly meta: {
         readonly name: "@sarj/eslint-plugin";
-        readonly version: "11.1.0";
+        readonly version: "11.2.0";
     };
     readonly rules: {
         readonly "duplicate-test-body": _typescript_eslint_utils_ts_eslint.RuleModule<"duplicateTestBody", readonly [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener> & {
@@ -614,6 +615,7 @@ declare const plugin: {
         readonly "prefer-module-level-schema": _typescript_eslint_utils_ts_eslint.RuleModule<"hoistSchema", readonly [{
             factories?: readonly string[];
             ignoreTestFiles?: boolean;
+            memoCallees?: readonly string[];
             minProperties?: number;
         }?], unknown, _typescript_eslint_utils_ts_eslint.RuleListener> & {
             name: string;

package/dist/index.js

@@ -827,6 +827,7 @@ var BANNER_RUN_RE = /={4,}|-{4,}|#{4,}|\*{4,}|~{4,}|[\u2500-\u257f]{4,}/;
 var REGION_MARKER_RE = /^#?(?:end)?region\b(.*)$/i;
 var REGION_TITLE_RE = /^[\s:\-\u2013\u2014]*\w[\w \-/&+]*$/;
 var REGION_TITLE_MAX_WORDS = 5;
+var REGION_PROSE_VERB_RE = /^(?:is|are|was|were|comes?|defaults?|derives?|inherits?|depends?|uses?|maps?|resolves?)\b/i;
 function stripCommentMarker(line) {
   return line.replace(/^\s*\/{1,2}/, "").replace(/^\s*\*+/, "").trim();
 }
@@ -888,7 +889,9 @@ var CALL_RE = /^[A-Za-z_$][\w.$]*\([^)]*\)\s*;?\s*$/;
 var ASSERTION_CODE_RE = /^(?:await\s+)?(?:expect(?:TypeOf)?|assert(?:\.\w+)?)\s*\(/;
 var HTTP_CONTRACT_RE = /\b(?:GET|HEAD|OPTIONS|PATCH|POST|PUT|DELETE)\s+(?:https?:\/\/|\/|\{[A-Za-z_$])/;
 var PROSE_ASSIGNMENT_RE = /^[A-Za-z_$][\w.$[\]]*\s*=\s*[A-Za-z][A-Za-z '-]{2,}\s+\((?:how|what|when|where|which|who|why)\b[^)]*\)[.!]?$/i;
-var PSEUDOCODE_RE = /%\w+%|\[opt\]|(?:^|\s)<[A-Za-z]\w*>|…|\.\.\./;
+var PSEUDOCODE_RE = /%\w+%|\[opt\]|(?:^|\s)<[A-Za-z]\w*>|\b[A-Za-z_$][\w$]*\s+x\d+\b|…|\.\.\./;
+var CALL_LABEL_RE = /^([A-Za-z_$][\w$]*(?:\.[A-Za-z_$][\w$]*)*)\([^;{}]*\)$/;
+var MULTIPLIER_PSEUDOCODE_RE = /\b[A-Za-z_$][\w$]*\s+x\d+\b/;
 function isBanner(text) {
   const t = text.trim();
   if (!t) return false;
@@ -900,6 +903,7 @@ function isRegionMarker(text) {
   if (match === null) return false;
   const title = (match[1] ?? "").trim();
   if (title.length === 0) return true;
+  if (REGION_PROSE_VERB_RE.test(title)) return false;
   if (!REGION_TITLE_RE.test(title)) return false;
   return title.split(/\s+/).length <= REGION_TITLE_MAX_WORDS;
 }
@@ -915,6 +919,15 @@ function looksLikeCode(text, allowCall = true) {
 function hasPseudocode(text) {
   return PSEUDOCODE_RE.test(text);
 }
+function testCallMatrixStems(comments) {
+  const stems = /* @__PURE__ */ new Set();
+  for (const comment of comments) {
+    const body2 = stripCommentMarker(comment.value);
+    const stem3 = CALL_LABEL_RE.exec(body2)?.[1];
+    if (stem3 !== void 0 && MULTIPLIER_PSEUDOCODE_RE.test(body2)) stems.add(stem3);
+  }
+  return stems;
+}
 function isEnumeratedProseItem(text) {
   const t = text.trim();
   return ENUMERATED_ITEM_RE.test(t) && t.split(/\s+/).length >= ENUMERATED_ITEM_MIN_WORDS;
@@ -1203,6 +1216,7 @@ var no_comment_cruft_default = createRule({
     return {
       Program() {
         const comments = sourceCode.getAllComments();
+        const callMatrixStems = isTestFile(context.filename) ? testCallMatrixStems(comments) : /* @__PURE__ */ new Set();
         const walls = findCommentWalls(comments);
         const wallByLeader = new Map(walls.map((wall) => [wall.leader, wall]));
         const wallMembers = new Set(walls.flatMap((wall) => [...wall.members]));
@@ -1240,6 +1254,8 @@ var no_comment_cruft_default = createRule({
             continue;
           }
           const firstText = texts[0];
+          const firstTextStem = firstText === void 0 ? void 0 : CALL_LABEL_RE.exec(firstText)?.[1];
+          if (firstTextStem !== void 0 && texts.length === 1 && callMatrixStems.has(firstTextStem)) continue;
           if (firstText !== void 0 && /^(?:todo|fixme)\b/i.test(firstText)) {
             if (!runCitesAReference(comments, i)) {
               context.report({
@@ -2169,6 +2185,20 @@ var PURE_METHODS = /* @__PURE__ */ new Set([
   "startsWith",
   "endsWith"
 ]);
+var SYNC_CALLBACK_METHODS = /* @__PURE__ */ new Set([
+  "every",
+  "filter",
+  "findIndex",
+  "findLast",
+  "findLastIndex",
+  "flatMap",
+  "forEach",
+  "map",
+  "reduce",
+  "reduceRight",
+  "some",
+  "sort"
+]);
 var PURE_NAMESPACES = /* @__PURE__ */ new Set([
   "Object",
   "Array",
@@ -2218,7 +2248,17 @@ function isPureCall(node) {
   if (callee.object.type === AST_NODE_TYPES10.Identifier && PURE_NAMESPACES.has(callee.object.name)) {
     return !IMPURE_NAMESPACE_METHODS.has(`${callee.object.name}.${property.name}`);
   }
-  return PURE_METHODS.has(property.name);
+  return PURE_METHODS.has(property.name) && (!SYNC_CALLBACK_METHODS.has(property.name) || !synchronousCallbackCanThrow(node));
+}
+function synchronousCallbackCanThrow(node) {
+  return node.arguments.some((argument) => {
+    if (argument.type !== AST_NODE_TYPES10.ArrowFunctionExpression && argument.type !== AST_NODE_TYPES10.FunctionExpression) return false;
+    if (argument.async) return false;
+    return subtreeMatches2(
+      argument.body,
+      (current) => current.type === AST_NODE_TYPES10.ThrowStatement || current.type === AST_NODE_TYPES10.CallExpression && current.callee.type === AST_NODE_TYPES10.MemberExpression && !current.callee.computed && current.callee.object.type === AST_NODE_TYPES10.Identifier && current.callee.object.name === "JSON" && current.callee.property.type === AST_NODE_TYPES10.Identifier && current.callee.property.name === "parse"
+    );
+  });
 }
 function isPureNew(node) {
   return node.callee.type === AST_NODE_TYPES10.Identifier && PURE_CONSTRUCTORS.has(node.callee.name);
@@ -3480,6 +3520,8 @@ function typedFunction(node) {
 
 // src/rules/no-long-comment.ts
 var EXCESSIVE_SENTENCE_COUNT = 8;
+var EXCESSIVE_WORD_COUNT = 120;
+var PROSE_WORD_RE = /[\p{L}\p{N}][\p{L}\p{N}'’-]*/gu;
 var VERSIONED_DEPENDENCY_TREE_RE = /(?:^|\/)lib\/[^/]*-?v?\d+\.\d+(?:\.\d+)?[^/]*\//iu;
 function documentsTypeOrMember(sourceCode, comment) {
   const token = sourceCode.getTokenAfter(comment, { includeComments: false });
@@ -3491,6 +3533,9 @@ function documentsTypeOrMember(sourceCode, comment) {
   }
   return false;
 }
+function wordUnits(text) {
+  return text.match(PROSE_WORD_RE)?.length ?? 0;
+}
 var no_long_comment_default = createRule({
   name: "no-long-comment",
   meta: {
@@ -3508,7 +3553,7 @@ var no_long_comment_default = createRule({
     return {
       Program() {
         for (const group of proseGroups(context.filename, context.sourceCode)) {
-          if (group.comment.type !== "Block" || !group.comment.value.startsWith("*") || group.hasTypedTags || hasDocumentationStructure(group.text) || hasTechnicalAnchor(group.text) || documentsTypedFunction(context.sourceCode, group.comment) || documentsTypeOrMember(context.sourceCode, group.comment) || sentenceUnits(group.text) < EXCESSIVE_SENTENCE_COUNT) continue;
+          if (group.comment.type !== "Block" || !group.comment.value.startsWith("*") || group.hasTypedTags || hasDocumentationStructure(group.text) || hasTechnicalAnchor(group.text) || documentsTypedFunction(context.sourceCode, group.comment) || documentsTypeOrMember(context.sourceCode, group.comment) || sentenceUnits(group.text) < EXCESSIVE_SENTENCE_COUNT && wordUnits(group.text) < EXCESSIVE_WORD_COUNT) continue;
           context.report({ node: group.comment, messageId: "tooLong" });
         }
       }
@@ -8515,6 +8560,11 @@ var prefer_module_level_schema_default = createRule({
             type: "boolean",
             description: "Skip test files, where a fixture schema belongs next to its assertion."
           },
+          memoCallees: {
+            type: "array",
+            items: { type: "string" },
+            description: "Wrappers that construct their callback at most once. Defaults to lazy, memo, once, and useMemo."
+          },
           minProperties: {
             type: "number",
             minimum: 0,
@@ -8532,6 +8582,7 @@ var prefer_module_level_schema_default = createRule({
   create(context, [options]) {
     const factories = new Set(options?.factories ?? DEFAULT_FACTORIES);
     const ignoreTestFiles = options?.ignoreTestFiles ?? true;
+    const memoCallees = new Set(options?.memoCallees ?? MEMO_CALLEES);
     const minProperties = options?.minProperties ?? DEFAULT_MIN_PROPERTIES;
     const sourceCode = context.sourceCode;
     const filename = context.filename;
@@ -8551,7 +8602,7 @@ var prefer_module_level_schema_default = createRule({
         if (current !== node && isZodCall(current) && current.callee.type === AST_NODE_TYPES42.MemberExpression && current.callee.property.type === AST_NODE_TYPES42.Identifier && factories.has(current.callee.property.name)) {
           return true;
         }
-        if (current.type === AST_NODE_TYPES42.CallExpression && (current.callee.type === AST_NODE_TYPES42.Identifier && MEMO_CALLEES.has(current.callee.name) || current.callee.type === AST_NODE_TYPES42.MemberExpression && !current.callee.computed && current.callee.property.type === AST_NODE_TYPES42.Identifier && MEMO_CALLEES.has(current.callee.property.name))) {
+        if (current.type === AST_NODE_TYPES42.CallExpression && (current.callee.type === AST_NODE_TYPES42.Identifier && memoCallees.has(current.callee.name) || current.callee.type === AST_NODE_TYPES42.MemberExpression && !current.callee.computed && current.callee.property.type === AST_NODE_TYPES42.Identifier && memoCallees.has(current.callee.property.name))) {
           return true;
         }
         current = current.parent ?? void 0;
@@ -8601,6 +8652,9 @@ var prefer_module_level_schema_default = createRule({
           if (definition.type === "ImportBinding") {
             continue;
           }
+          if (definition.node.type === AST_NODE_TYPES42.VariableDeclarator && definition.node.parent.type === AST_NODE_TYPES42.VariableDeclaration && definition.node.parent.kind !== "const") {
+            return false;
+          }
           const [defStart, defEnd] = definition.node.range;
           if (defStart >= schemaStart && defEnd <= schemaEnd) {
             continue;
@@ -8835,7 +8889,7 @@ function sameAccess(node, access) {
   return node.type === AST_NODE_TYPES44.MemberExpression && !node.computed && node.object.type === AST_NODE_TYPES44.Identifier && node.object.name === access.object && node.property.type === AST_NODE_TYPES44.Identifier && node.property.name === access.property;
 }
 function isNullGuard(node, access) {
-  if (node.type === AST_NODE_TYPES44.UnaryExpression && node.operator === "!" && sameAccess(node.argument, access)) return true;
+  if (node.type === AST_NODE_TYPES44.UnaryExpression && node.operator === "!" && (sameAccess(node.argument, access) || optionalMemberLengthOf(node.argument, access))) return true;
   if (node.type !== AST_NODE_TYPES44.BinaryExpression || !["==", "==="].includes(node.operator)) {
     return false;
   }
@@ -8851,7 +8905,11 @@ function isEmptyGuard(node, access) {
   return memberLengthOf(node.left, access) && zero(node.right) || memberLengthOf(node.right, access) && zero(node.left);
 }
 function memberLengthOf(node, access) {
-  return node.type === AST_NODE_TYPES44.MemberExpression && !node.computed && node.property.type === AST_NODE_TYPES44.Identifier && node.property.name === "length" && sameAccess(node.object, access);
+  const target = node.type === AST_NODE_TYPES44.ChainExpression ? node.expression : node;
+  return target.type === AST_NODE_TYPES44.MemberExpression && !target.computed && target.property.type === AST_NODE_TYPES44.Identifier && target.property.name === "length" && sameAccess(target.object, access);
+}
+function optionalMemberLengthOf(node, access) {
+  return node.type === AST_NODE_TYPES44.ChainExpression && node.expression.type === AST_NODE_TYPES44.MemberExpression && node.expression.optional && memberLengthOf(node, access);
 }
 function hasEquivalentLeadingGuard(fn, access, visitorKeys) {
   if (fn.body.type !== AST_NODE_TYPES44.BlockStatement) return false;
@@ -8859,6 +8917,7 @@ function hasEquivalentLeadingGuard(fn, access, visitorKeys) {
   if (first?.type !== AST_NODE_TYPES44.IfStatement) return false;
   const terminating = first.consequent.type === AST_NODE_TYPES44.ReturnStatement || first.consequent.type === AST_NODE_TYPES44.ThrowStatement || first.consequent.type === AST_NODE_TYPES44.BlockStatement && first.consequent.body.length === 1 && (first.consequent.body[0]?.type === AST_NODE_TYPES44.ReturnStatement || first.consequent.body[0]?.type === AST_NODE_TYPES44.ThrowStatement);
   if (!terminating) return false;
+  if (contains(first.consequent, visitorKeys, (node) => sameAccess(node, access))) return false;
   return contains(first.test, visitorKeys, (node) => isNullGuard(node, access)) && contains(first.test, visitorKeys, (node) => isEmptyGuard(node, access));
 }
 function contains(node, visitorKeys, predicate) {
@@ -12605,7 +12664,7 @@ var rules = {
 };
 var meta = {
   name: "@sarj/eslint-plugin",
-  version: "11.1.0"
+  version: "11.2.0"
 };
 var applicationOnlyRules = [
   "no-restricted-library-load",

package/package.json

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