@sarj/eslint-plugin 16.26.1 → 16.26.2

package/dist/index.cjs

@@ -883,7 +883,7 @@ function isSearchParamReceiver(call, services) {
 // package.json
 var package_default = {
   name: "@sarj/eslint-plugin",
-  version: "16.26.1",
+  version: "16.26.2",
   packageManager: "npm@12.2.0",
   description: "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   type: "module",
@@ -11084,6 +11084,7 @@ var no_bare_return_from_test_catch_default = createRule({
         const returnBlock = node.parent;
         if (parameter?.type === import_utils64.AST_NODE_TYPES.Identifier && returnBlock?.type === import_utils64.AST_NODE_TYPES.BlockStatement) {
           const errorBinding = import_utils64.ASTUtils.findVariable(context.sourceCode.getScope(parameter), parameter.name);
+          const errorReferences = errorBinding !== null && errorBinding.references.length > 32 ? new Set(errorBinding.references.map((reference) => reference.identifier)) : void 0;
           const assertedError = returnBlock.body.some((statement) => {
             if (statement.range[1] > node.range[0] || statement.type !== import_utils64.AST_NODE_TYPES.ExpressionStatement) return false;
             const expression = statement.expression;
@@ -11094,7 +11095,7 @@ var no_bare_return_from_test_catch_default = createRule({
             let operand = expression.callee.type === import_utils64.AST_NODE_TYPES.MemberExpression ? expression.callee.object : null;
             if (operand?.type === import_utils64.AST_NODE_TYPES.MemberExpression && staticMemberName3(operand) === "not") operand = operand.object;
             if (assertionName !== "assert" && (assertionName !== "expect" || operand?.type !== import_utils64.AST_NODE_TYPES.CallExpression || operand.callee !== root)) return false;
-            return walkOwnScope(expression, (current) => current.type === import_utils64.AST_NODE_TYPES.Identifier && errorBinding?.references.some((reference) => reference.identifier === current) === true);
+            return walkOwnScope(expression, (current) => current.type === import_utils64.AST_NODE_TYPES.Identifier && (errorReferences?.has(current) ?? errorBinding?.references.some((reference) => reference.identifier === current)) === true);
           });
           if (assertedError) return;
         }
@@ -17325,6 +17326,15 @@ var test_loops_over_literal_cases_default = createRule({
     if (!isTestFile(context)) {
       return {};
     }
+    const referenceIdentifiers = /* @__PURE__ */ new WeakMap();
+    const isReference = (variable, identifier) => {
+      let identifiers = referenceIdentifiers.get(variable);
+      if (identifiers === void 0) {
+        identifiers = new Set(variable.references.map((reference) => reference.identifier));
+        referenceIdentifiers.set(variable, identifiers);
+      }
+      return identifiers.has(identifier);
+    };
     const isFrameworkTest = (identifier) => TEST_CALLERS3.has(runtimeTestFrameworkName(context.sourceCode, identifier, "test") ?? "");
     const isFrameworkAssertion = (identifier) => ASSERTION_ROOTS.has(runtimeTestFrameworkName(context.sourceCode, identifier, "assertion") ?? "");
     return {
@@ -17348,7 +17358,7 @@ var test_loops_over_literal_cases_default = createRule({
         const capturesSetup = walkOwnScope2(node.body, (current) => {
           if (current.type !== import_utils99.AST_NODE_TYPES.Identifier) return false;
           const variable = import_utils99.ASTUtils.findVariable(context.sourceCode.getScope(current), current.name);
-          if (variable === null || !variable.references.some((reference) => reference.identifier === current)) return false;
+          if (variable === null || !isReference(variable, current)) return false;
           return variable.defs.some((definition) => {
             const declaration = definition.name;
             return declaration.range[0] >= enclosing.range[0] && declaration.range[1] <= enclosing.range[1] && (declaration.range[0] < node.range[0] || declaration.range[1] > node.range[1]);
@@ -27532,6 +27542,14 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
     defaultOptions: [],
     create(context) {
       if (!isTestFile(context) || isGeneratedFile(context, context.sourceCode.text)) return {};
+      const referenceStability = /* @__PURE__ */ new WeakMap();
+      const referenceStable = (variable) => {
+        const cached = referenceStability.get(variable);
+        if (cached !== void 0) return cached;
+        const stable = !variable.references.some((reference) => reference.isWrite() && reference.init !== true);
+        referenceStability.set(variable, stable);
+        return stable;
+      };
       const scopes = [newScope()];
       const reportedOrigins = /* @__PURE__ */ new Set();
       const currentScope = () => scopes.at(-1) ?? scopes[0];
@@ -27549,7 +27567,7 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
       };
       const visible = (kind, node) => {
         const name2 = bindingOf(node);
-        if (name2 === null || name2.references.some((reference) => reference.isWrite() && reference.init !== true)) return false;
+        if (name2 === null || !referenceStable(name2)) return false;
         for (let index = scopes.length - 1; index >= 0; index--) {
           const scope = scopes[index];
           if (scope.declared.has(name2)) return scope[kind].has(name2);
@@ -27558,7 +27576,7 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
       };
       const visibleRawOrigins = (node) => {
         const name2 = bindingOf(node);
-        if (name2 === null || name2.references.some((reference) => reference.isWrite() && reference.init !== true)) return /* @__PURE__ */ new Set();
+        if (name2 === null || !referenceStable(name2)) return /* @__PURE__ */ new Set();
         for (let index = scopes.length - 1; index >= 0; index--) {
           const scope = scopes[index];
           if (scope.declared.has(name2)) return scope.rawOrigins.get(name2) ?? /* @__PURE__ */ new Set();
@@ -27590,7 +27608,7 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
       };
       const nativeImportedName = (node, modules) => {
         const binding = bindingOf(node);
-        if (binding === null || binding.references.some((reference) => reference.isWrite() && reference.init !== true)) return null;
+        if (binding === null || !referenceStable(binding)) return null;
         for (const definition of binding.defs) {
           const imported = definition.node;
           if (imported.type === import_utils144.AST_NODE_TYPES.ImportSpecifier && imported.imported.type === import_utils144.AST_NODE_TYPES.Identifier && imported.parent.type === import_utils144.AST_NODE_TYPES.ImportDeclaration && modules.has(String(imported.parent.source.value))) {
@@ -27601,7 +27619,7 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
       };
       const stableInitializer = (node) => {
         const binding = bindingOf(node);
-        if (binding === null || binding.references.some((reference) => reference.isWrite() && reference.init !== true) || binding.defs.length !== 1) return null;
+        if (binding === null || !referenceStable(binding) || binding.defs.length !== 1) return null;
         const definition = binding.defs[0]?.node;
         return definition?.type === import_utils144.AST_NODE_TYPES.VariableDeclarator ? definition.init : null;
       };

package/dist/index.js

@@ -836,7 +836,7 @@ function isSearchParamReceiver(call, services) {
 // package.json
 var package_default = {
   name: "@sarj/eslint-plugin",
-  version: "16.26.1",
+  version: "16.26.2",
   packageManager: "npm@12.2.0",
   description: "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   type: "module",
@@ -11069,6 +11069,7 @@ var no_bare_return_from_test_catch_default = createRule({
         const returnBlock = node.parent;
         if (parameter?.type === AST_NODE_TYPES55.Identifier && returnBlock?.type === AST_NODE_TYPES55.BlockStatement) {
           const errorBinding = ASTUtils32.findVariable(context.sourceCode.getScope(parameter), parameter.name);
+          const errorReferences = errorBinding !== null && errorBinding.references.length > 32 ? new Set(errorBinding.references.map((reference) => reference.identifier)) : void 0;
           const assertedError = returnBlock.body.some((statement) => {
             if (statement.range[1] > node.range[0] || statement.type !== AST_NODE_TYPES55.ExpressionStatement) return false;
             const expression = statement.expression;
@@ -11079,7 +11080,7 @@ var no_bare_return_from_test_catch_default = createRule({
             let operand = expression.callee.type === AST_NODE_TYPES55.MemberExpression ? expression.callee.object : null;
             if (operand?.type === AST_NODE_TYPES55.MemberExpression && staticMemberName3(operand) === "not") operand = operand.object;
             if (assertionName !== "assert" && (assertionName !== "expect" || operand?.type !== AST_NODE_TYPES55.CallExpression || operand.callee !== root)) return false;
-            return walkOwnScope(expression, (current) => current.type === AST_NODE_TYPES55.Identifier && errorBinding?.references.some((reference) => reference.identifier === current) === true);
+            return walkOwnScope(expression, (current) => current.type === AST_NODE_TYPES55.Identifier && (errorReferences?.has(current) ?? errorBinding?.references.some((reference) => reference.identifier === current)) === true);
           });
           if (assertedError) return;
         }
@@ -17316,6 +17317,15 @@ var test_loops_over_literal_cases_default = createRule({
     if (!isTestFile(context)) {
       return {};
     }
+    const referenceIdentifiers = /* @__PURE__ */ new WeakMap();
+    const isReference = (variable, identifier) => {
+      let identifiers = referenceIdentifiers.get(variable);
+      if (identifiers === void 0) {
+        identifiers = new Set(variable.references.map((reference) => reference.identifier));
+        referenceIdentifiers.set(variable, identifiers);
+      }
+      return identifiers.has(identifier);
+    };
     const isFrameworkTest = (identifier) => TEST_CALLERS3.has(runtimeTestFrameworkName(context.sourceCode, identifier, "test") ?? "");
     const isFrameworkAssertion = (identifier) => ASSERTION_ROOTS.has(runtimeTestFrameworkName(context.sourceCode, identifier, "assertion") ?? "");
     return {
@@ -17339,7 +17349,7 @@ var test_loops_over_literal_cases_default = createRule({
         const capturesSetup = walkOwnScope2(node.body, (current) => {
           if (current.type !== AST_NODE_TYPES82.Identifier) return false;
           const variable = ASTUtils50.findVariable(context.sourceCode.getScope(current), current.name);
-          if (variable === null || !variable.references.some((reference) => reference.identifier === current)) return false;
+          if (variable === null || !isReference(variable, current)) return false;
           return variable.defs.some((definition) => {
             const declaration = definition.name;
             return declaration.range[0] >= enclosing.range[0] && declaration.range[1] <= enclosing.range[1] && (declaration.range[0] < node.range[0] || declaration.range[1] > node.range[1]);
@@ -27556,6 +27566,14 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
     defaultOptions: [],
     create(context) {
       if (!isTestFile(context) || isGeneratedFile(context, context.sourceCode.text)) return {};
+      const referenceStability = /* @__PURE__ */ new WeakMap();
+      const referenceStable = (variable) => {
+        const cached = referenceStability.get(variable);
+        if (cached !== void 0) return cached;
+        const stable = !variable.references.some((reference) => reference.isWrite() && reference.init !== true);
+        referenceStability.set(variable, stable);
+        return stable;
+      };
       const scopes = [newScope()];
       const reportedOrigins = /* @__PURE__ */ new Set();
       const currentScope = () => scopes.at(-1) ?? scopes[0];
@@ -27573,7 +27591,7 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
       };
       const visible = (kind, node) => {
         const name2 = bindingOf(node);
-        if (name2 === null || name2.references.some((reference) => reference.isWrite() && reference.init !== true)) return false;
+        if (name2 === null || !referenceStable(name2)) return false;
         for (let index = scopes.length - 1; index >= 0; index--) {
           const scope = scopes[index];
           if (scope.declared.has(name2)) return scope[kind].has(name2);
@@ -27582,7 +27600,7 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
       };
       const visibleRawOrigins = (node) => {
         const name2 = bindingOf(node);
-        if (name2 === null || name2.references.some((reference) => reference.isWrite() && reference.init !== true)) return /* @__PURE__ */ new Set();
+        if (name2 === null || !referenceStable(name2)) return /* @__PURE__ */ new Set();
         for (let index = scopes.length - 1; index >= 0; index--) {
           const scope = scopes[index];
           if (scope.declared.has(name2)) return scope.rawOrigins.get(name2) ?? /* @__PURE__ */ new Set();
@@ -27614,7 +27632,7 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
       };
       const nativeImportedName = (node, modules) => {
         const binding = bindingOf(node);
-        if (binding === null || binding.references.some((reference) => reference.isWrite() && reference.init !== true)) return null;
+        if (binding === null || !referenceStable(binding)) return null;
         for (const definition of binding.defs) {
           const imported = definition.node;
           if (imported.type === AST_NODE_TYPES123.ImportSpecifier && imported.imported.type === AST_NODE_TYPES123.Identifier && imported.parent.type === AST_NODE_TYPES123.ImportDeclaration && modules.has(String(imported.parent.source.value))) {
@@ -27625,7 +27643,7 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
       };
       const stableInitializer = (node) => {
         const binding = bindingOf(node);
-        if (binding === null || binding.references.some((reference) => reference.isWrite() && reference.init !== true) || binding.defs.length !== 1) return null;
+        if (binding === null || !referenceStable(binding) || binding.defs.length !== 1) return null;
         const definition = binding.defs[0]?.node;
         return definition?.type === AST_NODE_TYPES123.VariableDeclarator ? definition.init : null;
       };

package/package.json

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