Fix stan errors

Author: jaapio

packages/guides-cli/src/DevServer/RerenderListener.php

@@ -37,7 +37,7 @@
 
 final class RerenderListener
 {
-    /** @param array<string, DocumentNode> $documents */
+    /** @param DocumentNode[] $documents */
     public function __construct(
         private readonly OutputInterface $output,
         private readonly CommandBus $commandBus,
@@ -75,7 +75,15 @@ public function __invoke(FileModifiedEvent $event): void
         /** @var array<string, DocumentNode> $documents */
         $documents = $this->commandBus->handle(new CompileDocumentsCommand([$file => $document], new CompilerContext($this->projectNode)));
         $key = array_find_key($this->documents, static fn (DocumentNode $entry) => $entry->getFilePath() === $document->getFilePath());
-        $this->documents[$key] = current($documents);
+        $document = current($documents);
+        assert($document instanceof DocumentNode);
+        if ($key === null) {
+            // If the document is new, we add it to the list with its file path as key
+            $this->documents[] = $document;
+        }
+        else {
+            $this->documents[$key] = $document;
+        }
         $destinationFileSystem = FlySystemAdapter::createForPath($this->settings->getOutput());
 
         $documentIterator = DocumentListIterator::create(

packages/guides-restructured-text/composer.json

@@ -25,7 +25,7 @@
         "doctrine/lexer": "^3.0",
         "phpdocumentor/guides": "^1.0 || ^2.0",
         "webmozart/assert": "^1.11",
-        "league/csv": "^9.10.0"
+        "league/csv": "^9.27.0"
     },
     "extra": {
         "branch-alias": {

packages/guides-restructured-text/src/RestructuredText/Directives/CsvTableDirective.php

@@ -79,18 +79,10 @@ public function processNode(
                 return new GenericNode('csv-table');
             }
 
-            if (method_exists(Reader::class, 'from')) {
-                $csv = Reader::from($csvStream);
-            } else {
-                $csv = Reader::createFromStream($csvStream);
-            }
+            $csv = Reader::from($csvStream);
         } else {
             $lines = $blockContext->getDocumentIterator()->toArray();
-            if (method_exists(Reader::class, 'fromString')) {
-                $csv = Reader::fromString(implode("\n", $lines));
-            } else {
-                $csv = Reader::createFromString(implode("\n", $lines));
-            }
+            $csv = Reader::fromString(implode("\n", $lines));
         }
 
         if ($directive->getOption('header-rows')->getValue() !== null) {
@@ -99,11 +91,7 @@ public function processNode(
 
         $header = null;
         if ($directive->hasOption('header')) {
-            if (method_exists(Reader::class, 'fromString')) {
-                $headerCsv = Reader::fromString($directive->getOption('header')->toString());
-            } else {
-                $headerCsv = Reader::createFromString($directive->getOption('header')->toString());
-            }
+            $headerCsv = Reader::fromString($directive->getOption('header')->toString());
 
             $header = new TableRow();
             foreach ($headerCsv->first() as $column) {

packages/guides-restructured-text/src/RestructuredText/Parser/Productions/FieldListRule.php

@@ -103,8 +103,8 @@ private function parseField(string $line): array
     {
         if (preg_match('/^:([^:]+):( (.*)|)$/mUsi', $line, $match) > 0) {
             return [
-                $match[1] ?? '',
-                $match[2] ?? '',
+                $match[1],
+                $match[2],
             ];
         }
 

packages/guides/tests/unit/Interlink/InventoryLoaderTest.php

@@ -44,8 +44,6 @@ final class InventoryLoaderTest extends TestCase
     private JsonLoader&MockObject $jsonLoader;
     private DefaultInventoryRepository $inventoryRepository;
     private RenderContext&MockObject $renderContext;
-    /** @var array<string, mixed> */
-    private array $json;
 
     protected function setUp(): void
     {
@@ -65,9 +63,10 @@ public function loadObjectsJsonInv(string $filename): void
     {
         $jsonString = file_get_contents($filename);
         assertIsString($jsonString);
-        $this->json = (array) json_decode($jsonString, true, 512, JSON_THROW_ON_ERROR);
+        /** @var array<array-key, mixed> $json */
+        $json = (array) json_decode($jsonString, true, 512, JSON_THROW_ON_ERROR);
         $inventory = new Inventory('https://example.com/', new SluggerAnchorNormalizer());
-        $this->inventoryLoader->loadInventoryFromJson($inventory, $this->json);
+        $this->inventoryLoader->loadInventoryFromJson($inventory, $json);
         $this->inventoryRepository->addInventory('somekey', $inventory);
         $this->inventoryRepository->addInventory('some-key', $inventory);
     }
@@ -82,7 +81,9 @@ public function testInventoryLoaderLoadsInventory(): void
 
     public function testInventoryIsLoadedExactlyOnce(): void
     {
-        $this->jsonLoader->expects(self::once())->method('loadJsonFromUrl')->willReturn($this->json);
+        $this->jsonLoader->expects(self::once())->method('loadJsonFromUrl')->willReturn(
+            ['dummy' => 'data', 'items' => []]
+        );
         $inventory = new Inventory('https://example.com/', new SluggerAnchorNormalizer());
         $this->inventoryLoader->loadInventory($inventory);
         $this->inventoryLoader->loadInventory($inventory);