@blumintinc/eslint-plugin-blumint 1.19.5 → 1.19.7

package/lib/index.js

@@ -222,7 +222,7 @@ function noFrontendImportsFromFunctionsPatterns(pattern) {
 module.exports = {
     meta: {
         name: '@blumintinc/eslint-plugin-blumint',
-        version: '1.19.5',
+        version: '1.19.7',
     },
     parseOptions: {
         ecmaVersion: 2020,

package/lib/rules/no-redundant-this-params.js

@@ -77,8 +77,105 @@ exports.noRedundantThisParams = (0, createRule_1.createRule)({
             }
             return [];
         }
+        function isBindingExportedInScope(scopeBody, bindingName) {
+            // A class bound at module (or namespace) scope can be exported by a
+            // statement separate from its declaration — `export { Foo }`,
+            // `export { Foo as Bar }`, `export default Foo`, or `export = Foo`. Any of
+            // these exposes it to subclassing from another file just as an inline
+            // export does.
+            for (const statement of scopeBody) {
+                if (statement.type === utils_1.AST_NODE_TYPES.ExportNamedDeclaration &&
+                    !statement.source) {
+                    for (const specifier of statement.specifiers) {
+                        if (specifier.local.type === utils_1.AST_NODE_TYPES.Identifier &&
+                            specifier.local.name === bindingName) {
+                            return true;
+                        }
+                    }
+                }
+                if (statement.type === utils_1.AST_NODE_TYPES.ExportDefaultDeclaration &&
+                    statement.declaration.type === utils_1.AST_NODE_TYPES.Identifier &&
+                    statement.declaration.name === bindingName) {
+                    return true;
+                }
+                if (statement.type === utils_1.AST_NODE_TYPES.TSExportAssignment &&
+                    statement.expression.type === utils_1.AST_NODE_TYPES.Identifier &&
+                    statement.expression.name === bindingName) {
+                    return true;
+                }
+            }
+            return false;
+        }
+        function getEnclosingScopeBody(node) {
+            if (!node) {
+                return null;
+            }
+            if (node.type === utils_1.AST_NODE_TYPES.Program) {
+                return node.body;
+            }
+            if (node.type === utils_1.AST_NODE_TYPES.TSModuleBlock) {
+                return node.body;
+            }
+            return null;
+        }
+        function isExportedClass(node) {
+            const parent = node.parent;
+            if (!parent) {
+                return false;
+            }
+            if (parent.type === utils_1.AST_NODE_TYPES.ExportNamedDeclaration ||
+                parent.type === utils_1.AST_NODE_TYPES.ExportDefaultDeclaration) {
+                return true;
+            }
+            // A `class Foo {}` declaration exported by a later `export { Foo }` /
+            // `export default Foo` in the same scope is still reachable cross-file.
+            if (node.type === utils_1.AST_NODE_TYPES.ClassDeclaration &&
+                node.id &&
+                (parent.type === utils_1.AST_NODE_TYPES.Program ||
+                    parent.type === utils_1.AST_NODE_TYPES.TSModuleBlock)) {
+                const scopeBody = getEnclosingScopeBody(parent);
+                if (scopeBody && isBindingExportedInScope(scopeBody, node.id.name)) {
+                    return true;
+                }
+            }
+            // `export const Foo = class { ... }`, or `const Foo = class {}` exported by
+            // a later statement, both expose the class expression like an exported
+            // declaration does.
+            if (node.type === utils_1.AST_NODE_TYPES.ClassExpression &&
+                parent.type === utils_1.AST_NODE_TYPES.VariableDeclarator &&
+                parent.id.type === utils_1.AST_NODE_TYPES.Identifier) {
+                const declaration = parent.parent;
+                const exportNode = declaration?.parent;
+                if (exportNode?.type === utils_1.AST_NODE_TYPES.ExportNamedDeclaration ||
+                    exportNode?.type === utils_1.AST_NODE_TYPES.ExportDefaultDeclaration) {
+                    return true;
+                }
+                const scopeBody = getEnclosingScopeBody(exportNode);
+                if (scopeBody && isBindingExportedInScope(scopeBody, parent.id.name)) {
+                    return true;
+                }
+            }
+            return false;
+        }
         function collectClassInfo(node) {
             const methods = new Map();
+            // A method's `this.method(...)` call sites are confined to this file only
+            // when the class cannot be extended from another file. An `abstract` class
+            // exists to be subclassed, and an exported class can be subclassed
+            // anywhere; either way subclasses may call an inherited non-private method
+            // with a different `this.<member>`, so those methods are externally
+            // reachable and must not be reported. See issue #1309.
+            const classIsExtensible = Boolean(node.abstract) ||
+                isExportedClass(node);
+            function computeExternallyReachable(methodName, accessibility) {
+                if (!classIsExtensible) {
+                    return false;
+                }
+                // `private` (TS) and `#name` (ECMAScript) methods are never inherited,
+                // so all of their call sites live in the declaring class body.
+                const isPrivate = accessibility === 'private' || methodName.startsWith('#');
+                return !isPrivate;
+            }
             function setMethod(methodName, meta) {
                 /**
                  * Prefer instance methods over static methods when names collide.
@@ -120,6 +217,7 @@ exports.noRedundantThisParams = (0, createRule_1.createRule)({
                         params: getMethodParams(member),
                         isStatic: Boolean(member.static),
                         isAbstract: member.type === utils_1.AST_NODE_TYPES.TSAbstractMethodDefinition,
+                        externallyReachable: computeExternallyReachable(methodName, member.accessibility),
                     });
                 }
                 else if (member.type === utils_1.AST_NODE_TYPES.PropertyDefinition) {
@@ -137,6 +235,7 @@ exports.noRedundantThisParams = (0, createRule_1.createRule)({
                         params,
                         isStatic: false,
                         isAbstract: false,
+                        externallyReachable: computeExternallyReachable(methodName, member.accessibility),
                     });
                 }
             }
@@ -432,6 +531,12 @@ exports.noRedundantThisParams = (0, createRule_1.createRule)({
                         for (const [, propStats] of argProperties) {
                             if (propStats.callsWithProperty === methodStats.totalCalls) {
                                 for (const violation of propStats.violations) {
+                                    // Skip methods whose call sites are not provably confined to
+                                    // this file: a subclass elsewhere may thread a different
+                                    // `this.<member>` through the same parameter (issue #1309).
+                                    if (violation.methodMeta.externallyReachable) {
+                                        continue;
+                                    }
                                     reportAccess(methodName, violation.methodMeta, argIndex, violation.access);
                                 }
                             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blumintinc/eslint-plugin-blumint",
-  "version": "1.19.5",
+  "version": "1.19.7",
   "description": "Custom eslint rules for use within BluMint",
   "author": {
     "name": "Brodie McGuire",

package/release-manifest.json

@@ -1,4 +1,32 @@
 [
+  {
+    "version": "1.19.7",
+    "date": "2026-07-16T01:36:53.555Z",
+    "rules": [
+      {
+        "name": "no-redundant-this-params",
+        "changeType": "fix",
+        "issues": [
+          1309
+        ],
+        "summary": "treat trailing-export classes as reachable"
+      }
+    ]
+  },
+  {
+    "version": "1.19.6",
+    "date": "2026-07-16T01:27:07.130Z",
+    "rules": [
+      {
+        "name": "no-redundant-this-params",
+        "changeType": "fix",
+        "issues": [
+          1309
+        ],
+        "summary": "skip externally-reachable methods (closes #1309)"
+      }
+    ]
+  },
   {
     "version": "1.19.5",
     "date": "2026-07-15T15:24:16.132Z",