Tighten match type to drop (string) casts

PatternIterator declared its yielded match as array<mixed>, but
preg_match results (no PREG_OFFSET_CAPTURE / PREG_UNMATCHED_AS_NULL)
are always string-valued. Declaring array<array-key, string> reflects
that guarantee and lets buildQuery() return/concatenate the match
values directly, removing the (string) casts.

Co-Authored-By: Claude Code

Author: JanTvrdik

src/BaseMultiQueryParser.php

@@ -69,17 +69,15 @@ abstract public function parseStringStream(Iterator $stream): Iterator;
 	/**
 	 * Builds the yielded query string, prepending captured leading comments when enabled.
 	 *
-	 * @param array<mixed> $match
+	 * @param array<array-key, string> $match
 	 */
 	protected function buildQuery(array $match): string
 	{
-		$query = (string) $match['query'];
-
 		if (!$this->preserveLeadingComments) {
-			return $query;
+			return $match['query'];
 		}
 
-		return (string) ($match['leadingComments'] ?? '') . $query;
+		return ($match['leadingComments'] ?? '') . $match['query'];
 	}
 
 

src/PatternIterator.php

@@ -30,7 +30,7 @@
  * the regex engine commits to the construct — if the closing delimiter is missing (because
  * it is in a later chunk), the overall match fails, causing the iterator to load more data.
  *
- * @implements IteratorAggregate<int, array<mixed>>
+ * @implements IteratorAggregate<int, array<array-key, string>>
  */
 class PatternIterator implements IteratorAggregate
 {