Try to quit the child process only after internal errors were accounted for

Author: ondrejmirtes

.github/workflows/e2e-tests.yml

@@ -248,6 +248,13 @@ jobs:
               cd e2e/bad-exclude-paths
               OUTPUT=$(../../bin/phpstan analyse -c ignoreReportUnmatchedFalse.neon)
               echo "$OUTPUT"
+          - script: |
+              cd e2e/bug-11826
+              composer install
+              OUTPUT=$(../bashunit -a exit_code "1" "../../bin/phpstan")
+              echo "$OUTPUT"
+              ../bashunit -a contains 'Child process error (exit code 255): PHP Fatal error' "$OUTPUT"
+              ../bashunit -a contains 'Result is incomplete because of severe errors.' "$OUTPUT"
 
     steps:
       - name: "Checkout"

e2e/bug-11826/.gitignore

@@ -0,0 +1,2 @@
+/vendor
+/composer.lock

e2e/bug-11826/composer.json

@@ -0,0 +1,8 @@
+{
+    "autoload": {
+        "psr-4": {
+            "App\\": "src/",
+            "Rules\\": "rules/"
+        }
+    }
+}

e2e/bug-11826/phpstan.neon.dist

@@ -0,0 +1,8 @@
+parameters:
+    level: 9
+    paths:
+        - src
+        - rules
+
+rules:
+    - Rules\DummyRule

e2e/bug-11826/rules/DummyRule.php

@@ -0,0 +1,35 @@
+<?php declare(strict_types = 1);
+
+namespace Rules;
+
+use App\FatalErrorWhenAutoloaded;
+use PhpParser\Node;
+use PHPStan\Analyser\Scope;
+use PHPStan\Node\InClassNode;
+use PHPStan\Rules\IdentifierRuleError;
+use PHPStan\Rules\Rule;
+
+/**
+ * @implements Rule<InClassNode>
+ */
+class DummyRule implements Rule
+{
+
+    public function getNodeType(): string
+    {
+        return InClassNode::class;
+    }
+
+    /**
+     * @param InClassNode $node
+     * @return list<IdentifierRuleError>
+     */
+    public function processNode(
+        Node $node,
+        Scope $scope,
+    ): array
+    {
+        return [FatalErrorWhenAutoloaded::AUTOLOAD];
+    }
+
+}

e2e/bug-11826/src/FatalErrorWhenAutoloaded.php

@@ -0,0 +1,11 @@
+<?php declare(strict_types = 1);
+
+namespace App;
+
+interface FatalErrorWhenAutoloaded
+{
+
+    public const AUTOLOAD = 'autoload';
+    public const CLASS = 'error';
+
+}

src/Parallel/ParallelAnalyser.php

@@ -279,11 +279,12 @@ public function analyse(
 				}
 				$someChildEnded = true;
 
-				$this->processPool->tryQuitProcess($processIdentifier);
 				if ($exitCode === 0) {
+					$this->processPool->tryQuitProcess($processIdentifier);
 					return;
 				}
 				if ($exitCode === null) {
+					$this->processPool->tryQuitProcess($processIdentifier);
 					return;
 				}
 
@@ -294,6 +295,7 @@ public function analyse(
 							continue;
 						}
 
+						$this->processPool->tryQuitProcess($processIdentifier);
 						return;
 					}
 					$internalErrors[] = new InternalError(sprintf(
@@ -303,11 +305,13 @@ public function analyse(
 						'Increase your memory limit in php.ini or run PHPStan with --memory-limit CLI option.',
 					), 'running parallel worker', [], null, false);
 					$internalErrorsCount++;
+					$this->processPool->tryQuitProcess($processIdentifier);
 					return;
 				}
 
 				$internalErrors[] = new InternalError(sprintf('Child process error (exit code %d): %s', $exitCode, $output), 'running parallel worker', [], null, true);
 				$internalErrorsCount++;
+				$this->processPool->tryQuitProcess($processIdentifier);
 			});
 			$this->processPool->attachProcess($processIdentifier, $process);
 		}