feature #476 [Store] Rename __invoke methods in interfaces to descriptive names (OskarStark)

This PR was squashed before being merged into the main branch.

Discussion
----------

[Store] Rename `__invoke` methods in interfaces to descriptive names

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| Docs?         | no
| Issues        | --
| License       | MIT

Discussed with `@chr`-hertel upfront.

Replace generic `__invoke` methods with descriptive names:
- `TransformerInterface::__invoke` → `transform`
- `VectorizerInterface::__invoke` → `vectorize`
- `LoaderInterface::__invoke` → `load`

This improves code readability and makes the intent of each interface clearer. Updated all implementations and usages throughout the codebase.

Commits
-------

0f2b756 [Store] Rename `__invoke` methods in interfaces to descriptive names

Author: OskarStark

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

@@ -22,7 +22,7 @@
  */
 final readonly class TextFileLoader implements LoaderInterface
 {
-    public function __invoke(string $source, array $options = []): iterable
+    public function load(string $source, array $options = []): iterable
     {
         if (!is_file($source)) {
             throw new RuntimeException(\sprintf('File "%s" does not exist.', $source));

src/store/src/Document/LoaderInterface.php

@@ -22,5 +22,5 @@ interface LoaderInterface
      *
      * @return iterable<TextDocument> iterable of TextDocuments loaded from the source
      */
-    public function __invoke(string $source, array $options = []): iterable;
+    public function load(string $source, array $options = []): iterable;
 }

src/store/src/Document/Transformer/ChainTransformer.php

@@ -28,10 +28,10 @@ public function __construct(iterable $transformers)
         $this->transformers = $transformers instanceof \Traversable ? iterator_to_array($transformers) : $transformers;
     }
 
-    public function __invoke(iterable $documents, array $options = []): iterable
+    public function transform(iterable $documents, array $options = []): iterable
     {
         foreach ($this->transformers as $transformer) {
-            $documents = $transformer($documents, $options);
+            $documents = $transformer->transform($documents, $options);
         }
 
         return $documents;

src/store/src/Document/Transformer/ChunkDelayTransformer.php

@@ -33,7 +33,7 @@ public function __construct(
     /**
      * @param array{chunk_size?: int, delay?: int} $options
      */
-    public function __invoke(iterable $documents, array $options = []): iterable
+    public function transform(iterable $documents, array $options = []): iterable
     {
         $chunkSize = $options[self::OPTION_CHUNK_SIZE] ?? 50;
         $delay = $options[self::OPTION_DELAY] ?? 10;

src/store/src/Document/Transformer/TextSplitTransformer.php

@@ -32,7 +32,7 @@
     /**
      * @param array{chunk_size?: int, overlap?: int} $options
      */
-    public function __invoke(iterable $documents, array $options = []): iterable
+    public function transform(iterable $documents, array $options = []): iterable
     {
         $chunkSize = $options[self::OPTION_CHUNK_SIZE] ?? 1000;
         $overlap = $options[self::OPTION_OVERLAP] ?? 200;

src/store/src/Document/TransformerInterface.php

@@ -26,5 +26,5 @@ interface TransformerInterface
      *
      * @return iterable<TextDocument>
      */
-    public function __invoke(iterable $documents, array $options = []): iterable;
+    public function transform(iterable $documents, array $options = []): iterable;
 }

src/store/src/Document/Vectorizer.php

@@ -26,7 +26,7 @@ public function __construct(
     ) {
     }
 
-    public function __invoke(array $documents): array
+    public function vectorize(array $documents): array
     {
         $documentCount = \count($documents);
         $this->logger->info('Starting vectorization process', ['document_count' => $documentCount]);

src/store/src/Document/VectorizerInterface.php

@@ -23,5 +23,5 @@ interface VectorizerInterface
      *
      * @return VectorDocument[]
      */
-    public function __invoke(array $documents): array;
+    public function vectorize(array $documents): array;
 }

src/store/src/Indexer.php

@@ -45,13 +45,13 @@ public function index(TextDocument|iterable $documents, int $chunkSize = 50): vo
             ++$counter;
 
             if ($chunkSize === \count($chunk)) {
-                $this->store->add(...($this->vectorizer)($chunk));
+                $this->store->add(...$this->vectorizer->vectorize($chunk));
                 $chunk = [];
             }
         }
 
         if (\count($chunk) > 0) {
-            $this->store->add(...($this->vectorizer)($chunk));
+            $this->store->add(...$this->vectorizer->vectorize($chunk));
         }
 
         $this->logger->debug(0 === $counter ? 'No documents to index' : \sprintf('Indexed %d documents', $counter));

src/store/tests/Document/Loader/TextFileLoaderTest.php

@@ -27,14 +27,14 @@ public function testLoadWithInvalidSource()
         $this->expectException(RuntimeException::class);
         $this->expectExceptionMessage('File "/invalid/source.txt" does not exist.');
 
-        iterator_to_array($loader('/invalid/source.txt'));
+        iterator_to_array($loader->load('/invalid/source.txt'));
     }
 
     public function testLoadWithValidSource()
     {
         $loader = new TextFileLoader();
 
-        $documents = iterator_to_array($loader(\dirname(__DIR__, 5).'/fixtures/lorem.txt'));
+        $documents = iterator_to_array($loader->load(\dirname(__DIR__, 5).'/fixtures/lorem.txt'));
 
         $this->assertCount(1, $documents);
         $this->assertInstanceOf(TextDocument::class, $document = $documents[0]);
@@ -48,7 +48,7 @@ public function testSourceIsPresentInMetadata()
         $loader = new TextFileLoader();
 
         $source = \dirname(__DIR__, 5).'/fixtures/lorem.txt';
-        $documents = iterator_to_array($loader($source));
+        $documents = iterator_to_array($loader->load($source));
 
         $this->assertCount(1, $documents);
         $this->assertInstanceOf(TextDocument::class, $document = $documents[0]);