Revert "cleanup errors (rectorphp#7826)"

This reverts commit edc3ee2.

Author: calebdw

composer.json

@@ -56,7 +56,7 @@
         "rector/type-perfect": "^2.1.2",
         "shipmonk/composer-dependency-analyser": "^1.8",
         "symplify/phpstan-extensions": "^12.0.2",
-        "symplify/phpstan-rules": "^14.9.11",
+        "symplify/phpstan-rules": "^14.9.4",
         "symplify/vendor-patches": "^11.5",
         "tomasvotruba/class-leak": "^2.1",
         "tomasvotruba/unused-public": "^2.2",

phpstan.neon

@@ -5,8 +5,7 @@ includes:
 parameters:
     level: 8
 
-    # reportUnmatchedIgnoredErrors: false
-
+    reportUnmatchedIgnoredErrors: false
     errorFormat: symplify
 
     # see https://phpstan.org/writing-php-code/phpdoc-types#global-type-aliases
@@ -372,6 +371,11 @@ parameters:
 
         - '#Method Rector\\Utils\\Rector\\RemoveRefactorDuplicatedNodeInstanceCheckRector\:\:getInstanceofNodeClass\(\) should return class\-string<PhpParser\\Node>\|null but returns class\-string#'
 
+        # helper classes for bin script
+        -
+            path: scripts
+            identifier:  symplify.multipleClassLikeInFile
+
         # copied from /vendor, to keep as original as possible
         -
             identifier: missingType.iterableValue
@@ -402,6 +406,9 @@ parameters:
                 - rules/DeadCode/Rector/ConstFetch/RemovePhpVersionIdCheckRector.php
                 - rules/DeadCode/Rector/If_/UnwrapFutureCompatibleIfPhpVersionRector.php
 
+        # condition check, just to be sure
+        - '#Method Rector\\Rector\\AbstractRector\:\:enterNode\(\) never returns 3 so it can be removed from the return type#'
+
         # deprecated
         -
             identifier: public.method.unused
@@ -410,7 +417,14 @@ parameters:
                 - rules/CodeQuality/ValueObject/KeyAndExpr.php
         -
             identifier: symplify.forbiddenExtendOfNonAbstractClass
-            path: rules/Php81/Rector/Array_/FirstClassCallableRector.php
+            path: src/PhpParser/Node/FileNode.php
+        -
+            identifier: method.deprecatedClass
+            path: src/PhpParser/Node/FileNode.php
+
+        -
+            identifier: class.extendsDeprecatedClass
+            path: src/PhpParser/Node/FileNode.php
 
         -
             identifier: public.classConstant.unused
@@ -428,23 +442,38 @@ parameters:
         - '#Rule Rector\\Php81\\Rector\\Array_\\FirstClassCallableRector must implements Rector\\VersionBonding\\Contract\\MinPhpVersionInterface#'
         - '#Register "Rector\\Php81\\Rector\\Array_\\FirstClassCallableRector" service to "php81\.php" config set#'
         - '#Class "Rector\\CodingStyle\\Rector\\String_\\SymplifyQuoteEscapeRector" is missing @see annotation with test case class reference#'
+        - '#Access to constant on deprecated class Rector\\Php81\\Rector\\Array_\\FirstClassCallableRector#'
 
         # BC layer for FileWithoutNamespace node
+        - message: '#Use @see \\Rector\\PhpParser\\Node\\FileNode instead#'
+        - '#BC layer for FileNode and FileWithoutNamespace compat, use FileNode instead#'
         -
             path: src/PhpParser/Node/CustomNode/FileWithoutNamespace.php
             identifier: symplify.forbiddenExtendOfNonAbstractClass
+        -
+            message: '#Only abstract classes can be extended#'
+            path: rules/Php81/Rector/Array_/FirstClassCallableRector.php
 
+        - '#Method Rector\\Tests\\Issues\\FileWithoutNamespaceCompat\\Rector\\SubscribedToFileWithoutNamespaceRector\:\:refactor\(\) should return Rector\\PhpParser\\Node\\FileNode but returns Rector\\PhpParser\\Node\\CustomNode\\FileWithoutNamespace#'
+
+        # BC layer for FileWithoutNamespace node
         - message: '#Use @see \\Rector\\PhpParser\\Node\\FileNode instead#'
+        - '#Class Rector\\PhpParser\\Node\\CustomNode\\FileWithoutNamespace extends final class Rector\\PhpParser\\Node\\FileNode#'
+        -
+            path: src/PhpParser/Node/CustomNode/FileWithoutNamespace.php
+            identifier: symplify.forbiddenExtendOfNonAbstractClass
 
         # false positive
         -
             identifier: phpstanApi.varTagAssumption
             path: rules/TypeDeclarationDocblocks/Rector/ClassMethod/AddReturnDocblockForDimFetchArrayFromAssignsRector.php
 
-        # @todo fix in phpstan-rules
+        # bug, fix in symplify/phpstan-rules
         -
             identifier: symplify.noMissnamedDocTag
-            path: rules/TypeDeclarationDocblocks/TypeResolver/ConstantArrayTypeGeneralizer.php
+            paths:
+                 - rules/Php74/NodeAnalyzer/ClosureArrowFunctionAnalyzer.php
+                 - rules/TypeDeclarationDocblocks/TypeResolver/ConstantArrayTypeGeneralizer.php
 
         -
             identifier: typePerfect.narrowReturnObjectType

rules/TypeDeclarationDocblocks/TypeResolver/ConstantArrayTypeGeneralizer.php

@@ -17,8 +17,6 @@
 final class ConstantArrayTypeGeneralizer
 {
     /**
-     * @var int
-     *
      * Using 10-level array @return docblocks makes code very hard to read,
      * lets limit it to reasonable level
      */