@typescript-eslint/eslint-plugin 8.70.1-alpha.16 → 8.70.1-alpha.18

package/dist/rules/no-unnecessary-parameter-property-assignment.js

@@ -101,10 +101,6 @@ exports.default = (0, util_1.createRule)({
                 });
             },
             "MethodDefinition[kind='constructor'] > FunctionExpression AssignmentExpression"(node) {
-                const leftName = getPropertyName(node.left);
-                if (!leftName) {
-                    return;
-                }
                 let functionNode = findParentFunction(node);
                 if (functionNode && isArrowIIFE(functionNode)) {
                     functionNode = findParentFunction(functionNode.parent);
@@ -113,6 +109,18 @@ exports.default = (0, util_1.createRule)({
                     return;
                 }
                 const { assignedBeforeUnnecessary, unnecessaryAssignments } = (0, util_1.nullThrows)(reportInfoStack.at(reportInfoStack.length - 1), 'The top of stack should exist');
+                // A prior write to the parameter binding means a later `this.x = x`
+                // may copy a different value than the parameter property initialized.
+                if (node.left.type === utils_1.AST_NODE_TYPES.Identifier) {
+                    if (isReferenceFromParameter(node.left)) {
+                        assignedBeforeUnnecessary.add(node.left.name);
+                    }
+                    return;
+                }
+                const leftName = getPropertyName(node.left);
+                if (!leftName) {
+                    return;
+                }
                 if (!UNNECESSARY_OPERATORS.has(node.operator)) {
                     assignedBeforeUnnecessary.add(leftName);
                     return;

package/dist/rules/unbound-method.js

@@ -90,6 +90,19 @@ const SUPPORTED_GLOBAL_TYPES = [
     'Math',
     'JSON',
 ];
+/**
+ * Methods on instances of built-in classes that are defined by the ECMAScript
+ * spec as *bound* to their instance (i.e. they are safe to call without their
+ * receiver), but whose TypeScript declarations don't model that binding - they
+ * are declared as plain methods that would require a `this` value.
+ *
+ * For example, `Intl.Collator.prototype.compare` is defined as a bound getter
+ * function per the ECMA-402 spec:
+ * https://tc39.es/ecma402/#sec-intl.collator.prototype.compare
+ * ...but `lib.es5.d.ts` declares it as a regular method, so this rule would
+ * otherwise incorrectly flag usages such as `array.sort(collator.compare)`.
+ */
+const nativelyBoundInstanceMethods = new Set(['Collator.compare']);
 const isNotImported = (symbol, currentSourceFile) => {
     const { valueDeclaration } = symbol;
     if (!valueDeclaration) {
@@ -99,6 +112,18 @@ const isNotImported = (symbol, currentSourceFile) => {
     return (!!currentSourceFile &&
         currentSourceFile !== valueDeclaration.getSourceFile());
 };
+/**
+ * Returns `true` if `propertyName` is a member of a built-in class instance
+ * that is defined by the spec as bound to that instance (see
+ * `nativelyBoundInstanceMethods` above).
+ */
+function isSpecBoundBuiltinMethod(program, objectType, propertyName) {
+    const symbol = objectType.getSymbol();
+    if (!symbol || !(0, util_1.isSymbolFromDefaultLibrary)(program, symbol)) {
+        return false;
+    }
+    return nativelyBoundInstanceMethods.has(`${symbol.getName()}.${propertyName}`);
+}
 const BASE_MESSAGE = [
     `A method that is not declared with \`this: void\` may cause unintentional scoping of \`this\` when separated from its object.`,
     `Consider using an arrow function or explicitly \`.bind()\`ing the method to avoid calling the method with an unintended \`this\` value. `,
@@ -196,6 +221,12 @@ exports.default = (0, util_1.createRule)({
                     return true;
                 }
             }
+            if (property.type === utils_1.AST_NODE_TYPES.Identifier) {
+                const objectType = services.getTypeAtLocation(object);
+                if (isSpecBoundBuiltinMethod(services.program, objectType, property.name)) {
+                    return true;
+                }
+            }
             // if `${object.name}.${property.name}` doesn't match any of
             // the nativelyBoundMembers, then we fallback to type-level checks
             return ((0, util_1.isBuiltinSymbolLike)(services.program, services.getTypeAtLocation(object), SUPPORTED_GLOBAL_TYPES) &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@typescript-eslint/eslint-plugin",
-  "version": "8.70.1-alpha.16",
+  "version": "8.70.1-alpha.18",
   "description": "TypeScript plugin for ESLint",
   "files": [
     "dist",
@@ -47,10 +47,10 @@
   ],
   "dependencies": {
     "@eslint-community/regexpp": "^4.12.2",
-    "@typescript-eslint/scope-manager": "8.70.1-alpha.16",
-    "@typescript-eslint/type-utils": "8.70.1-alpha.16",
-    "@typescript-eslint/utils": "8.70.1-alpha.16",
-    "@typescript-eslint/visitor-keys": "8.70.1-alpha.16",
+    "@typescript-eslint/scope-manager": "8.70.1-alpha.18",
+    "@typescript-eslint/type-utils": "8.70.1-alpha.18",
+    "@typescript-eslint/utils": "8.70.1-alpha.18",
+    "@typescript-eslint/visitor-keys": "8.70.1-alpha.18",
     "ignore": "^7.0.5",
     "natural-compare": "^1.4.0",
     "ts-api-utils": "^2.5.0"
@@ -60,8 +60,8 @@
     "@types/mdast": "^4.0.4",
     "@types/natural-compare": "^1.4.3",
     "@types/react": "^18.3.21",
-    "@typescript-eslint/rule-schema-to-typescript-types": "8.70.1-alpha.16",
-    "@typescript-eslint/rule-tester": "8.70.1-alpha.16",
+    "@typescript-eslint/rule-schema-to-typescript-types": "8.70.1-alpha.18",
+    "@typescript-eslint/rule-tester": "8.70.1-alpha.18",
     "@typescript/native": "npm:typescript@^7.0.2",
     "@vitest/coverage-v8": "^4.0.18",
     "ajv": "^6.12.6",
@@ -81,7 +81,7 @@
     "vitest": "^4.0.18"
   },
   "peerDependencies": {
-    "@typescript-eslint/parser": "^8.70.1-alpha.16",
+    "@typescript-eslint/parser": "^8.70.1-alpha.18",
     "eslint": "^8.57.0 || ^9.0.0 || ^10.0.0",
     "typescript": ">=4.8.4 <6.1.0"
   },