Make AnalyserResult::processedFiles non-optional, remove unused callback parameter

Address review feedback:
- Remove default value from AnalyserResult::$processedFiles, add explicit
  processedFiles: [] at all construction sites
- Remove unused $processedFiles parameter from non-debug postFileCallback
- Revert postFileCallback type annotations to optional second parameter

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: phpstan-bot

src/Analyser/Analyser.php

@@ -36,7 +36,7 @@ public function __construct(
 	/**
 	 * @param string[] $files
 	 * @param Closure(string $file): void|null $preFileCallback
-	 * @param Closure(int, list<string>): void|null $postFileCallback
+	 * @param Closure(int, list<string>=): void|null $postFileCallback
 	 * @param string[]|null $allAnalysedFiles
 	 */
 	public function analyse(

src/Analyser/AnalyserResult.php

@@ -44,7 +44,7 @@ public function __construct(
 		private array $exportedNodes,
 		private bool $reachedInternalErrorsCountLimit,
 		private int $peakMemoryUsageBytes,
-		private array $processedFiles = [],
+		private array $processedFiles,
 	)
 	{
 	}

src/Command/AnalyseApplication.php

@@ -244,12 +244,13 @@ private function runAnalyser(
 				exportedNodes: [],
 				reachedInternalErrorsCountLimit: false,
 				peakMemoryUsageBytes: memory_get_peak_usage(true),
+				processedFiles: [],
 			);
 		}
 
 		if (!$debug) {
 			$preFileCallback = null;
-			$postFileCallback = static function (int $step, array $processedFiles) use ($errorOutput): void {
+			$postFileCallback = static function (int $step) use ($errorOutput): void {
 				$errorOutput->getStyle()->progressAdvance($step);
 			};
 
@@ -264,7 +265,7 @@ private function runAnalyser(
 			$postFileCallback = null;
 			if ($stdOutput->isDebug()) {
 				$previousMemory = memory_get_peak_usage(true);
-				$postFileCallback = static function (int $step, array $processedFiles) use ($stdOutput, &$previousMemory, &$startTime, &$linesOfCode): void {
+				$postFileCallback = static function (int $step, array $processedFiles = []) use ($stdOutput, &$previousMemory, &$startTime, &$linesOfCode): void {
 					if ($startTime === null) {
 						throw new ShouldNotHappenException();
 					}

src/Command/AnalyserRunner.php

@@ -37,7 +37,7 @@ public function __construct(
 	 * @param string[] $files
 	 * @param string[] $allAnalysedFiles
 	 * @param Closure(string $file): void|null $preFileCallback
-	 * @param Closure(int, list<string>): void|null $postFileCallback
+	 * @param Closure(int, list<string>=): void|null $postFileCallback
 	 */
 	public function runAnalyser(
 		array $files,
@@ -68,6 +68,7 @@ public function runAnalyser(
 				exportedNodes: [],
 				reachedInternalErrorsCountLimit: false,
 				peakMemoryUsageBytes: memory_get_peak_usage(true),
+				processedFiles: [],
 			);
 		}
 

src/Command/FixerWorkerCommand.php

@@ -404,6 +404,7 @@ private function runAnalyser(LoopInterface $loop, Container $container, array $f
 				exportedNodes: [],
 				reachedInternalErrorsCountLimit: false,
 				peakMemoryUsageBytes: memory_get_peak_usage(true),
+				processedFiles: [],
 			));
 		}
 

src/Parallel/ParallelAnalyser.php

@@ -58,7 +58,7 @@ public function __construct(
 	}
 
 	/**
-	 * @param Closure(int, list<string>): void|null $postFileCallback
+	 * @param Closure(int, list<string>=): void|null $postFileCallback
 	 * @param (callable(list<Error>, list<Error>, string[]): void)|null $onFileAnalysisHandler
 	 * @return PromiseInterface<AnalyserResult>
 	 */