@sarj/eslint-plugin 18.2.0 → 18.2.1

package/dist/index.cjs

@@ -1824,19 +1824,19 @@ function firstParameterBindings(source, node) {
 function collectParameterBindings(source, node) {
   if (hasDirectEval(source)) return /* @__PURE__ */ new Set();
   const parameter = node.params[0];
-  const identifiers = [];
+  const identifiers = /* @__PURE__ */ new Set();
   if (parameter?.type === import_utils14.AST_NODE_TYPES.Identifier)
-    identifiers.push(parameter);
+    identifiers.add(parameter);
   if (parameter?.type === import_utils14.AST_NODE_TYPES.ObjectPattern) {
     for (const property of parameter.properties) {
       if (property.type === import_utils14.AST_NODE_TYPES.Property && !property.computed && property.value.type === import_utils14.AST_NODE_TYPES.Identifier)
-        identifiers.push(property.value);
+        identifiers.add(property.value);
     }
   }
   return new Set(
     source.getDeclaredVariables(node).filter(
-      (variable) => identifiers.some(
-        (identifier) => variable.identifiers.includes(identifier)
+      (variable) => variable.identifiers.some(
+        (identifier) => identifiers.has(identifier)
       ) && stableValueBinding(variable)
     )
   );
@@ -1879,8 +1879,10 @@ function reactHook(source, callee, hook) {
   let identifier;
   let namespace = false;
   if (callee.type === import_utils14.AST_NODE_TYPES.Identifier) identifier = callee;
-  else if (callee.type === import_utils14.AST_NODE_TYPES.MemberExpression && !callee.optional && import_utils14.ASTUtils.getPropertyName(callee) === hook && callee.object.type === import_utils14.AST_NODE_TYPES.Identifier) {
-    identifier = callee.object;
+  else if (callee.type === import_utils14.AST_NODE_TYPES.MemberExpression && !callee.optional && import_utils14.ASTUtils.getPropertyName(callee) === hook) {
+    const object = unwrapExpression(callee.object);
+    if (object.type !== import_utils14.AST_NODE_TYPES.Identifier) return false;
+    identifier = object;
     namespace = true;
   } else return false;
   const variable = import_utils14.ASTUtils.findVariable(
@@ -1888,17 +1890,33 @@ function reactHook(source, callee, hook) {
     identifier.name
   );
   if (variable === null) return false;
-  const specifier = reactImportSpecifier(variable);
-  if (specifier === null) return false;
-  return namespace ? specifier.type === import_utils14.AST_NODE_TYPES.ImportDefaultSpecifier || specifier.type === import_utils14.AST_NODE_TYPES.ImportNamespaceSpecifier : specifier.type === import_utils14.AST_NODE_TYPES.ImportSpecifier && specifier.importKind !== "type" && importSpecifierName(specifier) === hook;
-}
-function reactImportSpecifier(variable) {
-  let specifier = REACT_IMPORTS.get(variable);
-  if (specifier !== void 0) return specifier;
+  const imported = reactImport(variable);
+  if (imported === null) return false;
+  const { specifier, replacedMembers } = imported;
+  return namespace ? (specifier.type === import_utils14.AST_NODE_TYPES.ImportDefaultSpecifier || specifier.type === import_utils14.AST_NODE_TYPES.ImportNamespaceSpecifier) && replacedMembers?.has(null) !== true && replacedMembers?.has(hook) !== true : specifier.type === import_utils14.AST_NODE_TYPES.ImportSpecifier && specifier.importKind !== "type" && importSpecifierName(specifier) === hook;
+}
+function reactImport(variable) {
+  let imported = REACT_IMPORTS.get(variable);
+  if (imported !== void 0) return imported;
   const definition = variable.defs[0];
-  specifier = variable.defs.length === 1 && definition?.type === "ImportBinding" && definition.parent.type === import_utils14.AST_NODE_TYPES.ImportDeclaration && definition.parent.source.value === "react" && definition.parent.importKind !== "type" && stableValueBinding(variable) ? definition.node : null;
-  REACT_IMPORTS.set(variable, specifier);
-  return specifier;
+  imported = null;
+  if (variable.defs.length === 1 && definition?.type === "ImportBinding" && definition.parent.type === import_utils14.AST_NODE_TYPES.ImportDeclaration && definition.parent.source.value === "react" && definition.parent.importKind !== "type") {
+    const specifier = definition.node;
+    const replacedMembers = specifier.type === import_utils14.AST_NODE_TYPES.ImportSpecifier ? void 0 : /* @__PURE__ */ new Set();
+    if (stableValueBinding(variable, replacedMembers === void 0 ? void 0 : (reference) => {
+      const object = outerExpression(reference.identifier);
+      const member3 = object.parent;
+      if (member3?.type !== import_utils14.AST_NODE_TYPES.MemberExpression || member3.object !== object)
+        return true;
+      const target = outerExpression(member3);
+      if (target.parent?.type === import_utils14.AST_NODE_TYPES.MemberExpression && target.parent.object === target)
+        return true;
+      if (writtenMember(member3)) replacedMembers.add(import_utils14.ASTUtils.getPropertyName(member3));
+      return true;
+    })) imported = { specifier, replacedMembers };
+  }
+  REACT_IMPORTS.set(variable, imported);
+  return imported;
 }
 function containsView(node) {
   if (node.type === import_utils14.AST_NODE_TYPES.JSXElement || node.type === import_utils14.AST_NODE_TYPES.JSXFragment)
@@ -1953,6 +1971,7 @@ var PREFER_SERVER_DATA_PREPARATION_DOCUMENTATION = {
     "Advisory only. Local syntax cannot establish whether a caller is a Server Component, whether props change interactively or whether server preparation reduces the total cost. This rule is a review lead, never proof that code can be moved to the server. No automatic fix is provided.",
     "Requires an explicit use client directive and parser-owned TypeScript services resolving map/filter/flatMap to the standard Array or ReadonlyArray library signature. The receiver beneath erased wrappers must itself have an Array, ReadonlyArray or tuple type after nullish narrowing; casts from opaque or custom collections do not establish this. Subclass and structural collection types outside those native array types are excluded. The containing view must have an independent React useState/useReducer call or a native callable type on an intrinsic DOM onClick/onChange/onInput/onSubmit handler. Nullish, noncallable and opaque handler types do not establish interaction. Noninteractive boundaries remain owned by no-unnecessary-use-client; custom hooks and indirect client requirements are unsupported.",
     "Every runtime reference to the source parameter binding must be a supported native data projection or a field-only JSX display loop. Inputs also needed for editing, whole-record forwarding, lookup, mutation, other fields or opaque helpers are excluded. Files containing direct eval are excluded because it can observe or change lexical bindings; indirect eval and properties named eval do not establish lexical access. Only direct first-parameter paths and synchronous single-expression callbacks are supported. Whole-row identity maps and constant maps are excluded; supported projections must read a fixed input field.",
+    "Field-only object and dense array literals can form nested data shapes, including readonly tuples and flatMap outputs. Array holes, spreads, captured values and JSX outputs remain unsupported. Hook recognition follows erased namespace wrappers and excludes direct writes to the requested imported member; reflective or external API patches are not modeled.",
     "JSX-rendering callbacks are never reported as preparation findings. Callback calls, captured values, destructuring defaults/rest/nesting, async callbacks, event handlers, effects, memo callbacks, opaque helpers, dynamic/optional paths, custom collection signatures, missing type services, generated source and tests are excluded. First-parameter provenance is syntactic; annotations cannot prove absence of runtime getters or patched array methods. Analysis does not claim callback purity or preserve semantics of a hypothetical move.",
     "Reports the direct source projection once; chained operations with a call receiver are not independently reported. A static subprojection may still be prepared by its producer while a union with pending local edits remains in the client."
   ],
@@ -2107,6 +2126,10 @@ function projection(source, callback, mode = "data") {
       return inspect(node.left, depth + 1) && inspect(node.right, depth + 1);
     if (node.type === import_utils15.AST_NODE_TYPES.ConditionalExpression)
       return inspect(node.test, depth + 1) && inspect(node.consequent, depth + 1) && inspect(node.alternate, depth + 1);
+    if (node.type === import_utils15.AST_NODE_TYPES.ArrayExpression)
+      return node.elements.every(
+        (element) => element !== null && element.type !== import_utils15.AST_NODE_TYPES.SpreadElement && inspect(element, depth + 1)
+      );
     if (node.type === import_utils15.AST_NODE_TYPES.ObjectExpression)
       return node.properties.every(
         (property) => property.type === import_utils15.AST_NODE_TYPES.Property && property.kind === "init" && !property.method && !property.computed && inspect(property.value, depth + 1)
@@ -2191,8 +2214,9 @@ var NO_READONLY_PROP_STATE_DOCUMENTATION = {
   category: "correctness",
   limitations: [
     "Advisory only: initial snapshots can be intentional. There is no automatic fix because replacing state with current props changes how later updates behave.",
-    "Recognizes direct React useState imports and default/namespace member calls, including renamed imports, through lexical binding identity. Checks a direct array destructure in a function containing its own JSX and a direct first-parameter property path, or a zero-argument synchronous lazy initializer returning that path.",
+    "Recognizes direct React useState imports and default/namespace member calls, including renamed imports and erased wrappers around imported namespaces, through lexical binding identity. Checks a direct array destructure in a function containing its own JSX and a direct first-parameter property path, or a zero-argument synchronous lazy initializer returning that path.",
     "Eager initializers require parser-owned TypeScript services and primitive types, including unions composed entirely of primitives. Object and void types can hide callable values through structural assignment or ignored return values, so eager nonprimitive values, Any, unknown and unresolved generics are excluded. React executes function arguments as lazy initializers. Explicit zero-argument lazy callbacks returning a prop directly remain supported for all value types without type services. Opaque casts cannot prove runtime identity; lifecycle changes stay manual.",
+    "Direct writes to the requested imported namespace member exclude its hook calls. Unknown computed writes exclude all namespace hooks; unrelated member writes and a hook function's own properties do not replace the callable. Reflective mutation and external API patches are not modeled.",
     "An updater read, escape, reassignment or nontrivial destructuring excludes the finding. Unused state values remain owned by unused-variable checks. Files containing direct eval are excluded because it can observe or change lexical bindings. Indirect eval and properties named eval do not establish lexical access. Opaque initialization, property aliases, optional/dynamic paths, rest/default bindings, non-JSX hooks, generated source and tests are outside scope. This does not ban interactive useState or replace the upstream set-state-in-effect rule."
   ],
   references: [
@@ -2481,18 +2505,8 @@ var FRONTEND_IMPORT = /^(?:react(?:-dom)?|next|preact|vue|svelte|@angular\/core|
 function isFrontendSource(context) {
   return /\.[jt]sx$/u.test(context.filename) || frontendModule(context.sourceCode.ast) || frontendPackage(context.filename);
 }
-var FRONTEND_PACKAGE_BY_DIRECTORY = /* @__PURE__ */ new Map();
 function frontendPackage(filename) {
-  const directory = (0, import_node_path2.dirname)((0, import_node_path2.resolve)(filename));
-  let cached = FRONTEND_PACKAGE_BY_DIRECTORY.get(directory);
-  if (cached === void 0) {
-    cached = nearestManifestDeclaresFrontend(directory);
-    FRONTEND_PACKAGE_BY_DIRECTORY.set(directory, cached);
-  }
-  return cached;
-}
-function nearestManifestDeclaresFrontend(start) {
-  let directory = start;
+  let directory = (0, import_node_path2.dirname)((0, import_node_path2.resolve)(filename));
   for (; ; ) {
     const manifestPath = (0, import_node_path2.join)(directory, "package.json");
     if ((0, import_node_fs.existsSync)(manifestPath)) {
@@ -2838,6 +2852,23 @@ function ownsTestComments(context) {
   const framework = context.sourceCode.ast.body.some((statement) => statement.type === import_utils22.AST_NODE_TYPES.ImportDeclaration && statement.importKind !== "type" && (statement.specifiers.length === 0 || statement.specifiers.some((specifier) => specifier.type !== import_utils22.AST_NODE_TYPES.ImportSpecifier || specifier.importKind !== "type")) && ["vitest", "@jest/globals", "@playwright/test", "bun:test", "node:test"].includes(statement.source.value));
   return unambiguous || framework;
 }
+function legalNoticeComments(comments) {
+  const legal = /* @__PURE__ */ new Set();
+  for (const [index, comment] of comments.entries()) {
+    if (legal.has(comment) || comment.type !== "Line" || !LEGAL.test(comment.value)) continue;
+    legal.add(comment);
+    for (const direction of [-1, 1]) {
+      let previous = comment;
+      for (let next = index + direction; next >= 0 && next < comments.length; next += direction) {
+        const neighbor = comments[next];
+        if (neighbor?.type !== "Line" || Math.abs(neighbor.loc.start.line - previous.loc.start.line) !== 1) break;
+        legal.add(neighbor);
+        previous = neighbor;
+      }
+    }
+  }
+  return legal;
+}
 var no_test_comments_default = createRule({
   name: "no-test-comments",
   documentation: NO_TEST_COMMENTS_DOCUMENTATION,
@@ -2856,23 +2887,10 @@ var no_test_comments_default = createRule({
     return {
       Program() {
         const comments = context.sourceCode.getAllComments();
-        const legalLines = /* @__PURE__ */ new Set();
-        for (const [index, comment] of comments.entries()) {
-          if (comment.type !== "Line" || !LEGAL.test(comment.value)) continue;
-          legalLines.add(index);
-          for (const direction of [-1, 1]) {
-            let previous = comment;
-            for (let next = index + direction; next >= 0 && next < comments.length; next += direction) {
-              const neighbor = comments[next];
-              if (neighbor?.type !== "Line" || Math.abs(neighbor.loc.start.line - previous.loc.start.line) !== 1) break;
-              legalLines.add(next);
-              previous = neighbor;
-            }
-          }
-        }
-        for (const [index, comment] of comments.entries()) {
+        const legal = frontend ? void 0 : legalNoticeComments(comments);
+        for (const comment of comments) {
           const body3 = comment.value.trim().replace(/^\*\s*/u, "");
-          if (!frontend && (DIRECTIVE.test(body3) || BEHAVIORAL_DIRECTIVE.test(comment.value) || LEGAL.test(comment.value) || legalLines.has(index) || TYPE_CONTRACT.test(comment.value) || hasDetectSecretsPragma(comment, context.sourceCode.text))) continue;
+          if (!frontend && (DIRECTIVE.test(body3) || BEHAVIORAL_DIRECTIVE.test(comment.value) || LEGAL.test(comment.value) || legal?.has(comment) || TYPE_CONTRACT.test(comment.value) || hasDetectSecretsPragma(comment, context.sourceCode.text))) continue;
           context.report({
             node: comment,
             messageId: "testComment",
@@ -6287,7 +6305,7 @@ function isSearchParamReceiver(call, services) {
 // package.json
 var package_default = {
   name: "@sarj/eslint-plugin",
-  version: "18.2.0",
+  version: "18.2.1",
   packageManager: "npm@12.2.1",
   description: "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   type: "module",

package/dist/index.js

@@ -1782,19 +1782,19 @@ function firstParameterBindings(source, node) {
 function collectParameterBindings(source, node) {
   if (hasDirectEval(source)) return /* @__PURE__ */ new Set();
   const parameter = node.params[0];
-  const identifiers = [];
+  const identifiers = /* @__PURE__ */ new Set();
   if (parameter?.type === AST_NODE_TYPES13.Identifier)
-    identifiers.push(parameter);
+    identifiers.add(parameter);
   if (parameter?.type === AST_NODE_TYPES13.ObjectPattern) {
     for (const property of parameter.properties) {
       if (property.type === AST_NODE_TYPES13.Property && !property.computed && property.value.type === AST_NODE_TYPES13.Identifier)
-        identifiers.push(property.value);
+        identifiers.add(property.value);
     }
   }
   return new Set(
     source.getDeclaredVariables(node).filter(
-      (variable) => identifiers.some(
-        (identifier) => variable.identifiers.includes(identifier)
+      (variable) => variable.identifiers.some(
+        (identifier) => identifiers.has(identifier)
       ) && stableValueBinding(variable)
     )
   );
@@ -1837,8 +1837,10 @@ function reactHook(source, callee, hook) {
   let identifier;
   let namespace = false;
   if (callee.type === AST_NODE_TYPES13.Identifier) identifier = callee;
-  else if (callee.type === AST_NODE_TYPES13.MemberExpression && !callee.optional && ASTUtils6.getPropertyName(callee) === hook && callee.object.type === AST_NODE_TYPES13.Identifier) {
-    identifier = callee.object;
+  else if (callee.type === AST_NODE_TYPES13.MemberExpression && !callee.optional && ASTUtils6.getPropertyName(callee) === hook) {
+    const object = unwrapExpression(callee.object);
+    if (object.type !== AST_NODE_TYPES13.Identifier) return false;
+    identifier = object;
     namespace = true;
   } else return false;
   const variable = ASTUtils6.findVariable(
@@ -1846,17 +1848,33 @@ function reactHook(source, callee, hook) {
     identifier.name
   );
   if (variable === null) return false;
-  const specifier = reactImportSpecifier(variable);
-  if (specifier === null) return false;
-  return namespace ? specifier.type === AST_NODE_TYPES13.ImportDefaultSpecifier || specifier.type === AST_NODE_TYPES13.ImportNamespaceSpecifier : specifier.type === AST_NODE_TYPES13.ImportSpecifier && specifier.importKind !== "type" && importSpecifierName(specifier) === hook;
-}
-function reactImportSpecifier(variable) {
-  let specifier = REACT_IMPORTS.get(variable);
-  if (specifier !== void 0) return specifier;
+  const imported = reactImport(variable);
+  if (imported === null) return false;
+  const { specifier, replacedMembers } = imported;
+  return namespace ? (specifier.type === AST_NODE_TYPES13.ImportDefaultSpecifier || specifier.type === AST_NODE_TYPES13.ImportNamespaceSpecifier) && replacedMembers?.has(null) !== true && replacedMembers?.has(hook) !== true : specifier.type === AST_NODE_TYPES13.ImportSpecifier && specifier.importKind !== "type" && importSpecifierName(specifier) === hook;
+}
+function reactImport(variable) {
+  let imported = REACT_IMPORTS.get(variable);
+  if (imported !== void 0) return imported;
   const definition = variable.defs[0];
-  specifier = variable.defs.length === 1 && definition?.type === "ImportBinding" && definition.parent.type === AST_NODE_TYPES13.ImportDeclaration && definition.parent.source.value === "react" && definition.parent.importKind !== "type" && stableValueBinding(variable) ? definition.node : null;
-  REACT_IMPORTS.set(variable, specifier);
-  return specifier;
+  imported = null;
+  if (variable.defs.length === 1 && definition?.type === "ImportBinding" && definition.parent.type === AST_NODE_TYPES13.ImportDeclaration && definition.parent.source.value === "react" && definition.parent.importKind !== "type") {
+    const specifier = definition.node;
+    const replacedMembers = specifier.type === AST_NODE_TYPES13.ImportSpecifier ? void 0 : /* @__PURE__ */ new Set();
+    if (stableValueBinding(variable, replacedMembers === void 0 ? void 0 : (reference) => {
+      const object = outerExpression(reference.identifier);
+      const member3 = object.parent;
+      if (member3?.type !== AST_NODE_TYPES13.MemberExpression || member3.object !== object)
+        return true;
+      const target = outerExpression(member3);
+      if (target.parent?.type === AST_NODE_TYPES13.MemberExpression && target.parent.object === target)
+        return true;
+      if (writtenMember(member3)) replacedMembers.add(ASTUtils6.getPropertyName(member3));
+      return true;
+    })) imported = { specifier, replacedMembers };
+  }
+  REACT_IMPORTS.set(variable, imported);
+  return imported;
 }
 function containsView(node) {
   if (node.type === AST_NODE_TYPES13.JSXElement || node.type === AST_NODE_TYPES13.JSXFragment)
@@ -1911,6 +1929,7 @@ var PREFER_SERVER_DATA_PREPARATION_DOCUMENTATION = {
     "Advisory only. Local syntax cannot establish whether a caller is a Server Component, whether props change interactively or whether server preparation reduces the total cost. This rule is a review lead, never proof that code can be moved to the server. No automatic fix is provided.",
     "Requires an explicit use client directive and parser-owned TypeScript services resolving map/filter/flatMap to the standard Array or ReadonlyArray library signature. The receiver beneath erased wrappers must itself have an Array, ReadonlyArray or tuple type after nullish narrowing; casts from opaque or custom collections do not establish this. Subclass and structural collection types outside those native array types are excluded. The containing view must have an independent React useState/useReducer call or a native callable type on an intrinsic DOM onClick/onChange/onInput/onSubmit handler. Nullish, noncallable and opaque handler types do not establish interaction. Noninteractive boundaries remain owned by no-unnecessary-use-client; custom hooks and indirect client requirements are unsupported.",
     "Every runtime reference to the source parameter binding must be a supported native data projection or a field-only JSX display loop. Inputs also needed for editing, whole-record forwarding, lookup, mutation, other fields or opaque helpers are excluded. Files containing direct eval are excluded because it can observe or change lexical bindings; indirect eval and properties named eval do not establish lexical access. Only direct first-parameter paths and synchronous single-expression callbacks are supported. Whole-row identity maps and constant maps are excluded; supported projections must read a fixed input field.",
+    "Field-only object and dense array literals can form nested data shapes, including readonly tuples and flatMap outputs. Array holes, spreads, captured values and JSX outputs remain unsupported. Hook recognition follows erased namespace wrappers and excludes direct writes to the requested imported member; reflective or external API patches are not modeled.",
     "JSX-rendering callbacks are never reported as preparation findings. Callback calls, captured values, destructuring defaults/rest/nesting, async callbacks, event handlers, effects, memo callbacks, opaque helpers, dynamic/optional paths, custom collection signatures, missing type services, generated source and tests are excluded. First-parameter provenance is syntactic; annotations cannot prove absence of runtime getters or patched array methods. Analysis does not claim callback purity or preserve semantics of a hypothetical move.",
     "Reports the direct source projection once; chained operations with a call receiver are not independently reported. A static subprojection may still be prepared by its producer while a union with pending local edits remains in the client."
   ],
@@ -2065,6 +2084,10 @@ function projection(source, callback, mode = "data") {
       return inspect(node.left, depth + 1) && inspect(node.right, depth + 1);
     if (node.type === AST_NODE_TYPES14.ConditionalExpression)
       return inspect(node.test, depth + 1) && inspect(node.consequent, depth + 1) && inspect(node.alternate, depth + 1);
+    if (node.type === AST_NODE_TYPES14.ArrayExpression)
+      return node.elements.every(
+        (element) => element !== null && element.type !== AST_NODE_TYPES14.SpreadElement && inspect(element, depth + 1)
+      );
     if (node.type === AST_NODE_TYPES14.ObjectExpression)
       return node.properties.every(
         (property) => property.type === AST_NODE_TYPES14.Property && property.kind === "init" && !property.method && !property.computed && inspect(property.value, depth + 1)
@@ -2149,8 +2172,9 @@ var NO_READONLY_PROP_STATE_DOCUMENTATION = {
   category: "correctness",
   limitations: [
     "Advisory only: initial snapshots can be intentional. There is no automatic fix because replacing state with current props changes how later updates behave.",
-    "Recognizes direct React useState imports and default/namespace member calls, including renamed imports, through lexical binding identity. Checks a direct array destructure in a function containing its own JSX and a direct first-parameter property path, or a zero-argument synchronous lazy initializer returning that path.",
+    "Recognizes direct React useState imports and default/namespace member calls, including renamed imports and erased wrappers around imported namespaces, through lexical binding identity. Checks a direct array destructure in a function containing its own JSX and a direct first-parameter property path, or a zero-argument synchronous lazy initializer returning that path.",
     "Eager initializers require parser-owned TypeScript services and primitive types, including unions composed entirely of primitives. Object and void types can hide callable values through structural assignment or ignored return values, so eager nonprimitive values, Any, unknown and unresolved generics are excluded. React executes function arguments as lazy initializers. Explicit zero-argument lazy callbacks returning a prop directly remain supported for all value types without type services. Opaque casts cannot prove runtime identity; lifecycle changes stay manual.",
+    "Direct writes to the requested imported namespace member exclude its hook calls. Unknown computed writes exclude all namespace hooks; unrelated member writes and a hook function's own properties do not replace the callable. Reflective mutation and external API patches are not modeled.",
     "An updater read, escape, reassignment or nontrivial destructuring excludes the finding. Unused state values remain owned by unused-variable checks. Files containing direct eval are excluded because it can observe or change lexical bindings. Indirect eval and properties named eval do not establish lexical access. Opaque initialization, property aliases, optional/dynamic paths, rest/default bindings, non-JSX hooks, generated source and tests are outside scope. This does not ban interactive useState or replace the upstream set-state-in-effect rule."
   ],
   references: [
@@ -2439,18 +2463,8 @@ var FRONTEND_IMPORT = /^(?:react(?:-dom)?|next|preact|vue|svelte|@angular\/core|
 function isFrontendSource(context) {
   return /\.[jt]sx$/u.test(context.filename) || frontendModule(context.sourceCode.ast) || frontendPackage(context.filename);
 }
-var FRONTEND_PACKAGE_BY_DIRECTORY = /* @__PURE__ */ new Map();
 function frontendPackage(filename) {
-  const directory = dirname(resolve(filename));
-  let cached = FRONTEND_PACKAGE_BY_DIRECTORY.get(directory);
-  if (cached === void 0) {
-    cached = nearestManifestDeclaresFrontend(directory);
-    FRONTEND_PACKAGE_BY_DIRECTORY.set(directory, cached);
-  }
-  return cached;
-}
-function nearestManifestDeclaresFrontend(start) {
-  let directory = start;
+  let directory = dirname(resolve(filename));
   for (; ; ) {
     const manifestPath = join(directory, "package.json");
     if (existsSync(manifestPath)) {
@@ -2796,6 +2810,23 @@ function ownsTestComments(context) {
   const framework = context.sourceCode.ast.body.some((statement) => statement.type === AST_NODE_TYPES20.ImportDeclaration && statement.importKind !== "type" && (statement.specifiers.length === 0 || statement.specifiers.some((specifier) => specifier.type !== AST_NODE_TYPES20.ImportSpecifier || specifier.importKind !== "type")) && ["vitest", "@jest/globals", "@playwright/test", "bun:test", "node:test"].includes(statement.source.value));
   return unambiguous || framework;
 }
+function legalNoticeComments(comments) {
+  const legal = /* @__PURE__ */ new Set();
+  for (const [index, comment] of comments.entries()) {
+    if (legal.has(comment) || comment.type !== "Line" || !LEGAL.test(comment.value)) continue;
+    legal.add(comment);
+    for (const direction of [-1, 1]) {
+      let previous = comment;
+      for (let next = index + direction; next >= 0 && next < comments.length; next += direction) {
+        const neighbor = comments[next];
+        if (neighbor?.type !== "Line" || Math.abs(neighbor.loc.start.line - previous.loc.start.line) !== 1) break;
+        legal.add(neighbor);
+        previous = neighbor;
+      }
+    }
+  }
+  return legal;
+}
 var no_test_comments_default = createRule({
   name: "no-test-comments",
   documentation: NO_TEST_COMMENTS_DOCUMENTATION,
@@ -2814,23 +2845,10 @@ var no_test_comments_default = createRule({
     return {
       Program() {
         const comments = context.sourceCode.getAllComments();
-        const legalLines = /* @__PURE__ */ new Set();
-        for (const [index, comment] of comments.entries()) {
-          if (comment.type !== "Line" || !LEGAL.test(comment.value)) continue;
-          legalLines.add(index);
-          for (const direction of [-1, 1]) {
-            let previous = comment;
-            for (let next = index + direction; next >= 0 && next < comments.length; next += direction) {
-              const neighbor = comments[next];
-              if (neighbor?.type !== "Line" || Math.abs(neighbor.loc.start.line - previous.loc.start.line) !== 1) break;
-              legalLines.add(next);
-              previous = neighbor;
-            }
-          }
-        }
-        for (const [index, comment] of comments.entries()) {
+        const legal = frontend ? void 0 : legalNoticeComments(comments);
+        for (const comment of comments) {
           const body3 = comment.value.trim().replace(/^\*\s*/u, "");
-          if (!frontend && (DIRECTIVE.test(body3) || BEHAVIORAL_DIRECTIVE.test(comment.value) || LEGAL.test(comment.value) || legalLines.has(index) || TYPE_CONTRACT.test(comment.value) || hasDetectSecretsPragma(comment, context.sourceCode.text))) continue;
+          if (!frontend && (DIRECTIVE.test(body3) || BEHAVIORAL_DIRECTIVE.test(comment.value) || LEGAL.test(comment.value) || legal?.has(comment) || TYPE_CONTRACT.test(comment.value) || hasDetectSecretsPragma(comment, context.sourceCode.text))) continue;
           context.report({
             node: comment,
             messageId: "testComment",
@@ -6248,7 +6266,7 @@ function isSearchParamReceiver(call, services) {
 // package.json
 var package_default = {
   name: "@sarj/eslint-plugin",
-  version: "18.2.0",
+  version: "18.2.1",
   packageManager: "npm@12.2.1",
   description: "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   type: "module",

package/package.json

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