Refactor from single to multiple accounts

Classes/AccountRecord.php

@@ -0,0 +1,30 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Sitegeist\Flow\OpenAiClientFactory;
+
+use Neos\Flow\Annotations as Flow;
+
+#[Flow\Proxy(false)]
+class AccountRecord
+{
+    public function __construct(
+        public readonly string $name,
+        public readonly string $apiKey,
+        public readonly string $organisation,
+    ) {
+    }
+
+    /**
+     * @param array<string,mixed> $configuration
+     */
+    public static function fromConfiguration(string $name, array $configuration): self
+    {
+        return new self(
+            $name,
+            $configuration['apiKey'],
+            $configuration['organisation'],
+        );
+    }
+}

Classes/AccountRepository.php

@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Sitegeist\Flow\OpenAiClientFactory;
+
+use Neos\Flow\Annotations as Flow;
+
+#[Flow\Scope('singleton')]
+class AccountRepository
+{
+    #[Flow\InjectConfiguration(path: 'accounts')]
+    protected ?array $accounts = [];
+
+    /**
+     * @return array<AccountRecord>
+     */
+    public function findAll(): array
+    {
+        return array_map(
+            fn (array $data, string $name): AccountRecord => AccountRecord::fromConfiguration($name, $data),
+            $this->accounts,
+            array_keys($this->accounts)
+        );
+    }
+
+    public function findById(string $id): AccountRecord
+    {
+        if (array_key_exists($id, $this->accounts)) {
+            return AccountRecord::fromConfiguration($id, $this->accounts[$id]);
+        }
+
+        throw new \Exception('Unknown account "' . $id . '"', 1707149688);
+    }
+}

Classes/OpenAiClientFactory.php

@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Sitegeist\Flow\OpenAiClientFactory;
@@ -10,22 +11,16 @@
 
 class OpenAiClientFactory
 {
-    #[Flow\InjectConfiguration(path: 'apiKey')]
-    protected ?string $apiKey = '';
-
-    #[Flow\InjectConfiguration(path: 'organisation')]
-    protected ?string $organisation = '';
-
     #[Flow\Inject]
-    protected ClientInterface $client;
+    protected ClientInterface $httpClient;
 
-    public function createClient(): ClientContract
+    public function createClientForAccountRecord(AccountRecord $record): ClientContract
     {
         $factory = (new Factory())
-            ->withHttpClient($this->client)
-            ->withApiKey($this->apiKey)
+            ->withHttpClient($this->httpClient)
+            ->withApiKey($record->apiKey)
             ->withHttpHeader('OpenAI-Beta', 'assistants=v1')
-            ->withOrganization($this->organisation);
+            ->withOrganization($record->organisation);
         return $factory->make();
     }
 }

Configuration/Settings.yaml

@@ -3,6 +3,5 @@ Sitegeist:
     OpenAiClientFactory:
       accounts: []
         # name:
-        #  discriminator: 'aja-%ENV:ENVIRONMENT_NAME%'
         #  apiKey: ~
         #  organisation: ~