Changes suggested in review.

Author: JakeQZ

src/Rule/Rule.php

@@ -114,11 +114,13 @@ public static function parse(ParserState $oParserState): Rule
     }
 
     /**
-     * @param string $sRule
-     *
-     * @return array<int, string>
+     * Returns a list of delimiters (or separators).
      * The first item is the innermost separator (or, put another way, the highest-precedence operator).
      * The sequence continues to the outermost separator (or lowest-precedence operator).
+     *
+     * @param string $sRule
+     *
+     * @return list<non-empty-string>
      */
     private static function listDelimiterForRule($sRule): array
     {

tests/Rule/RuleTest.php

@@ -17,7 +17,7 @@
 final class RuleTest extends TestCase
 {
     /**
-     * @return array<string, array{0: string, 1: array<int, string>}>
+     * @return array<string, array{0: string, 1:  list<class-string>}>
      */
     public static function provideRulesAndExpectedParsedValueListTypes(): array
     {
@@ -35,7 +35,7 @@ public static function provideRulesAndExpectedParsedValueListTypes(): array
     /**
      * @test
      *
-     * @param array<int, string> $expectedTypeClassnames
+     * @param list<class-string> $expectedTypeClassnames
      *
      * @dataProvider provideRulesAndExpectedParsedValueListTypes
      */
@@ -47,8 +47,11 @@ public function parsesValuesIntoExpectedTypeList(string $rule, array $expectedTy
         self::assertInstanceOf(ValueList::class, $value);
 
         $actualClassnames = \array_map(
-            function ($component): string {
-                return \get_class($component);
+            /**
+             * @param Value|string $component
+             */
+            static function ($component): string {
+                return \is_string($component) ? 'string' : \get_class($component);
             },
             $value->getListComponents()
         );