Fix PHPStan errors and apply code style improvements

- Fix method call from vectorize() to vectorizeTextDocuments() in Indexer
- Change null|string to ?string type hints in loader interfaces
- Remove unused imports from ReplaceTextTransformer
- Apply Yoda conditions in tests
- Use dedicated InvalidArgumentException for TextFileLoader null source
- Update parameter documentation for better clarity
- Remove trailing spaces and apply consistent formatting

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: OskarStark

demo/src/Blog/Embedder.php

@@ -11,15 +11,24 @@
 
 namespace App\Blog;
 
+use Symfony\AI\Store\Document\Loader\InMemoryLoader;
 use Symfony\AI\Store\Document\Metadata;
 use Symfony\AI\Store\Document\TextDocument;
-use Symfony\AI\Store\IndexerInterface;
+use Symfony\AI\Store\Document\TransformerInterface;
+use Symfony\AI\Store\Document\VectorizerInterface;
+use Symfony\AI\Store\Indexer;
+use Symfony\AI\Store\StoreInterface;
 
 final readonly class Embedder
 {
+    /**
+     * @param TransformerInterface[] $transformers
+     */
     public function __construct(
         private FeedLoader $loader,
-        private IndexerInterface $indexer,
+        private array $transformers,
+        private VectorizerInterface $vectorizer,
+        private StoreInterface $store,
     ) {
     }
 
@@ -30,6 +39,16 @@ public function embedBlog(): void
             $documents[] = new TextDocument($post->id, $post->toString(), new Metadata($post->toArray()));
         }
 
-        $this->indexer->index($documents);
+        // Use the new Indexer with InMemoryLoader for the complete pipeline
+        $inMemoryLoader = new InMemoryLoader($documents);
+        $indexer = new Indexer(
+            $inMemoryLoader,
+            $this->transformers,
+            $this->vectorizer,
+            $this->store
+        );
+        
+        // Process through the complete pipeline: load → transform → vectorize → store
+        $indexer->index(); // Uses default null source, perfect for InMemoryLoader
     }
 }

examples/indexer/index-inmemory-loader.php

