@blumintinc/eslint-plugin-blumint 1.19.4 → 1.19.5
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/lib/index.js
CHANGED
|
@@ -102,7 +102,14 @@ exports.noRedundantThisParams = (0, createRule_1.createRule)({
|
|
|
102
102
|
for (const member of node.body.body) {
|
|
103
103
|
if (member.type === utils_1.AST_NODE_TYPES.MethodDefinition ||
|
|
104
104
|
member.type === utils_1.AST_NODE_TYPES.TSAbstractMethodDefinition) {
|
|
105
|
-
|
|
105
|
+
// Only genuine methods (`kind: 'method'`) are invoked directly as
|
|
106
|
+
// `this.x(args)`. A `get`/`set` accessor is also a MethodDefinition,
|
|
107
|
+
// but `this.accessor(args)` evaluates the accessor with zero arguments
|
|
108
|
+
// and then calls its RETURN VALUE — so when that value is an
|
|
109
|
+
// externally-supplied function it legitimately needs instance state
|
|
110
|
+
// threaded in, and flagging it is a false positive (issue #1308). The
|
|
111
|
+
// constructor is likewise never called as `this.constructor(...)`.
|
|
112
|
+
if (member.kind !== 'method') {
|
|
106
113
|
continue;
|
|
107
114
|
}
|
|
108
115
|
const methodName = getNameFromPropertyName(member.key);
|
package/package.json
CHANGED
package/release-manifest.json
CHANGED
|
@@ -1,4 +1,18 @@
|
|
|
1
1
|
[
|
|
2
|
+
{
|
|
3
|
+
"version": "1.19.5",
|
|
4
|
+
"date": "2026-07-15T15:24:16.132Z",
|
|
5
|
+
"rules": [
|
|
6
|
+
{
|
|
7
|
+
"name": "no-redundant-this-params",
|
|
8
|
+
"changeType": "fix",
|
|
9
|
+
"issues": [
|
|
10
|
+
1308
|
|
11
|
+
],
|
|
12
|
+
"summary": "skip get/set accessors when resolving callees (closes #1308)"
|
|
13
|
+
}
|
|
14
|
+
]
|
|
15
|
+
},
|
|
2
16
|
{
|
|
3
17
|
"version": "1.19.4",
|
|
4
18
|
"date": "2026-07-14T18:24:36.895Z",
|