fix extractor erasing comments when they precede strict_types

Author: xHeaven

src/PhpGenerator/Extractor.php

@@ -243,18 +243,29 @@ public function extractAll(): PhpFile
 	{
 		$phpFile = new PhpFile;
 
+		$comments = [];
+		$firstStmt = $this->statements[0] ?? null;
+		if ($firstStmt instanceof Node\Stmt\Declare_) {
+			$comments = $firstStmt->getComments();
+			if (!$firstStmt->getDocComment()) {
+				$comments = [];
+				$firstStmt = $this->statements[1] ?? null;
+			}
+		}
+
 		if (
-			($firstStmt = $this->statements[0] ?? null)
-			&& ($firstStmt = $firstStmt instanceof Node\Stmt\Declare_ ? $this->statements[1] ?? null : $firstStmt)
+			!$comments
+			&& $firstStmt
 			&& !$firstStmt instanceof Node\Stmt\ClassLike
 			&& !$firstStmt instanceof Node\Stmt\Function_
 		) {
 			$comments = $firstStmt->getComments();
-			foreach ($comments as $i => $comment) {
-				if ($comment instanceof PhpParser\Comment\Doc) {
-					$phpFile->setComment(Helpers::unformatDocComment($comment->getReformattedText()));
-					break;
-				}
+		}
+
+		foreach ($comments as $comment) {
+			if ($comment instanceof PhpParser\Comment\Doc) {
+				$phpFile->setComment(Helpers::unformatDocComment($comment->getReformattedText()));
+				break;
 			}
 		}
 

tests/PhpGenerator/Extractor.extractAll.file.comments.phpt

@@ -30,6 +30,19 @@ $file = (new Extractor(<<<'XX'
 Assert::same('doc comment', $file->getComment());
 
 
+$file = (new Extractor(<<<'XX'
+	<?php
+
+	/** doc comment */
+
+	declare(strict_types=1);
+
+	namespace Abc;
+	XX))->extractAll();
+
+Assert::same('doc comment', $file->getComment());
+
+
 $file = (new Extractor(<<<'XX'
 	<?php