@@ -27,7 +27,7 @@
 
 $documents = [
     new TextDocument(
-        Uuid::v4(), 
+        Uuid::v4(),
         'Artificial Intelligence is transforming the way we work and live. Machine learning algorithms can now process vast amounts of data and make predictions with remarkable accuracy.',
         new Metadata(['title' => 'AI Revolution'])
     ),

src/store/src/Document/Loader/InMemoryLoader.php

@@ -30,7 +30,7 @@ public function __construct(
     ) {
     }
 
-    public function load(null|string $source, array $options = []): iterable
+    public function load(?string $source, array $options = []): iterable
     {
         yield from $this->documents;
     }

src/store/src/Document/Loader/TextFileLoader.php

@@ -14,6 +14,7 @@
 use Symfony\AI\Store\Document\LoaderInterface;
 use Symfony\AI\Store\Document\Metadata;
 use Symfony\AI\Store\Document\TextDocument;
+use Symfony\AI\Store\Exception\InvalidArgumentException;
 use Symfony\AI\Store\Exception\RuntimeException;
 use Symfony\Component\Uid\Uuid;
 
@@ -22,10 +23,14 @@
  */
 final readonly class TextFileLoader implements LoaderInterface
 {
-    public function load(null|string $source, array $options = []): iterable
+    public function load(?string $source, array $options = []): iterable
     {
-        if (null === $source || !is_file($source)) {
-            throw new RuntimeException(\sprintf('File "%s" does not exist.', $source ?? 'null'));
+        if (null === $source) {
+            throw new InvalidArgumentException('TextFileLoader requires a file path as source, null given.');
+        }
+        
+        if (!is_file($source)) {
+            throw new RuntimeException(\sprintf('File "%s" does not exist.', $source));
         }
 
         $content = file_get_contents($source);

src/store/src/Document/LoaderInterface.php

@@ -17,10 +17,10 @@
 interface LoaderInterface
 {
     /**
-     * @param null|string          $source  Identifier for the loader to load the documents from, e.g. file path, folder, or URL. Can be null for InMemoryLoader.
+     * @param string|null          $source  Identifier for the loader to load the documents from, e.g. file path, folder, or URL. Can be null for InMemoryLoader.
      * @param array<string, mixed> $options loader specific set of options to control the loading process
      *
      * @return iterable<TextDocument> iterable of TextDocuments loaded from the source
      */
-    public function load(null|string $source, array $options = []): iterable;
+    public function load(?string $source, array $options = []): iterable;
 }

src/store/src/Document/Transformer/ReplaceTextTransformer.php

@@ -11,11 +11,8 @@
 
 namespace Symfony\AI\Store\Document\Transformer;
 
-use Symfony\AI\Store\Document\Metadata;
-use Symfony\AI\Store\Document\TextDocument;
 use Symfony\AI\Store\Document\TransformerInterface;
 use Symfony\AI\Store\Exception\InvalidArgumentException;
-use Symfony\Component\Uid\Uuid;
 
 /**
  * Replaces specified text within document content.

src/store/src/Indexer.php

@@ -36,38 +36,7 @@ public function __construct(
     }
 
     /**
-     * @param TextDocument|iterable<TextDocument> $documents
-     * @param int                                 $chunkSize number of documents to vectorize and store in one batch
-     */
-    public function index(TextDocument|iterable $documents, int $chunkSize = 50): void
-    {
-        if ($documents instanceof TextDocument) {
-            $documents = [$documents];
-        }
-
-        $counter = 0;
-        $chunk = [];
-        foreach ($documents as $document) {
-            $chunk[] = $document;
-            ++$counter;
-
-            if ($chunkSize === \count($chunk)) {
-                $this->store->add(...$this->vectorizer->vectorize($chunk));
-                $chunk = [];
-            }
-        }
-
-        if (\count($chunk) > 0) {
-            $this->store->add(...$this->vectorizer->vectorize($chunk));
-        }
-
-        $this->logger->debug(0 === $counter ? 'No documents to index' : \sprintf('Indexed %d documents', $counter));
-    }
-
-    /**
-     * Process sources through the complete document pipeline: load → transform → vectorize → store.
-     * 
-     * @param null|string|array<string> $source  Source identifier (file path, URL, etc.), array of sources, or null for InMemoryLoader
+     * @param null|string|array<string> $source  Source identifier (file path, URL, etc.), array of sources, or null if the loader does not need a source, e.g. InMemoryLoader
      * @param array<string, mixed> $options Processing options
      */
     public function index(null|string|array $source = null, array $options = []): void
@@ -103,13 +72,13 @@ public function index(null|string|array $source = null, array $options = []): vo
             ++$counter;
 
             if ($chunkSize === \count($chunk)) {
-                $this->store->add(...$this->vectorizer->vectorize($chunk));
+                $this->store->add(...$this->vectorizer->vectorizeTextDocuments($chunk));
                 $chunk = [];
             }
         }
 
         if (\count($chunk) > 0) {
-            $this->store->add(...$this->vectorizer->vectorize($chunk));
+            $this->store->add(...$this->vectorizer->vectorizeTextDocuments($chunk));
         }
 
         $this->logger->debug('Document processing completed', [

src/store/src/IndexerInterface.php

@@ -19,7 +19,9 @@
 interface IndexerInterface
 {
     /**
-     * @param null|string|array<string> $source  Source identifier (file path, URL, etc.), array of sources, or null for InMemoryLoader
+     * Process sources through the complete document pipeline: load → transform → vectorize → store.
+     * 
+     * @param null|string|array<string> $source  Source identifier (file path, URL, etc.), array of sources, or null if the loader does not need a source, e.g. InMemoryLoader
      * @param array<string, mixed> $options Processing options
      */
     public function index(null|string|array $source = null, array $options = []): void;

src/store/tests/IndexerTest.php

@@ -69,10 +69,10 @@ public function testIndexEmptyDocumentList()
         $logger->expects($this->exactly(2))->method('debug')
             ->willReturnCallback(function ($message, $context) {
                 static $callCount = 0;
-                if ($callCount === 0) {
+                if (0 === $callCount) {
                     $this->assertSame('Starting document processing', $message);
                     $this->assertSame(['source' => null, 'options' => []], $context);
-                } elseif ($callCount === 1) {
+                } elseif (1 === $callCount) {
                     $this->assertSame('Document processing completed', $message);
                     $this->assertSame(['source' => null, 'documents_processed' => 0], $context);
                 }