Simplify public interface

Author: jaapio

packages/guides-cli/src/Command/Serve.php

@@ -28,7 +28,6 @@
 use phpDocumentor\Guides\Nodes\DocumentNode;
 use phpDocumentor\Guides\RenderContext;
 use phpDocumentor\Guides\Renderer\DocumentListIterator;
-use phpDocumentor\Guides\Renderer\DocumentTreeIterator;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
@@ -152,11 +151,8 @@ function (FileModifiedEvent $event) use ($documents, $sourceFileSystem, $project
  $documents = $this->commandBus->handle(new CompileDocumentsCommand($documents, new CompilerContext($projectNode)));
  $destinationFileSystem = FlySystemAdapter::createForPath($settings->getOutput());
 
- $documentIterator = new DocumentListIterator(
- new DocumentTreeIterator(
- [$projectNode->getRootDocumentEntry()],
- $documents,
- ),
+ $documentIterator = DocumentListIterator::create(
+ $projectNode->getRootDocumentEntry(),
  $documents,
  );
 

packages/guides/src/Handlers/RenderCommand.php

@@ -18,7 +18,6 @@
 use phpDocumentor\Guides\Nodes\DocumentNode;
 use phpDocumentor\Guides\Nodes\ProjectNode;
 use phpDocumentor\Guides\Renderer\DocumentListIterator;
-use phpDocumentor\Guides\Renderer\DocumentTreeIterator;
 
 final class RenderCommand
 {
@@ -33,11 +32,8 @@ public function __construct(
  private readonly ProjectNode $projectNode,
  private readonly string $destinationPath = '/',
  ) {
- $this->documentIterator = new DocumentListIterator(
- new DocumentTreeIterator(
- [$this->projectNode->getRootDocumentEntry()],
- $this->documentArray,
- ),
+ $this->documentIterator = DocumentListIterator::create(
+ $this->projectNode->getRootDocumentEntry(),
  $this->documentArray,
  );
  }

packages/guides/src/Renderer/DocumentListIterator.php

@@ -18,6 +18,7 @@
 use Iterator;
 use OutOfRangeException;
 use phpDocumentor\Guides\Nodes\DocumentNode;
+use phpDocumentor\Guides\Nodes\DocumentTree\DocumentEntryNode;
 use RecursiveIteratorIterator;
 use WeakMap;
 use WeakReference;
@@ -60,6 +61,17 @@ public function __construct(
  }
  }
 
+ public static function create(DocumentEntryNode $getRootDocumentEntry, array $documentArray): self
+ {
+ return new self(
+ new DocumentTreeIterator(
+ [$getRootDocumentEntry],
+ $documentArray,
+ ),
+ $documentArray,
+ );
+ }
+
  public function next(): void
  {
  $this->previousDocument = $this->currentDocument;