@zzzen/pyright-internal 1.2.0-dev.20260927 → 1.2.0-dev.20261004

package/dist/analyzer/checker.d.ts

@@ -148,7 +148,6 @@ export declare class Checker extends ParseTreeWalker {
     private _validateTypedDictOverrides;
     private _validateBaseClassOverrides;
     private _validateOverrideDecoratorPresent;
-    private _isMethodExemptFromLsp;
     private _validateOverrideDecoratorNotPresent;
     private _getCallableVariableOverrideComparison;
     private _markFunctionStatic;

package/dist/analyzer/checker.js

@@ -2803,7 +2803,10 @@ class Checker extends parseTreeWalker_1.ParseTreeWalker {
         }
         const action = rule === diagnosticRules_1.DiagnosticRule.reportUnusedImport ? { action: "pyright.unusedImport" /* Commands.unusedImport */ } : undefined;
         if (nameNode) {
-            this._fileInfo.diagnosticSink.addUnusedCodeWithTextRange(localize_1.LocMessage.unaccessedSymbol().format({ name: nameNode.d.value }), nameNode, action);
+            const isAllowedUnusedVariable = rule === diagnosticRules_1.DiagnosticRule.reportUnusedVariable && nameNode.d.value.startsWith('_');
+            if (!isAllowedUnusedVariable) {
+                this._fileInfo.diagnosticSink.addUnusedCodeWithTextRange(localize_1.LocMessage.unaccessedSymbol().format({ name: nameNode.d.value }), nameNode, action);
+            }
             if (rule !== undefined && message && diagnosticLevel !== 'none') {
                 this._evaluator.addDiagnostic(rule, message, nameNode);
             }
@@ -4706,7 +4709,7 @@ class Checker extends parseTreeWalker_1.ParseTreeWalker {
             return;
         }
         // Constructors are exempt.
-        if (this._isMethodExemptFromLsp(overrideFunction.shared.name)) {
+        if (SymbolNameUtils.isMethodExemptFromLsp(overrideFunction.shared.name)) {
             return;
         }
         // If the declaration for the override function is not the same as the
@@ -4724,11 +4727,6 @@ class Checker extends parseTreeWalker_1.ParseTreeWalker {
             className: this._evaluator.printType((0, typeUtils_1.convertToInstance)(baseMember.classType)),
         }), funcNode.d.name);
     }
-    // Determines whether the name is exempt from Liskov Substitution Principle rules.
-    _isMethodExemptFromLsp(name) {
-        const exemptMethods = ['__init__', '__new__', '__init_subclass__', '__post_init__'];
-        return exemptMethods.some((n) => n === name);
-    }
     // Determines whether the type is a function or overloaded function with an @override
     // decorator. In this case, an error is reported because no base class has declared
     // a method of the same name.
@@ -4877,7 +4875,7 @@ class Checker extends parseTreeWalker_1.ParseTreeWalker {
             // Also, skip this check if the class is a TypedDict. The methods for a TypedDict
             // are synthesized, and they can result in many overloads. We assume they
             // are correct and will not produce any errors.
-            if (this._isMethodExemptFromLsp(memberName) ||
+            if (SymbolNameUtils.isMethodExemptFromLsp(memberName) ||
                 SymbolNameUtils.isPrivateName(memberName) ||
                 types_1.ClassType.isTypedDictClass(childClassType)) {
                 return;