Use iterator_to_array instead of changing property type

This keeps the array type for usedDocuments and converts
the iterable from StoreInterface::query() to an array.

Author: OskarStark

src/agent/src/Bridge/SimilaritySearch/SimilaritySearch.php

@@ -23,9 +23,9 @@
 final class SimilaritySearch
 {
     /**
-     * @var iterable<VectorDocument>
+     * @var VectorDocument[]
      */
-    public iterable $usedDocuments = [];
+    public array $usedDocuments = [];
 
     public function __construct(
         private readonly VectorizerInterface $vectorizer,
@@ -39,17 +39,17 @@ public function __construct(
     public function __invoke(string $searchTerm): string
     {
         $vector = $this->vectorizer->vectorize($searchTerm);
-        $this->usedDocuments = $this->store->query($vector);
+        $this->usedDocuments = iterator_to_array($this->store->query($vector));
 
-        $result = '';
-        foreach ($this->usedDocuments as $document) {
-            $result .= json_encode($document->metadata);
+        if ([] === $this->usedDocuments) {
+            return 'No results found';
         }
 
-        if ('' === $result) {
-            return 'No results found';
+        $result = 'Found documents with following information:'.\PHP_EOL;
+        foreach ($this->usedDocuments as $document) {
+            $result .= json_encode($document->metadata);
         }
 
-        return 'Found documents with following information:'.\PHP_EOL.$result;
+        return $result;
     }
 }