@zzzen/pyright-internal 1.2.0-dev.20220724 → 1.2.0-dev.20220731

package/dist/analyzer/checker.js

@@ -2855,6 +2855,10 @@ class Checker extends parseTreeWalker_1.ParseTreeWalker {
         if (this._fileInfo.diagnosticRuleSet.reportUninitializedInstanceVariable === 'none') {
             return;
         }
+        // Protocol classes are exempted from this check.
+        if (types_1.ClassType.isProtocolClass(classType)) {
+            return;
+        }
         classType.details.fields.forEach((localSymbol, name) => {
             // This applies only to instance members.
             if (!localSymbol.isInstanceMember()) {
@@ -3519,6 +3523,7 @@ class Checker extends parseTreeWalker_1.ParseTreeWalker {
     // Performs checks on a function that is located within a class
     // and has been determined not to be a property accessor.
     _validateMethod(node, functionType, classNode) {
+        var _a, _b, _c, _d;
         const classTypeInfo = this._evaluator.getTypeOfClass(classNode);
         const classType = classTypeInfo === null || classTypeInfo === void 0 ? void 0 : classTypeInfo.classType;
         if (node.name && classType) {
@@ -3531,7 +3536,7 @@ class Checker extends parseTreeWalker_1.ParseTreeWalker {
                 }
             }
         }
-        if (node.name && node.name.value === '__new__') {
+        if (((_a = node.name) === null || _a === void 0 ? void 0 : _a.value) === '__new__') {
             // __new__ overrides should have a "cls" parameter.
             if (node.parameters.length === 0 ||
                 !node.parameters[0].name ||
@@ -3542,7 +3547,7 @@ class Checker extends parseTreeWalker_1.ParseTreeWalker {
                 this._validateClsSelfParameterType(functionType, classType, /* isCls */ true);
             }
         }
-        else if (node.name && node.name.value === '__init_subclass__') {
+        else if (((_b = node.name) === null || _b === void 0 ? void 0 : _b.value) === '__init_subclass__') {
             // __init_subclass__ overrides should have a "cls" parameter.
             if (node.parameters.length === 0 || !node.parameters[0].name || node.parameters[0].name.value !== 'cls') {
                 this._evaluator.addDiagnostic(this._fileInfo.diagnosticRuleSet.reportSelfClsParameterName, diagnosticRules_1.DiagnosticRule.reportSelfClsParameterName, localize_1.Localizer.Diagnostic.initSubclassClsParam(), node.parameters.length > 0 ? node.parameters[0] : node.name);
@@ -3551,7 +3556,7 @@ class Checker extends parseTreeWalker_1.ParseTreeWalker {
                 this._validateClsSelfParameterType(functionType, classType, /* isCls */ true);
             }
         }
-        else if (node.name && node.name.value === '__class_getitem__') {
+        else if (((_c = node.name) === null || _c === void 0 ? void 0 : _c.value) === '__class_getitem__') {
             // __class_getitem__ overrides should have a "cls" parameter.
             if (node.parameters.length === 0 || !node.parameters[0].name || node.parameters[0].name.value !== 'cls') {
                 this._evaluator.addDiagnostic(this._fileInfo.diagnosticRuleSet.reportSelfClsParameterName, diagnosticRules_1.DiagnosticRule.reportSelfClsParameterName, localize_1.Localizer.Diagnostic.classGetItemClsParam(), node.parameters.length > 0 ? node.parameters[0] : node.name);
@@ -3560,6 +3565,9 @@ class Checker extends parseTreeWalker_1.ParseTreeWalker {
                 this._validateClsSelfParameterType(functionType, classType, /* isCls */ true);
             }
         }
+        else if (((_d = node.name) === null || _d === void 0 ? void 0 : _d.value) === '_generate_next_value_') {
+            // Skip this check for _generate_next_value_.
+        }
         else if (types_1.FunctionType.isStaticMethod(functionType)) {
             // Static methods should not have "self" or "cls" parameters.
             if (node.parameters.length > 0 && node.parameters[0].name) {