@typescript-eslint/eslint-plugin 8.51.1-alpha.12 → 8.51.1-alpha.14

package/dist/rules/no-base-to-string.js

@@ -213,30 +213,19 @@ exports.default = (0, util_1.createRule)({
             if (checker.isArrayType(type)) {
                 return collectArrayCertainty(type, new Set([...visited, type]));
             }
-            const toString = checker.getPropertyOfType(type, 'toString') ??
-                checker.getPropertyOfType(type, 'toLocaleString');
-            if (!toString) {
-                // unknown
-                if (option.checkUnknown && type.flags === ts.TypeFlags.Unknown) {
-                    return Usefulness.Sometimes;
-                }
-                // e.g. any
-                return Usefulness.Always;
-            }
-            const declarations = toString.getDeclarations();
-            // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
-            if (declarations == null || declarations.length !== 1) {
-                // If there are multiple declarations, at least one of them must not be
-                // the default object toString.
-                //
-                // This may only matter for older versions of TS
-                // see https://github.com/typescript-eslint/typescript-eslint/issues/8585
-                return Usefulness.Always;
+            switch (isToStringLikeFromObject(type)) {
+                case undefined:
+                    // unknown
+                    if (option.checkUnknown && type.flags === ts.TypeFlags.Unknown) {
+                        return Usefulness.Sometimes;
+                    }
+                    // e.g. any
+                    return Usefulness.Always;
+                case true:
+                    return Usefulness.Never;
+                case false:
+                    return Usefulness.Always;
             }
-            const declaration = declarations[0];
-            const isBaseToString = ts.isInterfaceDeclaration(declaration.parent) &&
-                declaration.parent.name.text === 'Object';
-            return isBaseToString ? Usefulness.Never : Usefulness.Always;
         }
         function isBuiltInStringCall(node) {
             if (node.callee.type === utils_1.AST_NODE_TYPES.Identifier &&
@@ -250,6 +239,53 @@ exports.default = (0, util_1.createRule)({
             }
             return false;
         }
+        function isSymbolToPrimitiveMethod(node) {
+            return (ts.isMethodSignature(node) &&
+                ts.isComputedPropertyName(node.name) &&
+                ts.isPropertyAccessExpression(node.name.expression) &&
+                ts.isIdentifier(node.name.expression.expression) &&
+                node.name.expression.expression.text === 'Symbol' &&
+                ts.isIdentifier(node.name.expression.name) &&
+                node.name.expression.name.text === 'toPrimitive' &&
+                checker
+                    .getSymbolAtLocation(node.name.expression.expression)
+                    ?.valueDeclaration?.getSourceFile().hasNoDefaultLib);
+        }
+        function isToStringLikeFromObject(type) {
+            // An explicit [Symbol.toPrimitive] declaration is always user-defined
+            if (type
+                .getProperties()
+                .some(property => property.valueDeclaration &&
+                isSymbolToPrimitiveMethod(property.valueDeclaration))) {
+                return false;
+            }
+            // Otherwise, we check for known methods used in type coercion.
+            // We'll try to find one that's not declared on Object itself.
+            // Failing that, we'll fall back to one that is.
+            let foundFallbackOnObject = false;
+            for (const propertyName of ['toLocaleString', 'toString', 'valueOf']) {
+                const candidate = checker.getPropertyOfType(type, propertyName);
+                if (!candidate) {
+                    continue;
+                }
+                const declarations = candidate.getDeclarations();
+                // If there are multiple declarations, at least one of them must not be
+                // the default object toString.
+                //
+                // This may only matter for older versions of TS
+                // see https://github.com/typescript-eslint/typescript-eslint/issues/8585
+                if (declarations?.length !== 1) {
+                    continue;
+                }
+                // Not being the Object interface means this is user-defined.
+                if (!ts.isInterfaceDeclaration(declarations[0].parent) ||
+                    declarations[0].parent.name.text !== 'Object') {
+                    return false;
+                }
+                foundFallbackOnObject = true;
+            }
+            return foundFallbackOnObject ? true : undefined;
+        }
         return {
             'AssignmentExpression[operator = "+="], BinaryExpression[operator = "+"]'(node) {
                 const leftType = services.getTypeAtLocation(node.left);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@typescript-eslint/eslint-plugin",
-  "version": "8.51.1-alpha.12",
+  "version": "8.51.1-alpha.14",
   "description": "TypeScript plugin for ESLint",
   "files": [
     "dist",
@@ -59,10 +59,10 @@
   },
   "dependencies": {
     "@eslint-community/regexpp": "^4.12.2",
-    "@typescript-eslint/scope-manager": "8.51.1-alpha.12",
-    "@typescript-eslint/type-utils": "8.51.1-alpha.12",
-    "@typescript-eslint/utils": "8.51.1-alpha.12",
-    "@typescript-eslint/visitor-keys": "8.51.1-alpha.12",
+    "@typescript-eslint/scope-manager": "8.51.1-alpha.14",
+    "@typescript-eslint/type-utils": "8.51.1-alpha.14",
+    "@typescript-eslint/utils": "8.51.1-alpha.14",
+    "@typescript-eslint/visitor-keys": "8.51.1-alpha.14",
     "ignore": "^7.0.5",
     "natural-compare": "^1.4.0",
     "ts-api-utils": "^2.4.0"
@@ -70,8 +70,8 @@
   "devDependencies": {
     "@types/mdast": "^4.0.4",
     "@types/natural-compare": "*",
-    "@typescript-eslint/rule-schema-to-typescript-types": "8.51.1-alpha.12",
-    "@typescript-eslint/rule-tester": "8.51.1-alpha.12",
+    "@typescript-eslint/rule-schema-to-typescript-types": "8.51.1-alpha.14",
+    "@typescript-eslint/rule-tester": "8.51.1-alpha.14",
     "@vitest/coverage-v8": "^3.2.4",
     "ajv": "^6.12.6",
     "eslint": "*",
@@ -90,7 +90,7 @@
     "vitest": "^3.2.4"
   },
   "peerDependencies": {
-    "@typescript-eslint/parser": "^8.51.1-alpha.12",
+    "@typescript-eslint/parser": "^8.51.1-alpha.14",
     "eslint": "^8.57.0 || ^9.0.0",
     "typescript": ">=4.8.4 <6.0.0"
   },